@heyramzi/lint 1.32.2 → 1.32.4

package/bin/heyramzi-slop.js

@@ -12,7 +12,7 @@
  */
 
 /*
- * cat draft.txt | heyramzi-slop -            stdin in, cleaned text on stdout
+ * cat draft.txt | heyramzi-slop "-"          stdin in, cleaned text on stdout
  */
 
 /*

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heyramzi/lint",
-  "version": "1.32.2",
+  "version": "1.32.4",
   "description": "Shared oxlint rules, base config and gates for Studio projects",
   "license": "MIT",
   "repository": {
@@ -37,6 +37,7 @@
     "push-diff.js",
     "slop.d.ts",
     "slop.js",
+    "slop-views.js",
     "ui-rules.js",
     "svelte-markup.js",
     "svelte-template.js",

package/rules/no-em-dash.js

@@ -7,10 +7,14 @@ import { isTestFile } from "#lint/rules/test-file.js";
 
 const SPACED_HYPHEN = / - /;
 
+const EM_DASH = String.fromCharCode(0x2014);
+const EN_DASH = String.fromCharCode(0x2013);
+const SPACED_HYPHEN_TEXT = String.fromCharCode(0x20, 0x2d, 0x20);
+
 const PATTERNS = [
-  { regex: /\u2014/, name: "em dash (\u2014)" },
-  { regex: /\u2013/, name: "en dash (\u2013)" },
-  { regex: SPACED_HYPHEN, name: "spaced hyphen ( - ) used as dash" },
+  { regex: /\u2014/, name: `em dash (${EM_DASH})` },
+  { regex: /\u2013/, name: `en dash (${EN_DASH})` },
+  { regex: SPACED_HYPHEN, name: `spaced hyphen (${SPACED_HYPHEN_TEXT}) used as dash` },
 ];
 
 /**
@@ -18,7 +22,7 @@ const PATTERNS = [
  */
 
 /*
- * `calc((100% - var(--gap)) / 2)` needs the spaces around its operator, the spec requires them, so a `<style>` block reported a spaced hyphen on every Svelte component doing arithmetic and
+ * `calc()` arithmetic needs spaces around its minus operator, the spec requires them, so a `<style>` block reported a spaced hyphen on every Svelte component doing arithmetic and
  */
 
 /*
@@ -33,7 +37,7 @@ function isInsideStyleBlock(node) {
 }
 
 /**
- * SQL is not prose either, and for the same reason CSS is not: `now() - make_interval(...)` and `extract(epoch from a - lag(b))` need the spaces around the operator,
+ * SQL is not prose either, and for the same reason CSS is not: date subtraction in a query needs spaces around its minus operator,
  */
 
 /*
@@ -59,7 +63,7 @@ function taggedWithCode(node) {
 }
 
 /**
- * A markdown bullet is not punctuation. ` - item` at the head of a comment or template line is a list, and rewording it deletes the list.
+ * A markdown bullet is not punctuation. a hyphen then a space at the head of a comment or template line is a list, and rewording it deletes the list.
  */
 
 /*
@@ -175,7 +179,7 @@ export default {
       JSONLiteral(node) {
         checkString(node.value, node);
       },
-      // Svelte template body text — e.g. <p>Some prose with a dash</p>.
+      // Svelte template body text, e.g. <p>Some prose with a dash</p>.
       // Without this, em/en dashes inside .svelte HTML text slip through
       // because the rule otherwise only inspects JS string nodes.
       SvelteText(node) {
@@ -183,7 +187,7 @@ export default {
         if (PLACEHOLDER.test(node.value.trim())) return;
         checkValue(node.value, node);
       },
-      // Svelte attribute literal values — e.g. title="Heads up important".
+      // Svelte attribute literal values, e.g. title="Heads up important".
       SvelteLiteral(node) {
         if (isInsideStyleBlock(node)) return;
         if (PLACEHOLDER.test(String(node.value).trim())) return;

package/rules/no-hardcoded-models.js

@@ -14,7 +14,7 @@ const MODEL_PATTERNS = [
 
 const COMBINED = new RegExp(MODEL_PATTERNS.map((r) => r.source).join("|"));
 
-// WHY: files inside @heyramzi/ai define the canonical model catalog — they must use literal IDs.
+// WHY: files inside @heyramzi/ai define the canonical model catalog, so they must use literal IDs.
 function isModelCatalogFile(filename) {
   return /packages\/ai\//.test(filename);
 }

package/rules/no-overkill-fallback.js

@@ -24,7 +24,7 @@ export default {
       LogicalExpression(node) {
         if (node.operator !== "??") return;
 
-        // Rule 1: Triple ?? chain — a ?? b ?? c
+        // Rule 1: Triple ?? chain, a ?? b ?? c
         // The outermost ?? has left side that is also a ??
         if (node.left.type === "LogicalExpression" && node.left.operator === "??") {
           context.report({ node, messageId: "tripleChain" });

package/slop-views.js

@@ -0,0 +1,122 @@
+/**
+ * The views of a draft the audit judges: code, comments, strings and FAQ sections cut away, and the names its prose keeps.
+ */
+
+/**
+ * Code is not copy.
+ */
+
+/*
+ * `#if`, `#endif`, `#Preview` and `#available` are Swift, `#newlink` is a DOM id and `#sales-alerts` is a Slack channel: all 16 hashtag hits in the first corpus run were one of
+ */
+
+/*
+ * those, every one of them inside a fence or a backtick span. Only the hashtag check reads this, because only it is confused by code.
+ */
+export function withoutCode(text) {
+  return (
+    text
+      .replace(/```[\s\S]*?```/g, " ")
+      .replace(/`[^`\n]*`/g, " ")
+      // A tool directive is syntax the file has to carry: `/* eslint-enable ... */`
+      // failed the banned word `enable` in 2 files that contain no copy at all.
+      .replace(
+        /\/[*/][ \t]*(?:eslint|oxlint|prettier|biome|c8|v8|ts-(?:ignore|expect-error|nocheck))[^\n]*/g,
+        " ",
+      )
+  );
+}
+
+/** A source file with its comments blanked. Comments are notes to the next developer, not copy; `comment-length` gates them. */
+export function withoutComments(text) {
+  return text
+    .replace(/\/\*[\s\S]*?\*\//g, (m) => m.replace(/[^\n]/g, " "))
+    .replace(/(^|[\s;{}(,])\/\/[^\n]*/g, "$1");
+}
+
+/**
+ * The copy inside a source file: string literals of 2+ words, comments gone. The ledger read `get
+ * ours()` and `winner: "ours"` as first person: 24 false hits.
+ */
+export function copyStrings(text) {
+  const literals =
+    withoutComments(text).match(/"(?:[^"\\\n]|\\.)*"|`(?:[^`\\]|\\.)*`|'(?:[^'\\\n]|\\.)*'/g) ?? [];
+  return literals.filter((s) => /\p{L}{2,}\s+\p{L}{2,}/u.test(s)).join("\n");
+}
+
+/**
+ * A string literal's first sentence is sentence-initial, and every anchored pattern used to miss it.
+ */
+
+/*
+ * The anchors in `patterns` accept `^`, a newline, or `.!?` plus a space.
+ */
+
+/*
+ * In a copy module the sentence opens after `key: "`, which is none of the three, so `nobody-opener` and the 4 other anchored patterns never fired on the first sentence
+ */
+
+/*
+ * of any string. sitesmiths keeps every word of the page in `copy.ts`, ran the gate as its AGENTS.md tells it to, and shipped "Nobody
+ */
+
+/*
+ * pays us to start." past a pattern that had been lint-enforced since 5 Sep 2026. Opening the quote into a line break makes the anchors true again.
+ */
+
+/*
+ * Only for files the caller says are source, because in markdown `he said, "Nobody knows."` is a quotation rather than a key, and it stays one.
+ */
+export function withStringStarts(text) {
+  return text.replace(/([:(,=[]\s*)["'`]/g, "$1\n");
+}
+
+/**
+ * An FAQ is the one section where a question heading is the right heading: it is written to match the query the reader typed,
+ */
+
+/*
+ * and it holds its own subheadings by design. `question-heading` and `empty-section` skip it rather than asking a page to hide the question it answers.
+ */
+export const FAQ_TITLE =
+  /^(?:FAQ|F\.A\.Q\.?|Frequently asked questions|よくある質問|Questions? fréquentes?)$/i;
+
+export function withoutFaqSections(text) {
+  let depth = 0;
+  return text
+    .split("\n")
+    .map((line) => {
+      const heading = /^(#{1,6})[ \t]+(.+?)[ \t]*#*$/.exec(line);
+      if (heading) {
+        const level = heading[1].length;
+        if (depth && level <= depth) depth = 0;
+        if (!depth && FAQ_TITLE.test(heading[2].trim())) depth = level;
+      }
+      return depth ? "" : line;
+    })
+    .join("\n");
+}
+
+/**
+ * The words the body capitalises mid-sentence, which are names rather than styling.
+ */
+
+/*
+ * Without it every review page naming 2 products failed `title-case-heading`: "What Wispr Flow does better than Wavenote" counts 2 capitals and none of them is a styling choice.
+ */
+
+/*
+ * Headings are excluded from the scan so a title-cased heading cannot vouch for itself.
+ */
+export function properNouns(body) {
+  const found = new Set();
+  for (const line of body.replace(/^#{1,6}[ \t]+.*$/gm, " ").split("\n")) {
+    let start = true;
+    for (const raw of line.trim().split(/\s+/)) {
+      const word = raw.replace(/^[^\p{L}]+/u, "").replace(/[^\p{L}\p{N}'’-]+$/u, "");
+      if (word && !start && /^[A-ZÀ-Þ]/.test(word)) found.add(word);
+      start = /[.!?:][""''”’)\]]?$/.test(raw);
+    }
+  }
+  return found;
+}

package/slop.js

@@ -48,6 +48,15 @@ import {
 } from "#lint/data/invisibles.js";
 import { artifacts, patterns, phrases, words } from "#lint/data/slop-words.js";
 import { auditRegister } from "#lint/register.js";
+import {
+  FAQ_TITLE,
+  copyStrings,
+  properNouns,
+  withStringStarts,
+  withoutCode,
+  withoutComments,
+  withoutFaqSections,
+} from "#lint/slop-views.js";
 import { auditShorts } from "@heyramzi/social/shorts";
 
 const escapeRe = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
@@ -79,7 +88,7 @@ export function stripMarkup(raw) {
       .replace(/<(script|style)[\s\S]*?<\/\1>/gi, " ")
       // A markup comment is a note to the next developer, not copy.
 
-      // It goes with the script block, for the same reason: the CBAM pages rule their sections with `<!-- ————— name ————— -->`, which reported 8 em dashes nobody reads.
+      // It goes with the script block, for the same reason: the CBAM pages rule their sections with `<!-- name -->` wrapped in rows of em dashes, which reported 8 dashes nobody reads.
       .replace(/<!--[\s\S]*?-->/g, " ")
       .replace(/&[a-z]+;|&#\d+;/gi, " ")
       .replace(/<[^>]+>/g, " ")
@@ -213,125 +222,6 @@ const EMOJI =
  */
 const HASHTAG = /(?<=[^\n#]\s|^\s*\S[^\n]*?\s)#(?![0-9a-fA-F]{3,8}\b)(?!\d+\b)[A-Za-z][\w-]*/g;
 
-/**
- * Code is not copy.
- */
-
-/*
- * `#if`, `#endif`, `#Preview` and `#available` are Swift, `#newlink` is a DOM id and `#sales-alerts` is a Slack channel: all 16 hashtag hits in the first corpus run were one of
- */
-
-/*
- * those, every one of them inside a fence or a backtick span. Only the hashtag check reads this, because only it is confused by code.
- */
-function withoutCode(text) {
-  return (
-    text
-      .replace(/```[\s\S]*?```/g, " ")
-      .replace(/`[^`\n]*`/g, " ")
-      // A tool directive is syntax the file has to carry: `/* eslint-enable ... */`
-      // failed the banned word `enable` in 2 files that contain no copy at all.
-      .replace(
-        /\/[*/][ \t]*(?:eslint|oxlint|prettier|biome|c8|v8|ts-(?:ignore|expect-error|nocheck))[^\n]*/g,
-        " ",
-      )
-  );
-}
-
-/** A source file with its comments blanked. Comments are notes to the next developer, not copy; `comment-length` gates them. */
-function withoutComments(text) {
-  return text
-    .replace(/\/\*[\s\S]*?\*\//g, (m) => m.replace(/[^\n]/g, " "))
-    .replace(/(^|[\s;{}(,])\/\/[^\n]*/g, "$1");
-}
-
-/**
- * The copy inside a source file: string literals of 2+ words, comments gone. The ledger read `get
- * ours()` and `winner: "ours"` as first person: 24 false hits.
- */
-function copyStrings(text) {
-  const literals =
-    withoutComments(text).match(/"(?:[^"\\\n]|\\.)*"|`(?:[^`\\]|\\.)*`|'(?:[^'\\\n]|\\.)*'/g) ?? [];
-  return literals.filter((s) => /\p{L}{2,}\s+\p{L}{2,}/u.test(s)).join("\n");
-}
-
-/**
- * A string literal's first sentence is sentence-initial, and every anchored pattern used to miss it.
- */
-
-/*
- * The anchors in `patterns` accept `^`, a newline, or `.!?` plus a space.
- */
-
-/*
- * In a copy module the sentence opens after `key: "`, which is none of the three, so `nobody-opener` and the 4 other anchored patterns never fired on the first sentence
- */
-
-/*
- * of any string. sitesmiths keeps every word of the page in `copy.ts`, ran the gate as its AGENTS.md tells it to, and shipped "Nobody
- */
-
-/*
- * pays us to start." past a pattern that had been lint-enforced since 5 Sep 2026. Opening the quote into a line break makes the anchors true again.
- */
-
-/*
- * Only for files the caller says are source, because in markdown `he said, "Nobody knows."` is a quotation rather than a key, and it stays one.
- */
-function withStringStarts(text) {
-  return text.replace(/([:(,=[]\s*)["'`]/g, "$1\n");
-}
-
-/**
- * An FAQ is the one section where a question heading is the right heading: it is written to match the query the reader typed,
- */
-
-/*
- * and it holds its own subheadings by design. `question-heading` and `empty-section` skip it rather than asking a page to hide the question it answers.
- */
-const FAQ_TITLE =
-  /^(?:FAQ|F\.A\.Q\.?|Frequently asked questions|よくある質問|Questions? fréquentes?)$/i;
-
-function withoutFaqSections(text) {
-  let depth = 0;
-  return text
-    .split("\n")
-    .map((line) => {
-      const heading = /^(#{1,6})[ \t]+(.+?)[ \t]*#*$/.exec(line);
-      if (heading) {
-        const level = heading[1].length;
-        if (depth && level <= depth) depth = 0;
-        if (!depth && FAQ_TITLE.test(heading[2].trim())) depth = level;
-      }
-      return depth ? "" : line;
-    })
-    .join("\n");
-}
-
-/**
- * The words the body capitalises mid-sentence, which are names rather than styling.
- */
-
-/*
- * Without it every review page naming 2 products failed `title-case-heading`: "What Wispr Flow does better than Wavenote" counts 2 capitals and none of them is a styling choice.
- */
-
-/*
- * Headings are excluded from the scan so a title-cased heading cannot vouch for itself.
- */
-function properNouns(body) {
-  const found = new Set();
-  for (const line of body.replace(/^#{1,6}[ \t]+.*$/gm, " ").split("\n")) {
-    let start = true;
-    for (const raw of line.trim().split(/\s+/)) {
-      const word = raw.replace(/^[^\p{L}]+/u, "").replace(/[^\p{L}\p{N}'’-]+$/u, "");
-      if (word && !start && /^[A-ZÀ-Þ]/.test(word)) found.add(word);
-      start = /[.!?:][""''”’)\]]?$/.test(raw);
-    }
-  }
-  return found;
-}
-
 /**
  * Which language the draft is in, so the checks that only work in one of them pick the right list.
  */

package/ui-rules.js

@@ -158,7 +158,7 @@ function cssRules(source, file) {
         hits.push({ rule: "raw-hover", line, snippet: rule.selector.slice(0, 80) });
       }
       const selectors = rule.selector.split(",").map((s) => s.trim());
-      const own = selectors.some((s) => /(^|[\s>+~])\.t-acc-panel$/.test(s) || /(^|[\s>+~])\.t-acc-panel(?=[:.\[])/.test(s));
+      const own = selectors.some((s) => /(^|[\s>+~])\.t-acc-panel$/.test(s) || /(^|[\s>+~])\.t-acc-panel(?=[:.[])/.test(s));
       if (own && /(?<![\w-])padding(?:-[a-z]+)*\s*:/.test(rule.body) && !selectors.some((s) => /t-acc-panel-/.test(s))) {
         hits.push({ rule: "acc-panel-padding", line, snippet: `${rule.selector} { padding }` });
       }