@wix/origami-lint 1.34.19 → 1.34.21

package/dist/bi/read-stats.js

@@ -8,7 +8,10 @@ export function readStats(file, run) {
         const validKind = (kind) => kind === 'page' || kind === 'component';
         const count = (value) => typeof value === 'number' && Number.isFinite(value) && value >= 0;
         if (!Array.isArray(stats.rules) || !Array.isArray(stats.models)
-            || !stats.rules.every(s => s && ruleRegistry.some(rule => rule.name === s.rule) && validPage(s.page) && validKind(s.kind) && Number.isSafeInteger(s.violations) && s.violations >= 0 && count(s.duration_ms))
+            || !stats.rules.every(s => s && ruleRegistry.some(rule => rule.name === s.rule && s.checks && typeof s.checks === 'object' && !Array.isArray(s.checks)
+                && Object.keys(s.checks).length === rule.checks.length && rule.checks.every(check => Object.hasOwn(s.checks, check))
+                && Object.values(s.checks).every(value => Number.isSafeInteger(value) && value >= 0)
+                && Object.values(s.checks).reduce((total, value) => total + value, 0) === s.violations) && validPage(s.page) && validKind(s.kind) && Number.isSafeInteger(s.violations) && s.violations >= 0 && count(s.duration_ms))
             || !stats.models.every(s => s && validPage(s.page) && validKind(s.kind) && count(s.duration_ms) && count(s.components) && count(s.css_files) && (s.error_name === undefined || s.error_name === 'css_syntax_error' || s.error_name === 'unknown_error')))
             return;
         for (const model of stats.models) {
@@ -18,9 +21,13 @@ export function readStats(file, run) {
             });
         }
         for (const statsRow of stats.rules) {
-            run.recordRow('lint_rule', statsRow.rule, { item_type: statsRow.kind, item_id: statsRow.page, step_details: { violations: statsRow.violations } }, {
+            const row = run.recordRow('lint_rule', statsRow.rule, { item_type: statsRow.kind, item_id: statsRow.page, step_details: { violations: statsRow.violations } }, {
                 success: statsRow.violations === 0, duration_ms: statsRow.duration_ms,
             });
+            for (const [check, violations] of Object.entries(statsRow.checks)) {
+                if (violations > 0)
+                    run.recordRow('lint_rule', statsRow.rule, { item_type: statsRow.kind, item_id: statsRow.page, step_details: { check, violations } }, { success: false, duration_ms: 0 }, row);
+            }
         }
     }
     catch { /* Missing, truncated or malformed stats never affect lint. */ }

package/dist/bi/stats.js

@@ -18,7 +18,7 @@ export function pageId(filename, text, program) {
     return page;
 }
 /** Called once per rule and file by the adapter. Passing rules retain their zero count. */
-export function startRuleStats(rule, filename, text, program) {
+export function startRuleStats(rule, checks, filename, text, program) {
     const destination = process.env.ORIGAMI_LINT_STATS;
     if (!destination)
         return;
@@ -33,7 +33,8 @@ export function startRuleStats(rule, filename, text, program) {
             });
         }
         const kind = filename.startsWith(`${GLOBAL_COMPONENTS_FOLDER}/`) ? 'component' : 'page';
-        const stats = { rule, kind, page: pageId(filename, text, program), violations: 0, duration_ms: 0 };
+        const page = pageId(filename, text, program);
+        const stats = { rule, checks: Object.fromEntries(checks.map(check => [check, 0])), kind, page, violations: 0, duration_ms: 0 };
         rules.push(stats);
         return stats;
     }

package/dist/project-rules-runner/run-project-rules.js

