Skip to content

doc-audit: fix stale OpenSpec counts, dossier headers, Haven forge gate#760

Merged
stphung merged 2 commits into
mainfrom
claude/sharp-darwin-3yuvyy
Jul 15, 2026
Merged

doc-audit: fix stale OpenSpec counts, dossier headers, Haven forge gate#760
stphung merged 2 commits into
mainfrom
claude/sharp-darwin-3yuvyy

Conversation

@stphung

@stphung stphung commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Summary

Full 6-agent /doc-audit pass (root/architecture, core engine, content systems, progression systems, infrastructure, README) cross-referenced every CLAUDE.md, README.md, and docs/dossiers/*.md claim against current source. The codebase docs were largely clean; this fixes the handful of real discrepancies found:

  • CLAUDE.md / openspec/README.md: the OpenSpec baseline count had drifted (215→216 requirements, 517→525 scenarios). The capability index table in openspec/README.md was also missing a row for the challenges capability entirely and undercounted vessel-act2 by one requirement (19→20) — these two errors happened to offset and mask the drift in the summary line.
  • docs/dossiers/act2-pilgrimage.md: "Last refreshed" header was dated 2026-07-05, but the dossier's own Refresh History log (and a same-day docs: correct the "Act 1 idles beneath Act 2" premise; record the launch runway #757 correction to the Act 1/Act 2 interrelations diagram) show substantive edits through 2026-07-15. Bumped to the latest commit that touched the file.
  • docs/dossiers/act1-ascent.md: "Last refreshed" header was similarly one commit behind (doc-audit: fix stale docs, add missing mercenary promotion section #706's stale-constant fix on 2026-07-08).
  • src/haven/CLAUDE.md: the Storm Forge section described Stormbreaker forging as gated only on the Storm Leviathan achievement, omitting the Prestige Rank 25+ requirement that can_forge_stormbreaker() actually enforces alongside it.

Two additional findings were flagged but intentionally not auto-fixed (left for human judgment):

  • README.md claims an MIT license (badge + text) but no LICENSE file exists in the repo — adding one is a legal/business decision, not a doc correction.
  • src/challenges/CLAUDE.md deviates from the standard module-doc template (tutorial-style instead of Files/Key Types/How It Works sections) — content is accurate, just structurally different; a LOW-severity, cosmetic-only note.

Everything else audited — root CLAUDE.md's Key Constants, all module CLAUDE.md files under core/combat/character/zones, dungeon/fishing/items/challenges, haven/enhancement/ascension/stormglass/deep/power_cores/god_items/loom/vessel, ui/input/utils/main_helpers/achievements/history, and README.md's player-facing claims — checked out accurate against source.

Test plan

  • cargo fmt --check passes
  • No src/ files changed — docs-only diff, no risk to tests/clippy/coverage
  • cargo audit step of make check could not run in this environment (pre-existing toolchain mismatch: rustc 1.94.1 vs. a cargo-audit transitive dependency requiring 1.96.0), unrelated to this change

Generated by Claude Code

- CLAUDE.md / openspec/README.md: requirement/scenario counts had drifted
  (215→216 reqs, 517→525 scenarios); the capability table was missing a
  `challenges` row and undercounted `vessel-act2` by one requirement
- docs/dossiers/act2-pilgrimage.md, act1-ascent.md: "Last refreshed"
  headers were stale relative to each dossier's own latest substantive edit
- src/haven/CLAUDE.md: Storm Forge section omitted the Prestige Rank 25+
  gate that can_forge_stormbreaker() actually requires alongside the
  Storm Leviathan achievement

Full 6-agent doc-audit found the codebase docs otherwise clean.
@stphung stphung enabled auto-merge (squash) July 15, 2026 17:19
@stphung stphung merged commit df7e9a9 into main Jul 15, 2026
9 checks passed
@stphung stphung deleted the claude/sharp-darwin-3yuvyy branch July 15, 2026 17:26
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.

2 participants