Skip to content

Commit 3fb5101

Browse files
robphoenixclaude
andcommitted
chore(react): suppress inherited-prop oversight warnings and finish oversight CI wiring
- Add packages/react/oversight.config.json and disable prop-descriptions-missing for props inherited from third-party base types (mirrors packages/react-native) - Sync the same rule override into .storybook/manager.ts so the live addon panel matches CI - Simplify lint:oversight to read config from oversight.config.json - Fix react-storybook-oversight.yml to use rachelslurs/oversight-lint-action Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 13a33bb commit 3fb5101

5 files changed

Lines changed: 33 additions & 2 deletions

File tree

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
---
2+
'@utilitywarehouse/hearth-react-native': patch
3+
---
4+
5+
💅 [ENHANCEMENT]: `hearth-react-native` skill now defaults to the MCP server over local markdown
6+
7+
The skill guidance used by AI coding agents building with this package
8+
previously defaulted to reading local markdown files for component lookups.
9+
It now defaults to the `hearth-react-native` MCP server for component lookups and
10+
general/cross-cutting guidance, falling back to markdown only for a specific
11+
story's exact code beyond what `get-documentation` or
12+
`get-documentation-for-story` already surfaces.
13+
14+
**Developer changes**:
15+
16+
No action required — this only affects the guidance given to AI coding
17+
agents using this library, not the runtime API.

.github/workflows/react-storybook-oversight.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,5 +33,5 @@ jobs:
3333
- run: pnpm build:storybook:react
3434
env:
3535
NODE_OPTIONS: '--max_old_space_size=4096'
36-
- run: pnpm lint:oversight
36+
- uses: rachelslurs/oversight-lint-action@v1
3737
working-directory: ./packages/react

packages/react/.storybook/manager.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,12 @@ const theme = create(config);
88

99
addons.setConfig({
1010
theme,
11+
'storybook-addon-oversight': {
12+
expectedExtractor: 'react-docgen-typescript',
13+
rules: {
14+
'prop-descriptions-missing': 'off',
15+
},
16+
},
1117
layoutCustomisations: {
1218
showToolbar(state: State, defaultValue: boolean) {
1319
return state.viewMode === 'docs' ? false : defaultValue;
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
"manifest": "storybook-static/manifests/components.json",
3+
"expectedExtractor": "react-docgen-typescript",
4+
"maxWarnings": 0,
5+
"rules": {
6+
"prop-descriptions-missing": "off"
7+
}
8+
}

packages/react/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252
"lint:js:fix": "TIMING=1 eslint --fix --max-warnings 0 \"src/**/*.{js,ts*}\"",
5353
"lint:css": "stylelint \"src/**/*.css\"",
5454
"lint:css:fix": "stylelint \"src/**/*.css\" --fix",
55-
"lint:oversight": "npx oversight --max-warnings 0 --expected-extractor react-docgen-typescript",
55+
"lint:oversight": "oversight",
5656
"format": "prettier --write \"{src,public,docs,scripts}/**/*.{js,ts*,md*,css}\" {README,SKILL}.md",
5757
"format:ci": "prettier --list-different \"src/**/*.{ts*,mdx,css}\"",
5858
"figma": "figma connect",

0 commit comments

Comments
 (0)