Skip to content

docs(research): reconcile seven current records with the proposed validator - #923

Merged
tt-a1i merged 2 commits into
mainfrom
codex/knowledge-records-369
Nov 13, 2026
Merged

tt-a1i merged 2 commits into
mainfrom
codex/knowledge-records-369

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Problem

Refs #198 and #369. The proposed research/Benchmark validator at aed4643e merges cleanly with main ffc9c624, but its integrated check reports 18 errors across seven research records. Four lack machine-readable metadata. Three lack some required sections or a PR reference. Checking the newly recognized records also exposed a missing research index link.

Value

Make those records usable with the existing validator and preserve their stated evidence and revision boundaries. Correct the separate architecture index omission for the published Workflow artifact commit protocol.

Approach

Transcribe existing source boundaries and dates, identify the originating PRs, and distinguish verified observations, inferences, recommendations and open questions. Keep original research text and dated follow-ups. Add the two missing index links. The existing #369 retains ownership of the validator and legacy manifest.

Validation

  • Before this change, the validator's 22 focused tests passed but its merged-tree repository check failed. With these records and ci(docs): enforce governed knowledge records #369 applied together, the repository check accepts all seven and the 22 tests pass.
  • On this documentation branch, bun run check passed. The full suite reported 2,475 Node passes, 8 skips and 1,647 UI passes across 109 files.
  • This check concerns metadata, structure and discoverability. The historical experiments were not rerun.

Impact

Documentation only. Existing evidence states remain scoped to their original sources. The companion can land before #369; that PR still needs independent review. This PR introduces no new architecture publishing gate.

@tt-a1i
tt-a1i marked this pull request as ready for review November 13, 2026 16:32
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 13, 2026
@tt-a1i
tt-a1i merged commit 8600a7c into main Nov 13, 2026
20 checks passed

@Trantken101907-byte Trantken101907-byte left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

preview

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants