test(cli): the Connect-an-Agent wire pin counts the static Setup composition (Branding now travels with service-settings)

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

Author: claude

packages/cli/test/connect-agent-both-halves-wire.pin.test.ts

@@ -252,10 +252,17 @@ describe('#17647 — Connect-an-Agent visibility: BOTH halves, over the wire', (
         expect(allowedIds).toContain('nav_connect_agent');
         expect(groupChildren(setupAllowed.body?.item, 'group_integrations'))
             .toContain('nav_connect_agent');
-        // The Setup tree the platform admin is served: 9 authored groups less
-        // `group_approvals`, which nothing contributes into and the filter
-        // collapses, plus 23 contributed entries plus the MCP one.
-        expect(allowedIds).toHaveLength(32);
+        // The Setup tree the platform admin is served BY THIS COMPOSITION: 9
+        // authored groups less `group_approvals`, which nothing contributes
+        // into and the filter collapses, plus the 22 entries
+        // `SETUP_NAV_CONTRIBUTIONS` carries, plus the MCP one.
+        // `nav_settings_branding` is not among them, and not because of a
+        // tenancy wall (this harness boots none): since #24766 it is
+        // contributed by `@objectstack/service-settings` together with the
+        // `branding` namespace, offered only where no organization wall stands,
+        // and this harness composes platform-objects and the MCP bundle alone.
+        expect(allowedIds).toHaveLength(31);
+        expect(allowedIds).not.toContain('nav_settings_branding');
         // The two per-user stores are reached from the User record page, not
         // from Setup's menu (#23803).
         expect(allowedIds).not.toContain('nav_accounts');
@@ -277,7 +284,8 @@ describe('#17647 — Connect-an-Agent visibility: BOTH halves, over the wire', (
         expect(withAccountHalf.statusCode).toBe(200);
         expect(withoutAccountHalf.statusCode).toBe(200);
         expect(navIds(withAccountHalf.body?.item)).toEqual(navIds(withoutAccountHalf.body?.item));
-        expect(navIds(withAccountHalf.body?.item)).toHaveLength(32);
+        // 31: the composition the count in the test above states.
+        expect(navIds(withAccountHalf.body?.item)).toHaveLength(31);
 
         // The mechanism behind the invariance, asserted rather than assumed:
         // the bundle aims exactly one contribution at `setup` and one at