Skip to content

chore: delete the five unmapped root entries - #1097

Closed
hyperpolymath wants to merge 1 commit into
mainfrom
chore/1095-delete-unmapped-roots
Closed

hyperpolymath wants to merge 1 commit into
mainfrom
chore/1095-delete-unmapped-roots

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Closes #1095.

Owner ruling D241 (2026-09-30, selector): delete all five rather than map or relocate them.

entry files
ULTRAPLAN-2026-09-24.adoc 1
ULTRAPLAN-2026-09-29.adoc 1
arena-session-787/ 4
patches/ 1
ziz-drop/ 16

Git history keeps every byte, so any of them can be restored from 74d2f66.

Evidence

  • Gate D (scripts/check-standards-map.sh) on main 74d2f66: rc=1, GATE D FAILED — 5 violation(s).
  • The same gate on this tree: rc=0.
  • Positive control: an unmapped probe file at the root makes it fail again, GATE D FAILED — 1 violation(s).

Left as is

scripts/triage-2026-09-24-apply.sh and scripts/triage-2026-09-29-apply.sh cite the two ULTRAPLANs in comments and in issue-comment text. Those scripts have already run; the citations remain valid through history.

This is one of the two pre-existing main reds that #1088 is held on (ruling D240). The other is #1094.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QYY8Gp4v4x2J7iSNn1vZ57

Owner ruling D241 (2026-09-30): delete all five rather than map or relocate.
Removed from the repository root (git history keeps every byte):
ULTRAPLAN-2026-09-24.adoc, ULTRAPLAN-2026-09-29.adoc, arena-session-787/,
patches/, ziz-drop/.

Gate D (scripts/check-standards-map.sh): rc=1 with 5 violations on
main 74d2f66, rc=0 on this tree. Positive control: an unmapped probe file
at the root makes it fail again with 1 violation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QYY8Gp4v4x2J7iSNn1vZ57
@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 6e2236ca-4c24-4289-8fc7-e700f5ef1e15

📥 Commits

Reviewing files that changed from the base of the PR and between 74d2f66 and a805686.

📒 Files selected for processing (23)
  • ULTRAPLAN-2026-09-24.adoc
  • ULTRAPLAN-2026-09-29.adoc
  • arena-session-787/787-execution-kit.md
  • arena-session-787/RULESET-B-PROTOCOL.md
  • arena-session-787/owner-rulings-comment-2026-09-26.md
  • arena-session-787/ruleset-23787415-backup-2026-09-27.json
  • patches/airborne-submarine-squadron.patch
  • ziz-drop/DESIGN.adoc
  • ziz-drop/Justfile.ziz-fragment
  • ziz-drop/README.adoc
  • ziz-drop/TRIAGE.adoc
  • ziz-drop/bootstrap/ziz0.chpl
  • ziz-drop/docs/JEG.adoc
  • ziz-drop/examples/fact.ziz
  • ziz-drop/examples/hello.ziz
  • ziz-drop/examples/layout.ziz
  • ziz-drop/examples/quote-eval.ziz
  • ziz-drop/examples/reflexive.ziz
  • ziz-drop/grammar/grammar.js
  • ziz-drop/grammar/package.json
  • ziz-drop/grammar/src/scanner.c
  • ziz-drop/grammar/test/corpus/basics.txt
  • ziz-drop/grammar/tree-sitter.json
