feat(engine): discussion map into the manifest — engine map commands, adapter display, migration 045#385
Open
leeovery wants to merge 8 commits into
Open
feat(engine): discussion map into the manifest — engine map commands, adapter display, migration 045#385leeovery wants to merge 8 commits into
leeovery wants to merge 8 commits into
Conversation
…anifests Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…mapState Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…sponses Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
DISCUSSION_GLYPH vocabulary (deferred = ⊙), discussionMap projection over the kernel tree, and the discussion-process read gateway (map verb: DATA counts/all_decided/unresolved/review_cycles + DISPLAY block). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…trip Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ds, adapter display The file template drops its Discussion Map section; subtopics are recorded via engine map add/set, the status display is the adapter's DISPLAY block, and convergence/conclusion read all_decided, unresolved, and review_cycles from one adapter call. deferred joins the lifecycle for conclude-anyway. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…subtopics Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
leeovery
added a commit
that referenced
this pull request
Jun 12, 2026
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…scribe map at resume The first branch row ticks up into the header (hang-off-header grammar, matching the epic dashboard's no-blank rule). The resume-detection path now renders the map before the continue/restart gate — previously the model improvised this display; now it's prescribed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
PR 3 of the engine stack (on #384). The discussion map becomes typed state in the manifest, and the engine gains its first write commands.
kernel/manifest.cjs: load + atomic save (temp-file rename).domain/map.cjs—addSubtopic/setSubtopicState(enumpending|exploring|converging|decided|deferred, two-level nesting, loud validation) +mapState(decision-ready: counts,all_decided,unresolved).engine map add/set— JSON responses carry the derived state (all_decided,unresolved_count) so the session never needs a follow-up read. No internal commits — the session's existing commit cadence picks the manifest change up.workflow-discussion-processgets its gateway (map {wu} {topic}): DATA (counts,all_decided,unresolved,review_cycles) + DISPLAY (the tree, hang-off-header — the old┌─opener is normalised away).discussion-session.mdre-pointed — judgment decides,map setrecords; the file holds knowledge only (template's map section dropped); convergence + the twice-stated manual cache count collapse into one adapter call; conclude-anyway sets unresolved subtopicsdeferredand notes them in Open Threads. Full survey of every "Discussion Map" reference across the skill, shared references, and agents — all re-pointed.Judgment calls to review
⊙for deferred — collides visually with the discovery tier's⊙ handled(separate vocabularies, same symbol). Alternatives:◌/▫.addSubtopicenforces kebab-case slugs at write time.review_cyclescountsreview-*.mdonly (perspective/synthesis files would falsely satisfy the review safety net).subtopics(stale map state would corrupt a fresh start).Tests
82/82 engine (incl. 27 new map tests: domain, projection goldens, manifest IO, CLI round-trip); epic discovery suite 98/98 unchanged; migration 045 15/15 (044 still green); typecheck clean.
🤖 Generated with Claude Code