Skip to content

Add second slot to two-by-two-table, server-closet, the-archive#43

Merged
harshithmullapudi merged 1 commit into
mainfrom
fix/expand-more-variant-slots
Jul 9, 2026
Merged

Add second slot to two-by-two-table, server-closet, the-archive#43
harshithmullapudi merged 1 commit into
mainfrom
fix/expand-more-variant-slots

Conversation

@harshithmullapudi

Copy link
Copy Markdown
Member

Summary

Follow-up to #42. The slot-collision validator caught the same pattern in three more catalog variants that ship a single npcPosition — any town pairing two NPCs in a positioning room, a growth war-room, or an archive would hit "collides with npcs[N]" on the second MDX.

Same pattern as office.pitch-wall: keep the existing tile as slot "" (single-NPC MDXs stay bound to the same visual position) and add one "second" slot per variant on a walkable tile.

  • office.two-by-two-table — adds "second" at (11, 5). Row 5 walkable between the middle workstation (cols 7-8) and the right one (cols 12-13); plant snake (cols 10-11) ends at row 4.
  • workshop.the-server-closet — adds "second" at (8, 4). Middle workstation feet — mirrors the existing (13, 4) position on the right workstation.
  • library.the-archive — adds "second" at (10, 7). Second reading table on row 7, well inside the LIBRARY walkable rect.

Effect on bookface-town

After this + slotId edits on the MDXs, the four multi-NPC buildings render everyone:

Building Variant NPCs
officehours office.pitch-wall (3 slots) dalton "", pg left, seibel right
positioning office.two-by-two-table (2 slots) Dan "", Maya second
growthfloor workshop.the-server-closet (2 slots) Tomás "", Priya second
customerlab library.the-archive (2 slots) Frank "", Dr. Novak second

Test plan

  • JSON parses cleanly (verified with node -e "require(...)").
  • Existing single-NPC towns using any of these three variants keep rendering at the same tile.
  • Deploy bookface-town after adding slotId: second to metrics-mirror.mdx (Priya), one-liner.mdx (Maya), user-interview.mdx (Dr. Novak) — passes the validator.
  • Confirm no NPC sprite intersects a wall or gets stranded outside the walkable rect in-game.

🤖 Generated with Claude Code

…two slots

Same pattern as office.pitch-wall — keep the existing position as slot
"" (backward compat for MDXs without a slotId frontmatter) and add a
second walkable slot each. Unblocks bookface-town's positioning,
growthfloor, and customerlab, which each pair a specialist with a
challenger and were hitting the new slot-collision error.

- office.two-by-two-table: add "second" at (11, 5) — walkable strip
  next to plant snake.
- workshop.the-server-closet: add "second" at (8, 4) — visually at
  the middle workstation feet.
- library.the-archive: add "second" at (10, 7) — at the second
  reading table row.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant