fix(objectql,cli): carry boot's falsy-key skip into the shared view-container judge; fixtures boot accepts

viewContainerNameRefusal now answers undefined for a container whose
derived object key is falsy, the entry registerMetadataCollections warns
about and skips before the name check, so os validate cannot refuse what
boot skips. Pinned by a control that boot registers nothing and throws
nothing.

Every CLI test fixture whose views: container carried a name other than
its bound object (two red integration pins among them) now names the
object, so each is a stack the server loads. The changeset grades
@objectstack/objectql minor with Clause-② yes for its two new root
exports, and the gate-parity remedy text names VALIDATE_ONLY_GATES.

Claude-Session: https://claude.ai/code/session_01UYBdGBzWSrAMzpW8ah3GbP
Co-authored-by: Claude <noreply@anthropic.com>

Author: claude

.changeset/20331-validate-view-container-name.md

@@ -1,11 +1,11 @@
 ---
 '@objectstack/cli': patch
-'@objectstack/objectql': patch
+'@objectstack/objectql': minor
 ---
 
 fix(cli): `os validate` refuses a `views:` container whose own `name` disagrees with the object it binds to, the stack the server refuses at boot (#20331)
 
-Clause-②: no
+Clause-②: yes
 
 A view container is registered under the object it binds to. When its own `name`
 is set to something else, for example `{ name: 'order_line', object: 'my_app_order_line', list: { … } }`,
@@ -21,10 +21,13 @@ unchanged.
 
 **Fix:** remove the container's `name`, or set it to the object name the message names.
 
-`@objectstack/objectql` exports the check as `viewContainerNameRefusal(container, sourceLabel, ownerId)`,
-with its type `ViewContainerNameRefusal`. It returns the refusal the boot registrar
-throws, or `undefined`. The boot registrar now calls this function. What it refuses,
-its message and its `VALIDATION_ERROR` / `400` envelope are unchanged.
+**New in `@objectstack/objectql` (the widening):** two new exports on the package's
+root entry, `viewContainerNameRefusal(container, sourceLabel, ownerId)` and its
+return type `ViewContainerNameRefusal`. The function returns the refusal the boot
+registrar throws, or `undefined`. It returns `undefined` for a container whose
+derived object key is empty, because the boot registrar skips that entry with a
+warning and never refuses it. The boot registrar now calls this function. What it
+refuses, its message and its `VALIDATION_ERROR` / `400` envelope are unchanged.
 
 Not changed: `os build` does not run this check, so it still writes an artifact
 carrying such a container, and the server refuses that artifact when it loads it.

packages/cli/test/build-text-face-advisory-count.test.ts

@@ -187,11 +187,11 @@ const ordersObjects = [{
 }];
 const ordersViews = [
   {
-    name: 'bc_account_list', label: 'Account List', object: 'bc_account',
+    name: 'bc_account', label: 'Account List', object: 'bc_account',
     list: { label: 'Account List', columns: ['name', 'industry'] },
   },
   {
-    name: 'bc_order_list', label: 'Order List', object: 'bc_order',
+    name: 'bc_order', label: 'Order List', object: 'bc_order',
     list: { label: 'Order List', columns: ['name', 'account'] },
   },
 ];

packages/cli/test/format-zod-union.test.ts

@@ -185,7 +185,7 @@ const TOOLTIP_ALIAS_STACK = {
   manifest: { id: 'com.example.union-probe', name: 'Union Probe', namespace: 'union_probe', version: '1.0.0', type: 'app' },
   views: [
     {
-      name: 'union_probe_view',
+      name: 'union_probe_obj',
       object: 'union_probe_obj',
       list: {
         name: 'union_probe_list',

packages/cli/test/info-detail-package-fold.test.ts

@@ -91,7 +91,7 @@ const OBJECTS = [
 
 const VIEWS = [
   {
-    name: 'ob_order_views',
+    name: 'ob_order',
     object: 'ob_order',
     list: {
       label: 'Orders',

packages/cli/test/lint-handwritten-checks-package-fold.test.ts

@@ -103,7 +103,7 @@ const definitions = () => ({
   ],
   views: [
     {
-      name: 'probe_view',
+      name: 'probe_order',
       object: 'probe_order',
       list: { label: 'order list', columns: ['number'] }, // convention/label-case
     },

packages/cli/test/lint-label-case-localized.test.ts

@@ -92,13 +92,13 @@ const stackWithApp = (label: unknown) => ({
 
 const stackWithListLabel = (label: unknown) => ({
   manifest: MANIFEST,
-  views: [{ name: 'invoice_views', object: 'invoice', list: { label, type: 'grid', columns: ['name'] } }],
+  views: [{ name: 'invoice', object: 'invoice', list: { label, type: 'grid', columns: ['name'] } }],
 });
 
 const stackWithNamedListLabel = (label: unknown) => ({
   manifest: MANIFEST,
   views: [{
-    name: 'invoice_views',
+    name: 'invoice',
     object: 'invoice',
     listViews: { all: { label, type: 'grid', columns: ['name'] } },
   }],

packages/cli/test/lint-per-package-authoring-parity.test.ts

@@ -152,11 +152,11 @@ const ordersObjects = [{
 }];
 const ordersViews = [
   {
-    name: 'pp_account_list', label: 'Account List', object: 'pp_account',
+    name: 'pp_account', label: 'Account List', object: 'pp_account',
     list: { label: 'Account List', columns: ['name', 'industry'] },
   },
   {
-    name: 'pp_order_list', label: 'Order List', object: 'pp_order',
+    name: 'pp_order', label: 'Order List', object: 'pp_order',
     list: { label: 'Order List', columns: ['name', 'account'] },
   },
 ];

packages/cli/test/lint-per-package-authoring-seam.test.ts

@@ -136,13 +136,13 @@ function twoPackageArtifact(): Record<string, unknown> {
     ],
     views: [
       {
-        name: 'pp_account_list',
+        name: 'pp_account',
         label: 'Account List',
         object: 'pp_account',
         list: { label: 'Account List', columns: ['name', 'industry'] },
       },
       {
-        name: 'pp_order_list',
+        name: 'pp_order',
         label: 'Order List',
         object: 'pp_order',
         list: { label: 'Order List', columns: ['name', 'account'] },

packages/cli/test/union-fold-command-parity.test.ts

@@ -203,13 +203,17 @@ const perPackageOnlyStack = (): Record<string, unknown> => {
       account: { name: 'account', type: 'lookup', label: 'Account', reference: 'ob_account' },
     },
   }];
+  // [#20331] Each container's `name` IS the object it binds to: the boot
+  // registrar refuses a container whose own `name` disagrees with that key,
+  // and `os validate` now refuses it too, so a `*_list` name here would make
+  // this a stack the server cannot load.
   const ordersViews = [
     {
-      name: 'ob_account_list', label: 'Account List', object: 'ob_account',
+      name: 'ob_account', label: 'Account List', object: 'ob_account',
       list: { label: 'Account List', columns: ['name', 'industry'] },
     },
     {
-      name: 'ob_order_list', label: 'Order List', object: 'ob_order',
+      name: 'ob_order', label: 'Order List', object: 'ob_order',
       list: { label: 'Order List', columns: ['name', 'account'] },
     },
   ];

packages/cli/test/validate-build-gate-parity.test.ts

@@ -836,9 +836,10 @@ describe('os validate is the read-only superset of os build (#3782, #4409)', ()
         `${unclassified.join(', ')}.\n` +
         `Every call site must land in exactly one ledger. If it is an artifact-level gate, wire it ` +
         `into BOTH commands and add it to SHARED_NON_REGISTRY_GATES; if it genuinely cannot run ` +
-        `read-only, add it to BUILD_ONLY_GATES with a reason; if it is not a gate at all, add it to ` +
-        `NOT_A_GATE under the reason that says so. Registering it in ` +
-        `packages/lint/src/authoring-rules.ts instead is better than all three — then all THREE ` +
+        `read-only, add it to BUILD_ONLY_GATES with a reason; if validate.ts runs it and compile.ts ` +
+        `deliberately does not, add it to VALIDATE_ONLY_GATES with the reason the build lacks it; if ` +
+        `it is not a gate at all, add it to NOT_A_GATE under the reason that says so. Registering it in ` +
+        `packages/lint/src/authoring-rules.ts instead is better than all four — then all THREE ` +
         `authoring commands get it and no roster row is needed.`,
     ).toEqual([]);
   });