Skip to content

Delete toolchain.json - #24

Merged
kreneskyp merged 2 commits into
mainfrom
chore/delete-toolchain-json
Oct 1, 2026
Merged

kreneskyp merged 2 commits into
mainfrom
chore/delete-toolchain-json

Conversation

@kreneskyp

@kreneskyp kreneskyp commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Deletions:

  • spec_objects_security/schemas/toolchain.json
  • In scripts/generate-schemas.mjs: the toolchain record (compiler, emitter and semantic-core name+version), the normalization record, the overall sha256 over emitted files, the emitted file inventory, the dependencyVersion helper, the node:crypto import, the rewritten-files tracking in normalize, and the toolchain.json write, compare and exclusion paths. Schema emission and the --check comparison of emitted vs committed schemas are unchanged.
  • Tests: TC-020 (tests/test_schema_emission.py), the toolchain() helper and EXPECTED_MODELS, the toolchain.json assertions in TC-021, TC-027 and TC-030, and the toolchain.json skip clauses in tests/conftest.py, tests/test_security_rules.py and tests/test_role_schemas.py. TC-025 and TC-026 (wheel and npm tarball contents) are kept; they now compare against the committed schemas/*.json files.
  • Spec: FR-002 Outputs bullet for toolchain.json, the "record each rewrite" and "applied: false" behavior clauses, the check clause "or toolchain.json differs", toolchain.json mentions in the $id rationale, bump procedure, CON-5 and AC-8, and AC FR-002-AC-1; TC-020 row in spec/tests.md. AC-6 and AC-7 keep their packaging requirement with the "every file toolchain.json lists" wording replaced by "every emitted schema".

🤖 Generated with Claude Code

…escribed it

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kreneskyp
kreneskyp requested a review from a team as a code owner October 1, 2026 03:27
…oolchain.json

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kreneskyp
kreneskyp merged commit 81c8060 into main Oct 1, 2026
3 checks passed
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.

1 participant