@topolo/sdk 0.4.10 → 0.5.0

package/src/modules/identity.test.ts

@@ -9,7 +9,7 @@ function stubFetch(responses: Array<{ status?: number; body?: unknown }>) {
   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_auth' } }), {
+      return new Response(JSON.stringify({ success: true, data: { app_id: 'service_auth' } }), {
         status: 200,
         headers: { 'Content-Type': 'application/json' },
       });
@@ -48,12 +48,12 @@ function clientWith(
     credential,
     agent: AGENT,
     fetch: impl,
-    serviceIds: { auth: 'service_auth' },
+    appIds: { auth: 'service_auth' },
   });
 }
 
 beforeEach(() => {
-  TopoloClient.clearServiceIdCache();
+  TopoloClient.clearAppIdCache();
 });
 
 describe('IdentityModule.whoami', () => {
@@ -93,7 +93,7 @@ describe('IdentityModule.whoami', () => {
       {
         body: {
           credentialType: 'api_key',
-          user: { id: 'svc_1', email: 'svc@example.com' },
+          user: { id: 'usr_1', email: 'user@example.com' },
           organization: { id: 'org_2', slug: 'beta' },
         },
       },

package/src/modules/p2p.test.ts

@@ -9,7 +9,7 @@ function stubFetch(responses: Array<{ status?: number; body?: unknown }>) {
   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' } }), {
+      return new Response(JSON.stringify({ success: true, data: { app_id: 'service_p2p' } }), {
         status: 200,
         headers: { 'Content-Type': 'application/json' },
       });
@@ -19,14 +19,14 @@ function stubFetch(responses: Array<{ status?: number; body?: unknown }>) {
         data: {
           services: [
             {
-              service_id: 'srv_p2p',
+              app_id: 'app_p2p',
               app_slug: 'topolo-p2p',
               name: 'Topolo P2P',
               api_base_url: 'https://p2p.topolo.app',
               launch_url: 'https://p2p.topolo.app',
               surface_kind: 'api',
               status: 'active',
-              granted_permissions: ['srv_p2p.directory:read'],
+              granted_permissions: ['app_p2p.directory:read'],
             },
           ],
         },
@@ -61,7 +61,7 @@ function clientWith(impl: typeof fetch) {
 
 describe('P2PModule.listDirectory', () => {
   beforeEach(() => {
-    TopoloClient.clearServiceIdCache();
+    TopoloClient.clearAppIdCache();
   });
 
   it('GETs /api/directory/capabilities and unwraps the envelope', async () => {
@@ -75,7 +75,6 @@ describe('P2PModule.listDirectory', () => {
                 capability_id: 'cap_1',
                 publisher_org_id: 'org_a',
                 app_id: 'app_x',
-                service_id: 'service_x',
                 name: 'Demo',
                 description: 'A capability',
                 category: 'other',
@@ -117,7 +116,6 @@ describe('P2PModule.publishCapability', () => {
               capability_id: 'cap_z',
               publisher_org_id: 'org_p',
               app_id: 'app_x',
-              service_id: 'service_x',
               name: 'Demo',
               description: 'A capability',
               category: 'other',
@@ -140,7 +138,6 @@ describe('P2PModule.publishCapability', () => {
       capability_id: 'cap_z',
       publisher_org_id: 'org_p',
       app_id: 'app_x',
-      service_id: 'service_x',
       name: 'Demo',
       description: 'A capability',
       category: 'other',
@@ -174,7 +171,6 @@ describe('P2PModule.publishCapability', () => {
       capability_id: 'cap_other',
       publisher_org_id: 'org_target',
       app_id: 'app_x',
-      service_id: 'service_x',
       name: 'Demo',
       description: '...',
       category: 'other',

package/src/modules/p2p.ts

@@ -6,7 +6,6 @@ export interface P2PCapability {
   capability_id: string;
   publisher_org_id: string;
   app_id: string;
-  service_id: string;
   name: string;
   description: string;
   category: string;
@@ -26,7 +25,6 @@ export interface PublishCapabilityInput {
   capability_id: string;
   publisher_org_id: string;
   app_id: string;
-  service_id: string;
   name: string;
   description: string;
   category: string;
@@ -231,7 +229,6 @@ export class P2PModule {
         capability_id: input.capability_id,
         publisher_org_id: input.publisher_org_id,
         app_id: input.app_id,
-        service_id: input.service_id,
         name: input.name,
         description: input.description,
         category: input.category,

package/src/services.test.ts

@@ -93,14 +93,14 @@ describe('service registry', () => {
       Object.values(APPLICATIONS).flatMap((application) => [...application.services]),
     );
 
-    for (const serviceId of Object.keys(DEFAULT_SERVICE_URLS)) {
-      expect(applicationServices.has(serviceId)).toBe(true);
+    for (const appId of Object.keys(DEFAULT_SERVICE_URLS)) {
+      expect(applicationServices.has(appId)).toBe(true);
     }
   });
 
   it('resolves services from explicit catalog API URLs', () => {
     expect(resolveCatalogServiceUrl({
-      serviceId: 'srv_crm',
+      appId: 'app_crm',
       slug: 'topolo-crm',
       name: 'Topolo CRM',
       launchUrl: 'https://crm.topolo.app',
@@ -110,7 +110,7 @@ describe('service registry', () => {
 
   it('does not infer callable URLs from launcher URLs', () => {
     expect(resolveCatalogServiceUrl({
-      serviceId: 'srv_crm',
+      appId: 'app_crm',
       slug: 'topolo-crm',
       name: 'Topolo CRM',
       launchUrl: 'https://crm.stg.topolo.us',
@@ -119,7 +119,7 @@ describe('service registry', () => {
 
   it('prefers explicit catalog API URLs over launch URLs', () => {
     expect(resolveCatalogServiceUrl({
-      serviceId: 'srv_future',
+      appId: 'app_future',
       slug: 'topolo-future',
       name: 'Future',
       launchUrl: 'https://future.topolo.app',
@@ -137,11 +137,11 @@ describe('application catalog', () => {
     const registeredServices = new Set<string>(Object.keys(DEFAULT_SERVICE_URLS));
 
     for (const application of Object.values(APPLICATIONS)) {
-      for (const serviceId of application.services) {
-        expect(registeredServices.has(serviceId)).toBe(true);
+      for (const appId of application.services) {
+        expect(registeredServices.has(appId)).toBe(true);
       }
       for (const target of application.deployTargets) {
-        if (target.serviceId) expect(registeredServices.has(target.serviceId)).toBe(true);
+        if (target.appId) expect(registeredServices.has(target.appId)).toBe(true);
       }
     }
   });
@@ -152,19 +152,19 @@ describe('application catalog', () => {
     // studio web worker (web.topolo.app/api); the API worker is de-routed and
     // no longer carries a public service registration.
     expect(
-      APPLICATIONS.web.deployTargets.find((target) => target.name === 'topolo-web-studio')?.serviceId,
+      APPLICATIONS.web.deployTargets.find((target) => target.name === 'topolo-web-studio')?.appId,
     ).toBe('web');
     expect(
-      APPLICATIONS.web.deployTargets.find((target) => target.name === 'topolo-web-api')?.serviceId,
+      APPLICATIONS.web.deployTargets.find((target) => target.name === 'topolo-web-api')?.appId,
     ).toBeNull();
     // topolo-web-runtime (Astro, sites.topolo.app) deploys from its own wrangler
     // config and is intentionally listed as non-callable, so it can never be an
     // agent-callable service.
     expect(
-      APPLICATIONS.web.deployTargets.find((target) => target.name === 'topolo-web-runtime')?.serviceId,
+      APPLICATIONS.web.deployTargets.find((target) => target.name === 'topolo-web-runtime')?.appId,
     ).toBeNull();
     const callableTargetNames = APPLICATIONS.web.deployTargets
-      .filter((target) => target.serviceId)
+      .filter((target) => target.appId)
       .map((target) => target.name);
     expect(callableTargetNames).not.toContain('topolo-web-runtime');
   });
@@ -174,14 +174,14 @@ describe('application catalog', () => {
     // Front-door standardization: the callable `feed` service is served by the
     // feed web worker (feed.topolo.app/api); the API worker is de-routed.
     expect(
-      APPLICATIONS.feed.deployTargets.find((target) => target.name === 'topolo-feed-web')?.serviceId,
+      APPLICATIONS.feed.deployTargets.find((target) => target.name === 'topolo-feed-web')?.appId,
     ).toBe('feed');
     expect(
-      APPLICATIONS.feed.deployTargets.find((target) => target.name === 'topolo-feed-api')?.serviceId,
+      APPLICATIONS.feed.deployTargets.find((target) => target.name === 'topolo-feed-api')?.appId,
     ).toBeNull();
     // The analytics ingestion worker keeps its own host and is non-callable.
     expect(
-      APPLICATIONS.feed.deployTargets.find((target) => target.name === 'topolo-feed-analytics-api')?.serviceId,
+      APPLICATIONS.feed.deployTargets.find((target) => target.name === 'topolo-feed-analytics-api')?.appId,
     ).toBeNull();
   });
 });

package/src/services.ts

@@ -48,7 +48,7 @@ export type { ApplicationCatalogEntry, ApplicationDeployTarget, ApplicationId };
 export const DEFAULT_SERVICE_URLS: Record<ServiceId, string> = SERVICES;
 
 export interface ServiceCatalogUrlEntry {
-  serviceId?: string | null;
+  appId?: string | null;
   slug?: string | null;
   name?: string | null;
   apiBaseUrl?: string | null;
@@ -85,7 +85,7 @@ export function resolveServiceUrl(
 }
 
 function catalogServiceKeyCandidates(entry: ServiceCatalogUrlEntry, preferred?: string): string[] {
-  const values = [preferred, entry.slug, entry.name, entry.serviceId].filter(
+  const values = [preferred, entry.slug, entry.name, entry.appId].filter(
     (value): value is string => typeof value === 'string' && value.trim().length > 0,
   );
   const keys = new Set<string>();

package/src/widgets.test.ts

@@ -13,7 +13,7 @@ import {
 } from "./widgets.js";
 
 const BASE = {
-  serviceId: "service_x",
+  appId: "service_x",
   serviceName: "X",
   lastUpdated: "2026-04-25T00:00:00.000Z",
   snapshot: { summary: "A summary that is long enough to read." },
@@ -27,7 +27,7 @@ describe("Topolo widget contract", () => {
 
   it("stamps service metadata and timestamp onto widget drafts", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "service_bytes",
+      appId: "service_bytes",
       serviceName: "Bytes",
       now: "2026-04-25T00:00:00.000Z",
       widgets: [
@@ -45,7 +45,7 @@ describe("Topolo widget contract", () => {
       success: true,
       widgets: [
         {
-          serviceId: "service_bytes",
+          appId: "service_bytes",
           serviceName: "Bytes",
           lastUpdated: "2026-04-25T00:00:00.000Z",
           type: "stats",
@@ -60,12 +60,12 @@ describe("Topolo widget contract", () => {
 
   it("keeps explicit widget metadata when an app needs multiple service cards", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "service_platform",
+      appId: "service_platform",
       serviceName: "Platform",
       now: "2026-04-25T00:00:00.000Z",
       widgets: [
         {
-          serviceId: "service_crm",
+          appId: "service_crm",
           serviceName: "CRM",
           lastUpdated: "2026-04-24T23:00:00.000Z",
           type: "stats",
@@ -78,7 +78,7 @@ describe("Topolo widget contract", () => {
     });
 
     expect(response.widgets[0]).toMatchObject({
-      serviceId: "service_crm",
+      appId: "service_crm",
       serviceName: "CRM",
       lastUpdated: "2026-04-24T23:00:00.000Z",
     });
@@ -94,7 +94,7 @@ describe("Topolo widget contract", () => {
 
   it("validates a compliant response", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "service_crm",
+      appId: "service_crm",
       serviceName: "CRM",
       now: "2026-04-25T00:00:00.000Z",
       widgets: [
@@ -126,7 +126,7 @@ describe("Topolo widget contract", () => {
 
   it("keeps required snapshot metadata for configurable TopoloOne datapoints", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "service_admin",
+      appId: "service_admin",
       serviceName: "Admin",
       now: "2026-04-28T00:00:00.000Z",
       widgets: [
@@ -172,7 +172,7 @@ describe("Topolo widget contract", () => {
       success: true,
       widgets: [
         {
-          serviceId: "",
+          appId: "",
           serviceName: "Bad App",
           lastUpdated: "",
           type: "calendar",
@@ -182,7 +182,7 @@ describe("Topolo widget contract", () => {
 
     expect(result.ok).toBe(false);
     expect(result.errors).toEqual([
-      "widgets[0].serviceId must be a non-empty string",
+      "widgets[0].appId must be a non-empty string",
       "widgets[0].lastUpdated must be a non-empty string",
       "widgets[0].snapshot must be an object",
       "widgets[0].type must be a supported Topolo widget type",
@@ -213,7 +213,7 @@ describe("Topolo widget contract", () => {
 
   it("derives lastUpdated from a Date instance passed as now", () => {
     const response = createTopoloWidgetResponse({
-      serviceId: "service_x",
+      appId: "service_x",
       serviceName: "X",
       now: new Date("2026-05-01T12:00:00.000Z"),
       widgets: [
@@ -230,7 +230,7 @@ describe("Topolo widget contract", () => {
   it("generates a current timestamp when now is omitted", () => {
     const before = Date.now();
     const response = createTopoloWidgetResponse({
-      serviceId: "service_x",
+      appId: "service_x",
       serviceName: "X",
       widgets: [
         {
@@ -362,7 +362,7 @@ describe("Topolo widget contract", () => {
       success: true,
       widgets: [
         {
-          serviceId: "s",
+          appId: "s",
           serviceName: "n",
           lastUpdated: "2026-04-25T00:00:00.000Z",
           type: "stats",
@@ -398,7 +398,7 @@ describe("Topolo widget contract", () => {
       success: true,
       widgets: [
         {
-          serviceId: "service_bad",
+          appId: "service_bad",
           serviceName: "Bad App",
           lastUpdated: "2026-04-25T00:00:00.000Z",
           type: "stats",

package/src/widgets.ts

@@ -9,7 +9,7 @@ export type TopoloWidgetType =
   | "chart";
 
 export interface TopoloWidgetBase {
-  serviceId: string;
+  appId: string;
   serviceName: string;
   serviceIcon?: string;
   type: TopoloWidgetType;
@@ -114,9 +114,9 @@ export type TopoloWidget =
 
 type TopoloWidgetWithOptionalMetadata<T extends TopoloWidget> = Omit<
   T,
-  "serviceId" | "serviceName" | "lastUpdated"
+  "appId" | "serviceName" | "lastUpdated"
 > &
-  Partial<Pick<TopoloWidgetBase, "serviceId" | "serviceName" | "lastUpdated">>;
+  Partial<Pick<TopoloWidgetBase, "appId" | "serviceName" | "lastUpdated">>;
 
 export type TopoloWidgetDraft = TopoloWidget extends infer Widget
   ? Widget extends TopoloWidget
@@ -131,7 +131,7 @@ export interface TopoloWidgetApiResponse {
 }
 
 export interface TopoloWidgetResponseOptions {
-  serviceId: string;
+  appId: string;
   serviceName: string;
   widgets: TopoloWidgetDraft[];
   now?: Date | string;
@@ -143,7 +143,7 @@ export interface TopoloWidgetValidationResult {
 }
 
 export interface TopoloWidgetConfig {
-  serviceId: string;
+  appId: string;
   serviceName: string;
   baseUrl: string;
   widgetEndpoint: string;
@@ -182,7 +182,7 @@ export function defineTopoloChartWidget(
 }
 
 export function createTopoloWidgetResponse({
-  serviceId,
+  appId,
   serviceName,
   widgets,
   now,
@@ -193,7 +193,7 @@ export function createTopoloWidgetResponse({
     success: true,
     widgets: widgets.map((widget) => ({
       ...widget,
-      serviceId: widget.serviceId?.trim() || serviceId,
+      appId: widget.appId?.trim() || appId,
       serviceName: widget.serviceName?.trim() || serviceName,
       lastUpdated: widget.lastUpdated?.trim() || lastUpdated,
     })) as TopoloWidget[],
@@ -249,7 +249,7 @@ function validateWidget(widget: unknown, path: string, errors: string[]) {
     return;
   }
 
-  validateNonEmptyString(widget.serviceId, `${path}.serviceId`, errors);
+  validateNonEmptyString(widget.appId, `${path}.appId`, errors);
   validateNonEmptyString(widget.serviceName, `${path}.serviceName`, errors);
   validateNonEmptyString(widget.lastUpdated, `${path}.lastUpdated`, errors);
   validateSnapshotMetadata(widget.snapshot, `${path}.snapshot`, errors);