Repository navigation
chore: retire Deno, canonicalise JSON, align CLAUDE.md to D305-D307 - #230
Merged
Merged
Conversation
Deno (owner ruling 2026-08-26, standards#655): drop the deno branch in release.yml, port the e2e template to Bun, and replace "grandfathered" with "retired" in the agent rules files and the arrival-pack source. The runtime-policy Deno detector stays: it is what flags Deno. JSON (D306): canonicalise the four template-owned files with ijson-jcs. Files copied verbatim from standards, vendored, generated or JSONC are left alone and listed in the PR. CLAUDE.md hand-authored block: UUIDv8 (D305), KYAML/.kyaml (D307), JSON-POLICY (D306), descriptiles as deed clauses and contractiles as k9 verb directories (D312). The deed's :contractiles_required now names those directories instead of the retired Mustfile/Trustfile/Intentfile. Arrival pack regenerated; verify.sh reports it in sync. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D6JVLvmCeSFGcY6jSNesbL
Contributor
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (16)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
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
Brings the template in line with three owner rulings recorded on
hyperpolymath/standards#787:.kyaml, descriptiles as deed clauses, contractiles as k9 verb directories (D305, D307, D312).The template scaffolds every new repo, so text here that contradicts a ruling gets copied into every repo made from it.
📌 New pins
Head SHA:
0509dbfec1c8e60aff47b3de1b06406dda18fd57.No action, lockfile or container pin is added or changed.
oven-sh/setup-bun@v2.2.0appears only inside commented-out blocks ofe2e.yml.template. Its comment saysactions.lockresolves it when the template is instantiated. It is not a liveuses:.Changes
Deno
release.yml: removed thedeno.jsonbranch, and removeddeno.jsonfrom the "Expected one of" message.e2e.yml.template: the DENO/E2E block and the Playwright block now use Bun (bun install --frozen-lockfile,bun test,bunx playwright).runtime-policy.yml,workflow-linter.ymlandscripts/check-action-pinning.jsordering comments now read "Bun > pnpm > npm; Deno retired".::warningto an error is logged as a finding, not done here..clinerules,.cursorrules,.windsurfrules: "Deno grandfathered" and "TypeScript IS PERMITTED" are replaced by "Deno retired, migrate to Bun; TypeScript and ReScript migrate to AffineScript"..machine_readable/arrival-pack/arrival-pack.ncl: the language-policy source of CLAUDE.md, corrected the same way. CLAUDE.md was regenerated withgenerate.sh..github/pull_request_template.md: the "banned language" checkbox read "(no , no npm/bun, no Go/Python)": a word was missing and it banned Bun. It now names the banned set. The SPDX header is untouched.JSON (D306)
ijson-jcs fix(pinned0fdeea71), the four template-owned files only:.github/rulesets/Immutable-Tags.jsonwww/.well-known/aibdp.jsonwww/tests/controls/aibdp-bad-status.control.jsonwww/tests/controls/aibdp-enforce-430.control.jsonbase.json,branch-floor.json,require-linear-history.json,require-signed-commits.json: byte-identical copies ofstandards/config/rulesets, said to be sourced verbatim. Canonicalise upstream first.www/schemas/aibdp-schema-v0.2.json: vendored unmodified from consent-aware-web..github/labels.json,.github/label-classifier.json: generated ("do not edit") by generators in another repo. Logged as a finding.devcontainer.jsonfiles: tool-demanded JSONC whose comments carry SPDX headers.CLAUDE.md and the deed
#u5, and existing IDs are not rewritten..kyamlfor estate config,.ymlonly where a tool demands it. Workflows wait for the lock-coverage shim (D280, D307).standards/3-practice/JSON-POLICY.adoc. That file lands with feat(json): JSON-POLICY (D306) + report-only ijson-jcs gate metadatastician/standards#1189 and does not exist on standardsmainuntil that PR merges.rsr-template-repo_chora.deed:contractiles_requirednow lists the retired names' real directories:("Mustfile" "Trustfile" "Intentfile")→.machine_readable/contractiles/{must,trust,intend}/.(contractiles …)attestation clause yet. CLAUDE.md names it as the D312 target and does not claim it exists.Left as it is, on purpose:
.github/copilot-instructions.mdkeeps the narrow TypeScript carve-out thatstandards/3-practice/LANGUAGE-POLICY.adoc§1.2 still states. Reconciling it with the Bun-only ruling is pending in standards.RSR Quality Checklist
Required
www/tests/check-aibdp.shgives rc=0, and its output is byte-identical before and after the JSON change; both planted controls are still rejected.ijson-jcs check(rc=0 on all four), andjq -Sshows the same data before and after.actionlinton the three edited workflows reports two shellcheck warnings (release.ymlSC2046,workflow-linter.ymlSC2034). Both are present onmainunchanged; this PR adds none.unsafeblocks: not applicable, no Rust touched..envfiles.As Applicable
(status)/(maturity): no project state change. Only:contractiles_requiredchanged, as above.(ecosystem): no integration change.(meta)/docs/decisions/: no new decision. This applies rulings already recorded on standards#787.Testing
bash .machine_readable/arrival-pack/verify.sh→OK: CLAUDE.md arrival pack is in sync with the deed.Run before the change (baseline) and after regeneration. Run with nickel 1.16.0.bash scripts/validate-template.sh→Validation PASSED, 0 errors. The 7 warnings are placeholders and SPDX headers on files this PR does not touch.ijson-jcs check→ rc=0 on the four canonicalised files.standards/.githooks/docstring-scan.sh --worktree→ no functions touched.denoinrelease.ymlreturns nothing.deed-validateruns a remote action (deed-ecosystem/validate-action). CI is the check for the deed edit, which reuses the multi-line string-list form already used by:machine_readable_required.Screenshots
Not applicable.
🤖 Generated with Claude Code
https://claude.ai/code/session_01D6JVLvmCeSFGcY6jSNesbL
Summary by CodeRabbit