feat(shared): add a browser-safe @finos/calm-shared/browser entry point

cli/src/browser-manifest.spec.ts

@@ -0,0 +1,30 @@
+import { describe, it, expect } from 'vitest';
+import { Command } from 'commander';
+import { setupCLI } from './cli';
+import { BROWSER_COMMAND_SUPPORT } from '@finos/calm-shared/browser';
+
+function registeredCommandKeys(): string[] {
+    const program = new Command();
+    setupCLI(program);
+    const keys: string[] = [];
+    for (const command of program.commands) {
+        if (command.name() === 'hub') {
+            for (const sub of command.commands) {
+                keys.push(`hub ${sub.name()}`);
+            }
+        } else {
+            keys.push(command.name());
+        }
+    }
+    return keys.sort();
+}
+
+describe('browser capability manifest matches the CLI', () => {
+    // Granularity: top-level commands plus the `hub` subgroups (`hub pull`, `hub list`, ...);
+    // `workspace` subcommands are intentionally not enumerated, they're covered by the single
+    // `workspace` entry.
+    it('lists every registered command exactly once', () => {
+        const manifest = BROWSER_COMMAND_SUPPORT.map((entry) => entry.command).sort();
+        expect(manifest).toEqual(registeredCommandKeys());
+    });
+});

cli/src/cli.e2e.spec.ts

@@ -2,7 +2,7 @@ import { execSync } from 'child_process';
 import path from 'path';
 import * as fs from 'fs';
 import { parseStringPromise } from 'xml2js';
-import { expectDirectoryMatch, expectFilesMatch } from '@finos/calm-shared';
+import { expectDirectoryMatch, expectFilesMatch } from '../../shared/src/test/file-comparison';
 import { installPackedCli, type CliInstall } from './test_helpers/cli-runner';
 import { patchJson } from './test_helpers/json-file';
 import { STATIC_GETTING_STARTED_MAPPING_PATH } from './test_helpers/getting-started-url-mapping';

cli/src/cli.spec.ts

@@ -14,8 +14,7 @@ let templateModule: typeof import('./command-helpers/template');
 let optionsModule: typeof import('./command-helpers/generate-options');
 let diffModule: typeof import('./command-helpers/diff');
 let hubCommandsModule: typeof import('./command-helpers/hub-commands');
-let _fileSystemDocLoaderModule: typeof import('@finos/calm-shared/dist/document-loader/file-system-document-loader');
-let documentLoaderModule: typeof import('../../shared/src/document-loader/document-loader');
+let documentLoaderModule: typeof import('../../shared/src/document-loader/node-document-loader');
 let setupCLI: typeof import('./cli').setupCLI;
 let cliConfigModule: typeof import('./cli-config');
 
