docs(impeccable): three new derivative-template DON'T rules (hedged type stacks, flat tech-blue, default pill chip)#103
Conversation
Adds three net-new DON'T lines to the impeccable skill's existing
typography, color, and visual-details rule blocks. Each targets a
2020s derivative-template tell not yet covered by the skill:
1. Typography — bans hedged type stacks that list two display/body
families as interchangeable choices ("Family A or Family B").
Complements the existing "overused fonts" rule by catching the
deferred-commitment pattern rather than the individual font.
2. Color — bans the saturated tech-product-blue accent band
(hue ~215–225, high chroma, lightness ~50%). Complements the
existing AI-color-palette rule, which only catches gradient/neon
combos, not flat default blue.
3. Visual Details — bans the default rounded-full pill chip with
faint tinted background and hairline border, at 11–12px, as a
"Beta"/"New" or section-label treatment. No existing rule covers
badge/chip shape.
Scope: source/skills/impeccable/SKILL.md only. No reference file,
no detector code, no generated-mirror edits. Wording is
time-agnostic and product-agnostic (no specific product names in
the final DON'T lines) so the rules don't age out of the skill.
Evidence for each rule comes from three documented design-review
dogfoods that surfaced the same patterns across unrelated projects.
Co-Authored-By: Paperclip <noreply@paperclip.ing>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f8303b779c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| DO vary your font choices across projects. If you used a serif display font on the last project, look for a sans, monospace, or display face on this one. | ||
|
|
||
| DO NOT use overused fonts like Inter, Roboto, Arial, Open Sans, or system defaults — but also do not simply switch to your second-favorite. Every font in the reflex_fonts_to_reject list above is banned. Look further. | ||
| DO NOT hedge a type stack by listing two display or body families as interchangeable choices ("Family A or Family B"). That is deferred commitment, not a type system. Commit to one primary family per role; a second family is only legitimate as a named license fallback. |
There was a problem hiding this comment.
Regenerate provider dist outputs after source skill edit
This commit changes source/skills/impeccable/SKILL.md but leaves all generated provider artifacts untouched, so the new rules never reach users who install from committed dist/ bundles (dist/cursor, dist/claude-code, dist/gemini, dist/codex). In this repo, source edits are expected to be followed by bun run build and committing the regenerated outputs; otherwise source and distribution drift, and published downloads/API bundle content remain stale.
Useful? React with 👍 / 👎.
…cedure Rewrites the typography DON'T line added in the previous commit so it targets the reflex-pairing mechanism rather than the "Family A or B" surface symptom. Anchors explicitly to font_selection_procedure and reflex_fonts_to_reject, and requires every role in a type stack to clear the procedure independently. A pair where only one role survived the procedure is still monoculture, displaced. Keeps the rule product- and time-agnostic (no font or year named) so it still reads correctly when the specific reflex-font list rotates.
Summary
Adds three net-new DON'T rules to
source/skills/impeccable/SKILL.md, inserted into the existing typography, color, and visual-details rule blocks. Each rule targets a 2020s derivative-template tell the skill does not currently catch.<typography_rules>— closes a reflex-pairing loophole infont_selection_procedure. Each role in a type stack (display, body, any third) must independently clearreflex_fonts_to_rejectand Steps 1–4; a pair where only one role survived the procedure is still monoculture, just displaced into the other role.<color_rules>— bans the saturated tech-product-blue accent band (hue ~215–225, high chroma, lightness ~50%). Complements the existing AI-color-palette rule (which catches gradient/neon combos, not flat default blue)."Beta"/"New"tags or section labels. No existing rule covers badge/chip shape.Diff: 1 file, 3 insertions, 0 deletions on the original commit, with the typography rule rewritten in a follow-up amendment to anchor it to
font_selection_procedure/reflex_fonts_to_reject(see below).Why this is scoped the way it is
source/skills/impeccable/SKILL.mdis touched.source/is the editable source perDEVELOP.md; the.agents/,.claude/,.codex/,.cursor/,.gemini/,.kiro/,.opencode/,.pi/,.rovodev/,.trae*/mirrors are build artifacts — not hand-edited here.Evidence
Each rule was surfaced by at least one documented design-review dogfood where the absence of the rule let the tell ship or nearly ship:
reflex_fonts_to_reject, but runningfont_selection_procedureon each role independently would have surfaced that neither was selected from a brief — both were reflex grabs. The existing "overused fonts" rule only catches individual families, not the reflex-pairing shape, so a designer who committed to a single reflex font in each of two roles would still pass it.#0066FFas the primary accent passed the AI-color-palette rule (no gradient, no neon) but read as a generic dev-tool landing page. The existing rule targets glow/gradient combos, not the flat-high-chroma-blue default."Beta"/"New"treatments across unrelated pages reached for the faint-tinted-pill-with-hairline-border at 11–12px. No existing rule addressed badge shape, so the pill shipped by default on every project until flagged by hand.Scope: two tells held local, not proposed upstream
Two additional derivative-template tells from our internal checklist are intentionally not in this PR:
If either generalizes with more external evidence, we'll scope a dedicated follow-up PR. Not this one.
Test plan
source/skills/impeccable/SKILL.md.git diff --statagainstupstream/mainshows 1 file, 3 insertions..agents/,.claude/,.codex/,.cursor/,.gemini/,.kiro/,.opencode/,.pi/,.rovodev/,.trae*/) is touched; they regenerate viabun run build.DO NOT …inside<typography_rules>and<color_rules>,DO NOT: …inside Visual Details bullets).font_selection_procedure/reflex_fonts_to_rejectblocks so it still reads correctly in a 2027+ context.~215–225, high chroma, ~50% L) concrete enough, or would you prefer an OKLCH-specific phrasing given the skill's existing OKLCH guidance?Follow-ups
If this lands cleanly, there is one candidate for a scoped second PR: a
reference/template-tells-2025.mdwith rationale and rewrite patterns for each of the three tells in this PR. Holding it back to keep review surface minimal. The two tells named in the Scope section above are not candidates for a follow-up PR — they stay local unless new external evidence generalizes them.🤖 Generated with Claude Code