update skill to say dont make up Tabler icons - #2703
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ Deploy Preview for agent-native-content canceled.
|
✅ Deploy Preview for agent-native-forms canceled.
|
✅ Deploy Preview for agent-native-dispatch canceled.
|
✅ Deploy Preview for agent-native-mail canceled.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ Deploy Preview for agent-native-slides canceled.
|
![builder-io-integration[bot]](https://avatars.githubusercontent.com/in/353721?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Visual recap — skipped (diff too small)The change in this pull request is too small to be worth a visual recap. This is informational only and does not block the PR. |
There was a problem hiding this comment.
Builder reviewed your changes — looks good ✅
Review Details
Incremental Code Review Summary
The latest PR head still contains the corrected Tabler declaration path, node_modules/@tabler/icons-react/dist/tabler-icons-react.d.ts, with no regression of the prior dist/esm/ issue. Independent reviewers verified that the path is valid for the repository's current Tabler package versions and that the example names are not exported. The guidance accurately explains the Vite failure mode and provides a practical verification step before importing unfamiliar icons.
No new or unresolved actionable issues were found. This remains a low-risk, documentation-only change with no runtime or user-facing browser impact. 🧪 Browser testing: Skipped — PR only modifies backend/config/docs, no UI impact.
3 participants
silly agent