Skip to content

docs: fix .claude/CLAUDE.md drift#112

Merged
hfoffani merged 1 commit into
mainfrom
claude-md-107
Jul 12, 2026
Merged

docs: fix .claude/CLAUDE.md drift#112
hfoffani merged 1 commit into
mainfrom
claude-md-107

Conversation

@hfoffani

Copy link
Copy Markdown
Owner

Closes #107.

What

Rewrite .claude/CLAUDE.md to match the post-1.1.x project. Assessment before the fix: 63/100 (C) — currency was the worst axis.

Drift fixed

  • Intro said "parsing only, no planners" — directly contradicted by the planning layer (and now the CLI/MCP surfaces). Now: four deliverables (library, CLI, MCP server, Agent Skill).
  • Key Files: added planning/, cli.py+serialize.py, mcpserver.py, diagnostics.py, binding.py, docs/; corrected tests location (tests/ + tests/corpus/; pddlpy/test.py marked legacy).
  • Architecture diagram extended above DomainProblem: planning layer (with the test-enforced layering rule) and the CLI/MCP surfaces.
  • Quick Commands: added lint/typecheck/coverage/examples, the CLI and MCP entry points, and the fresh-clone warning (make pyparser before uv sync).
  • Gotchas captured: Atom has no value equality (atom_tuple), UTF-8 reads (UnicodeDecodeError: PDDL files with non-ASCII bytes crash DomainProblem #103), mcp optional extra, CI-enforced 100% coverage.
  • Release section: added the pypi-environment approval gate and CI-built artifacts detail.

Kept the original terse structure and the YAGNI/coverage style rules unchanged.

🤖 Generated with Claude Code

The file predated most of 1.1.x: intro claimed 'parsing only, no
planners' while a Planning-layer section sat below it; Key Files named
pddlpy/test.py as the unit tests (legacy — the suite is 191 tests in
tests/) and examples-pddl/ as the corpus (now tests/corpus/); the
architecture diagram stopped at DomainProblem. Rewritten to cover the
four deliverables (library, CLI, MCP server, skill), the layered
architecture with the CLI/MCP/diagnostics modules, the Atom
no-value-equality and pyparser-before-uv-sync gotchas, CI enforcement,
and the approval gate in the release flow.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@hfoffani
hfoffani merged commit d2d2fd9 into main Jul 12, 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.

Fix .claude/CLAUDE.md drift

1 participant