@7365admin1/layer-common 3.2.8-staging.206 → 3.2.8-staging.208

package/components/HidReaderManagement.vue

@@ -112,7 +112,6 @@
       :mode="selectedReader ? 'edit' : 'add'"
       :reader="selectedReader"
       :loading="saving"
-      :site="site"
       @submit="saveReader"
     />
 

package/package.json

@@ -2,7 +2,7 @@
   "name": "@7365admin1/layer-common",
   "license": "MIT",
   "type": "module",
-  "version": "3.2.8-staging.206",
+  "version": "3.2.8-staging.208",
   "author": "7365admin1",
   "main": "./nuxt.config.ts",
   "//files": "What a consumer extending this layer actually loads. Without this npm ships the whole working tree - the changesets, the CI workflows, the render harness in tools/ and any scratch directory that happened to exist at publish time. Nuxt resolves a layer by directory, so every runtime directory below has to stay listed; adding a new top-level runtime directory means adding it here too.",

package/utils/hid-discover.test.ts

@@ -96,7 +96,20 @@ test("the reader form takes site as a prop and puts it in the discover call", ()
   assert.match(source, /discoverReader\(\{[\s\S]*?site:\s*props\.site[\s\S]*?\}\)/);
 });
 
-test("reader management passes its site prop down to the form", () => {
+test("reader management passes its site prop down to the form, ONCE", () => {
   const source = readFileSync(join(root, "components", "HidReaderManagement.vue"), "utf8");
-  assert.match(source, /<HidReaderForm[\s\S]*?:site="site"[\s\S]*?\/>/);
+  const tag = source.match(/<HidReaderForm[\s\S]*?\/>/)?.[0] ?? "";
+
+  // Bound to `props.site`, which is the only name for it that exists in this
+  // component - a bare `site` resolves to nothing.
+  assert.match(tag, /:site="props\.site"/);
+
+  /*
+   * And exactly once. It was written twice - `props.site` and then a bare
+   * `site` - and Vite's Vue plugin treats a duplicate attribute as a COMPILE
+   * ERROR, so every consuming application's `nuxt build` failed on this file.
+   * The layer's own playground build does not import this component, so
+   * nothing here caught it; the first sign was a consumer failing to build.
+   */
+  assert.equal((tag.match(/:site=/g) ?? []).length, 1, tag);
 });