Skip to content

[ENH] Claude config#148

Open
amaury901130 wants to merge 2 commits into
masterfrom
enh/claude-config
Open

[ENH] Claude config#148
amaury901130 wants to merge 2 commits into
masterfrom
enh/claude-config

Conversation

@amaury901130
Copy link
Copy Markdown

Jira board reference:

  • N/A

What does this do?

Expands AGENTS.md with missing glossary entries (APIProvider, authStorage), three new critical constraints (forms, animations, token refresh), and a provider hierarchy reference. Updates CLAUDE.md to require agents to read the key architecture docs before starting any non-trivial task. Also refines the pr-description skill trigger description for more precise matching.


Why did you do this?

Agents were missing enough context to correctly apply the project's constraints around forms, animations, and token handling. The pre-task reading list in CLAUDE.md ensures agents orient themselves before writing code, reducing the chance of convention violations.

Who/what does this impact?

AI agents (Claude Code) working in this repository. No runtime code, screens, or APIs are affected.

Amaury added 2 commits June 3, 2026 10:11
…PIProvider and authStorage glossary entries, forms/animations/token-refresh

  critical constraints, and provider hierarchy to AGENTS.md. Add pre-task reading
  list to CLAUDE.md so agents orient themselves before writing code.
@amaury901130 amaury901130 requested a review from a team as a code owner June 3, 2026 16:47
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

💯 Test Coverage

Lines Statements Branches Functions
Coverage: 56%
56.23% (347/617) 59.18% (174/294) 48.76% (118/242)

😎 Tests Results

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

Copy link
Copy Markdown
Collaborator

@fernandatoledo fernandatoledo left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🫶

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.

2 participants