FE-544: Turn tree schema and API by lunelson · Pull Request #13 · hashintel/brunch

lunelson · 2026-04-01T11:29:26Z

chore: align Linear issues with PLAN.md slices

feat: migrate from flat message table to turn-tree schema

Replace the message table with the full schema.dbml entity model (13 tables:
project, turn, option, decision, assumption, requirement, criterion + 6 join
tables). Project gains active_turn_id as HEAD pointer. POST /api/chat creates
turns with parent chain and advances HEAD. GET /api/projects/current returns
turns on the active path via recursive CTE. Client hydrates useChat from turns.

Integer autoincrement PKs replace TEXT UUIDs. Entity/join tables beyond
turn/option are created but unpopulated until later slices.

39 tests passing (18 db + 9 app + 12 sse-adapter).

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

docs: record transitional turn-field mapping and integer PK decisions

Add D15 (transitional turn-field inversion — answer=user, question=agent
during pre-structured-interview phase) and D16 (integer autoincrement PKs
replace TEXT UUIDs). Both surfaced during slice 3 build.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

linear · 2026-04-01T11:29:31Z

FE-544 Turn tree schema + API

project_snapshot table. Auto-snapshot at phase transitions. Manual snapshot button. Restore from previous. Client-side diff view.

lunelson · 2026-04-01T11:29:42Z

This stack of pull requests is managed by Graphite. Learn more about stacking.

augmentcode · 2026-04-01T12:42:46Z

🤖 Augment PR Summary

Summary: This PR migrates Brunch’s persistence layer from a flat message log to a turn-tree schema and updates the API/client hydration to follow the project’s active-path HEAD model.

Changes:

Introduces a 13-table SQLite schema (project, turn, option, decision, assumption, requirement, criterion + join tables) with integer autoincrement IDs.
Adds project.active_turn_id as a HEAD pointer and implements active-path retrieval via a recursive CTE.
Updates POST /api/chat to create a turn row, stream the assistant response, then persist it onto the turn and advance HEAD.
Updates GET /api/projects/current to return turns (active path) instead of messages.
Updates the React client to hydrate useChat from turns (mapping turn.answer→user, turn.question→assistant per the transitional inversion).
Reworks server/db tests to validate turn CRUD, option constraints, HEAD advancement, and active-path resolution (including fork scenarios).
Expands planning/spec docs (PLAN/SPEC) and adds new agent skills docs for cmux/cdp-based manual verification workflows.

Technical Notes: Active-path resolution is performed in SQLite using a recursive query from active_turn_id back through parent_turn_id, and the server now persists turn records rather than role-tagged message rows.

_{🤖 Was this summary useful? React with 👍 or 👎}

augmentcode

Review completed. 3 suggestions posted.

Comment augment review to trigger a new review at any time.

augmentcode · 2026-04-01T12:42:47Z

-			id TEXT PRIMARY KEY,
+			id INTEGER PRIMARY KEY AUTOINCREMENT,
 			name TEXT NOT NULL,
+			active_turn_id INTEGER,


project.active_turn_id isn’t declared as a foreign key, and SQLite won’t enforce any REFERENCES constraints unless PRAGMA foreign_keys=ON is set, so HEAD/parent pointers can silently become invalid and break getActivePath() semantics.

Severity: medium

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

augmentcode · 2026-04-01T12:42:48Z

