Update Claude / Agent MD and add React skill#2273
Open
alexandrudanpop wants to merge 4 commits intomainfrom
Open
Update Claude / Agent MD and add React skill#2273alexandrudanpop wants to merge 4 commits intomainfrom
alexandrudanpop wants to merge 4 commits intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates repository AI/agent guidance docs and introduces a new Claude “react” skill to standardize frontend development practices across packages/react-ui and packages/ui-components.
Changes:
- Updated
CLAUDE.mdandAGENTS.mdstructure/package listings and consolidated frontend guidance by referencing the newreactskill. - Added a new
.claude/skills/react/SKILL.mdwith React-specific conventions for components, hooks, react-query, styling, and quality checks. - Tweaked testing/lint command examples and formatting consistency in the docs.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
CLAUDE.md |
Updates repo/package guidance and points frontend work to the new react skill; adjusts testing/examples formatting. |
AGENTS.md |
Aligns agent guidance with updated repo structure and the new react skill; updates Nx command examples. |
.claude/skills/react/SKILL.md |
Adds a dedicated React/frontend “skill” document defining conventions for UI work in react-ui and ui-components. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Soften useCallback/inline handler guidance to prefer extraction only when referential stability matters or logic is non-trivial - Add radix 10 to all Number.parseInt calls in examples - Replace non-existent qa-agent reference with generic browser testing guidance - Update anti-patterns checklist to match softened rules Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace domain-specific Number.parseInt examples with generic inline vs extracted handler patterns that illustrate the guideline more clearly. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
alexandrudanpop
commented
May 8, 2026
| @@ -7,15 +7,25 @@ | |||
| - Update documentation for user-facing changes | |||
Contributor
Author
There was a problem hiding this comment.
modiefied agents.md to be the same as claude.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes OPS-4263