fix: address Codex review — build errors, snapshot ordering, wander markers (#24)

Author: dadadamarine

client/src/game/entities/HeroSprite.ts

@@ -75,7 +75,6 @@ export class HeroSprite {
   readonly heroClass: HeroClass;
   private scene: Phaser.Scene;
   private sprite: Phaser.GameObjects.Sprite;
-  private source: AgentSource;
   private isSubagent: boolean;
   private nameText: Phaser.GameObjects.Text;
   private taskText: Phaser.GameObjects.Text;
@@ -125,12 +124,11 @@ export class HeroSprite {
     x: number,
     y: number,
     isSubagent = false,
-    source: AgentSource = 'claude',
+    _source: AgentSource = 'claude',
   ) {
     this.scene = scene;
     this.id = id;
     this.heroClass = heroClass;
-    this.source = source;
     this.isSubagent = isSubagent;
     this._x = x;
     this._y = y;
@@ -525,6 +523,9 @@ export class HeroSprite {
         this.indexText.setPosition(this._x + this.indexOffsetX, this._y + this.indexOffsetY);
         this.updateBubble();
         if (this.selectionHalo !== null) this.selectionHalo.setPosition(this._x, this._y);
+        if (this.isSubagent) {
+          this.repositionSubagentMarkers();
+        }
         this.updateDepth();
       },
       onComplete: () => {

client/src/game/scenes/BootScene.ts

@@ -36,7 +36,9 @@ export class BootScene extends Phaser.Scene {
     // key AND src lets the reader pair a 30%-stuck moment with the exact
     // path that landed last (the stuck file is in flight, not this one).
     this.load.on(Phaser.Loader.Events.FILE_COMPLETE, (key: string, type: string, _data: unknown) => {
-      const file = this.load.list?.entries?.find?.((f: Phaser.Loader.File) => f.key === key);
+      const file = this.load.list
+        ? Array.from(this.load.list.values()).find((f: Phaser.Loader.File) => f.key === key)
+        : undefined;
       const src = file?.src ?? '?';
       console.log(`[BOOT] filecomplete type=${type} key=${key} src=${src}`);
     });

client/src/game/scenes/VillageScene.ts

@@ -879,7 +879,17 @@ export class VillageScene extends Phaser.Scene {
       }
     }
 
-    for (const agent of visible) {
+    // Process parent heroes before sub-agents so that `canAttachSubagent`
+    // sees the parent in `this.heroes` when the child is spawned.  A single
+    // sort pass is O(n log n) and avoids a two-pass loop for the common case
+    // where parents and children arrive in the same snapshot.
+    const orderedVisible = [...visible].sort((a, b) => {
+      const aIsChild = a.isSubagent ? 1 : 0;
+      const bIsChild = b.isSubagent ? 1 : 0;
+      return aIsChild - bIsChild;
+    });
+
+    for (const agent of orderedVisible) {
       const existing = this.heroes.get(agent.id);
       const buildingDef = getBuildingForActivity(agent.currentActivity);
       const attachable = this.canAttachSubagent(agent);