update skill to say dont make up Tabler icons

[emmacramer]

silly agent

[github-actions]

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.

[emmacramer]

@builderio-bot why is the Chat template E2E test failing?

[builder-io-integration]

I looked into PR #2703 and the CI run for the failing job.

What's failing: The Generate + run standalone Chat job fails inside pnpm qa:sign-in (scripts/qa-sign-in-matrix-smoke.ts), specifically a Playwright click on .tab[data-tab="signup"]:

Error: page.click: Timeout 30000ms exceeded.
- locator resolved to <button data-tab="signup" class="tab active" ...>Create account</button>
- element is not visible

The element exists (it's already marked active) but Playwright can't get it into a stable/visible state within 30s — a rendering/timing race in the sign-in test harness, not a missing element.

Why it's not related to this PR: This PR only touches .agents/skills/frontend-design/SKILL.md. There's no app code, template, or test change in the diff, so it can't be the cause of a browser-rendering timing issue in the sign-in smoke test.

Confirmation it's not a broken/consistently-failing test: The same workflow has been passing reliably on main in recent runs, so this looks like CI flakiness in this run (a slow/unstable render during the standalone Chat E2E) rather than a real regression.

Recommendation: Re-run the failed job — this should be safe to retry since the PR content is docs-only. Want me to trigger a re-run via gh run rerun --failed?

---

⠀
🟡 Diagnosed the failing Chat E2E job as an unrelated CI flake (docs-only PR); awaiting confirmation to re-run it.

---

  

[builder-io-integration]

Builder reviewed your changes — looks good ✅

Review Details

Incremental Code Review Summary

The latest PR head remains a documentation-only addition to .agents/skills/frontend-design/SKILL.md. The Tabler icon verification guidance is unchanged and remains technically correct: the declaration path is node_modules/@tabler/icons-react/dist/tabler-icons-react.d.ts, the package layout supports it, and the previously corrected dist/esm/ path has not regressed.

Independent reviews found no new or unresolved actionable issues. The guidance is consistent with the surrounding frontend-design rules and should help prevent invalid named imports that fail Vite builds. This remains low risk with no user-facing runtime impact. 🧪 Browser testing: Skipped — PR only modifies backend/config/docs, no UI impact.