Block slop before it ships.
-A detector you can wire into PR checks. 49 deterministic rules, no LLM, exit codes the build can read.
+A detector you can wire into PR checks. 51 deterministic rules, no LLM, exit codes the build can read.
@@ -799,7 +799,7 @@ import '../styles/testimonials.css';
CLI for CI
-
diff --git a/tests/detect-antipatterns-fixtures.test.mjs b/tests/detect-antipatterns-fixtures.test.mjs
index 597adc45d..0ecaf992d 100644
--- a/tests/detect-antipatterns-fixtures.test.mjs
+++ b/tests/detect-antipatterns-fixtures.test.mjs
@@ -375,6 +375,39 @@ describe('detectHtml — static HTML/CSS fixtures', () => {
);
assert.match(numbered[0].snippet, /01, 02, 03/);
});
+
+ it('numbered-section-labels: tiny repeated index labels flag, deliberate/list/card numbering passes', async () => {
+ const f = await detectHtml(path.join(FIXTURES, 'numbered-section-labels.html'));
+ const labels = f.filter(r => r.antipattern === 'numbered-section-labels');
+ const snippets = labels.map(r => r.snippet).join(' | ');
+ assert.equal(
+ labels.length,
+ 4,
+ `expected 4 numbered-label findings, got ${labels.length}: ${snippets}`
+ );
+ for (const heading of ['Alpha ships first', 'Beta earns trust', 'Gamma holds the line', 'Delta closes the loop']) {
+ assert.match(snippets, new RegExp(heading), `expected label beside "${heading}" to flag`);
+ }
+ for (const heading of ['Epsilon', 'Zeta', 'Eta', 'Theta', 'Iota', 'Kappa', 'Lambda', 'Mu']) {
+ assert.doesNotMatch(snippets, new RegExp(heading), `label beside "${heading}" should pass`);
+ }
+ });
+
+ it('repeated-container-text: same string in 3+ distinct slots of one card flags; structural repetition passes', async () => {
+ const f = await detectHtml(path.join(FIXTURES, 'repeated-container-text.html'));
+ const repeats = f.filter(r => r.antipattern === 'repeated-container-text');
+ const snippets = repeats.map(r => r.snippet).join(' | ');
+ assert.equal(
+ repeats.length,
+ 2,
+ `expected 2 repeated-text findings, got ${repeats.length}: ${snippets}`
+ );
+ assert.match(snippets, /Suspended.*3×|Suspended" rendered 3/, 'expected the 3-slot status word to flag');
+ assert.match(snippets, /Unavailable" rendered 4/, 'expected the 4-slot status word to flag');
+ for (const passText of ['Rolled back', 'On schedule', 'Overview page', 'Standby mode', 'Open slot', 'Rescheduled', '2026']) {
+ assert.doesNotMatch(snippets, new RegExp(passText), `"${passText}" should pass`);
+ }
+ });
});
describe('detectHtml — icon-tile-stack', () => {
diff --git a/tests/detect-antipatterns.test.js b/tests/detect-antipatterns.test.js
index 4edb8d164..fab7a7650 100644
--- a/tests/detect-antipatterns.test.js
+++ b/tests/detect-antipatterns.test.js
@@ -12,9 +12,12 @@ import {
} from '../cli/engine/detect-antipatterns.mjs';
import { filterByScopes } from '../cli/engine/registry/antipatterns.mjs';
import {
+ checkColors,
checkElementTextOverflowDOM,
checkHeroEyebrow,
checkHoverContrast,
+ checkNumberedSectionLabels,
+ parseNumberedLabelText,
checkHtmlPatterns,
checkPageTypography,
isScreenReaderOnlyTextStyle,
@@ -1022,6 +1025,22 @@ describe('side-tab — pseudo-element stripe variant', () => {
expect(scanCssTextForPseudoStripe(css)).toHaveLength(1);
});
+ test('detects floating stripe inset a few px from each end', () => {
+ // The evasion shape from human review: same left-edge accent bar, but
+ // backed off the card's corners by a small top/bottom inset so it never
+ // touches an edge (and needs no corner rounding to read as a side tab).
+ const css = '.row::before { content: ""; position: absolute; left: 0; top: 12px; bottom: 12px; width: 3px; border-radius: 3px; background: oklch(0.65 0.19 15); }';
+ const f = scanCssTextForPseudoStripe(css);
+ expect(f).toHaveLength(1);
+ expect(f[0].id).toBe('side-tab');
+ expect(f[0].snippet).toContain('(left: 0)');
+ });
+
+ test('skips deeply-inset partial rail (not an edge-spanning stripe)', () => {
+ const css = '.rail::before { position: absolute; left: 0; top: 40px; bottom: 40px; width: 4px; background: #3b82f6; }';
+ expect(scanCssTextForPseudoStripe(css)).toHaveLength(0);
+ });
+
test('unresolvable custom-property color errs toward detection', () => {
const css = '.card::before { position: absolute; left: 0; top: 0; bottom: 0; width: 5px; background: var(--from-external-sheet); }';
expect(scanCssTextForPseudoStripe(css)).toHaveLength(1);
@@ -1106,6 +1125,63 @@ describe('side-tab — pseudo-element stripe variant', () => {
});
});
+// ---------------------------------------------------------------------------
+// Low contrast — modern computed-color serializations (browser adapter path)
+// ---------------------------------------------------------------------------
+
+describe('checkColors — oklch computed colors', () => {
+ test('flat dark-on-dark oklch CTA pair parses and fails contrast', () => {
+ // Real browsers hand back oklch() strings from getComputedStyle for
+ // colors authored in modern spaces; the adapters must not lose them.
+ const textColor = parseAnyColor('oklch(0.34 0.01 70)');
+ const bgColor = parseAnyColor('oklch(0.22 0.01 70)');
+ expect(textColor).toBeTruthy();
+ expect(bgColor).toBeTruthy();
+ const f = checkColors({
+ tag: 'a',
+ textColor,
+ bgColor,
+ effectiveBg: bgColor,
+ effectiveBgStops: null,
+ fontSize: 14.4,
+ fontWeight: 500,
+ hasDirectText: true,
+ isEmojiOnly: false,
+ bgClip: '',
+ bgImage: '',
+ classList: 'btn btn-primary',
+ });
+ expect(f.some(r => r.id === 'low-contrast')).toBe(true);
+ });
+});
+
+// ---------------------------------------------------------------------------
+// Numbered section labels — pure helpers
+// ---------------------------------------------------------------------------
+
+describe('numbered-section-labels — pure helpers', () => {
+ test('parseNumberedLabelText accepts zero-padded and separator forms only', () => {
+ expect(parseNumberedLabelText('01')).toEqual({ index: 1, text: '01' });
+ expect(parseNumberedLabelText('12')).toEqual({ index: 12, text: '12' });
+ expect(parseNumberedLabelText('04 / rollout')).toMatchObject({ index: 4 });
+ expect(parseNumberedLabelText('6 · getting started')).toMatchObject({ index: 6 });
+ expect(parseNumberedLabelText('7')).toBeNull();
+ expect(parseNumberedLabelText('Step 3')).toBeNull();
+ expect(parseNumberedLabelText('12 minute read')).toBeNull();
+ expect(parseNumberedLabelText('50% off everything')).toBeNull();
+ expect(parseNumberedLabelText('')).toBeNull();
+ });
+
+ test('checkNumberedSectionLabels needs 2+ candidates with 2+ distinct indices', () => {
+ const candidate = (index) => ({ index, labelText: String(index).padStart(2, '0'), headingTag: 'h2', headingText: 'Heading' });
+ expect(checkNumberedSectionLabels({ candidates: [candidate(1)] })).toHaveLength(0);
+ expect(checkNumberedSectionLabels({ candidates: [candidate(1), candidate(1)] })).toHaveLength(0);
+ const flagged = checkNumberedSectionLabels({ candidates: [candidate(1), candidate(2)] });
+ expect(flagged).toHaveLength(2);
+ expect(flagged[0].id).toBe('numbered-section-labels');
+ });
+});
+
// ---------------------------------------------------------------------------
// Radial-gradient background halo
// ---------------------------------------------------------------------------
diff --git a/tests/fixtures/antipatterns/numbered-section-labels.html b/tests/fixtures/antipatterns/numbered-section-labels.html
new file mode 100644
index 000000000..a14495a9c
--- /dev/null
+++ b/tests/fixtures/antipatterns/numbered-section-labels.html
@@ -0,0 +1,151 @@
+
+
+
+
+ Numbered Section Labels Fixture
+
+
+
+
+
+ Repeated Container Text Fixture
+
+
+
+
+
+
+
+
npx impeccable detect src/ in a PR check. 49 deterministic rules. JSON output, exit codes for build gates.
+ npx impeccable detect src/ in a PR check. 51 deterministic rules. JSON output, exit codes for build gates.
View on npm →
+
+
+ 01
+
+
+
+ 02
+
+
+
+ 03
+
+
+
+ 04 / ROLLOUT
+
+
+
+
+
+ Alpha ships first
+Direct previous-sibling label: tiny, mono, bold, zero-padded index.
+Beta earns trust
+Second section repeating the same index scaffold.
+
+
+ Gamma holds the line
+Label precedes the wrapper; the heading leads the wrapper.
+Delta closes the loop
+Index joined to a tracked micro-label by a separator glyph.
+
+
+
+ 05
+
+
+
+ 12 · minute read
+
+
+
+ Step 6
+
+
+
+ 7
+
+
+
+ 10
+
+
+
+
+
+
+
+
diff --git a/tests/fixtures/antipatterns/repeated-container-text.html b/tests/fixtures/antipatterns/repeated-container-text.html
new file mode 100644
index 000000000..fa04b91a6
--- /dev/null
+++ b/tests/fixtures/antipatterns/repeated-container-text.html
@@ -0,0 +1,129 @@
+
+
+
+
+ Epsilon reads large
+16px index is a deliberate display device, not a micro label.
+Zeta stays plain
+Parses as an index but carries no micro-label styling: regular weight, neutral color, no tracking, serif body face.
+Eta walks steps
+Word-first label is not a numeric index.
+Theta counts casually
+A bare unpadded single digit is ordinary list numbering.
+-
+
-
+ 08
+
Iota lives in a list
+
+ -
+ 09
+
Kappa lives in a list too
+
+
Lambda sits in a card
+Per-card indices are item numbering, not section scaffolding.
+
+
+
+
+ Museumplein departure
+Suspended
+
+ This service is Suspended until further notice.
+
+
+
+
+
+
+ Evening show
+ +Unavailable
+
+ Currently Unavailable at this venue.
+
+
+
+
+
+
+ Deploy history
+| build 12 | Rolled back |
| build 13 | Rolled back |
| build 14 | Rolled back |
+
+
+
+
+
+
+ Departures
+On schedule
+ On schedule
+ On schedule
+
+
+
+
+ Toggles
+Off
+ Off
+ Off
+
+
+
+
+ Scores
+2026
+ 2026
+ 2026
+ Standby mode
+ Standby mode
+ Standby mode
+
+
+
+
+
+
+ Availability
+
+
+ Open slot
+ Open slot
+ Open slot
+
+
+
+
+
Corner case
+Rescheduled
+
+