Skip to content

chore: fill derivable placeholders, drop false ARCHITECTURE, surface the rest #89

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
hyperpolymath wants to merge 1 commit into from
Closed

chore: fill derivable placeholders, drop false ARCHITECTURE, surface the rest #89

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
47 changes: 0 additions & 47 deletions ARCHITECTURE.md
View file
Open in desktop

This file was deleted.

4 changes: 2 additions & 2 deletions docs/status/CICD-REMEDIATION.adoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ in CI today because the checks that would catch them are either vacuous or unwir
| `Types.idr` / `Foreign.idr` / `Layout.idr` all fail `idris2 --check` with
`Expected a capitalised identifier, got: the` — the `module the-nash-equilibrium.ABI.*`
declaration has hyphens + a lowercase-leading segment (invalid namespace). *Never
compiled.* `Layout.idr` also has 3 unfilled proof holes. Root cause: `{{PROJECT_NAME}}`
compiled.* `Layout.idr` also has 3 unfilled proof holes. Root cause: `The Nash Equilibrium (Econosphere)`
substituted literally into identifier position.
Comment on lines 38 to 40

@gitar-bot gitar-bot Bot Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Quality: Placeholder fill makes "hyphenated" descriptions inaccurate

The token was replaced with the human-readable name The Nash Equilibrium (Econosphere), but the surrounding prose still describes it as a "hyphenated" namespace identifier that was "substituted literally into identifier position." The literal thing that broke compilation was the hyphenated the-nash-equilibrium form (as shown in the module line above), not the spaced/parenthesized display name. Consider using the actual offending identifier (e.g. the-nash-equilibrium) so the root-cause text stays factually consistent.

Was this helpful? React with 👍 / 👎

| PUSHABLE-NOW (rename namespace) + follow-up (fill proofs)

Expand Down Expand Up @@ -115,7 +115,7 @@ file/grep contractiles.
== Remediation buckets

*PUSHABLE-NOW* (non-workflow files; no scope needed)::
Rename the Idris2/Zig namespaces off the hyphenated `{{PROJECT_NAME}}`; wire the Zig
Rename the Idris2/Zig namespaces off the hyphenated `The Nash Equilibrium (Econosphere)`; wire the Zig
test step; fix `validate-state`/`validate-rsr`/`verify` paths to `6a2/`; fix
`crg-grade`/`crg-badge` `$$` bug; fill placeholder tokens (where owner values are
known); create `docs/AI_INSTALLATION_GUIDE.adoc`; de-`rsr-template-repo` the Justfile.
Expand Down
Loading