@topolo/sdk 0.3.0 → 0.4.0

package/src/modules/p2p.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, it, vi } from 'vitest';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { TopoloClient } from '../client.js';
 import { P2PModule } from './p2p.js';
 
@@ -8,6 +8,12 @@ function stubFetch(responses: Array<{ status?: number; body?: unknown }>) {
   const calls: Array<{ url: string; init: RequestInit }> = [];
   const impl = vi.fn(async (url: string, init: RequestInit) => {
     calls.push({ url, init });
+    if (String(url).includes('/api/services/by-slug/')) {
+      return new Response(JSON.stringify({ success: true, data: { service_id: 'service_p2p' } }), {
+        status: 200,
+        headers: { 'Content-Type': 'application/json' },
+      });
+    }
     const next = responses.shift() ?? { status: 200, body: { success: true, data: {} } };
     return new Response(JSON.stringify(next.body ?? { success: true, data: {} }), {
       status: next.status ?? 200,
@@ -17,6 +23,10 @@ function stubFetch(responses: Array<{ status?: number; body?: unknown }>) {
   return { impl: impl as unknown as typeof fetch, calls };
 }
 
+function requestCalls(calls: Array<{ url: string; init: RequestInit }>) {
+  return calls.filter((call) => !call.url.includes('/api/services/by-slug/'));
+}
+
 function clientWith(impl: typeof fetch) {
   return new TopoloClient({
     credential: { kind: 'access_token', accessToken: 'tok_test' },
@@ -26,6 +36,10 @@ function clientWith(impl: typeof fetch) {
 }
 
 describe('P2PModule.listDirectory', () => {
+  beforeEach(() => {
+    TopoloClient.clearServiceIdCache();
+  });
+
   it('GETs /api/directory/capabilities and unwraps the envelope', async () => {
     const { impl, calls } = stubFetch([
       {
@@ -37,7 +51,7 @@ describe('P2PModule.listDirectory', () => {
                 capability_id: 'cap_1',
                 publisher_org_id: 'org_a',
                 app_id: 'app_x',
-                service_id: 'srv_x',
+                service_id: 'service_x',
                 name: 'Demo',
                 description: 'A capability',
                 category: 'other',
@@ -60,10 +74,11 @@ describe('P2PModule.listDirectory', () => {
     const capabilities = await p2p.listDirectory({ limit: 10 });
     expect(capabilities).toHaveLength(1);
     expect(capabilities[0]!.capability_id).toBe('cap_1');
-    const url = new URL(calls[0]!.url);
+    const call = requestCalls(calls)[0]!;
+    const url = new URL(call.url);
     expect(url.pathname).toBe('/api/directory/capabilities');
     expect(url.searchParams.get('limit')).toBe('10');
-    expect((calls[0]!.init.method ?? 'GET').toUpperCase()).toBe('GET');
+    expect((call.init.method ?? 'GET').toUpperCase()).toBe('GET');
   });
 });
 
@@ -78,7 +93,7 @@ describe('P2PModule.publishCapability', () => {
               capability_id: 'cap_z',
               publisher_org_id: 'org_p',
               app_id: 'app_x',
-              service_id: 'srv_x',
+              service_id: 'service_x',
               name: 'Demo',
               description: 'A capability',
               category: 'other',
@@ -101,7 +116,7 @@ describe('P2PModule.publishCapability', () => {
       capability_id: 'cap_z',
       publisher_org_id: 'org_p',
       app_id: 'app_x',
-      service_id: 'srv_x',
+      service_id: 'service_x',
       name: 'Demo',
       description: 'A capability',
       category: 'other',
@@ -114,13 +129,14 @@ describe('P2PModule.publishCapability', () => {
     });
     expect(capability.capability_id).toBe('cap_z');
 
-    const sent = JSON.parse(String(calls[0]!.init.body));
+    const call = requestCalls(calls)[0]!;
+    const sent = JSON.parse(String(call.init.body));
     expect(sent.publisher_org_id).toBe('org_p');
     expect(sent.acting_for_org_id).toBe('org_p');
     expect(sent.public_directory_visible).toBe(true);
     expect(sent.requires_human_approval).toBe(false);
     expect(sent.trust_level).toBe('standard');
-    const url = new URL(calls[0]!.url);
+    const url = new URL(call.url);
     expect(url.pathname).toBe('/api/p2p/internal/capabilities/sync');
     expect((calls[0]!.init.method ?? '').toUpperCase()).toBe('POST');
   });
@@ -134,7 +150,7 @@ describe('P2PModule.publishCapability', () => {
       capability_id: 'cap_other',
       publisher_org_id: 'org_target',
       app_id: 'app_x',
-      service_id: 'srv_x',
+      service_id: 'service_x',
       name: 'Demo',
       description: '...',
       category: 'other',
@@ -144,7 +160,7 @@ describe('P2PModule.publishCapability', () => {
       source_revision: 'rev_1',
       acting_for_org_id: 'org_target',
     });
-    const sent = JSON.parse(String(calls[0]!.init.body));
+    const sent = JSON.parse(String(requestCalls(calls)[0]!.init.body));
     expect(sent.acting_for_org_id).toBe('org_target');
   });
 });
@@ -211,7 +227,7 @@ describe('P2PModule.releaseHold', () => {
     const result = await p2p.releaseHold('hold_1', 'seller', { reason: 'arbitration' });
     expect(result.resolution).toBe('seller');
     expect(result.cancel_entries[0]!.entry_id).toBe('led_1');
-    const sent = JSON.parse(String(calls[0]!.init.body));
+    const sent = JSON.parse(String(requestCalls(calls)[0]!.init.body));
     expect(sent.resolution).toBe('seller');
     expect(sent.reason).toBe('arbitration');
   });

package/src/services.generated.ts

@@ -36,12 +36,14 @@ export const SERVICES = {
   p2p: "https://p2p.topolo.app",
   pay: "https://pay.topolo.app",
   people: "https://people.topolo.app",
+  pulse: "https://pulse.topolo.app",
   "quro.api": "https://api.ol0.me",
   "quro.redirect": "https://ol0.me",
   roadmapper: "https://roadmapper-api.topolo.app",
   sign: "https://sign.topolo.app",
   social_studio: "https://studio-api.topolo.app",
   socialize: "https://socialize-api.topolo.app",
+  spaces: "https://spaces.topolo.app",
   status: "https://status.topolo.app",
   support: "https://support.topolo.app",
   survey: "https://survey.topolo.app",
@@ -50,35 +52,3 @@ export const SERVICES = {
 } as const;
 
 export type ServiceId = keyof typeof SERVICES;
-
-/**
- * @deprecated since 0.3.0 — opaque service ids are no longer baked into the
- * SDK. Use `TopoloClient.getServiceIdBySlug(slug)` which hits
- * `GET /api/services/by-slug/:slug` on TopoloAuth at runtime. This map is
- * still populated as a build-time snapshot for backward compatibility and
- * will be removed in 0.4.0. Stale entries are expected — do not rely on
- * presence/absence for catalog membership.
- */
-export const PLATFORM_SERVICE_IDS: Partial<Record<ServiceId, string>> = {
-  books: "srv_0g260hbN8oqv",
-  bugfix: "srv_kQ1t8qUSDgNs",
-  compose: "srv_aR3ETYmgygmd",
-  consent: "srv_8FIGR7Q7zP9r",
-  crm: "srv_iCwM4jGXcwlj",
-  customer_success: "srv_RWpOxkgZdZbH",
-  developers: "srv_NjDq9G9qyyjr",
-  director: "srv_wRCnLLbtvmVT",
-  flow: "srv_GBCuKkHhnRXv",
-  forms: "srv_UE7IcGe80J5o",
-  home: "srv_hE6HBldVzjth",
-  insights: "srv_DgbBMwWSIhmC",
-  inventory: "srv_hmvjbv3ZxDnh",
-  localize: "srv_Ze1g9OmXuZjF",
-  notify: "srv_9pRdkIjYyKX0",
-  p2p: "srv_4BzeLqH0wCve",
-  people: "srv_VDMBAonA5fKb",
-  sign: "srv_Bd8pRw5ZqRBT",
-  status: "srv_topolo_status",
-  survey: "srv_RCOi7CJkwf7R",
-  voice: "srv_BZANHkshGAK5",
-};

package/src/services.test.ts

@@ -14,20 +14,19 @@ const EXPECTED_APPLICATION_IDS = [
   'admin',
   'agent',
   'auth',
+  'backup',
   'books',
   'bugfix',
   'bytes',
   'calendar',
   'capacity',
   'chat',
-  'cli',
   'cloudcontrol',
   'commerce',
   'compose',
   'consent',
   'crm',
   'customer_success',
-  'd1_backup',
   'developers',
   'director',
   'docs',
@@ -41,7 +40,6 @@ const EXPECTED_APPLICATION_IDS = [
   'learn',
   'localize',
   'mail',
-  'mcp',
   'mdm',
   'messaging',
   'nexus',
@@ -57,6 +55,7 @@ const EXPECTED_APPLICATION_IDS = [
   'sign',
   'social_studio',
   'socialize',
+  'spaces',
   'status',
   'support',
   'survey',
@@ -84,20 +83,10 @@ describe('service registry', () => {
     expect(DEFAULT_SERVICE_URLS).not.toHaveProperty('web.runtime');
   });
 
-  it('still ships PLATFORM_SERVICE_IDS as a deprecated build-time snapshot', async () => {
-    // Deprecated in 0.3.0, removed in 0.4.0. Source of truth has moved to
-    // TopoloAuth's GET /api/services/by-slug/:slug — see
-    // TopoloClient.getServiceIdBySlug. We only sanity-check that the snapshot
-    // is non-empty and that the values look like opaque service ids; specific
-    // ids are no longer load-bearing.
-    const { PLATFORM_SERVICE_IDS } = await import('./services.js');
-
-    const entries = Object.entries(PLATFORM_SERVICE_IDS);
-    expect(entries.length).toBeGreaterThan(0);
-    for (const [, id] of entries) {
-      expect(typeof id).toBe('string');
-      expect(id).toMatch(/^srv_/);
-    }
+  it('does not bake opaque Auth service ids into the generated registry', async () => {
+    const services = await import('./services.js');
+
+    expect(services).not.toHaveProperty('PLATFORM_SERVICE_IDS');
   });
 
   it('maps every callable service back to an application catalog entry', () => {
@@ -169,23 +158,17 @@ describe('application requirements', () => {
     expect(applicationRequirementScopes(APPLICATIONS.mail)).toEqual(
       expect.arrayContaining(['all', 'browser', 'api', 'agent_surface']),
     );
-    expect(applicationRequirementScopes(APPLICATIONS.cli)).toEqual(
-      expect.arrayContaining(['all', 'tooling', 'agent_surface']),
-    );
   });
 
   it('filters requirements for a target application', () => {
     const adminRequirementIds = requirementsForApplication('admin').map((requirement) => requirement.id);
     const mailRequirementIds = requirementsForApplication('mail').map((requirement) => requirement.id);
-    const cliRequirementIds = requirementsForApplication('cli').map((requirement) => requirement.id);
 
     expect(adminRequirementIds).toContain('shared-shell-and-launcher');
     expect(adminRequirementIds).not.toContain('organization-scoped-data');
     expect(mailRequirementIds).toContain('organization-scoped-data');
     expect(mailRequirementIds).toContain('service-registration-and-scopes');
     expect(mailRequirementIds).toContain('native-dashboard-widget');
-    expect(cliRequirementIds).toContain('agent-and-operator-surfaces');
-    expect(cliRequirementIds).not.toContain('service-registration-and-scopes');
   });
 
   it('audits one application against the catalog-backed requirements', () => {

package/src/services.ts

@@ -15,7 +15,6 @@
  */
 import {
   SERVICES,
-  PLATFORM_SERVICE_IDS,
   type ServiceId,
 } from './services.generated.js';
 import {
@@ -41,7 +40,7 @@ export {
   type ApplicationRequirementsAuditReport,
 } from './application-requirements.js';
 
-export { SERVICES, PLATFORM_SERVICE_IDS };
+export { SERVICES };
 export type { ServiceId };
 export { APPLICATIONS };
 export type { ApplicationCatalogEntry, ApplicationDeployTarget, ApplicationId };

package/src/widgets.test.ts

@@ -15,7 +15,7 @@ describe("Topolo widget contract", () => {
 
   it("stamps service metadata and timestamp onto widget drafts", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "srv_bytes",
+      serviceId: "service_bytes",
       serviceName: "Bytes",
       now: "2026-04-25T00:00:00.000Z",
       widgets: [
@@ -30,7 +30,7 @@ describe("Topolo widget contract", () => {
       success: true,
       widgets: [
         {
-          serviceId: "srv_bytes",
+          serviceId: "service_bytes",
           serviceName: "Bytes",
           lastUpdated: "2026-04-25T00:00:00.000Z",
           type: "stats",
@@ -42,12 +42,12 @@ describe("Topolo widget contract", () => {
 
   it("keeps explicit widget metadata when an app needs multiple service cards", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "srv_platform",
+      serviceId: "service_platform",
       serviceName: "Platform",
       now: "2026-04-25T00:00:00.000Z",
       widgets: [
         {
-          serviceId: "srv_crm",
+          serviceId: "service_crm",
           serviceName: "CRM",
           lastUpdated: "2026-04-24T23:00:00.000Z",
           type: "stats",
@@ -57,7 +57,7 @@ describe("Topolo widget contract", () => {
     });
 
     expect(response.widgets[0]).toMatchObject({
-      serviceId: "srv_crm",
+      serviceId: "service_crm",
       serviceName: "CRM",
       lastUpdated: "2026-04-24T23:00:00.000Z",
     });
@@ -73,7 +73,7 @@ describe("Topolo widget contract", () => {
 
   it("validates a compliant response", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "srv_crm",
+      serviceId: "service_crm",
       serviceName: "CRM",
       now: "2026-04-25T00:00:00.000Z",
       widgets: [
@@ -102,7 +102,7 @@ describe("Topolo widget contract", () => {
 
   it("keeps optional snapshot metadata for configurable TopoloOne datapoints", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "srv_admin",
+      serviceId: "service_admin",
       serviceName: "Admin",
       now: "2026-04-28T00:00:00.000Z",
       widgets: [