Repository navigation
Bridge 1c: own-words descriptions - #4
Merged
Merged
Conversation
The description step (pipeline.Executor.Describe, not an engine stage) writes a spoiler-free description from the fact notes of the book's opening chapters only: the staged dir holds opening.md and nothing else. Validated (shape, length, voice, meta's description schema, the n-gram check against the transcripts left) and kept as sidecars/description.json with the run's work and sources, its spend on a describing stage run that reconcile and the supervisor skip. sidecars.generate runs it once at its finish and writes description.json with the others, never failing, parking or holding for it; sidecars.describe writes one for a book already generated here, parking for the agent's parks. The page labels a describe job and shows a generate result's description outcome.
…rk handling, safe provenance read)
…stays vouched for
This was referenced Oct 16, 2026
Bridge 1c: generated descriptions in .audiosilo, /meta and the console
KodeStar/audiosilo-server#148
Merged
Merged
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.
Bridge Phase 1c (BRIDGE-PLAN.md): own-words, spoiler-free descriptions.
What it does
pipeline.Describe, promptdescription.md): one more agent call written from the run's synthesized knowledge, never the transcript. The agent sees onlyopening.md, the fact notes of the opening chapters (logical positions 0..K: a quarter of the story chapters, two at least, never the last; a one-chapter book gets none). Positions are saved aschapter_positions.jsonso the bound holds after transcript retention.{"text"}, 200-1500 code points, no em dash or control character, at most two paragraphs, no second person, question endings or sales words (the book's own title exempt), meta'sdescription.schema.json, andextract.NGramagainst the source text.sidecars.generatewrites.audiosilo/description.json(meta-community'sdescriptionmember format, local only:contribute: issuenever files one). It runs at the start of the engine's contributing stage, while an ebook's text still exists, and the done-time write reuses it. It never fails, parks or holds the job; the result gainsdescriptionanddescription_error.sidecars.describe(new kind): description-only from the notes still on the Bridge (they outlive the transcript retention). Agent parks wait and retry; a book with no notes fails with a sentence saying to generate again.Scheduler.Hold: the description runs under the scheduler's reservation, so Retry, Delete, PurgeScratch and the retention sweep can't act mid-describe.describingstage runs carry the spend; the scheduler and supervisor skip them throughstate.IsStage.bridge.json. Model routedescribing(default sonnet).Pairs with KodeStar/audiosilo-server#148, KodeStar/audiosilo-frontend#111, KodeStar/audiosilo-docs#141 and KodeStar/audiosilo#62.
Quality passes
/simplify(4 reviewers): applied all but rebuilding the n-gram shingles once per call (meta'sextract.NGramonly takes paths)./code-review max --fix: 15 findings, all fixed with regression tests, among them two spoiler-bound holes (a misheard chapter announcement inflating K; K reaching a short book's last chapter), races between generate, describe and the purge, park flapping, and a provenance read that followed links. The last two (ebook n-gram, a kept description's provenance) were fixed by a follow-up commit.Test plan
go build ./... && go vet ./... && go test -race ./... && golangci-lint run(0 issues)cd web && npm run typecheck && npm run lint && npm run format && npm test(130 tests)Left open
internal/workersTestRouterWorkerConnectsWhileLocalPreparesis a pre-existing ~1-in-10 Phase 1b flake.