chore: remove old TODOs

Author: azaleacolburn

fission/src/mirabuf/MirabufParser.ts

@@ -389,7 +389,7 @@ class MirabufParser {
     private loadGlobalTransforms() {
         const root = this._designHierarchyRoot
         const parts = this._assembly.data?.parts
-        if (!parts) return // TODO not sure if we should return or provide a default value
+        if (!parts) return
 
         const partInstances = new Map<string, mirabuf.IPartInstance>(Object.entries(parts.partInstances!))
         const partDefinitions = parts.partDefinitions!

fission/src/ui/modals/configuring/SettingsModal.tsx

@@ -165,7 +165,6 @@ const GeneralTab: React.FC<GeneralTabProps> = ({ writePreference }) => (
                 onClick={checked => {
                     writePreference("RenderScoreboard", checked)
                     if (checked) {
-                        // TODO: figure out scoreboard - I think it should be its own component and not a panel
                         // openPanel("scoreboard");
                     }
                 }}

fission/src/ui/modals/mirabuf/ImportLocalMirabufModal.tsx

@@ -51,11 +51,9 @@ const ImportLocalMirabufModal: React.FC<ModalImplProps<void, void>> = ({ modal }
                 World.physicsSystem.holdPause(PAUSE_REF_ASSEMBLY_SPAWNING)
                 await MirabufCachingService.cacheAndGetLocalWithInfo(hashBuffer, miraType)
                     .then(x => {
-                        if (x) {
-                            // TODO This function shouldn't cache game pieces when imported locally!!!
-                            return createMirabuf(x.assembly, x.cacheInfo.id, miraType)
-                        }
-                        return undefined
+                        if (!x) return undefined
+
+                        return createMirabuf(x.assembly, x.cacheInfo.id, miraType)
                     })
                     .then(x => {
                         if (x) {

fission/src/ui/panels/simulation/DriverStationPanel.tsx

@@ -9,13 +9,8 @@ const DriverStationPanel: React.FC<PanelImplProps<void, void>> = ({ panel }) =>
     const [enabled, setEnabled] = useState(false)
 
     useEffect(() => {
-        // TODO: update Not Connected dynamically when implemented
         configureScreen(panel!, { title: "Driver Station (Not Connected)" }, {})
-    }, [
-        // TODO: update Not Connected dynamically when implemented
-        configureScreen,
-        panel,
-    ])
+    }, [configureScreen, panel])
 
     return (
         <Stack direction="row">