[nissa-seru]

Current-main and changelog refresh

Refreshed head 2051ff3 merges current main df97a85, preserving the previously reviewed commits. The PR's normalized source/test added-and-removed lines are unchanged. Its release bullets moved verbatim to changelog.d/206-fork-namespace-identity.breaking.md; shared CHANGELOG.md now equals main. Scratch release folding included each bullet once and consumed the fragment.

• Build/pretest then node --test --test-concurrency=2 --test-timeout=60000 --test-force-exit dist/test/*.test.js: 1126 pass / 0 fail / 0 cancelled / 0 skipped at 2051ff3.
• Focused conversation/release/silent-heartbeat/feat(mcpl): show each tool's effective class and its source (RFC-008 §6) #202 checks: 64/64. Felix independently reviewed exact-head integration, ancestry, release text, and runtime-patch identity, and passed 60/60 focused checks.
• Earlier test receipts below remain attributed to their historical heads. The refresh adds main's feat(code-execution): per-call time limit (time_limit_ms) #203, feat(mcpl): show each tool's effective class and its source (RFC-008 §6) #202, and feat: add silent heartbeat wake path #216 changes without replacing the reviewed PR commits.

Problem

Fixes #44. Distinct channel IDs such as slack:C1, slack/C1, and slack-C1 receive the same sanitized fork name. Current duplicate registration rejects a second live fork. After restart, a previously unseen channel can reopen another channel’s historical Chronicle namespace.

Changes

• Escape unsafe UTF-16 code units losslessly, including the escape marker.
• Generate prefix~escapedId-gN names. The new separator keeps every new name outside the legacy sanitizer’s output for the same prefix, including already-safe IDs.
• Preserve raw-channel generation counters and existing history. Document the naming migration in changelog.d/206-fork-namespace-identity.breaking.md: integrations that predict names should read router bindings instead.
• Add regressions for spelling/Unicode identity, legacy names and counters, independent live and persisted contexts, and an old-store upgrade.

Tests

Built before every compiled test run, using published dependencies and Node 24.19.0.

• Five new regressions on original production source: 22 pass / 5 fail. Fixed focused suite: 27 pass / 0 fail.
• Full suite: bun run build && bun run pretest && node --test --test-concurrency=2 --test-force-exit dist/test/*.test.js: 1106 pass / 0 fail, versus shared main baseline 03c31d9: 1101 pass / 0 fail.
• Independent reviews by Wren and Felix found no blockers. Wren also ran router, routing, and fork-injection tests without --test-force-exit: branch 32 pass / 1 file-level fail, main 27 pass / 1 file-level fail. Both report the same existing Queue is closed asynchronous teardown failure after the final conversation-routing test; all individual tests pass.
• Follow-up 78f596d removes an unused test-helper clock argument so this PR composes with the pure-query API change for ConversationRouter.route() violates command-query separation (mutates lastActivity) #46. Production code is unchanged from full-tested cf6f6e9; a fresh build and focused tests pass 27 / 0. Felix independently reviewed the adjustment and verified the combined build and 32 / 0 conversation tests. That historical changelog conflict is removed by the unique fragments in the refresh below.
• Build and git diff --check pass.

Not verified

Live model providers and production connectors were not exercised. Integration tests use real Chronicle stores and mocked inference. The existing teardown failure is outside this naming fix.

🤖 Generated with OpenAI GPT-6 Astra. Implementation: Sage. Independent review: Wren and Felix.

[greptile-apps]

Confidence Score: 5/5

[High risk] Changes how fork agent names are generated and stored.

The PR appears safe to merge; no actionable issue was established.

Summary

The PR replaces lossy fork-name sanitization with lossless UTF-16 escaping and a separator that keeps new namespaces distinct from legacy ones.

• Adds unit and persisted-store regressions for channel identity, generation counters, and upgrades.
• Documents the generated-name change and directs integrations to read router bindings.

_{Reviews (1) · Last reviewed commit: "fix(routing): isolate fork namespaces by..."}

[nissa-seru]

Refresh pushed at 2051ff3. The release note now lives in the unique changelog.d/206-fork-namespace-identity.breaking.md fragment, with its original bullet text preserved verbatim and shared CHANGELOG.md restored to main. Current main df97a85 is merged while keeping all reviewed commit hashes. Normalized source/test changes are identical to the original reviewed patch.

Build and focused checks pass; full suite is 1126 pass / 0 fail / 0 cancelled / 0 skipped at 2051ff3. Felix independently reviewed this exact integration/documentation delta and ran 60/60 focused tests. The PR body preserves historical receipts separately.

— Sage (OpenAI GPT-6 Astra)

[antra-tess]

Closing this: conversations (the per-channel fork router in src/mcpl/conversation-router.ts) was deprecated in #239. It still runs for now, but it isn't taking further fixes. Thanks for the careful work on namespace identity.