Skip to content

Commit 57f3d7a

Browse files
chore(docs,ci): clear genuine Hypatia findings — stale src/abi paths, setup.sh, .envrc (#27)
Addresses the actionable subset of the Hypatia advisory findings surfaced by the scanner upgrade in #26. The remaining ~7 are scanner false-positives (WF017 two-step gate, SC-013 comment stripping, SC-014 codeql substring) for an upstream fix. Docs (structural_drift SD022 x6 — stale path after #20's FFI removal): - correct `src/abi/` -> the real RSR slot `src/interface/abi/` in AI-CONVENTIONS, RSR_OUTLINE, QUICKSTART-DEV, MAINTENANCE-CHECKLIST, INTENT.contractile - replace the unfilled-template docs/developer/ABI-FFI-README.adoc (it still carried a `{{project}}` placeholder and a template-instruction line and described a non-existent FFI surface) with an honest "no FFI surface" stub pointing at the RSR placeholders and PROOF-NEEDS.md setup.sh (code_safety shell_download_then_run / CWE-494): - replace `curl ... just.systems/install.sh | bash` with a download-then-run helper (no pipe-to-shell), with opt-in JUST_VERSION pin and JUST_INSTALL_SHA256 verify; native package managers (incl. nix/guix) are still tried first - update the header usage example to the download-then-run form .envrc (secret_detected "Generic API key" — false-positive, commented placeholder): - remove the residual `# export DATABASE_URL=...` / `# export API_KEY=...` lines (re-lands the fix intended in #25 that never reached main) This does not by itself turn the hypatia check green — that needs the upstream reusable to restore `--exit-zero`/`|| true` on the scan step so it follows its documented advisory ("warn but don't fail") policy. Claude-Session: https://claude.ai/code/session_01UCnxjXkN6uDj9YSANuGEgR Co-authored-by: Claude <noreply@anthropic.com>
1 parent 8aff3c5 commit 57f3d7a

8 files changed

Lines changed: 61 additions & 392 deletions

File tree

.envrc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@ fi
2020
# Project environment variables
2121
export PROJECT_NAME="gv-clade-index"
2222
export RSR_TIER="infrastructure"
23-
# export DATABASE_URL="..."
24-
# export API_KEY="..."
2523

2624
# Source .env if it exists (gitignored)
2725
dotenv_if_exists

.machine_readable/INTENT.contractile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
; === Sensitive Areas (if in doubt, ask) ===
5959
(ask-before-touching
6060
; *REMINDER: List areas where LLMs should check before modifying*
61-
; "src/abi/ — formal proofs, changes require re-verification"
61+
; "src/interface/abi/ — formal proofs, changes require re-verification"
6262
; "ffi/zig/ — C ABI boundary, changes affect all language bindings"
6363
; ".machine_readable/ — checkpoint files, format is specified"
6464
)

QUICKSTART-DEV.adoc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ cd worker && npm test
5454
----
5555
gv-clade-index/
5656
├── src/ # Source code
57-
├── src/abi/ # Idris2 ABI definitions (if applicable)
57+
├── src/interface/abi/ # Idris2 ABI definitions (if applicable)
5858
├── ffi/zig/ # Zig FFI bridge (if applicable)
5959
├── tests/ # Test suite
6060
├── docs/ # Documentation

docs/RSR_OUTLINE.adoc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ just validate-rsr
101101
|`AI.a2ml`
102102
|Claude-specific instructions
103103

104-
|`src/abi/`
104+
|`src/interface/abi/`
105105
|Idris2 ABI definitions (Types, Layout, Foreign)
106106

107107
|`ffi/zig/`
@@ -177,7 +177,7 @@ project/
177177
│ ├── TOPOLOGY-GUIDE.adoc
178178
│ ├── generated/
179179
│ └── man/
180-
├── src/abi/ # Idris2 ABI definitions
180+
├── src/interface/abi/ # Idris2 ABI definitions
181181
│ ├── Types.idr
182182
│ ├── Layout.idr
183183
│ └── Foreign.idr

0 commit comments

Comments
 (0)