Skip to content

fix(gates): gate on the DEED grammar, per the template and standards#837 - #88

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/deed-gate-grammar
Sep 21, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/deed-gate-grammar

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Why

The guard in dogfood-gate.yml counted .a2ml files only:

COUNT=$(find . -name '*.a2ml' -not -path './.git/*' | wc -l)

so a repo whose manifest is written in the current grammar is reported as having none, and the
summary/compliance row is skipped by the same predicate (if find . -name '*.a2ml' ... | grep -q .).
The validator itself has scanned both grammars for a while -- the caller's predicate is the bug.

Authority

  • rsr-template-repo/.github/workflows/deed-validate.yml (the mint source) is what this wording is
    copied from, byte for byte, including -type f-free find and the dual-accept stance.
  • standards#837 — "DEED conversion campaign", campaign step 4: *"flip ... RSR gates from .a2ml
    presence to .deed presence alongside". Gate predicates, not document contents.
  • deed-ecosystem/README.adoc: "The DEED format name and .deed extension are final, not A2ML."

Deliberately not done

  • No .a2ml file is translated or renamed. #837 makes classification into the four DEED forms and
    per-family mapping specs a prerequisite for mass translation; that is not this PR.
  • No gate that reads a retained file is touched (CLAIMS.a2ml, .machine_readable/Debtfile.a2ml,
    contractiles/INDEX.a2ml); rewriting those would fail gates that currently pass.
  • No job-id or shell-var renames (a2ml-validate, A2ML_COUNT): they are referenced from needs:
    and ${{ needs.*.outputs }}. The Checks-UI label is updated, as it has no references.

Estate-wide .deed gate convergence, 2026-09-21. Script: estate-audit/engine/deed_gate_migration.py.

Mirrors `rsr-template-repo/.github/workflows/deed-validate.yml` and standards#837 step 4. No
`.a2ml` document is translated and no retained-input gate is touched.
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 35 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: e3baacd8-3818-4684-9f8b-37d55b45ddc7

📥 Commits

Reviewing files that changed from the base of the PR and between 0d806b5 and 8756c3d.

📒 Files selected for processing (1)
  • .github/workflows/dogfood-gate.yml

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath merged commit 79eba85 into main Sep 21, 2026
24 of 28 checks passed
@hyperpolymath
hyperpolymath deleted the fix/deed-gate-grammar branch September 21, 2026 11:23
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