Skip to content

Commit 4542d5a

Browse files
committed
fix: component-architecture.md
1 parent d8939ef commit 4542d5a

1 file changed

Lines changed: 14 additions & 14 deletions

File tree

skills/react-clean-architecture-add-component/references/component-architecture.md

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,20 @@ This reference does not define full application-wide architecture. It focuses on
1919
- `ui`: render-only, stateless components.
2020
- `features`: components that include logic.
2121

22+
## Reclassification Rule
23+
24+
If the user requests `ui` but the implementation contains any of the following, treat it as `features` and ask for confirmation before creating files:
25+
26+
- `useState`, `useReducer`, or `useEffect`.
27+
- Async behavior (API calls, timers, subscriptions).
28+
- Reading from or writing to context/store.
29+
- Business/data transformation logic.
30+
31+
Ask using these options:
32+
33+
- `Create as features`
34+
- `Keep ui and move logic/state to parent or features`
35+
2236
## Layer Responsibilities
2337

2438
This skill defines layers in two stages.
@@ -42,20 +56,6 @@ Notes:
4256
- `ui` is composed of presentation only.
4357
- `features` must separate container and presentation.
4458

45-
## Reclassification Rule
46-
47-
If the user requests `ui` but the implementation contains any of the following, treat it as `features` and ask for confirmation before creating files:
48-
49-
- `useState`, `useReducer`, or `useEffect`.
50-
- Async behavior (API calls, timers, subscriptions).
51-
- Reading from or writing to context/store.
52-
- Business/data transformation logic.
53-
54-
Ask using these options:
55-
56-
- `Create as features`
57-
- `Keep ui and move logic/state to parent or features`
58-
5959
## Implementation Rules
6060

6161
### ui

0 commit comments

Comments
 (0)