💤 Files with no reviewable changes (23)
  • ziz-drop/examples/quote-eval.ziz
  • ziz-drop/grammar/test/corpus/basics.txt
  • ziz-drop/README.adoc
  • ziz-drop/examples/fact.ziz
  • ziz-drop/grammar/tree-sitter.json
  • ziz-drop/TRIAGE.adoc
  • arena-session-787/787-execution-kit.md
  • arena-session-787/ruleset-23787415-backup-2026-09-27.json
  • ziz-drop/grammar/grammar.js
  • ziz-drop/examples/layout.ziz
  • ziz-drop/DESIGN.adoc
  • arena-session-787/owner-rulings-comment-2026-09-26.md
  • ziz-drop/grammar/package.json
  • ULTRAPLAN-2026-09-24.adoc
  • ziz-drop/docs/JEG.adoc
  • ziz-drop/Justfile.ziz-fragment
  • ziz-drop/grammar/src/scanner.c
  • patches/airborne-submarine-squadron.patch
  • ULTRAPLAN-2026-09-29.adoc
  • ziz-drop/bootstrap/ziz0.chpl
  • ziz-drop/examples/hello.ziz
  • ziz-drop/examples/reflexive.ziz
  • arena-session-787/RULESET-B-PROTOCOL.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (15)
  • GitHub Check: governance / Validate Hypatia Baseline
  • GitHub Check: scan / shell-secrets
  • GitHub Check: scan / gitleaks
  • GitHub Check: governance / Workflow security linter
  • GitHub Check: analyze-actions / analyze
  • GitHub Check: governance / Language / package anti-pattern policy
  • GitHub Check: governance / UUID v7 conformance
  • GitHub Check: analyze-js / analyze
  • GitHub Check: scorecard / Run Scorecard PR
  • GitHub Check: scan / Hypatia Neurosymbolic Analysis
  • GitHub Check: K9-SVC contractile validation
  • GitHub Check: Detect proof changes
  • GitHub Check: Repo self-tests
  • GitHub Check: Registry + topology in sync
  • GitHub Check: semgrep-cloud-platform/scan

📝 Summary

Summary by CodeRabbit

  • Removals
    • Removed the Žiz language prototype, including its interpreter, examples, grammar and supporting documentation.
    • Removed planning, triage and execution records, along with a ruleset backup and a proposed patch.
    • No changes to live repository rules or remote issue records are indicated.

Walkthrough

The pull request deletes two issue-triage documents, four session records, a stored patch, and the Žiz design, interpreter, examples, and Tree-sitter grammar materials.

Changes

Root entry removals

