[AbirAbbas]

Summary

The workstream store from the Workstreams plan, WP-1, built on the v0 wire contract that #2263 registered.

• Storage (schema 3 of the collections database). The first write rebuilds a schema 1 or 2 file by copying its tables (SQLite cannot change a CHECK in place). Every collection and membership keeps its sequence number, id, name and times. New tables: workstreams, workstream_jobs (its unique index keeps a Factory job in one workstream), workstream_pages, workstream_page_versions, workstream_records, workstream_goaheads, workstream_plans.
• factory_job ref kind in the SQLite CHECK, workspace.Ref.Validate and appwire.CheckRef, with factory on the ref. The CLI takes --factory.
• Domain package internal/workstream/: stage, numbers, plan, pages with versions, records and go-aheads. page.get's markdown comes from internal/workstream/pagedoc (app: workstream pages render from editor JSON to markdown (pagedoc) #2277), the one page renderer in the engine.
• Handlers: all 25 methods serve from the store, with bounds checks at the wire's one door. ServesWorkstreams() is true, so hello announces workstreams. Every write sends workstream.changed.
• Create makes the collection, the row, an Overview page and one bare agent with no first turn, so no model runs. A repeated client_id returns the first workstream.
• Numbers: stage, merged, needs_you and spend come from bounded Factory queue reads plus the agent's recorded spend. Unknown values are left out, never sent as 0.
• Collections: workstream: true on listings; deleting a workstream is refused (mark a workstream done instead); a job another workstream holds is refused with in_other_workstream.

Validation contract

item	test
(a) migration keeps collections and memberships, adds tables and kind	internal/workspace/schema3_test.go
(b) job in another workstream → in_other_workstream, nothing changes	internal/workspace/workstreams_test.go, cmd/codeaf/appworkstreams_test.go
(c) create makes one workstream, collection and agent in Shaping; repeat client_id returns the first	internal/workstream/store_test.go, cmd/codeaf/appworkstreams_test.go
(d) Shaping → Building → Done only by mark_done; reopen → Building	internal/workstream/store_test.go, cmd/codeaf/appworkstreams_test.go
(e) mark_done ends every go-ahead	internal/workstream/records_goaheads_test.go, cmd/codeaf/appworkstreams_test.go
(f) stale page.save → version_conflict, nothing changes; history/restore add a version	internal/workstream/pages_test.go, cmd/codeaf/appworkstreams_test.go
(g) records propose → keep/drop/supersede; superseded cannot change	internal/workstream/records_goaheads_test.go, cmd/codeaf/appworkstreams_test.go
(h) go-ahead without scope, cap or end date refused	internal/workstream/records_goaheads_test.go, internal/appwire/workstreams_check_test.go, cmd/codeaf/appworkstreams_test.go
(i) every write emits workstream.changed	cmd/codeaf/appworkstreams_test.go
(j) unknown numbers omitted, measured zero kept	internal/workstream/store_test.go, cmd/codeaf/appworkstreams_test.go

Stubbed or left

• Go-ahead spent_usd stays absent, and go-aheads and wave auto-start are stored but not enforced. Both wait on the Factory side (plan §3.3 asks 4 and 3).
• The workstream agent gets the plain prompt. WP-10 adds its tools and system prompt at the hook in workstreamAgentAsk.

CI

• PR gate: the only reds are the known baselines: the laws ratchet in internal/session, TestAppChatRowsR2SavedTurnTime, TestAppChatStartupR3QuietRelease and TestRecipeDoorWritesTheWorktreeWhenOnlyTheCheckoutIsHeld. TestAppIdleReleaseI1I2* flaked once and passed on rerun.
• Full check on the branch compared with the integration branch's full runs: no new failure repeats. The one-offs passed on rerun or on another run of the same commit, and none is in code this change touches: TestAProposedTaskCarriesTheMessageItWasMadeFromAndKeepsItAcrossAReopen, TestARequestOlderThanTheRunningTurnWaitsForIt, TestSilentRouterCanRecoverOnTheNextRoutingCompletion, and two temp-dir cleanup races in Factory tests (TestAppFactoryJobWithoutIssueHasAnHonestReference, TestFactoryStopAnyVersion).

Contract changes

None. The factory ref field, workstream: true and in_other_workstream are the additive collection behaviour the v0 contract already describes.

🤖 Generated with Claude Code