update skill to say dont make up Tabler icons

[emmacramer]

silly agent

[netlify]

✅ Deploy Preview for agent-native-content canceled.

Name	Link
🔨 Latest commit	b157950
🔍 Latest deploy log	https://app.netlify.com/projects/agent-native-content/deploys/6a74f0eae722110007cb42df

[netlify]

✅ Deploy Preview for agent-native-forms canceled.

Name	Link
🔨 Latest commit	b157950
🔍 Latest deploy log	https://app.netlify.com/projects/agent-native-forms/deploys/6a74f0eaeaa18a00085406fd

[netlify]

✅ Deploy Preview for agent-native-dispatch canceled.

Name	Link
🔨 Latest commit	b157950
🔍 Latest deploy log	https://app.netlify.com/projects/agent-native-dispatch/deploys/6a74f0ea609869000824dc5e

[netlify]

✅ Deploy Preview for agent-native-mail canceled.

Name	Link
🔨 Latest commit	b157950
🔍 Latest deploy log	https://app.netlify.com/projects/agent-native-mail/deploys/6a74f0eab0fb1200081da100

[netlify]

✅ Deploy Preview for agent-native-slides canceled.

Name	Link
🔨 Latest commit	b157950
🔍 Latest deploy log	https://app.netlify.com/projects/agent-native-slides/deploys/6a74f0ea5b46200008f600bd

[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.

[builder-io-integration]

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.

[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.

---