@@ -9,7 +9,8 @@ export function runProjectRules(files, { rules = projectRuleRegistry, readFile =
     log(`${rules.length} project rules to run`);
     let passed = true;
     for (const rule of rules) {
-        const row = parentRow?.run.openRow('lint_rule', rule.name, { item_type: 'site', step_details: { scope: 'project', severity: rule.severity ?? 'error' } });
+        const started = performance.now();
+        const details = { scope: 'project', severity: rule.severity ?? 'error' };
         let problems;
         try {
             problems = rule.check(createProjectModel(rule.name, files, readFile));
@@ -17,12 +18,17 @@ export function runProjectRules(files, { rules = projectRuleRegistry, readFile =
         catch (error) {
             // A rule that couldn't finish (a file it can't parse, a path out of the site) fails the run, whatever its severity.
             print(formatProblem({ name: rule.name }, { file: '.', message: `the rule couldn't finish, so this run can't pass: ${error instanceof Error ? error.message : String(error)}` }));
-            row?.end(false, { error_name: 'rule_crashed' });
+            parentRow?.run.recordRow('lint_rule', rule.name, { item_type: 'site', step_details: details }, { success: false, failure: { error_name: 'rule_crashed' }, duration_ms: performance.now() - started });
             passed = false;
             continue;
         }
-        row?.update({ step_details: { violations: problems.length } });
-        row?.end(problems.length === 0);
+        const duration = performance.now() - started;
+        const row = parentRow?.run.recordRow('lint_rule', rule.name, { item_type: 'site', step_details: { ...details, violations: problems.length } }, { success: problems.length === 0, duration_ms: duration });
+        for (const check of rule.checks) {
+            const violations = problems.filter(problem => problem.check === check).length;
+            if (violations > 0 && row)
+                row.run.recordRow('lint_rule', rule.name, { item_type: 'site', step_details: { ...details, check, violations } }, { success: false, duration_ms: 0 }, row);
+        }
         for (const problem of problems) {
             print(formatProblem(rule, problem));
             if ((rule.severity ?? 'error') === 'error')

package/dist/rules/index.js

@@ -13,5 +13,6 @@ import slotAttributes from "./site-app/slot-attributes.js";
 import themeFontIds from "./site-app/theme-font-ids.js";
 import themeRoot from "./site-app/theme-root.js";
 import noFreeCode from "./site-app/no-free-code.js";
+// Execution registries combine already-checked rules; only this boundary erases their individual key unions.
 export const ruleRegistry = [allowedImports, noFreeCode, pageStructure, noAbsoluteSection, themeFontIds, slotAttributes, themeRoot, breakpoints, cssFileNesting, globalStyles, applyAndPresets, bindings, dataValues, cssLayoutProperties, cssSelectors];
 export { defineRule } from "./site-app/define-rule.js";

package/dist/rules/project/home-page-folder.js

@@ -4,6 +4,7 @@
 import { findPages } from "../../page-model/index.js";
 import { PAGES_FOLDER, POPUPS_FOLDER } from "../../shared/site-files-tree-mapping.js";
 import { defineProjectRule } from "./define-project-rule.js";
+const CHECKS = ['missing_home', 'multiple_home_folders', 'nested_home_folder', 'popup_in_parentheses'];
 const EXPECTED_ONE_HOME = `Expected exactly one home page folder "pages/(<slug>)" in ${PAGES_FOLDER}`;
 export const NO_HOME = `${EXPECTED_ONE_HOME}; found 0. Name the home page's folder in parentheses, directly under ${PAGES_FOLDER}, like ${PAGES_FOLDER}/(home) (rule P2-R01; tracker OUR-035). The import stops without it.`;
 export const moreHomes = (count, folder) => `${EXPECTED_ONE_HOME}; found ${count}. Keep the parentheses on the home page's folder only: rename ${folder} to ${withoutParentheses(folder)} (rule P2-R01; tracker OUR-035).`;
@@ -12,6 +13,7 @@ export const onlyHomeInParentheses = (folder) => `Only the home page is written
 const HOME_FOLDER = /^\(.+\)$/;
 export default defineProjectRule({
     name: 'home-page-folder',
+    checks: CHECKS,
     editors: ['studio2'],
     check: (project) => [...checkOneHome(project), ...checkNoNestedHome(project), ...checkNoPopupInParentheses(project)],
 });
@@ -20,14 +22,14 @@ export default defineProjectRule({
 function checkOneHome(project) {
     const homes = getPages(project).filter((page) => page.folders.length === 1 && HOME_FOLDER.test(page.folders[0]));
     if (homes.length === 0)
-        return [{ file: PAGES_FOLDER, message: NO_HOME }];
-    return homes.slice(1).map((home) => ({ file: home.jsxFile, message: moreHomes(homes.length, home.folders[0]) }));
+        return [{ check: 'missing_home', file: PAGES_FOLDER, message: NO_HOME }];
+    return homes.slice(1).map((home) => ({ check: 'multiple_home_folders', file: home.jsxFile, message: moreHomes(homes.length, home.folders[0]) }));
 }
 /** Reports every page with a folder in parentheses below the top of pages/ on its path, as import-site refuses them. */
 function checkNoNestedHome(project) {
     return getPages(project).flatMap((page) => {
         const nested = page.folders.slice(1).find((folder) => HOME_FOLDER.test(folder));
-        return nested ? [{ file: page.jsxFile, message: onlyHomeInParentheses(nested) }] : [];
+        return nested ? [{ check: 'nested_home_folder', file: page.jsxFile, message: onlyHomeInParentheses(nested) }] : [];
     });
 }
 /** Reports each folder in parentheses under popups/, once, as the converter refuses them. */
@@ -39,7 +41,7 @@ function checkNoPopupInParentheses(project) {
         if (index !== -1)
             folders.add(`${POPUPS_FOLDER}/${segments.slice(0, index + 1).join('/')}`);
     }
-    return [...folders].sort().map((folder) => ({ file: folder, message: onlyHomeInParentheses(folder.slice(folder.lastIndexOf('/') + 1)) }));
+    return [...folders].sort().map((folder) => ({ check: 'popup_in_parentheses', file: folder, message: onlyHomeInParentheses(folder.slice(folder.lastIndexOf('/') + 1)) }));
 }
 /** The site's pages under pages/, sorted by path. */
 function getPages(project) {

package/dist/rules/project/index.js

@@ -1,3 +1,4 @@
 import homePageFolder from "./home-page-folder.js";
 export { defineProjectRule } from "./define-project-rule.js";
+// Execution registries combine already-checked rules; only this boundary erases their individual key unions.
 export const projectRuleRegistry = [homePageFolder];

package/dist/rules/site-app/allowed-imports.js

@@ -6,11 +6,13 @@ import { CSS_FILES, JSX_FILES } from "../../page-model/index.js";
 import { BUILDERS, RUNTIME_ONLY_BUILDERS, RUNTIME_PACKAGE } from "../../rules-api/index.js";
 import { GLOBAL_COMPONENTS, GLOBAL_COMPONENTS_FOLDER, GLOBAL_FOLDER, SITE_SRC } from "../../shared/site-files-tree-mapping.js";
 import { defineRule } from "./define-rule.js";
+const CHECKS = ['disallowed_import_source', 'interaction_import', 'missing_global_import_name', 'invalid_global_import_name', 'default_builder_import', 'aliased_builder_import', 'wrong_builder_source'];
 /** The interactions packages, which the import reserves: structure.jsx imports nothing from them. Matched by source, not by name: `Interact` from @site/site-code is the site's own. */
 const INTERACTIONS_PACKAGES = ['interactReact', 'interact', 'motionPresets'];
 const SOURCE_HINT = `Only packages, files in ${path.posix.basename(GLOBAL_FOLDER)}/, and ${CSS_FILES.map((file) => `'./${file}'`).join(' or ')} can be.`;
 export default defineRule({
     name: 'allowed-imports',
+    checks: CHECKS,
     editors: 'all',
     callbacks: {
         ImportDeclaration: (input) => {
@@ -28,7 +30,7 @@ function hasAllowedSource({ node, rulesApi, report }) {
     const source = node.source.value;
     if (isAllowedSource(source, rulesApi.page.jsxFile))
         return true;
-    report({ loc: node.loc, message: `\`${source}\` can't be imported. ${SOURCE_HINT}` });
+    report({ check: 'disallowed_import_source', loc: node.loc, message: `\`${source}\` can't be imported. ${SOURCE_HINT}` });
     return false;
 }
 /** Reports an import from @wix/interact or @wix/motion-presets; an `import type` is erased, so it passes. */
@@ -37,7 +39,7 @@ function isNotInteractionsPackage({ node, rulesApi, report }) {
     const kind = rulesApi.getImportKind(source);
     if (!kind || !INTERACTIONS_PACKAGES.includes(kind) || (node.specifiers.length > 0 && getValueSpecifiers(node).length === 0))
         return true;
-    report({ loc: node.loc, message: `\`${source}\` can't be imported: a page's interactions go in its servicesConfigs/interactions.ts, with the element's id as \`key\`. Remove the import and any <Interaction> tag around an element (keep the element).` });
+    report({ check: 'interaction_import', loc: node.loc, message: `\`${source}\` can't be imported: a page's interactions go in its servicesConfigs/interactions.ts, with the element's id as \`key\`. Remove the import and any <Interaction> tag around an element (keep the element).` });
     return false;
 }
 /** An import that reaches global/components must be `import <Folder> from '…/global/components/<Folder>'`, as the converter reads it. */
@@ -48,13 +50,13 @@ function checkGlobalComponentImport({ node, rulesApi, report }) {
     if (!folder)
         return;
     if (node.specifiers.length === 0) {
-        report({ loc: node.loc, message: `${source} is imported without a name. Import a global component under its folder's name: \`import ${folder} from '${getGlobalComponentImportPath(folder, jsxFile)}'\`, and write \`<${folder}>\`.` });
+        report({ check: 'missing_global_import_name', loc: node.loc, message: `${source} is imported without a name. Import a global component under its folder's name: \`import ${folder} from '${getGlobalComponentImportPath(folder, jsxFile)}'\`, and write \`<${folder}>\`.` });
     }
     for (const specifier of getValueSpecifiers(node)) {
         const name = specifier.type === 'ImportNamespaceSpecifier' ? '*' : specifier.local.name;
         if (withoutLeadingDots(source) === `${GLOBAL_COMPONENTS}/${name}`)
             continue;
-        report({
+        report({ check: 'invalid_global_import_name',
             loc: specifier.loc,
             message: `\`${specifier.local.name}\` is imported from ${source}. Import a global component under its folder's name: \`import ${folder} from '${getGlobalComponentImportPath(folder, jsxFile)}'\`, and write \`<${folder}>\`.`,
         });
@@ -67,14 +69,14 @@ function checkBuilderImports({ node, rulesApi, report }) {
         const importedName = rulesApi.getImportedName(specifier);
         if (source === RUNTIME_PACKAGE) {
             if (importedName === 'default') {
-                report({ loc: specifier.loc, message: `\`${localName}\` is a default import of ${RUNTIME_PACKAGE}. Import each builder by name: \`import { ctx } from '${RUNTIME_PACKAGE}'\`.` });
+                report({ check: 'default_builder_import', loc: specifier.loc, message: `\`${localName}\` is a default import of ${RUNTIME_PACKAGE}. Import each builder by name: \`import { ctx } from '${RUNTIME_PACKAGE}'\`.` });
             }
             else if (BUILDERS.includes(importedName) && localName !== importedName) {
-                report({ loc: specifier.loc, message: `The builder \`${importedName}\` is imported as \`${localName}\`. Import it under its own name: \`import { ${importedName} } from '${RUNTIME_PACKAGE}'\`, and call it as \`${importedName}(…)\`.` });
+                report({ check: 'aliased_builder_import', loc: specifier.loc, message: `The builder \`${importedName}\` is imported as \`${localName}\`. Import it under its own name: \`import { ${importedName} } from '${RUNTIME_PACKAGE}'\`, and call it as \`${importedName}(…)\`.` });
             }
         }
         else if (RUNTIME_ONLY_BUILDERS.includes(localName)) {
-            report({ loc: specifier.loc, message: `\`${localName}\` is imported from ${source}, but \`${localName}\` is a builder of ${RUNTIME_PACKAGE}. Import it from there: \`import { ${localName} } from '${RUNTIME_PACKAGE}'\`.` });
+            report({ check: 'wrong_builder_source', loc: specifier.loc, message: `\`${localName}\` is imported from ${source}, but \`${localName}\` is a builder of ${RUNTIME_PACKAGE}. Import it from there: \`import { ${localName} } from '${RUNTIME_PACKAGE}'\`.` });
         }
     }
 }

package/dist/rules/site-app/apply-and-presets.js

@@ -1,7 +1,9 @@
 import { createCssFileReporter, describeAtRule, getCssChildren, getDashedId, getMixinId, getSegmentClass, getSelectorSegments, isOneEditAway, isPresetProperty, isSafeCssIdentifier } from "../../rules-api/index.js";
 import { defineRule } from "./define-rule.js";
+const CHECKS = ['apply_with_declarations', 'nested_part_preset', 'duplicate_apply', 'invalid_apply_target', 'missing_apply_target', 'invalid_preset_name', 'missing_preset_value', 'duplicate_default_style', 'duplicate_preset', 'invalid_apply_form'];
 export default defineRule({
     name: 'apply-and-presets',
+    checks: CHECKS,
     editors: ['studio2'],
     callbacks: {
         Program: ({ rulesApi, report }) => {
@@ -57,7 +59,7 @@ function checkElementRule(node, breakpoint, media, reportInFile) {
     const declarations = children.filter((child) => child.type === 'decl');
     const values = declarations.filter((declaration) => !isPresetProperty(declaration.prop));
     if (apply && values.length > 0) {
-        reportInFile(values[0], `${where}: \`${describeAtRule(apply)}\` can't share a rule with style declarations (${describeProperties(values)}). Put them in the global style (\`@mixin ${apply.params.trim()}\` in theme.css), or under another breakpoint or state; only \`--preset…\` may stay beside the \`@apply\` (tracker CONV-astParser.ts:117).`);
+        reportInFile(values[0], 'apply_with_declarations', `${where}: \`${describeAtRule(apply)}\` can't share a rule with style declarations (${describeProperties(values)}). Put them in the global style (\`@mixin ${apply.params.trim()}\` in theme.css), or under another breakpoint or state; only \`--preset…\` may stay beside the \`@apply\` (tracker CONV-astParser.ts:117).`);
         return null;
     }
     return { node, selector: node.selector, breakpoint, media, apply, presets: declarations.filter((declaration) => isPresetProperty(declaration.prop)), values };
@@ -81,7 +83,7 @@ function checkMixinRule(node, mixin, reportInFile) {
         return;
     const declarations = children.filter((child) => child.type === 'decl');
     if (declarations.length > 0)
-        return reportInFile(declarations[0], `${where}: \`${describeAtRule(apply)}\` can't share a rule with declarations (${describeProperties(declarations)}). Move them into \`@mixin ${apply.params.trim()}\` (tracker CONV-astParser.ts:117).`);
+        return reportInFile(declarations[0], 'apply_with_declarations', `${where}: \`${describeAtRule(apply)}\` can't share a rule with declarations (${describeProperties(declarations)}). Move them into \`@mixin ${apply.params.trim()}\` (tracker CONV-astParser.ts:117).`);
     if (!isApplyWritten(apply))
         reportApplyForm(apply, where, reportInFile);
 }
@@ -91,7 +93,7 @@ function checkSelectorRules(group, componentClasses, reportInFile) {
     const picks = group.flatMap((rule) => rule.presets.flatMap((declaration) => checkPresetName(declaration, where, reportInFile)));
     if (isInnerPartRule(group[0].selector, componentClasses)) {
         for (const { declaration } of picks.filter(({ path }) => path.length > 0)) {
-            reportInFile(declaration, `${where}: \`${declaration.prop}\` can't be set on an inner part's rule, which picks only the part's own preset (\`--preset\`). Write it on the component's own rule, with the part's whole path: \`--preset-<part>-${declaration.prop.slice('--preset-'.length)}\` (tracker CONV-styleParser.ts:583).`);
+            reportInFile(declaration, 'nested_part_preset', `${where}: \`${declaration.prop}\` can't be set on an inner part's rule, which picks only the part's own preset (\`--preset\`). Write it on the component's own rule, with the part's whole path: \`--preset-<part>-${declaration.prop.slice('--preset-'.length)}\` (tracker CONV-styleParser.ts:583).`);
         }
     }
     for (const rules of groupByBreakpoint(group)) {
@@ -100,12 +102,12 @@ function checkSelectorRules(group, componentClasses, reportInFile) {
         const breakpointDescription = rules[0].media ? `in \`${describeAtRule(rules[0].media)}\`` : 'outside any @media';
         const applies = rules.filter((rule) => rule.apply);
         for (const rule of applies.slice(1)) {
-            reportInFile(rule.apply, `${where}: two \`@apply\` for this selector ${breakpointDescription} (\`${describeAtRule(applies[0].apply)}\` on line ${getLine(applies[0].apply)}, \`${describeAtRule(rule.apply)}\`): keep one (tracker CONV-styleParser.ts:179).`);
+            reportInFile(rule.apply, 'duplicate_apply', `${where}: two \`@apply\` for this selector ${breakpointDescription} (\`${describeAtRule(applies[0].apply)}\` on line ${getLine(applies[0].apply)}, \`${describeAtRule(rule.apply)}\`): keep one (tracker CONV-styleParser.ts:179).`);
         }
         const ruleWithValues = rules.find((rule) => rule.values.length > 0);
         if (applies.length > 0 && ruleWithValues) {
             const { apply } = applies[0];
-            reportInFile(ruleWithValues.values[0], `${where} has both \`${describeAtRule(apply)}\` (line ${getLine(apply)}) and style declarations ${breakpointDescription}: keep one, or move the declarations into the global style (\`@mixin ${apply.params.trim()}\` in theme.css) (tracker CONV-styleParser.ts:204).`);
+            reportInFile(ruleWithValues.values[0], 'apply_with_declarations', `${where} has both \`${describeAtRule(apply)}\` (line ${getLine(apply)}) and style declarations ${breakpointDescription}: keep one, or move the declarations into the global style (\`@mixin ${apply.params.trim()}\` in theme.css) (tracker CONV-styleParser.ts:204).`);
         }
     }
 }
@@ -121,12 +123,12 @@ function checkApplyTargets(elementRules, declaredIds, kind, reportInFile) {
         // the build strips a trailing `()`; the import doesn't, and reads `btn()` as the id (astParser.ts MIXIN_ID_PATTERN)
         const called = id.endsWith('()') ? id.slice(0, -2) : null;
         if (called !== null && declaredIds.has(called)) {
-            reportInFile(apply, `rule \`${selector.trim()}\`: \`@apply --${id}\` reads as the global style \`${id}\`, which the site doesn't have: write \`@apply --${called}\`, without the parentheses (tracker OUR-006).`);
+            reportInFile(apply, 'invalid_apply_target', `rule \`${selector.trim()}\`: \`@apply --${id}\` reads as the global style \`${id}\`, which the site doesn't have: write \`@apply --${called}\`, without the parentheses (tracker OUR-006).`);
             continue;
         }
         const nearMiss = [...declaredIds].find((declared) => isOneEditAway(id, declared));
         const fix = nearMiss ? `Did you mean \`@apply --${nearMiss}\`? Otherwise apply` : 'Apply';
-        reportInFile(apply, `rule \`${selector.trim()}\`: \`@apply --${id}\` names no global style: no \`@mixin --${id}\` in theme.css${kind === 'style' ? ' or style.css' : ''}. ${fix} one theme.css declares, or declare \`@mixin --${id}\` in theme.css (tracker OUR-006).`);
+        reportInFile(apply, 'missing_apply_target', `rule \`${selector.trim()}\`: \`@apply --${id}\` names no global style: no \`@mixin --${id}\` in theme.css${kind === 'style' ? ' or style.css' : ''}. ${fix} one theme.css declares, or declare \`@mixin --${id}\` in theme.css (tracker OUR-006).`);
     }
 }
 /** A `--preset-<path>` names each part as a CSS identifier (CONV-styleParser.ts:62); returns the pick, or none after reporting. */
@@ -136,7 +138,7 @@ function checkPresetName(declaration, where, reportInFile) {
     const path = decodePresetPath(declaration.prop.slice('--preset-'.length));
     if (path.every(isSafeCssIdentifier))
         return [{ declaration, path }];
-    reportInFile(declaration, `${where}: \`${declaration.prop}\` doesn't name an inner part: write \`--preset-<part>\`, each part a CSS identifier, parts joined by \`-\` and a \`-\` inside a part's name written \`--\` (\`close-button\` → \`--preset-close--button\`) (tracker CONV-styleParser.ts:62).`);
+    reportInFile(declaration, 'invalid_preset_name', `${where}: \`${declaration.prop}\` doesn't name an inner part: write \`--preset-<part>\`, each part a CSS identifier, parts joined by \`-\` and a \`-\` inside a part's name written \`--\` (\`close-button\` → \`--preset-close--button\`) (tracker CONV-styleParser.ts:62).`);
     return [];
 }
 /** The last pick of each path at one breakpoint is the one the import reads: it has a value (CONV-styleParser.ts:88). */
@@ -148,7 +150,7 @@ function checkPresetValues(picks, where, reportInFile) {
             continue;
         seen.add(key);
         if (!declaration.value.trim())
-            reportInFile(declaration, `${where}: \`${declaration.prop}\` has no value. Write a preset name, or remove the declaration to use no preset (tracker CONV-styleParser.ts:88).`);
+            reportInFile(declaration, 'missing_preset_value', `${where}: \`${declaration.prop}\` has no value. Write a preset name, or remove the declaration to use no preset (tracker CONV-styleParser.ts:88).`);
     }
 }
 /**
@@ -168,7 +170,7 @@ function checkOneSelectorPerElement(groups, reportInFile) {
         if (stylingRule) {
             const first = firstStylingRules.get(target.join(' '));
             if (first && first.selector !== stylingRule.selector) {
-                reportInFile(stylingRule.node, `\`${stylingRule.selector.trim()}\` and \`${first.selector.trim()}\` (line ${getLine(first.node)}) are one element, and both set its default style: write it under one selector (tracker CONV-styleParser.ts:109).`);
+                reportInFile(stylingRule.node, 'duplicate_default_style', `\`${stylingRule.selector.trim()}\` and \`${first.selector.trim()}\` (line ${getLine(first.node)}) are one element, and both set its default style: write it under one selector (tracker CONV-styleParser.ts:109).`);
             }
             else if (!first)
                 firstStylingRules.set(target.join(' '), stylingRule);
@@ -185,7 +187,7 @@ function checkOneSelectorPerElement(groups, reportInFile) {
             if (!first)
                 firstPicks.set(key, { selector: group[0].selector, declaration });
             else
-                reportInFile(declaration, `${describePresetOwner(key)}'s preset is picked twice: \`${first.declaration.prop}\` on \`${first.selector.trim()}\` (line ${getLine(first.declaration)}) and \`${declaration.prop}\` on \`${group[0].selector.trim()}\`. Keep one (tracker CONV-styleParser.ts:109).`);
+                reportInFile(declaration, 'duplicate_preset', `${describePresetOwner(key)}'s preset is picked twice: \`${first.declaration.prop}\` on \`${first.selector.trim()}\` (line ${getLine(first.declaration)}) and \`${declaration.prop}\` on \`${group[0].selector.trim()}\`. Keep one (tracker CONV-styleParser.ts:109).`);
         }
     }
 }
@@ -317,12 +319,12 @@ function groupByBreakpoint(rules) {
 // ── reporting ──
 /** Reports the second `@apply` of one rule (CONV-astParser.ts:99); returns null, as the rule isn't read further. */
 function reportTwoApplies(applies, where, why, reportInFile) {
-    reportInFile(applies[1], `${where}: ${applies.length} \`@apply\` in one rule (${applies.map((apply) => `\`${describeAtRule(apply)}\``).join(', ')}). Keep one: ${why} (tracker CONV-astParser.ts:99).`);
+    reportInFile(applies[1], 'duplicate_apply', `${where}: ${applies.length} \`@apply\` in one rule (${applies.map((apply) => `\`${describeAtRule(apply)}\``).join(', ')}). Keep one: ${why} (tracker CONV-astParser.ts:99).`);
     return null;
 }
 /** Reports an `@apply` that isn't one dashed id with no block (CONV-astParser.ts:41); returns null, as the rule isn't read further. */
 function reportApplyForm(apply, where, reportInFile) {
-    reportInFile(apply, `${where}: \`${describeAtRule(apply)}\` must name one global style by its dashed id. Write \`@apply --<global-style-id>;\`, with no block after it (tracker CONV-astParser.ts:41).`);
+    reportInFile(apply, 'invalid_apply_form', `${where}: \`${describeAtRule(apply)}\` must name one global style by its dashed id. Write \`@apply --<global-style-id>;\`, with no block after it (tracker CONV-astParser.ts:41).`);
     return null;
 }
 /** A pick's key → the element it picks the preset of: '.comp-x|label' → '`label` of `.comp-x`'; '.comp-x' → '`.comp-x`'. */