Repository navigation
Rendering slice 2: committed collector (#338 fix) + legacy deletion manifest - #425
Merged
Juliusolsson05 merged 1 commit intoJul 6, 2026
Conversation
…manifest observations/committed.ts — the anti-corruption boundary from raw transcript entries to ledger candidates. Visibility decided here with decision records for HIDDEN entries too (meta-entry, not-conversation, synthetic-user-filtered) so bundles can answer 'hidden or never ingested'. Includes the #338 fix as the centralized predicate: Claude non-meta user rows carrying <command-name>/<local-command-stdout>/ <environment_context> scaffolding (no permissionMode, text starts '<') are filtered with reason — claude-scoped so a codex/opencode user message legitimately starting with '<' is untouched. Ingest-time stable ids (never visible-index); producer-clock timestamps only; exact + NFKC-normalized text keys on assistant rows. docs/rendering/legacy-deletion-manifest.md — the strip-out contract: every legacy rendering file enumerated with fate (DELETE/GUT/ABSORB/ KEEP), LOC, deletion point, and a grep-based completeness check the Stage 3 cutover PR must pass. The old system is not patched during the rewrite (the new pipeline imports nothing from it) and does not survive it. 17/17 tests green (vitest unit); tsc node+web 0 errors gated on main checkout with the full integration diff applied. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Second slice into
integration/rendering-pipeline.observations/committed.ts— raw entries → candidates with visibility decided at the boundary; hidden entries keep decision records (meta-entry/not-conversation/synthetic-user-filtered). Carries the Claude local-command and synthetic transcript rows render as user prompt rows #338 fix: Claude command scaffolding no longer renders as user prompts, with the predicate centralized exactly as that issue requested. Ingest-time stable ids; producer-clock timestamps; conservative text keys.docs/rendering/legacy-deletion-manifest.md— the strip-out contract for the old system: per-file fate (DELETE/GUT/ABSORB/KEEP) + a grep completeness check the Stage 3 cutover PR must pass. Guarantees "start fresh" without breaking the app or the Stage-2 shadow oracle mid-rewrite.Next slice: semantic + ghost + optimistic collectors, then the first bundle-derived fixture directories.
🤖 Generated with Claude Code