Skip to content

Update CLAUDE.md to match current test conventions and layout - #356

Merged
CatarinaGamboa merged 1 commit into
mainfrom
docs/claude-md-update
Oct 6, 2026
Merged

CatarinaGamboa merged 1 commit into
mainfrom
docs/claude-md-update

Conversation

@CatarinaGamboa

Copy link
Copy Markdown
Collaborator

CLAUDE.md had drifted from the README and the code. Docs-only change.

  • Test discovery: any .java file outside a leaf directory is a test, and so is every leaf directory. Correct…/Error… names are only a convention (see TestExamples#sourcePaths).
  • Expectations: comments now use // Expect: <Title> Error / // Expect: Warning. The old // <Title> Error form no longer matches TestUtils.EXPECTED_DIAGNOSTIC. Warnings are matched by line only.
  • Package map: ast/ lives under rj_language/ (with parsing/, opt/, visitors/). errors/ and warnings/ live under diagnostics/.
  • Z3: it is bundled in-process through z3-turnkey; the verifier does not shell out to it.
  • Removed the reference to a co-specifying-liquidjava skill, which is not in the repo.
  • Added ./mvnw, the liquidjava script's recompile-only-when-changed behaviour, and the CLI options (as in the README).

🤖 Generated with Claude Code

- Test discovery: any .java file outside a leaf dir, and every leaf dir;
  names are only a convention
- Expectations use // Expect: <Title> Error / Warning comments
- Package map: ast under rj_language, errors/warnings under diagnostics
- Z3 bundled via z3-turnkey Java bindings; drop missing skill reference
- Mention ./mvnw, script recompile behaviour, and CLI options

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@CatarinaGamboa
CatarinaGamboa merged commit f4f65d7 into main Oct 6, 2026
1 check passed
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.

1 participant