FE-691: Multi-chat substrate — chat table + reconciliation_edge#90
Merged
Conversation
FE-691 Multi-chat substrate — chat table + reconciliation_edge
SummarySubstrate-only data-model RFC. First phase of the substrate evolution toward A71 (patch ledger) and ontology sharpening. No user-visible change. Scope
Out of scopePatch ledger (A71); phase-route de-emphasis; ontology sharpening ( Design doc
Unblocks
Relation to FE-656Independent of FE-656 (side-chat user surface). Peer of side-chat track on Track B (Infrastructure). FE-656 sub-items V3 and V4 will reference this issue as a blocker. |
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
kostandinang
changed the title
kostandinang
added a commit
that referenced
this pull request
May 5, 2026
Branch parked, not merging. Replaced by docs/design/MULTI_CHAT.md (PR #90, FE-691). Kept in-tree for review-thread context.
PR SummaryLow Risk Overview The doc also outlines the intended Drizzle/SQLite migration sequence and invariants, but does not implement any schema, migration, or code changes. Reviewed by Cursor Bugbot for commit 33db9aa. Bugbot is set up for automated code reviews on this repo. Configure here. |
🤖 Augment PR SummarySummary: Adds an RFC-style design doc describing Phase 1 of a “multi-chat” substrate evolution (doc-only; no runtime changes). Changes:
Technical Notes: Emphasizes “chat threads as substrate, not workflow state” and preserves existing knowledge graph tables/turn IDs while repointing the upward FK path. 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
kostandinang
force-pushed
the
ka/fe-691-multi-chat-substrate
branch
from
40f7926 to
fee726e
Compare
kostandinang
changed the base branch from
graphite-base/90
to
ka/fe-656-side-chat-v1-2e
This was referenced May 6, 2026
lunelson
previously approved these changes
May 6, 2026
Contributor
Author
Merge activity
|
kostandinang
changed the base branch from
ka/fe-656-side-chat-v1-2e
to
graphite-base/90
3 tasks
kostandinang
force-pushed
the
ka/fe-691-multi-chat-substrate
branch
from
7818813 to
7d46a40
Compare
kostandinang
force-pushed
the
graphite-base/90
branch
from
b4f4d33 to
292b62a
Compare
kostandinang
changed the base branch from
graphite-base/90
to
ka/fe-656-side-chat-v1-2e
5 tasks
lunelson
previously approved these changes
May 7, 2026
Captures the §11 fidelity work as its own sub-slice of the V1.2 frontier item. Implementation will follow on this branch. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…the composer Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… canUndo Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replaces center-pill positioning (left-1/2 + translateX) with anchored left-9 right-9 so the toast bar fills the available area between the + and send buttons. Content stays centered inside via justify-between (check + label on the left, Undo on the right). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…em heights - Put h-72 directly on the <ul> for guaranteed scroll (the flex-1 + min-h-0 pattern wasn't propagating height through the absolute-positioned wrapper). - Drop px-2 from the <ul> so each item spans the full popover width and the hover background paints edge-to-edge. - Move horizontal padding inside each item (px-3 on summary/div); body section uses pl-7 to stay aligned with summary text after the chevron. - Tighten vertical rhythm: py-2.5 → py-2 between items. - Bump body text size to text-sm with leading-relaxed and pt-1 spacing for comfortable reading when an item is expanded. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Clarify substrate invariants and migration verification expectations so the design doc distinguishes DB constraints from application-layer checks. Co-authored-by: Cursor <cursoragent@cursor.com>
kostandinang
force-pushed
the
ka/fe-691-multi-chat-substrate
branch
from
7d46a40 to
ad95ed0
Compare
kostandinang
force-pushed
the
ka/fe-656-side-chat-v1-2e
branch
from
292b62a to
a44c6bb
Compare
lunelson
previously approved these changes
May 7, 2026
graphite-app
Bot
changed the base branch from
ka/fe-656-side-chat-v1-2e
to
graphite-base/90
Co-authored-by: Kostandin Angjellari <kostandinang@users.noreply.github.com>
github-actions
Bot
dismissed
lunelson’s stale review
Manual conflict resolution detected after approval in merge commit 33db9aa for files: src/client/components/tests/side-chat-host.test.tsx,src/client/components/side-chat-host.tsx,src/client/routes/specification/$id/tests/structured-list-view.test.tsx
lunelson
approved these changes
May 7, 2026
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
3 participants
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
Adds
docs/design/MULTI_CHAT.md— RFC for the multi-chat data-model substrate. Doc-only.Why
Phase 1 of the substrate evolution toward the patch ledger (A71). Repoints
turnfromspecificationto a newchattable, addsspecification.primary_chat_idandchat.active_turn_id, and introducesreconciliation_edgefor the cross-surface "this item changed, that needs attention" signal. No user-visible change.Supersedes the deprecated ADR on
ka/fe-656-side-chat_v4(parked, not merging).Unblocks
reconciliation_edge, replacing ad-hoc REVISIT state).Track placement
Independent of FE-656; peer of the side-chat track on Track B (Infrastructure). FE-656 sub-items V3 and V4 will reference this issue as a blocker.
Linear
FE-691