eslint-plugin-kerfjs 5.0.0-beta.50 → 5.0.0-beta.51

package/docs/rules/ui-css-values.md

@@ -11,5 +11,14 @@ Stable diagnostics are `KUI-L013` (raw or unknown shorthand), `KUI-L014`
 spacing shorthand). Messages name the preferred shorthand or helper. Dynamic
 values are left to TypeScript rather than guessed.
 
+Only component exports carry an entry's contracts. A catalog entry's
+`publicExports` also lists the helpers its props consume (`Select` ships
+`uiColor`, `List` ships `px`, `rem`, and `flex`); a helper call such as
+`flex({ gap: '12px' })` is not a `List` call and is never checked as one. A
+component export is one whose name starts with an uppercase letter, the rule
+JSX itself uses to tell a component tag from an intrinsic element; a default
+import keeps its subpath's entry. `@kerfjs/ui`'s catalog check enforces the
+convention for the package, and `kerf-ui-analyze` applies the same rule.
+
 Third-party component suites can provide the same contracts in a composition
 catalog through `settings.kerfjs.ui.catalog` and its matching selection catalog.

package/index.js

@@ -13,7 +13,7 @@ import uiPublicBoundaries from './lib/rules/ui-public-boundaries.js';
 import uiWiring from './lib/rules/ui-wiring.js';
 
 const plugin = {
-  meta: { name: 'eslint-plugin-kerfjs', version: '5.0.0-beta.50' },
+  meta: { name: 'eslint-plugin-kerfjs', version: '5.0.0-beta.51' },
   rules: {
     'no-inline-jsx-event-handlers': noInlineJsxEventHandlers,
     'no-raw-with-dynamic-arg': noRawWithDynamicArg,

package/lib/rules/ui-css-values.js

@@ -100,7 +100,7 @@ export default {
             ...(cssContract.canonicalShorthands ?? []).map(
               (item) => `\`${item}\``,
             ),
-            ...(cssContract.helpers ?? []).map((item) => `\`${item}()`),
+            ...(cssContract.helpers ?? []).map((item) => `\`${item}()\``),
           ].join(' or ') || 'a cataloged component prop';
         for (const value of valuesFor(cssContract.path)) {
           if (!value) continue;

package/lib/ui-contract.js

@@ -9,6 +9,20 @@ const loadCommonJs = createRequire(import.meta.url);
 
 export const UI_CONTRACT_LOAD_CODE = 'KUI-L090';
 
+// A catalog entry's `publicExports` mixes the component render function with
+// helpers the component's props consume (`Select` ships `uiColor`, `List`
+// ships `px`/`rem`/`flex`). Only the render function stands for the entry, so
+// only it may carry the entry's contracts. A component export is one whose
+// name starts with an uppercase letter: JSX itself treats a lowercase tag as an
+// intrinsic element, so a lowercase export can never be a component tag, and
+// `@kerfjs/ui`'s catalog check (`check-component-catalog.mjs`) enforces the
+// converse for the package: every uppercase runtime export returns `SafeHtml`
+// and every lowercase one does not. A default import has no name to inspect
+// and keeps its subpath's entry.
+export function isComponentExport(name) {
+  return name === 'default' || /^[A-Z]/.test(name);
+}
+
 const readJson = (path) => JSON.parse(readFileSync(path, 'utf8'));
 
 function packageAsset(cwd, name) {
@@ -135,7 +149,7 @@ export function loadUiContract(context) {
       if (entry.delivery?.moduleImport)
         imports.set(entry.delivery.moduleImport, key);
       for (const name of entry.publicExports ?? []) {
-        exports.set(name, key);
+        if (isComponentExport(name)) exports.set(name, key);
         addHelperSource(name, selection.package);
       }
       for (const wiring of entry.wiring ?? [])
@@ -185,11 +199,12 @@ export function importRegistry(program, contract) {
         specifier.type === 'ImportDefaultSpecifier'
           ? 'default'
           : (specifier.imported.name ?? specifier.imported.value);
-      const key =
-        directKey ??
-        (source === contract.package
-          ? contract.exports?.get(imported)
-          : undefined);
+      const key = !isComponentExport(imported)
+        ? undefined
+        : (directKey ??
+          (source === contract.package
+            ? contract.exports?.get(imported)
+            : undefined));
       if (key) locals.set(specifier.local.name, key);
       helpers.set(specifier.local.name, { imported, source });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-kerfjs",
-  "version": "5.0.0-beta.50",
+  "version": "5.0.0-beta.51",
   "description": "ESLint rules that enforce kerf's hard rules — catches AI-shaped bugs at edit time.",
   "type": "module",
   "license": "MIT",