Conversation
…state as if current; preserved with addendum (Audit 2026-04-30)
…te; preserved with addendum (Audit 2026-04-30)
…framing; preserved with addendum (Audit 2026-04-30)
…vation (Audit 2026-04-30, structural fix for staleness chain)
… code observation at klappy/oddkit@1a1f093 (Audit 2026-04-30)
…+ Open dedup bug, target oddkit 0.28.0 (Audit 2026-04-30)
…lemetry-gated removal target 0.30.0 (Audit 2026-04-30)
…ns, one principle graduated, CLI deprecation queued, Open dedup bug surfaced (Audit 2026-04-30)
Canon Quality —
|
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.
Summary
Audit 2026-04-30 cleanup. Three documents shipped in PR #157 (E0008.4 Phase 1, merged earlier today) described code state that PR #96 in
klappy/oddkithad retired on 2026-04-16. The audit traced a six-link citation chain — TruthKit-KB brief → P1.3.4 H-01 → user memory → planning conversation → PR #157 — in which no reader verified against current code. This PR is the corrective.The corrective has two layers:
code-claims-require-code-observation) that names the failure mode so it cannot recur silently.The audit also surfaced a previously undetected production bug:
discoverEncodingTypesin the worker dedupes by letter alone, silently dropping Open's quality criteria when both Observation and Open register letterO. Verified live: an[O-open P1]artifact returnsquality.maxScore: 4(Observation's max) instead of5(Open's max). Fix is queued in the revised Phase 2 handoff.Files (8)
Modified — supersession in place (3)
These three artifacts shipped in PR #157 with stale framing. Pattern: status flipped to
superseded,superseded_byandsupersession_reasonfrontmatter added, addendum block prepended explaining what was wrong, what's still true, what to read instead. Original bodies preserved verbatim below the addendum.docs/architecture/encode-architecture-problem-and-gaps.mdodd/handoffs/2026-04-30-encode-vodka-refactor-alternative-d.mdodd/ledger/2026-04-30-e0008-4-phase-1-encode-governance-migration-landed.mdNew — accurate replacements + structural fix (5)
canon/principles/code-claims-require-code-observation.md— NEW tier-1 principle. Canon governs intent; canon does not govern code behavior. Any claim of the form "the X currently does Y" must be backed by direct code observation against current HEAD, not citation of another document.docs/architecture/encode-current-state-2026-04-30.md— accurate current-state architecture doc, sourced by direct code observation atklappy/oddkit@1a1f093. Includesdescribes_state_atfrontmatter naming the exact commits read. Maps the predecessor brief's six gaps against current reality.odd/handoffs/2026-04-30-encode-vodka-refactor-alternative-d-revised.md— revised Phase 2 handoff scoping the actual five small worker items (envelope plural alignment, dedup-by-letter bug, fallback baseline gap, self-teaching surface, schema-driven check evaluator) plus the Open dedup bug surfaced by the audit. Target oddkit 0.28.0.odd/handoffs/2026-04-30-cli-encode-deprecation.md— CLI encode deprecation track per operator decision. Telemetry-gated removal target: 0.30.0+.odd/ledger/2026-04-30-audit-cleanup-encode-artifacts-landed.md— closeout ledger for this PR (Decisions, Observations, Learnings, Constraints, Handoffs, Encodes), per the C-03 constraint graduated in PR feat(canon): E0008.4 Phase 1 — Encode governance from truthkit-kb (field schemas, quality criteria, architecture brief, Phase 2 handoff) #157.Validation
This is canon-only — no code changes, no Bugbot validator agent gate required. Standard canon-doc gauntlet applies: frontmatter conventions, voice, claim auditability, cross-link integrity. Cross-references between the eight files were verified before push.
The Phase 2 PR (separate, in
klappy/oddkit) will require the full release-validation-gate perklappy://canon/constraints/release-validation-gate.What this PR does not do
klappy/oddkitsource. Phase 2 is the next PR; it will fix the Open dedup bug and the four envelope/self-teaching items per the revised handoff.Provenance
Authored by direct code observation at:
klappy/oddkit@1a1f093:workers/src/orchestrate.ts(governance discovery, parsers, scorer, action handler)klappy/oddkit@1a1f093:src/tasks/encode.js(legacy CLI parser, confirmed unchanged from pre-feat: add ?consumer= query param to getting started URLs and telemetry governance #96)oddkit_encodecall against production worker (Open dedup bug verification)klappy/klappy.dev@125cf8d1:odd/encoding-types/open.md(criteria count comparison)Every claim of code behavior in the new architecture doc corresponds to a specific function and line range observed at the cited commits.
Note
Low Risk
Documentation-only changes that update canon status/supersession metadata and add new guidance; no runtime code paths are modified, so risk is limited to confusion from broken links or misapplied process.
Overview
Canon cleanup after Audit 2026-04-30: flips three encode-related artifacts (architecture brief, Phase 2 handoff, Phase 1 ledger) to
status: supersededwith frontmatter pointers and addendum blocks explaining the staleness and where to look instead.Adds a new tier-1 principle,
canon/principles/code-claims-require-code-observation.md, requiring direct HEAD code reads (anddescribes_state_atprovenance) before making claims about current code behavior.Publishes replacement docs:
docs/architecture/encode-current-state-2026-04-30.md(current encode state + remaining items), a revised Phase 2 handoff scoping the real remaining work and surfacing the Open dedup bug, a separate handoff queuing Node CLI encode deprecation, and an audit closeout ledger recording the supersession decisions.Reviewed by Cursor Bugbot for commit 1dc7798. Bugbot is set up for automated code reviews on this repo. Configure here.