mikser-io-layouts 2.11.1 → 2.11.2

package/lib/mcp.js

@@ -566,13 +566,36 @@ export function registerCheckTool({ runtime, findEntity, findEntities, useDataba
                 // declared key no layout reads.
                 const knows = (k) => req.has(k) || opt.has(k)
                     || [...req, ...opt].some(d => d.startsWith(`${k}.`) || d.startsWith(`${k}[`))
-                const isRead = (k) => consumed.has(k)
-                    || [...consumed].some(c => c.startsWith(`${k}.`) || c.startsWith(`${k}[`))
+
+                // `declaredButNotRead` asks whether a declared key is read by
+                // ANY layout, so it is answered against the UNSCOPED contract —
+                // the opposite of `missing`, which asks what THIS page needs.
+                //
+                // Scoped, a shorter edition of a page looked broken: every
+                // section it does not use made the schema's keys for that
+                // section look dead, and one such page reported 118 of them
+                // while its longer sibling reported none. The document's only
+                // difference was using fewer sections.
+                const wide = await runtime.options.layouts.inspect(layoutId, { samples: 0 })
+                const anyLayout = new Set([
+                    ...(wide.references?.contract?.meta ?? []),
+                    ...observed,
+                ])
+                const readAnywhere = (k) => anyLayout.has(k)
+                    || [...anyLayout].some(c => c.startsWith(`${k}.`) || c.startsWith(`${k}[`))
                 drift = {
                     schema: declaredSchema.name,
                     readButNotDeclared: [...consumed].filter(k => k && !knows(k)).sort(),
-                    declaredButNotRead: [...req].filter(k => !isRead(k)).sort(),
+                    declaredButNotRead: [...req].filter(k => !readAnywhere(k)).sort(),
                 }
+
+                // And the same noise in the softer list: a key this page's
+                // layout cannot reach is not an omission, it is not applicable.
+                // Reporting 139 of them is the noise the scoping fix removed
+                // from `missing`, arriving by another route.
+                const reachable = (k) => consumed.has(k)
+                    || [...consumed].some(c => c.startsWith(`${k}.`) || c.startsWith(`${k}[`))
+                missingOptional = missingOptional.filter(reachable)
             }
 
             // A declared section that resolved to no template under this

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mikser-io-layouts",
-  "version": "2.11.1",
+  "version": "2.11.2",
   "description": "SSG-flavor task-production policy for mikser-io. Multi-match layout assignment, per-layout destinations, postprocess chain attachment, layout inspection.",
   "main": "index.js",
   "type": "module",

package/test/check-entity.test.js

@@ -373,6 +373,27 @@ describe('mikser_check_entity: with a declared schema', () => {
             `missingOptional: ${r.missingOptional.join(', ')}`)
     })
 
+    it('answers declaredButNotRead against ANY layout, not just this page', async () => {
+        // The mirror of the scoping fix. `missing` asks what THIS page needs,
+        // so it is scoped; "is this key read anywhere" is the opposite question
+        // and must not be. Scoped, a shorter edition of a page reported every
+        // key of every section it does not use as dead — 118 of them on a page
+        // whose only difference was using fewer sections than its sibling.
+        const r = await checkWithSchema('/documents/schemed.md')
+        assert.ok(!r.drift.declaredButNotRead.some(k => k.startsWith('hero.')),
+            `hero keys are read by a layout, just not this render: ${r.drift.declaredButNotRead.join(', ')}`)
+    })
+
+    it('does not list an optional key this page could not use anyway', async () => {
+        // Same noise by another route: a key outside what this page renders is
+        // not an omission, it is not applicable.
+        const r = await checkWithSchema('/documents/schemed.md')
+        for (const k of r.missingOptional) {
+            assert.ok(!/^(specs|series|results)\./.test(k),
+                `${k} belongs to a section this page does not render`)
+        }
+    })
+
     it('reports drift in both directions', async () => {
         const r = await checkWithSchema('/documents/schemed.md')
         assert.equal(r.drift.schema, 'article')