+
+	return db.prepare(`
+		WITH RECURSIVE path AS (
+			SELECT * FROM turn WHERE id = ?


getActivePath()’s recursive CTE doesn’t constrain by project_id, so if a parent_turn_id chain ever crosses projects (which the current schema allows), this can traverse into another project’s turns and leak history.

Severity: medium

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

augmentcode · 2026-04-01T12:42:48Z

-			saveMessage(db, project.id, 'assistant', assistantText);
+			updateTurn(db, turn.id, { question: assistantText });
 		}
+		advanceHead(db, project.id, turn.id);


In the error path, assistantText stays empty but advanceHead() still runs, leaving HEAD pointing at a turn with question=''; subsequent formatHistory() will omit the assistant side of that exchange and degrade prompt context.

Severity: medium

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

lunelson · 2026-04-02T06:20:27Z

Merge activity

Apr 2, 6:20 AM UTC: A user started a stack merge that includes this pull request via Graphite.
Apr 2, 6:23 AM UTC: Graphite rebased this pull request as part of a merge.
Apr 2, 6:24 AM UTC: @lunelson merged this pull request with Graphite.

Replace the message table with the full schema.dbml entity model (13 tables: project, turn, option, decision, assumption, requirement, criterion + 6 join tables). Project gains active_turn_id as HEAD pointer. POST /api/chat creates turns with parent chain and advances HEAD. GET /api/projects/current returns turns on the active path via recursive CTE. Client hydrates useChat from turns. Integer autoincrement PKs replace TEXT UUIDs. Entity/join tables beyond turn/option are created but unpopulated until later slices. 39 tests passing (18 db + 9 app + 12 sse-adapter). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Add D15 (transitional turn-field inversion — answer=user, question=agent during pre-structured-interview phase) and D16 (integer autoincrement PKs replace TEXT UUIDs). Both surfaced during slice 3 build. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Fully superseded by docs/design/schema.dbml (entity model) and memory/SPEC.md (domain semantics, decisions, lexicon). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Grill session findings added to SPEC.md: - D17: two invalidation mechanisms (path exclusion + flag propagation) - D18: Drizzle ORM replaces raw DDL (single source of truth) - D19: layered architecture with DomainEvent streaming (core → adapters) - D20: CLI executable with subcommands - A18/A19: Drizzle migrations and AsyncIterable assumptions - Lexicon: branch, checkout, path exclusion, flag propagation, core, domain event; soft invalidation updated - Architecture section rewritten for db → core → adapters layering - Constraint updated: "no automatic deletion cascading" replaces "no belief invalidation cascading" Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Maps 3 places (project list, interview workspace, export preview), UI affordances per place, 20 API routes grouped by concern, data stores, and 5 key wiring flows (page load, interview turn, decision revisit, assumption falsification, phase transition). Identifies 5 candidate slices and 3 open uncertainties for ln-plan. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Type-safe routing with route loaders for data fetching on navigation. Three MVP routes per breadboard: project list, interview workspace, export preview. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Major re-plan incorporating grill + breadboard findings: - Phase 2 gains 3c (Drizzle + core extraction) and 3d (multi-project routing with TanStack Router) as architecture infrastructure - Phase 5 restructured: 11 (branch+checkout), 12 (entity lifecycle API with flag propagation), 13 (export) - Horizon updated: CLI interview, MCP adapter, graph visualizations - Dependencies and parallelism updated for new slice topology - Breadboard cross-references added to UI-touching slices Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

- Add .agents/skills/cli-cdp/ and .agents/skills/cli-cmux/ skill defs - Update AGENTS.md manual testing section: cli-cmux for terminal panes only (not browser), cli-cdp for Chrome DevTools interaction - Include design scratch docs Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

lunelson changed the title ~~chore: align Linear issues with PLAN.md slices~~ FE-544: Turn tree schema and API Apr 1, 2026

lunelson mentioned this pull request Apr 1, 2026

FE-534: Walking skeleton SDK to SSE to React #10

Merged

lunelson mentioned this pull request Apr 1, 2026

FE-535: SQLite foundation and project persistence #11

Merged

lunelson marked this pull request as ready for review April 1, 2026 12:39

augmentcode Bot reviewed Apr 1, 2026

View reviewed changes

This was referenced Apr 1, 2026

FE-552: Drizzle ORM and core extraction #14

Merged

FE-553: Multi-project routing #16

Merged

FE-541: Rich chat UI with tool calls and reasoning rendering #15

Merged

lunelson self-assigned this Apr 1, 2026

lunelson mentioned this pull request Apr 1, 2026

FE-554: Structured interview: scope phase #17

Merged

lunelson changed the base branch from ln/fe-535-sqlite-persistence to graphite-base/13 April 2, 2026 06:21

lunelson changed the base branch from graphite-base/13 to main April 2, 2026 06:22

lunelson and others added 9 commits April 2, 2026 06:23

chore: align Linear issues with PLAN.md slices

54a806a

chore: remove superseded REMODEL.md

d284c6e

Fully superseded by docs/design/schema.dbml (entity model) and memory/SPEC.md (domain semantics, decisions, lexicon). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

docs: add TanStack Router to tech stack (D9 updated)

5b7e23d

Type-safe routing with route loaders for data fetching on navigation. Three MVP routes per breadboard: project list, interview workspace, export preview. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

lunelson force-pushed the ln/fe-544-turn-tree-schema branch from ea42210 to 95d8dd0 Compare April 2, 2026 06:23

lunelson merged commit afe51ce into main Apr 2, 2026
1 check passed

This was referenced Apr 2, 2026

FE-555: Parts-based persistence + context builders #19

Merged

FE-556: Structured interview: client UI #20

Merged

FE-557: Spike: observer extraction fidelity #21

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

FE-544: Turn tree schema and API#13

FE-544: Turn tree schema and API#13
lunelson merged 9 commits into
mainfrom
ln/fe-544-turn-tree-schema

lunelson commented Apr 1, 2026

Uh oh!

linear Bot commented Apr 1, 2026

Uh oh!

lunelson commented Apr 1, 2026 •

edited

Loading

Uh oh!

augmentcode Bot commented Apr 1, 2026

Uh oh!

augmentcode Bot left a comment

Uh oh!

augmentcode Bot Apr 1, 2026

Uh oh!

augmentcode Bot Apr 1, 2026

Uh oh!

augmentcode Bot Apr 1, 2026

Uh oh!

lunelson commented Apr 2, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

lunelson commented Apr 1, 2026

Uh oh!

linear Bot commented Apr 1, 2026

Uh oh!

lunelson commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

augmentcode Bot commented Apr 1, 2026

Uh oh!

augmentcode Bot left a comment

Choose a reason for hiding this comment

Uh oh!

augmentcode Bot Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode Bot Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode Bot Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

lunelson commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

lunelson commented Apr 1, 2026 •

edited

Loading

lunelson commented Apr 2, 2026 •

edited

Loading