Skip to content

docs: sync AGENTS.md and CLAUDE.md with undocumented test files - #149

Merged
slittycode merged 1 commit into
mainfrom
claude/peaceful-hamilton-hSbX8
Jun 6, 2026
Merged

docs: sync AGENTS.md and CLAUDE.md with undocumented test files#149
slittycode merged 1 commit into
mainfrom
claude/peaceful-hamilton-hSbX8

Conversation

@slittycode

Copy link
Copy Markdown
Owner

Summary

Audit of documentation vs. actual codebase found 10 test files in apps/backend/tests/ with no entry in either CLAUDE.md or apps/backend/AGENTS.md, plus 2 file-map gaps in apps/ui/AGENTS.md. All documented files and modules were confirmed to exist — no stale references to removed code were found.

apps/backend/AGENTS.md — added to File Map:

  • test_audio_fixture.py / test_audio_fixture_smoke.py — WAV fixture helpers (no model load)
  • test_bootstrap_scripts.pybin/asa install/bootstrap script contracts
  • test_cleanup.pyutils/cleanup.py artifact-cleanup helpers
  • test_genre_check.pyscripts/genre_check.py
  • test_phase1_evaluation_transcription.py — Layer 2 transcription evaluation harness path
  • test_phase2_grammar_fix.py — Phase 2 gerund-fix post-process in server_phase2.py
  • test_phase2_prompt_catalog.py — prompt examples vs. Live 12 catalogue
  • test_root_dev_script.py — root scripts/dev.sh env-loading behavior
  • test_root_e2e_script.py — root test-e2e{,-integration}.sh script contracts
  • test_separation_ab.pyseparation_ab.py SI-SDR aggregation math
  • Added "Covered by tests/test_cleanup.py" to the utils/cleanup.py entry
  • Added "Covered by tests/test_separation_ab.py" to the separation_ab.py entry

apps/ui/AGENTS.md — file-map gaps:

  • src/services/midi/ was missing types.ts
  • src/services/sessionMusician/ was missing index.ts

CLAUDE.md — added inline test references at the relevant callout sites:

  • Item 17 (utils/cleanup.py): tests/test_cleanup.py
  • Item 16 (eval harnesses): tests/test_phase1_evaluation_transcription.py
  • Phase 2 block: tests/test_phase2_grammar_fix.py, tests/test_phase2_prompt_catalog.py
  • Scripts section: test_root_dev_script.py, test_root_e2e_script.py, test_bootstrap_scripts.py, test_genre_check.py, test_separation_ab.py

Test plan

  • No code changed — documentation-only PR
  • All referenced test files confirmed to exist in apps/backend/tests/
  • All referenced service files confirmed to exist in apps/ui/src/services/

https://claude.ai/code/session_01YQ4niA1KyaRrHMPXwFzJYm


Generated by Claude Code

Ten test files existed in apps/backend/tests/ with no entry in either
CLAUDE.md or apps/backend/AGENTS.md:

  test_audio_fixture{,_smoke}.py – WAV fixture helpers (no model load)
  test_bootstrap_scripts.py      – bin/asa install/bootstrap contracts
  test_cleanup.py                – utils/cleanup.py artifact helpers
  test_genre_check.py            – scripts/genre_check.py
  test_phase1_evaluation_transcription.py – Layer 2 eval harness path
  test_phase2_grammar_fix.py     – Phase 2 gerund-fix post-process
  test_phase2_prompt_catalog.py  – prompt examples vs Live 12 catalogue
  test_root_dev_script.py        – scripts/dev.sh env-loading
  test_root_e2e_script.py        – test-e2e{,-integration}.sh contracts
  test_separation_ab.py          – separation_ab.py SI-SDR math

Also two file-map gaps in apps/ui/AGENTS.md:
  src/services/midi/ was missing types.ts
  src/services/sessionMusician/ was missing index.ts

https://claude.ai/code/session_01YQ4niA1KyaRrHMPXwFzJYm

@slittycode slittycode left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

Pure documentation sync — adds 10 previously undocumented backend test files to apps/backend/AGENTS.md, backfills inline test references in CLAUDE.md, and plugs two file-map gaps in apps/ui/AGENTS.md (midi/types.ts, sessionMusician/index.ts). Every single referenced file was verified to exist in the repo. No code touched, no contracts changed.

Findings

None.

Test results

CI still running at review time, but immaterial — no source files changed. Docs-only diffs cannot break tests.

Phase boundary check

Clean. No phase code touched.


Generated by Claude Code

@slittycode slittycode left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

Docs-only sync: adds 10 previously undocumented backend test files to apps/backend/AGENTS.md and CLAUDE.md, fixes two file-map gaps in apps/ui/AGENTS.md (midi/types.ts, sessionMusician/index.ts). All 4 CI checks pass. All referenced files confirmed to exist on disk.

Findings

None.

Test results

Frontend ✓ / Backend ✓ / Chromatic ✓ / Loudness WASM ✓ — 4/4 green.

Phase boundary check

Clean — no code changed.


Generated by Claude Code

@slittycode
slittycode merged commit 60b3c74 into main Jun 6, 2026
4 checks passed
@slittycode
slittycode deleted the claude/peaceful-hamilton-hSbX8 branch June 6, 2026 21:05
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