@terpjs/eslint-boundaries 0.22.0 → 0.24.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@terpjs/eslint-boundaries",
-  "version": "0.22.0",
+  "version": "0.24.0",
   "type": "module",
   "description": "Terp frontend boundary rules (as data) + the ESLint adapter: no cross-module imports, no package internals, design-token-only styling (no style/className/module stylesheets), token-styled components for raw HTML tags, router-only in-app links, generated-client-only, and browser XSS/navigation sink bans. Strict-only (no modes); governed opt-outs via terp-allow markers + the escape-hatch budget ratchet (terp-boundaries-budget).",
   "main": "./src/index.js",
@@ -25,7 +25,7 @@
     "typescript-eslint": "^8.69.0"
   },
   "devDependencies": {
-    "@terpjs/spec": "0.34.0",
+    "@terpjs/spec": "0.35.0",
     "eslint": "^10.10.0",
     "vitest": "^5.0.0"
   },

package/src/i18n.test.js

@@ -144,6 +144,18 @@ describe("locale catalog completeness", () => {
     expect(dynamicTrans.find((message) => message.ruleId === "terp/locale-catalogs-complete")?.message)
       .toContain("requires static non-empty id and message");
 
+    // A spread is still refused - the catalog is inventoried statically - but it is the
+    // one shape with a sanctioned alternative, so the refusal names it. Sharing one
+    // descriptor between two screens is ordinary; only its JSX-body spelling is not.
+    const spreadTrans = await lintWithCatalog(
+      { sourceLocale: "en", locales: { en: {}, nl: {} } },
+      'export const W = () => <Trans {...TITLE} />;',
+    );
+    const spreadMessage = spreadTrans
+      .find((message) => message.ruleId === "terp/locale-catalogs-complete")?.message;
+    expect(spreadMessage).toContain("useUiText");
+    expect(spreadMessage).toContain("text(DESCRIPTOR)");
+
     const staleAllowlist = await lintWithCatalog(
       {
         sourceLocale: "en",

package/src/index.js

@@ -788,9 +788,23 @@ const localeCatalogsComplete = {
         if (jsxName(node.name) !== "Trans") return;
         const descriptor = transDescriptor(node);
         if (descriptor === null) {
+          // A spread is the one shape with a sanctioned alternative, and the generic
+          // message sent authors looking for a way to make the spread work instead.
+          // One descriptor shared by two screens is ordinary; what cannot be shared is
+          // the JSX-body spelling of it, because the catalog is inventoried statically
+          // and `{...DESCRIPTOR}` carries no attributes to read. The resolver renders
+          // the same copy from the same constant, so name it rather than the refusal.
+          const spread = node.attributes.some(
+            (attribute) => attribute.type === "JSXSpreadAttribute",
+          );
           context.report({
             node,
-            message: "<Trans> requires static non-empty id and message attributes.",
+            message: spread
+              ? "<Trans> reads its id and message as static attributes, so a shared UiText " +
+                "constant cannot be spread into it. Render the same constant through the " +
+                "resolver instead: const text = useUiText() in the component, then " +
+                "{text(DESCRIPTOR)} where the copy goes."
+              : "<Trans> requires static non-empty id and message attributes.",
           });
           return;
         }

package/src/spec.js

@@ -16,7 +16,7 @@
  * NOT by this package's own suite, which certification runs against candidate spec releases
  * whose version is allowed to be newer).
  */
-export const SPEC_VERSION = "0.34.0";
+export const SPEC_VERSION = "0.35.0";
 
 export const BOUNDARY_SPEC = {
   /** Every app-authored TypeScript source file whose user-facing copy must be cataloged. */