Skip to content

Chore/agents config#144

Open
fernandatoledo wants to merge 3 commits into
masterfrom
chore/agents_config
Open

Chore/agents config#144
fernandatoledo wants to merge 3 commits into
masterfrom
chore/agents_config

Conversation

@fernandatoledo
Copy link
Copy Markdown
Collaborator

What does this do?

Adds AGENTS.md and REVIEW.md as the main instructions for AI agents and code review, plus agent_docs/ (architecture.md, commands.md, conventions.md) for deeper context. CLAUDE.md now points agents at AGENTS.md. The repo gains a pr-description skill under .agents/skills/, .gitignore excludes PR_DESCRIPTION.md, and .claude/settings.json runs a short pnpm type-check after Edit/Write tool use. cli/setup-project.js calls a new updateAgentDocs() step so new projects get tailored copy in AGENTS.md and agent_docs/, removes PROMPT_agent_docs.md during cleanup, and adjusts commands.md placeholders for the app id.


Why did you do this?

Context was scattered or missing for agents and reviewers; centralizing it keepsClaude/Codex/Cursor aligned with Rootstrap conventions and makes onboarding and review repeatable. Automating agent-doc rewrites in the setup CLI keeps new client projects from shipping template-only prose.

Who/what does this impact?

Developers and anyone using AI tools against this repo; teams running cli/setup-project.js for new apps. No end-user app screens or runtime behavior changed.

How did you test this?

No app or device runs for this change set.


@fernandatoledo fernandatoledo requested a review from a team as a code owner May 13, 2026 13:21
@fernandatoledo fernandatoledo self-assigned this May 13, 2026
@github-actions
Copy link
Copy Markdown

💯 Test Coverage

Lines Statements Branches Functions
Coverage: 61%
60.4% (415/687) 57.4% (159/277) 48.52% (115/237)

😎 Tests Results

Tests Skipped Failures Errors Time
139 0 💤 0 ❌ 0 🔥 34.476s ⏱️
👀 Tests Details • (61%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files60.457.448.5261.02 
report-only-changed-files is enabled. No files were changed in this commit :)

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