Skip to content

chore(census): refresh the row of system-et (the Agda track is now on its main branch) - #48

Merged
hyperpolymath merged 1 commit into
mainfrom
claude/refresh-system-et-row
Nov 10, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
claude/refresh-system-et-row

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Nov 10, 2026 •

Copy link
Copy Markdown
Owner

Summary

The catalogue row of hyperpolymath/system-et was from 2026-11-06: 21 Lean files and no Agda file. The wager experiment in Cubical Agda is on the main branch of that repository since 2026-11-10 (stages 0 to 4). This pull request refreshes that one row and renders the catalogue and the atlas again.

Changes

  • docs/catalogue/census/catalogue-rows.json: the row of hyperpolymath/system-et, from scripts/census/update-rows.sh hyperpolymath/system-et, at the commit 04a8b6e of that repository.

    Field Before After
    Own proof files 21 86
    Provers lean 21 agda 65, lean 21
    Declarations 164 612
    Holes, axioms 0, 0 0, 0
    Status file docs/status/PROOF-NEEDS.adoc docs/status/PROOF-STATUS.adoc
  • docs/catalogue/catalogue.adoc and the 4 files in docs/atlas/generated/: rendered again from the rows.

  • The render removes a second line for metadatastician/hvn-dehallucinator. The committed catalogue had 2 lines for that repository, and the rows have 1, so catalogue-check failed on main. The rows are the source. The removed line named PROOF-STATUS.adoc as the status file; the row names PROOF-NEEDS.adoc.

Not in this pull request

  • The row of metadatastician/hvn-dehallucinator is not changed. A trial refresh gave: 19 own files, 640 declarations, status file docs/status/PROOF-STATUS.adoc, and 1 hole. The hole is a false positive of scripts/census/proof-holes.sh: the word "admit" in the comment /-! The evidence contexts admit the same worlds -/ (verification/proofs/lean4/Haruspex.lean, line 882). I did not commit a row that says "1 hole" for it. The scanner needs a repair at its source before that row is refreshed.
  • Stages 5 to 7 of the wager experiment are not in the row. They are in pull request 89 of system-et, which is open. The census reads the main branch. After that pull request is merged, the row needs one more refresh (79 Agda modules).
  • No claim of the wager experiment is in the claims registry (docs/registry). A claim needs a receipt in the home repository (ADR 0004), and system-et has none.
  • No full census. Only the named repository was read.

RSR Quality Checklist

Required

  • Tests pass (just test or equivalent). just is not available in the session. Run in the session: the catalogue check (PASS), bun scripts/atlas/census-atlas.mjs check (PASS), bun test tests/atlas/census-atlas.test.mjs (10 pass), bash tests/census/build_rows_status_test.sh (PASS).
  • Code is formatted. Not applicable: generated data and views only.
  • Linter is clean (no new warnings or errors)
  • No banned language patterns
  • No unsafe blocks without // SAFETY: comments (none exist)
  • No banned functions
  • SPDX license headers present on all new/modified source files (no new file)
  • No secrets, credentials, or .env files included

As Applicable

  • Repo deed (status) / (maturity) updated. Not applicable.
  • Repo deed (ecosystem) updated. Not applicable.
  • Repo deed (meta) and/or docs/decisions/ updated. Not applicable.
  • docs/status/ROADMAP.adoc updated. Not done: no milestone changes.
  • Documentation updated for user-facing changes. Not applicable.
  • TOPOLOGY.md updated. Not applicable.
  • CHANGELOG or release notes updated. Not done: a data refresh.
  • New dependencies reviewed for license compatibility. Not applicable.
  • ABI/FFI changes validated. Not applicable.

Testing

  • The rows file differs from main in 1 row (a comparison of the 248 rows, one by one).
  • The rows file has the byte form that the census script writes (jq -cS).
  • The catalogue check fails on main (the second line for one repository) and passes on this branch.
  • Not run: the registry check (it reads receipts in other repositories), the full census.

Screenshots

Not applicable.


Generated by Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the catalogue to list hyperpolymath/system-et under Theory, with its current file and declaration counts and proof-status link.
    • Removed its previous Engineering listing.
    • Updated the hvn-dehallucinator proof-status link to point to PROOF-NEEDS.adoc.

… its main branch)

The row of hyperpolymath/system-et was from 2026-11-06: 21 Lean files
and no Agda file. The wager experiment in Cubical Agda is on the main
branch of that repository since 2026-11-10 (stages 0 to 4).

`scripts/census/update-rows.sh hyperpolymath/system-et` gives, at the
commit 04a8b6e of that repository: 86 own proof files (agda 65,
lean 21), 612 declarations, 0 holes, 0 axioms, status file
docs/status/PROOF-STATUS.adoc.

The catalogue and the 4 atlas views are rendered again from the rows.

The render also removes a second line for
metadatastician/hvn-dehallucinator. The committed catalogue had 2 lines
for that repository and the rows have 1, so `catalogue-check` failed on
main. The rows are the source; the catalogue says "do not edit by
hand". The removed line named PROOF-STATUS.adoc as the status file,
and the row names PROOF-NEEDS.adoc. The row of that repository is not
changed here: a trial refresh gave 1 hole that is a false positive of
the scanner (the word "admit" in a comment).

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

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Explain Change →

Walkthrough

The catalogue adds system-et to the Theory table with updated counts and a proof-status link. It removes the former Engineering entry and changes the hvn-dehallucinator proof-status link.

Changes: Catalogue census and links

Layer / File(s) Summary
Catalogue census and status links
docs/catalogue/catalogue.adoc
The Theory table lists system-et with 86 files and 612 declarations. Its former Engineering entry is removed. The hvn-dehallucinator link changes from PROOF-STATUS to PROOF-NEEDS.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Possibly related issues

Possibly related PRs

Suggested reviewers: claude

Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed The title clearly identifies the census refresh for system-et and notes that its Agda track is now on the main branch.
Description check Passed The description includes the required Summary, Changes, checklist, Testing, and Screenshots sections. It explains the data changes and scope, and reports which checks passed and which were not run.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Review coverage is incomplete: 1 file could not be fully reviewed. Findings from completed review steps are included; see review info for details.

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

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 8ee6354 and 6e9180e.

Files ignored due to path filters: 4

  • docs/atlas/generated/az.adoc is excluded by !**/generated/**
  • docs/atlas/generated/classes.adoc is excluded by !**/generated/**
  • docs/atlas/generated/index.json is excluded by !**/generated/**
  • docs/atlas/generated/organisations.adoc is excluded by !**/generated/**
Files selected for processing: 2

  • docs/catalogue/catalogue.adoc
  • docs/catalogue/census/catalogue-rows.json
Files not fully reviewed because their changes were unavailable: 1

  • docs/catalogue/census/catalogue-rows.json

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

@sonarqubecloud

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath marked this pull request as ready for review November 10, 2026 15:36
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-10T15:36:40.946288Z
We couldn’t complete this review. Comment @codex review to try again.
6e9180e Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@hyperpolymath
hyperpolymath merged commit db209bf into main Nov 10, 2026
44 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.

3 participants