Skip to content

chore: add CLAUDE.md and AGENTS.md for AI agent guidelines - #1707

Merged
sanchitmehtagit merged 1 commit into
mainfrom
chore/add-claude-md
Sep 11, 2026
Merged

chore: add CLAUDE.md and AGENTS.md for AI agent guidelines#1707
sanchitmehtagit merged 1 commit into
mainfrom
chore/add-claude-md

Conversation

@sanchitmehtagit

Copy link
Copy Markdown
Contributor

Summary

  • Adds CLAUDE.md with project structure, three-tier boundaries (Always Do / Ask First / Never Do), security considerations, and 6 lazy-loaded reference sections
  • Adds AGENTS.md that imports @./CLAUDE.md for non-Claude AI agents
  • Adds references/ directory with commands.md, testing.md, code-style.md, git-workflow.md, pitfalls.md, and docs-update.md

Key guidelines captured

  • Terraform provider patterns: expand/flatten, NewResource() skeleton, HandleReadAPIError vs legacy HandleAPIError, mandatory schema Description fields
  • Test tiers: unit (make test-unit), recorded acceptance (make test-acc), live e2e (make test-acc-e2e) with cassette workflow
  • Critical boundaries: never edit docs/ manually, always run make docs after schema changes, verify cassette redaction before committing
  • CI-enforced lint rules: godot, errcheck, goimports local prefix

Test plan

  • Review CLAUDE.md for accuracy against current codebase conventions
  • Verify all make targets listed in references/commands.md exist in the Makefile
  • Confirm references/ paths match actual repo structure

Adds CLAUDE.md with project structure, boundaries, security considerations,
and lazy-loaded reference docs covering commands, testing conventions,
code style, git workflow, pitfalls, and docs-update rules. AGENTS.md
imports CLAUDE.md for non-Claude agents.
@sanchitmehtagit
sanchitmehtagit requested a review from a team as a code owner September 10, 2026 14:16
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.23%. Comparing base (25ae527) to head (282779c).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1707      +/-   ##
==========================================
- Coverage   86.10%   85.23%   -0.87%     
==========================================
  Files         205      209       +4     
  Lines       31382    30080    -1302     
==========================================
- Hits        27022    25640    -1382     
- Misses       3379     3444      +65     
- Partials      981      996      +15     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@developerkunal developerkunal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@sanchitmehtagit
sanchitmehtagit merged commit f5b8ff8 into main Sep 11, 2026
4 checks passed
@sanchitmehtagit
sanchitmehtagit deleted the chore/add-claude-md branch September 11, 2026 08:30
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.

3 participants