Skip to content

Re-pin Spec 1bed024; read_records follows §4.6 (R_RECORDS_FILE_INVALID) - #242

Merged
guijacquemet merged 5 commits into
mainfrom
claude/spec-1bed024-records
Nov 9, 2026
Merged

guijacquemet merged 5 commits into
mainfrom
claude/spec-1bed024-records

Conversation

@guijacquemet

@guijacquemet guijacquemet commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Re-pin Spec 1bed024; read_records follows §4.6 (R_RECORDS_FILE_INVALID)

  • Re-pin the vendored Spec to MorphoBricks-Spec main 1bed024e278d104ba4b8f63ae76c020fb2e3470d (Spec PR 16), following PR 230: PIN.toml, SPEC_COMMIT, docs/conformance.md, vendored fixtures/ (adds records/), experiment-v1.md, refval.py, plus the new refrecords.py that refval.py imports (also pinned). Drafts saved under the previous pin are refused at resume (spec_commit_mismatch).
  • read_records refuses every file the Spec's reference validator refuses, under R_RECORDS_FILE_INVALID with the E_RECORDS_* reasons (new pure module records_file.py; the canonical check is the line bytes equal json.dumps(record, ensure_ascii=False, sort_keys=True) + "\n", which _append_records writes, now through the same encode_record).
  • tests/test_spec_records_goldens.py runs all 37 records goldens through read_records against the catalogue and the reference validator; it fails on main.
  • Gate: python tools/check.py passed, 4919 passed, 4 skipped (the 4 pre-existing environment skips).

🤖 Generated with Claude Code

https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
@guijacquemet
guijacquemet marked this pull request as ready for review November 9, 2026 07:14
@guijacquemet
guijacquemet merged commit 1d605b8 into main Nov 9, 2026
7 checks passed
@guijacquemet
guijacquemet deleted the claude/spec-1bed024-records branch November 9, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants