Skip to content

[ENH] Claude config#148

Merged
amaury901130 merged 3 commits into
masterfrom
enh/claude-config
Jun 19, 2026
Merged

[ENH] Claude config#148
amaury901130 merged 3 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

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

💯 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 🔥 34.007s ⏱️
👀 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 :)

@fernandatoledo fernandatoledo left a comment

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.

🚀

Comment thread .claude/skills/review-changes/SKILL.md
@amaury901130 amaury901130 merged commit 99e3ba3 into master Jun 19, 2026
8 checks passed
@amaury901130 amaury901130 deleted the enh/claude-config branch June 19, 2026 13:23
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