Skip to content

docs: add pitfalls section and grant Edit tool access#1

Merged
dvhthomas merged 1 commit intoCalcMark:mainfrom
dvhthomas:docs/agent-pitfalls-and-tool-access
Mar 13, 2026
Merged

docs: add pitfalls section and grant Edit tool access#1
dvhthomas merged 1 commit intoCalcMark:mainfrom
dvhthomas:docs/agent-pitfalls-and-tool-access

Conversation

@dvhthomas
Copy link
Contributor

Summary

  • Adds a "Common Pitfalls" section to CALCMARK-AGENT.md covering immutable variables, unit propagation, NL-first guidance, and stderr error handling
  • Adds a condensed "Common Pitfalls" subsection to the Claude Code SKILL.md
  • Grants Edit tool access in SKILL.md frontmatter so agents can incrementally fix .cm files instead of rewriting them entirely

Context

These changes are based on real agent usage where:

  1. The agent used raw arithmetic (customers / 10) instead of NL capacity form, producing wrong units
  2. The agent reassigned an immutable variable and needed to Edit the .cm file to fix it — but lacked Edit permission and had to rewrite the whole file
  3. The agent piped stderr errors into a JSON parser, causing a silent failure

Companion PR in go-calcmark adds the same pitfalls to the full agent-integration docs.

Test plan

  • Verify allowed-tools in SKILL.md now includes Edit
  • Confirm pitfalls sections are correctly placed before Security/Error Handling sections

🤖 Generated with Claude Code

Add Common Pitfalls covering immutable variables, unit propagation,
and NL-first guidance. Grant Edit tool access in SKILL.md so agents
can fix .cm files incrementally.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dvhthomas dvhthomas merged commit 1604732 into CalcMark:main Mar 13, 2026
1 check failed
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