@@ -32,8 +31,7 @@ describe('CLI Commands', () => {
         templateModule = await import('./command-helpers/template');
         optionsModule = await import('./command-helpers/generate-options');
         diffModule = await import('./command-helpers/diff');
-        _fileSystemDocLoaderModule = await import('@finos/calm-shared/dist/document-loader/file-system-document-loader');
-        documentLoaderModule = await import('../../shared/src/document-loader/document-loader');
+        documentLoaderModule = await import('../../shared/src/document-loader/node-document-loader');
 
         vi.spyOn(calmShared, 'runGenerate').mockResolvedValue(undefined);
         vi.spyOn(calmShared.TemplateProcessor.prototype, 'processTemplate').mockResolvedValue(undefined);

cli/src/command-helpers/ai-tools.ts

@@ -1,5 +1,5 @@
 import { initLogger } from '@finos/calm-shared';
-import { Logger } from '@finos/calm-shared/src/logger.js';
+import { Logger } from '@finos/calm-shared';
 import { mkdir, writeFile, readFile, stat } from 'fs/promises';
 import { dirname, join, resolve } from 'path';
 

cli/src/command-helpers/hub-commands.spec.ts

@@ -14,11 +14,7 @@ import { runCreateNamespace, runListArchitectures, runListNamespaces,
 // We stub the @finos/calm-shared HTTP client so no real HTTP is made, but keep the
 // real (pure) document-id-utils helpers that orchestratePush relies on.
 vi.mock('@finos/calm-shared', async () => {
-    const documentIdUtils = await vi.importActual<Record<string, unknown>>('@finos/calm-shared/dist/hub/document-id-utils');
-    // Real (pure) semver helpers used by pushDocument's version-bump path.
-    const semver = await vi.importActual('@finos/calm-shared/dist/hub/semver');
-    // Real (pure) canonical-equality helper used by pushDocument's fail-if-modified path.
-    const canonical = await vi.importActual('@finos/calm-shared/dist/hub/canonical');
+    const actual = await vi.importActual<typeof import('@finos/calm-shared')>('@finos/calm-shared');
     const mockClient = {
         createNamespace: vi.fn(),
         listNamespaces: vi.fn(),
@@ -41,10 +37,11 @@ vi.mock('@finos/calm-shared', async () => {
         createControlConfigurationVersion: vi.fn()
     };
     return {
-        ...documentIdUtils,
-        extractDocumentMetadata: vi.fn(documentIdUtils['extractDocumentMetadata'] as (...args: unknown[]) => unknown),
-        ...semver,
-        ...canonical,
+        // Keep all the real (pure) helpers — document-id-utils, semver and canonical — that
+        // orchestratePush relies on, then override just the HTTP-touching pieces below. The
+        // vi.fn(...) overrides must come after this spread, or the spread would clobber them.
+        ...actual,
+        extractDocumentMetadata: vi.fn(actual.extractDocumentMetadata),
         CalmHubClient: vi.fn(function () { return mockClient; }),
         HubClientError: class HubClientError extends Error {
             constructor(public status: number, public error: string, public request: string) {

cli/src/command-helpers/workspace/bump.spec.ts

@@ -1,11 +1,12 @@
 import { describe, it, expect, beforeAll, afterAll, beforeEach, vi } from 'vitest';
 import { detectChangedResources, bumpWorkspace, canonicalEqual, maxIncrement } from './bump';
 import { saveManifest } from './bundle';
-import { CalmHubClient, ResourceChangeType } from '@finos/calm-shared/src/hub/calm-hub-client';
+import { CalmHubClient, ResourceChangeType } from '@finos/calm-shared';
 import { mkdir, writeFile, rm, readFile } from 'fs/promises';
 import path from 'path';
 
-vi.mock('@finos/calm-shared/src/logger', () => ({
+vi.mock('@finos/calm-shared', async (importOriginal) => ({
+    ...(await importOriginal<typeof import('@finos/calm-shared')>()),
     initLogger: () => ({ info: vi.fn(), warn: vi.fn(), error: vi.fn(), debug: vi.fn() }),
 }));
 

cli/src/command-helpers/workspace/bump.ts

@@ -2,15 +2,18 @@ import { readFile, writeFile } from 'fs/promises';
 import { existsSync } from 'fs';
 import { loadManifest, resolveFilePath } from './bundle';
 import { buildRefRulesFromDiskIds, syncReferences, RefUpdateResult } from './ref-rewrite';
-import { CalmHubClient, ResourceChangeType } from '@finos/calm-shared/src/hub/calm-hub-client';
 import {
+    CalmHubClient,
+    ResourceChangeType,
     DocumentMetadata,
     extractDocumentMetadata,
     constructDocumentId,
-} from '@finos/calm-shared/src/hub/document-id-utils';
-import { computeSemVerBump, sortSemVer } from '@finos/calm-shared/src/hub/semver';
-import { canonicalEqual } from '@finos/calm-shared/src/hub/canonical';
-import { initLogger, Logger } from '@finos/calm-shared/src/logger';
+    computeSemVerBump,
+    sortSemVer,
+    canonicalEqual,
+    initLogger,
+    Logger,
+} from '@finos/calm-shared';
 
 // Re-exported for existing consumers (push.ts, tests) that import it from here.
 export { canonicalEqual };

cli/src/command-helpers/workspace/commands.spec.ts

@@ -93,19 +93,10 @@ vi.mock('../../cli-config', () => ({
     loadAuthPlugin: mocks.loadAuthPlugin,
 }));
 
-vi.mock('@finos/calm-shared/src/hub/calm-hub-client', () => ({
-    CalmHubClient: mocks.CalmHubClient,
-}));
-
 vi.mock('./document-id-prompt', () => ({
     promptForDocumentId: mocks.promptForDocumentId,
 }));
 
-vi.mock('@finos/calm-shared/src/hub/document-id-utils', () => ({
-    isConformantDocumentId: mocks.isConformantDocumentId,
-    namespaceFromDocumentId: mocks.namespaceFromDocumentId,
-}));
-
 vi.mock('fs/promises', async (importOriginal) => {
     const actual = await importOriginal<typeof import('fs/promises')>();
     return { ...actual, readFile: mocks.readFile, writeFile: mocks.writeFile };
@@ -116,7 +107,11 @@ vi.mock('@inquirer/prompts', () => ({
     input: mocks.input,
 }));
 
-vi.mock('@finos/calm-shared/src/logger', () => ({
+vi.mock('@finos/calm-shared', async (importOriginal) => ({
+    ...(await importOriginal<typeof import('@finos/calm-shared')>()),
+    CalmHubClient: mocks.CalmHubClient,
+    isConformantDocumentId: mocks.isConformantDocumentId,
+    namespaceFromDocumentId: mocks.namespaceFromDocumentId,
     initLogger: () => ({
         info: vi.fn(),
         warn: vi.fn(),

cli/src/command-helpers/workspace/commands.ts

@@ -11,11 +11,9 @@ import { detectChangedResources, bumpWorkspace } from './bump';
 import { runPostBumpValidation } from './post-bump-validate';
 import { loadWorkspaceConfig } from './config';
 import { findWorkspaceManifestPath, findGitRoot } from '../../workspace-resolver';
-import { initLogger, Logger } from '@finos/calm-shared/src/logger';
+import { initLogger, Logger, CalmHubClient, ResourceChangeType, isConformantDocumentId, namespaceFromDocumentId } from '@finos/calm-shared';
 import { select, input } from '@inquirer/prompts';
 import { CALM_DOCUMENT_TYPES_LIST, isValidCalmDocumentType } from '@finos/calm-models/types';
-import { CalmHubClient, ResourceChangeType } from '@finos/calm-shared/src/hub/calm-hub-client';
-import { isConformantDocumentId, namespaceFromDocumentId } from '@finos/calm-shared/src/hub/document-id-utils';
 import { loadCliConfig } from '../../cli-config';
 import { resolveCalmHubOptions } from '../hub-commands';
 

cli/src/command-helpers/workspace/config.ts

@@ -1,7 +1,7 @@
 import path from 'path';
 import { readFile } from 'fs/promises';
 import { existsSync } from 'fs';
-import type { ResourceChangeType } from '@finos/calm-shared/src/hub/calm-hub-client';
+import type { ResourceChangeType } from '@finos/calm-shared';
 
 /**
  * Central, repo-level workspace configuration. Committed at

cli/src/command-helpers/workspace/document-id-prompt.ts

@@ -5,8 +5,9 @@ import {
     isConformantDocumentId,
     DocumentMetadata,
     ControlDocumentMetadata,
-} from '@finos/calm-shared/src/hub/document-id-utils';
-import { RESOURCE_TYPES, ResourceType } from '@finos/calm-shared/src/hub/calm-hub-client';
+    RESOURCE_TYPES,
+    ResourceType,
+} from '@finos/calm-shared';
 
 const DEFAULT_VERSION = '1.0.0';
 

cli/src/command-helpers/workspace/push.spec.ts

@@ -1,7 +1,7 @@
 import { describe, it, expect, beforeAll, afterAll, beforeEach, vi } from 'vitest';
 import { pushWorkspaceToHub } from './push';
 import { loadManifest, saveManifest } from './bundle';
-import { CalmHubClient, HubClientError } from '@finos/calm-shared/src/hub/calm-hub-client';
+import { CalmHubClient, HubClientError } from '@finos/calm-shared';
 import { mkdir, writeFile, rm } from 'fs/promises';
 import path from 'path';
 import { existsSync } from 'fs';
@@ -15,7 +15,8 @@ const makeClient = (
     ...overrides,
 }) as unknown as CalmHubClient;
 
-vi.mock('@finos/calm-shared/src/logger', () => ({
+vi.mock('@finos/calm-shared', async (importOriginal) => ({
+    ...(await importOriginal<typeof import('@finos/calm-shared')>()),
     initLogger: () => ({
         info: vi.fn(),
         warn: vi.fn(),

cli/src/command-helpers/workspace/push.ts

@@ -1,9 +1,7 @@
 import { readFile } from 'fs/promises';
 import { existsSync } from 'fs';
 import { loadManifest, saveManifest, resolveFilePath } from './bundle';
-import { CalmHubClient } from '@finos/calm-shared/src/hub/calm-hub-client';
-import { DocumentMetadata, extractDocumentMetadata } from '@finos/calm-shared/src/hub/document-id-utils';
-import { initLogger, Logger } from '@finos/calm-shared/src/logger';
+import { CalmHubClient, DocumentMetadata, extractDocumentMetadata, initLogger, Logger } from '@finos/calm-shared';
 import { canonicalEqual } from './bump';
 
 const logger: Logger = initLogger(false, 'workspace');

cli/src/command-helpers/workspace/ref-rewrite.spec.ts

@@ -12,7 +12,8 @@ import path from 'path';
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const loadJson = async (p: string): Promise<any> => JSON.parse(await readFile(p, 'utf8'));
 
-vi.mock('@finos/calm-shared/src/logger', () => ({
+vi.mock('@finos/calm-shared', async (importOriginal) => ({
+    ...(await importOriginal<typeof import('@finos/calm-shared')>()),
     initLogger: () => ({ info: vi.fn(), warn: vi.fn(), error: vi.fn(), debug: vi.fn() }),
 }));
 

cli/src/command-helpers/workspace/ref-rewrite.ts

@@ -1,7 +1,7 @@
 import { readFile, writeFile } from 'fs/promises';
 import { existsSync } from 'fs';
 import { REFERENCE_PROPERTIES, WorkspaceManifest, resolveFilePath } from './bundle';
-import { initLogger, Logger } from '@finos/calm-shared/src/logger';
+import { initLogger, Logger } from '@finos/calm-shared';
 
 const logger: Logger = initLogger(false, 'workspace');
 

cli/src/command-helpers/workspace/rm.ts

@@ -1,4 +1,4 @@
-import { initLogger } from '@finos/calm-shared/src/logger';
+import { initLogger } from '@finos/calm-shared';
 import { loadManifest, saveManifest } from './bundle';
 
 const logger = initLogger(false, 'workspace-rm');

cli/tsconfig.json

@@ -1,8 +1,8 @@
 {
   "extends": "../tsconfig.base.json",
-  "module": "Preserve",
-  "moduleResolution": "bundler",
   "compilerOptions": {
+    "module": "Preserve",
+    "moduleResolution": "bundler",
     "strict": true,
     "outDir": "dist",
   },