Repository navigation
WP-06: Orchestration + workspace UI - #10
Merged
Merged
Conversation
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
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.
WP-06: Orchestration + workspace UI
Built by the integrator (subagents were rate-limited). Base: main @6d96242 (all of WP-00..05, 07, 08 merged).
Acceptance criteria
db: U6 — the full package generates and artifacts appear (deterministic fallback)— 8/8 artifacts, honest fallback banner (AI keys are owner-blocked placeholders), well under 5 min (≈1 s in e2e)▶ Play from mm:ss(db: MK-7 — quote popover shows the verified context and play affordance); actual audio playback needs BLOB_READ_WRITE_TOKEN (owner blocker — honest "Audio isn't available in this preview." state)failed artifacts can retry without an instruction;fallback → deterministic package(U10) unit-proven, live-proven in WP-04db: U11 — edits autosave and survive a reloaddb: U12 — regenerate with an instruction and undo restores the snapshot; cap 20/artifact + 60 regen/h/user unit-testeddb: U13 — copy for Apple Podcasts…; unitU13: Apple/Spotify over 4000 chars need shorteningdb: mobile — chip row and bottom tabs,db: transcript panel searches,db: ready flag ticks an artifact (M8)Verification (all local, production build)
pnpm test: 436 passed / 2 skipped (skips = WP-04 CI-excluded live tests) — 22 new pipeline/export tests + segment-id fixpnpm e2e:db, embedded Postgres): 8/8 passed — real pipeline against a real databasepnpm lint0 errors ·pnpm typecheckclean ·pnpm build✓ · CI extended with the DB-mode e2e stepReal bugs found and fixed by the DB-mode e2e
segments.idis the global PK — per-episodeseg_000000ids would collide on the second episode ever; now namespacedseg_<episodeId[:8]>_<idx>Decisions logged (DECISIONS.md, 5 entries)
PGlite e2e database behind
pglite:URL scheme + seed script; pglite-scopedE2E_AUTH_USER_IDseam (can never fire against a real DB); PGlite globalThis singleton +serverExternalPackages(+ sequential fallback without neon.batch);cacheComponents/partialPrefetchingdisabled app-wide (PPR replay guard broke streamed per-user pages — all pages already use connection()+Suspense); segment-id namespacing.Left for later WPs