eslint-plugin-kerfjs 5.0.0-beta.58 → 5.0.0-beta.59

package/docs/rules/ui-composition.md

@@ -30,7 +30,9 @@ cataloged roots, or nothing) is checked as those roots:
 
 - **Zones (`KUI-L202`):** the zone must accept every declared root.
 - **Parents (`KUI-L201`):** each root's listed parents apply where the wrapper
-  is placed, and a wrapper used as a parent counts as each of its roots.
+  is placed, and a wrapper used as a parent counts as each of its roots. A
+  placement reports once, using the first violated root requirement before a
+  separate wrapper requirement if both apply.
 - **Cardinality (`KUI-L203`):** the wrapper counts as zero or one child, since
   it may render nothing.
 

package/index.js

@@ -14,7 +14,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.58' },
+  meta: { name: 'eslint-plugin-kerfjs', version: '5.0.0-beta.59' },
   rules: {
     'no-inline-jsx-event-handlers': noInlineJsxEventHandlers,
     'no-raw-with-dynamic-arg': noRawWithDynamicArg,

package/lib/rules/ui-composition.js

@@ -167,34 +167,37 @@ export default {
         const entry = contract.entries.get(key);
         if (!entry) return;
         // A wrapper answers to its own parent contract and to each root's.
+        // Report one violated requirement per placement, with roots first.
         const placed = [
-          { entry, child: key },
           ...(entry.rendersAs ?? []).flatMap((root) => {
             const rootEntry = contract.entries.get(root);
             return rootEntry
               ? [{ entry: rootEntry, child: `${key} (renders ${root})` }]
               : [];
           }),
+          { entry, child: key },
         ].filter(
           ({ entry: placedEntry }) => placedEntry.parents?.mode === 'listed',
         );
         if (placed.length && !isExcepted(contract, PARENT_CODE, filename)) {
           const parent = directParentKey(node, registry, contract);
-          if (parent)
-            for (const { entry: placedEntry, child } of placed)
-              if (
+          if (parent) {
+            const violation = placed.find(
+              ({ entry: placedEntry }) =>
                 !rootsOf(parent.key, contract).every((root) =>
                   placedEntry.parents.entries.includes(root),
-                )
-              )
-                context.report({
-                  node: node.name,
-                  messageId: 'parent',
-                  data: {
-                    child,
-                    parents: placedEntry.parents.entries.join(', '),
-                  },
-                });
+                ),
+            );
+            if (violation)
+              context.report({
+                node: node.name,
+                messageId: 'parent',
+                data: {
+                  child: violation.child,
+                  parents: violation.entry.parents.entries.join(', '),
+                },
+              });
+          }
         }
         for (const zone of entry.zones ?? []) {
           const bound = boundZone(node, zone, registry, contract);

package/package.json

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