Remove now-unnecessary Partial<T> usage (#1115)

These were workarounds for not having `noUncheckedIndexAccess` enabled.
We have it enabled now.

Author: EzraBrooks

packages/roslib/src/actionlib/ActionClient.ts

@@ -26,7 +26,7 @@ export default class ActionClient<
 > extends EventEmitter<{
   timeout: undefined;
 }> {
-  goals: Partial<Record<string, Goal<TGoal>>> = {};
+  goals: Record<string, Goal<TGoal>> = {};
   /** flag to check if a status has been received */
   receivedStatus = false;
   ros: Ros;

packages/roslib/src/tf/BaseTFClient.ts

@@ -7,8 +7,9 @@ import type { std_msgs } from "../types/std_msgs.ts";
  * Base class for TF Clients that provides common functionality.
  */
 export default class BaseTFClient {
-  frameInfos: Partial<
-    Record<string, { transform?: Transform; cbs: ((tf: Transform) => void)[] }>
+  frameInfos: Record<
+    string,
+    { transform?: Transform; cbs: ((tf: Transform) => void)[] }
   > = {};
   republisherUpdateRequested = false;
   ros: Ros;