Load prompts page sections closed with native toggles

[JacobStephens2]

Summary

Load the Prompts and skills page with its sections and job sheets closed. Native controls let readers open and close each independently; fragment links reveal their destination.

 Prompts and skills
-  Command line, prompts, skills and licence displayed in full
+  Four closed sections with clickable, keyboard-accessible headings
+    Prompt and skill sheets also start closed
+  Fragment links open enclosing sections and the linked sheet

Closes #666

Evidence

• Before: cargo test prompts_page_sections_and_job_sheets_start_closed failed: 0 native section disclosures, expected 4. The browser regression also failed on initial state, then reproduced in-page fragment navigation leaving its destination closed.
  After: all 11 prompts_page tests pass. The retained browser regression passes at 1440px and 390px for default state, pointer/keyboard toggles, independent sheets, direct/in-page/repeated fragment links, and layout; native toggles also pass with JavaScript disabled.

• Validation: cargo check --all-targets, cargo fmt --check, cargo clippy --all-targets -- -D warnings, and cargo test (1,736 passed, 2 ignored, 0 failed).

• The final diff against main matches the independently reviewed diff byte-for-byte; all implementation and test sources match the full-suite-tested sources. Typechecking, focused Rust tests, formatting, and browser checks also passed.

• Browser regression: install puppeteer-core outside the repo, then run NODE_PATH=<install>/node_modules CHROME_BIN=<chromium executable> node site/tests/prompts.cjs.

• Seams under test: the served Prompts page's disclosure and fragment-link interactions; generated HTML contract and checked-in fragment synchronization.

• Independent review against main: Standards 0 findings; Spec 0 findings. Both reviewers read all five changed files. Changed files left unread: none.

• Independent final validation: cargo nextest run passed all 1,736 tests (2 skipped); the final main...HEAD diff has 5 changed files, all read by both reviewers.

Merge Danger

Door: two-way

Revert the PR to restore the expanded page. No data migration.

Blast Radius: site

Disclosure styling and fragment navigation affect the Prompts and skills page. The Rust changes generate its HTML.

Unaddressed findings

Standards

Requested review of their commits (37c78b4645367ba38d02eeb26eed4c0c059123fc...a5be601f252780466fdef34d089658d3d01bd08c): the diff is empty; a5be601 only records a previous review and changes no files. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Standards reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-L4x2eI/standards.md.

Requested review of their commits (08e26e01d9199ebd0fb4e3ac1bed6afea5aaacce...37c78b4645367ba38d02eeb26eed4c0c059123fc): the diff is empty; 37c78b4 only records a previous review and changes no files. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Standards reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-w3blOF/standards.md.

Requested review of their commits (0b8cbc3c8174e16a75f46759687f53abb51f55b1...08e26e01d9199ebd0fb4e3ac1bed6afea5aaacce): the diff is empty; 08e26e0 only records a previous review and changes no files. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Standards reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-1P2kb8/standards.md.

Requested review of their commits (1303a81a685fde875139d83db92386c326158f13...0b8cbc3c8174e16a75f46759687f53abb51f55b1): the diff is empty; 0b8cbc3 only records a previous review. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Standards reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-uHdLgF/standards.md.

Latest requested review of their commits (8f5ac1e095db3a36d8265d7156e9b0b3caf4e374...1303a81a685fde875139d83db92386c326158f13): the diff is empty; the only later commit records the preceding review and changes no files. The review skill stopped at its non-empty-diff preflight, so no Standards reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none.

None. Review of their commits (764adf4af166738cc63af27e9855762dceb0b729...8f5ac1e095db3a36d8265d7156e9b0b3caf4e374) found no Standards findings. Both changed files (Cargo.toml, Cargo.lock) were read; changed files left unread: none.

Spec

Requested review of their commits (37c78b4645367ba38d02eeb26eed4c0c059123fc...a5be601f252780466fdef34d089658d3d01bd08c): the diff is empty; a5be601 only records a previous review and changes no files. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Spec reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-L4x2eI/spec.md.

Requested review of their commits (08e26e01d9199ebd0fb4e3ac1bed6afea5aaacce...37c78b4645367ba38d02eeb26eed4c0c059123fc): the diff is empty; 37c78b4 only records a previous review and changes no files. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Spec reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-w3blOF/spec.md.

Requested review of their commits (0b8cbc3c8174e16a75f46759687f53abb51f55b1...08e26e01d9199ebd0fb4e3ac1bed6afea5aaacce): the diff is empty; 08e26e0 only records a previous review and changes no files. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Spec reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-1P2kb8/spec.md.

Requested review of their commits (1303a81a685fde875139d83db92386c326158f13...0b8cbc3c8174e16a75f46759687f53abb51f55b1): the diff is empty; 0b8cbc3 only records a previous review. Both endpoints have tree ca00e49cd9dd344d80806f1a591a1521753604bf. The skill stopped at its non-empty-diff preflight, so no Spec reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none. Preflight record: .thirdshift-review-uHdLgF/spec.md.

Latest requested review of their commits (8f5ac1e095db3a36d8265d7156e9b0b3caf4e374...1303a81a685fde875139d83db92386c326158f13): the diff is empty; the only later commit records the preceding review and changes no files. The review skill stopped at its non-empty-diff preflight, so no Spec reviewer ran and no findings were produced or skipped. Changed files left unread in this range: none.

None. Review of their commits over the same fixed range found no Spec findings. Both changed files (Cargo.toml, Cargo.lock) were read; changed files left unread: none. The incoming changes only align the package version at 0.11.2 with upstream release PR #676; the prompts-page implementation is unchanged.

Security

None. Guidance review of d7c6df157ef762cbb89652ff30802cd046a0d18c...764adf4af166738cc63af27e9855762dceb0b729 read all five changed files and supporting rendering, escaping, skill embedding, and static-site deployment code. Relevant attack-class, client-side, and generated-input guidance was applied; no repository SECURITY.md or threat-model document was present. No introduced security finding was identified.

Bounded network-isolated checks of the production JavaScript passed on HEAD and the untouched pinned merge base: 12 harmless fragment inputs, literal ID lookup, malformed encoding, and initial/changed/repeated fragment handling. These checks used a minimal DOM harness; they do not claim browser end-to-end coverage. No security fix was needed.

Built with codex · gpt-6.1-sol · high

Security review outcome

No unaddressed introduced findings.