docs(cleanup): adding clarifying comments

Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>

Author: Dhruv-0-Arora

fission/src/mirabuf/MirabufThumbnail.ts

@@ -11,6 +11,10 @@ import {
 import World from "@/systems/World"
 import { unzipMira } from "@/util/Utility"
 
+/**
+ * wire tag protobuf has for thumbnails
+ * derived so renumbering the schema can't desync from the generated code
+ */
 const ASSEMBLY_THUMBNAIL_TAG = Reader.create(
     mirabuf.Assembly.encode(new mirabuf.Assembly({ thumbnail: new mirabuf.Thumbnail() })).finish()
 ).uint32()

fission/src/ui/panels/configuring/initial-config/InitialConfigPanel.tsx

@@ -33,7 +33,7 @@ const InitialConfigPanel: React.FC<PanelImplProps<void, void>> = ({ panel }) =>
 
     useHoldPhysicsPause()
 
-    // bug: pressing 'enter' will open the library modal. Instead, we focus on this panel
+    // The Add Assembly button keeps DOM focus after spawning, so Enter would reopen the Library.
     useEffect(() => {
         if (document.activeElement instanceof HTMLElement) document.activeElement.blur()
     }, [])

fission/src/ui/tour/TourOverlay.tsx

@@ -15,6 +15,8 @@ const SCRIM_COLOR = "rgba(0,0,0,0.5)"
 // Gap from the top bar / viewport edge for an anchorless card that is pinned to a corner.
 const SCREEN_EDGE_GAP = 12
 const SPOTLIGHT_PAD = 6
+
+// after step changes we remeasure points. Anchors can move without resizing because of MUI stuff / panels
 const SETTLE_DELAYS = [0, 100, 250, 450]
 
 /** Fixed-position style for an anchorless card, keyed by its {@link ScreenPosition}. */

fission/src/ui/tour/tourConditions.ts

@@ -74,6 +74,7 @@ export interface TourResult {
     toast?: string
 }
 
+/** The nearest preceding step whose `advanceOn` establishes this condition. */
 function producerOf(condition: TourCondition, before: number): number | undefined {
     for (let i = before - 1; i >= 0; i--) {
         const advanceOn = TOUR_STEPS[i].advanceOn
@@ -87,6 +88,11 @@ function countFor(stepIndex: number, snapshot: TourSnapshot): number | undefined
     return condition ? CONDITIONS[condition].count?.(snapshot) : undefined
 }
 
+/**
+ * given the current step and the progress of the simulator, returns what step the user should be on.
+ *
+ * Used for rewinding in the event of a user being outside the tour and advancing when step condition met
+ */
 export function reconcile(stepIndex: number, snapshot: TourSnapshot, previous: TourRuntime): TourResult {
     const step = TOUR_STEPS[stepIndex]