Found 3 issues. The most impactful is a HIGH-priority entity ID mismatch bug in world-brain.ts where the revelation reassignment code compares entity database IDs against petId UUIDs — they never match, making the entire reassignment block dead code and wasting ~15 LLM calls per 30 minutes on rejected revelations. Also proposed: diversifying the hardcoded social think fallbacks (3 creatures producing identical “I feel frustrated and tense right now.” text), and fixing group activity grammar (“as they mourning together” → “while mourning together”).
(HIGH): Entity ID mismatch makes revelation reassignment dead code
Fix: in `world-brain.ts`: (code change) This makes the reassignment actually work — when the LLM picks a cooldown creature, a random eligible creature is chosen instead. Eliminates ~15 wasted LLM calls per 30 minutes. —
(MEDIUM): Generic social think fallbacks break immersion
Fix: in `pet-brain.ts`, replace lines 4336-4346 with personality-aware fallbacks: (code change) This provides 3 variants per emotion so different creatures won’t produce identical thoughts. —
(MEDIUM): Group activity grammar bug for ritual_mourning
Fix: in `bootstrap.ts`: (code change) This produces correct grammar: “as they mourning” → “as they mourn together” — wait, that’s still wrong. The templates use “as they {activityLabel}” and “while {activityLabel}”, both of which need gerund form. The problem is actually that “mourning together” becomes…
Files Modified
bootstrap.tspet-brain.tsworld-brain.ts