Layer / File(s) Summary
Issue triage records
ULTRAPLAN-2026-09-24.adoc, ULTRAPLAN-2026-09-29.adoc
Both issue inventories and their recorded classifications, plans, and execution notes were deleted.
Session records and ruleset files
arena-session-787/*
The execution kit, ruleset protocol, owner-rulings record, and ruleset backup were deleted.
Patch payload
patches/airborne-submarine-squadron.patch
The stored patch containing workflow, linter, game, and compatibility-test changes was deleted.
Žiz design, interpreter, and examples
ziz-drop/DESIGN.adoc, ziz-drop/README.adoc, ziz-drop/TRIAGE.adoc, ziz-drop/bootstrap/*, ziz-drop/docs/*, ziz-drop/examples/*, ziz-drop/Justfile.ziz-fragment
The Žiz design and project documents, bootstrap interpreter, build recipes, JEG documentation, and five examples were deleted.
Žiz grammar and parser support
ziz-drop/grammar/*
The Tree-sitter grammar, external scanner, manifests, and parse corpus were deleted.

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~20 minutes

Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to a8056

The deletions implement the chosen repository cleanup without breaking the triage scripts. No actionable merge-blocking risk was identified; the change is ready for normal checks.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning Issue [#1095] requires the five unmapped root entries to be mapped, moved, or removed. The PR removes ULTRAPLAN-2026-09-24.adoc, ULTRAPLAN-2026-09-29.adoc, arena-session-787/, patches/, and `z… Add an explicit decision for the Gate D trigger policy. If the decision requires a workflow change, add and test that workflow. Re-run Gate D and the positive control.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: deletion of the five unmapped root entries.
Description check ✅ Passed The description directly explains the deletion decision, affected entries, Gate D results, issue linkage, and restoration point in Git history.
Out of Scope Changes check ✅ Passed The change removes only the five root entries named by [#1095], including their contained files. The deleted ULTRAPLAN records, execution-kit records, patch payload, and Žiz payload are therefore with…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check

Explanation

Issue [#1095] requires the five unmapped root entries to be mapped, moved, or removed. The PR removes ULTRAPLAN-2026-09-24.adoc, ULTRAPLAN-2026-09-29.adoc, arena-session-787/, patches/, and ziz-drop/. The reported Gate D result is 0 on the PR tree. The reported positive control fails as required. The PR does not record the required decision about running Gate D on every PR or on pushes to main.

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

A rabbit checks the burrow’s rows,
And notes which paths no longer show.
The old plans rest; the grammar’s gone,
The Žiz kit takes a quiet yawn.
Then paws the page and hops along.

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

@hyperpolymath

Copy link
Copy Markdown
Owner Author

Superseded by #1088. Its commit a805686 was cherry-picked and signed there as a5061ff. Under the fully-green rule this PR could never pass on its own: its three reds (registry sync, Repo self-tests, Hypatia Baseline) were main's own, and only #1088 cures them. #1088 in turn needed this deletion, so the two cures had to land together.

The branch is kept; refs/pull/1097/head is permanent either way.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QYY8Gp4v4x2J7iSNn1vZ57

hyperpolymath added a commit that referenced this pull request Sep 30, 2026
…uuid-v7, map roots, canon 2.1.2, docstring calibration (#1088)

Restores `standards` main to green. The reds on main and on this PR hold
each other in a cycle, so every cure is in this one PR: under the
fully-green rule, no smaller PR can pass CI alone.

## What each commit fixes

1. **Lock-gate staging pin bump.** The `ref:` under "Checkout standards
for the lock gate" in `governance-reusable.yml` moves to `5f82b635`.
`scripts/check-lock-gate-pin-freshness.sh` returns rc=1 on main and rc=0
on this branch. Each tree's own copy of the script was run; running one
tree's copy against another tree reads the wrong workflow and passes
vacuously.
2. **Registry `source_hash` regeneration** (#1092). #1072 and #1075
changed files under the RSR spec home without regenerating
`.machine_readable/REGISTRY.a2ml`, so `build-registry.sh --check` fails.
That failure reds Self-tests, and the pin guard step never runs because
it comes after the failing suite. The change is one regenerated hash
line. Under D231 it is a regeneration only; migrating off the generated
file is a later piece of work.
3. **`uuid-v7.yml` hardening.** It adds `timeout-minutes: 10`, a
`concurrency` group, and a `push` trigger bounded to `main`. These are
the three unfiltered Hypatia Baseline findings on main:
missing_timeout_minutes, d_burn_double_trigger, and WH006.
4. **Delete the five unmapped root entries** (D241, cherry-picked and
signed from #1097). This cures the map-integrity red that woke on this
PR.
5. **Canon PATCH 2.1.1 → 2.1.2.** #1041 added a KNOWN-TENSIONS row under
`0-canon/constitution/` without the same-commit bump and sha256 rewrite
that `canon.lock` requires. Gate A is path-filtered, so main never ran
it and the drift stayed invisible until this PR woke the gate. This
commit bumps the version, sets released to 2026-09-30 and the tag to
`canon-v2.1.2`, rewrites the constitution hash to `be48496f…`, and adds
a header note. The spine's dogfood red was a stale hypatia compile
error, fixed upstream at 9d2e6de3. Re-running rsr-template-repo run
36475038466 turned it green.

6. **Fetch the docstring calibration commit by SHA** (#1099). #1073's
`docstring-scan-test.sh` calibrates against `1cc72cdc80c9`, PR #1034's
pre-squash head, which no clone of main can contain. Self Test on main
(run 36741131926) failed on this as well as the stale registry. I read
only the first failure, so my earlier claim that this PR cured every
main red was wrong. The test now fetches the commit by full SHA on a
miss, without `--depth` (which would make a complete clone shallow), and
still fails closed if the fetch fails. The branch was rebased onto main
74d2f66 (signed) so the test file is present.

Closes #1092
Closes #1094
Closes #1095
Closes #1099

## Local verification on 6192c92 (rebased on main 74d2f66)

| check | result |
|---|---|
| `bash scripts/run-shell-test-suite.sh` | 63 of 63 test files passed |
| `check-lock-gate-pin-freshness.sh origin/main` | PASS |
| `build-registry.sh --check` | clean |
| `check-canon-lockstep.sh --spine rsr-template-repo@8256a6e --base
origin/main` | GATE A PASSED (9 passed, 1 skipped: hypatia oracle not
local) |
| `check-standards-map.sh` | GATE D PASSED, 124 entries |
| **positive control:** one appended byte in `KNOWN-TENSIONS.adoc`,
`canon.lock` untouched | GATE A FAILED, naming `constitution` |
| docstring calibration: watched failing locally (object absent, 25
passed / 1 failed), then green after the fix | files=2, functions=13,
documented=0, skipped=3, coverage=0.00%; clone not shallow afterwards |

#1097 is superseded by commit 4. This PR lands under the fully-green
rule only when every check is green.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_01QYY8Gp4v4x2J7iSNn1vZ57

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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.

Standards map integrity red on main: 5 unmapped top-level entries (2 ULTRAPLANs, arena-session-787, patches, ziz-drop)

1 participant