@shipfox/api-integration-core 13.1.0 → 14.0.0

package/src/presentation/routes/manage-connections.test.ts

@@ -1,6 +1,10 @@
+import {INTEGRATION_CONNECTION_AVAILABLE} from '@shipfox/api-integration-core-dto';
+import {sql} from 'drizzle-orm';
 import {createIntegrationProviderRegistry} from '#core/providers/registry.js';
 import {processIntegrationSecretCleanups} from '#core/secret-cleanup.js';
 import {getIntegrationConnectionById, upsertIntegrationConnection} from '#db/connections.js';
+import {db} from '#db/db.js';
+import {integrationsOutbox} from '#db/schema/outbox.js';
 import {listIntegrationSecretCleanups} from '#db/secret-cleanups.js';
 import {createTestApp, sourceProvider, useIntegrationRouteTest} from '#test/route-utils.js';
 
@@ -15,6 +19,7 @@ describe('PATCH /integration-connections/:connectionId', () => {
       externalAccountId: 'gitea-owner',
       slug: 'gitea_owner',
       displayName: 'Gitea',
+      capabilities: ['source_control'],
     });
 
     const res = await app.inject({
@@ -31,6 +36,41 @@ describe('PATCH /integration-connections/:connectionId', () => {
     expect(reloaded?.lifecycleStatus).toBe('disabled');
   });
 
+  it('publishes provider capabilities when reactivating a connection', async () => {
+    const app = await createTestApp([sourceProvider()]);
+    const connection = await upsertIntegrationConnection({
+      workspaceId: context.workspaceId,
+      provider: 'gitea',
+      externalAccountId: 'gitea-owner',
+      slug: 'gitea_owner',
+      displayName: 'Gitea',
+      lifecycleStatus: 'disabled',
+    });
+
+    const res = await app.inject({
+      method: 'PATCH',
+      url: `/integration-connections/${connection.id}`,
+      headers: {authorization: 'Bearer user'},
+      payload: {lifecycle_status: 'active'},
+    });
+
+    const [event] = await db()
+      .select({payload: integrationsOutbox.payload})
+      .from(integrationsOutbox)
+      .where(
+        sql`${integrationsOutbox.eventType} = ${INTEGRATION_CONNECTION_AVAILABLE} AND ${integrationsOutbox.payload}->>'connectionId' = ${connection.id}`,
+      );
+
+    expect(res.statusCode).toBe(200);
+    expect(event?.payload).toEqual({
+      provider: 'gitea',
+      workspaceId: context.workspaceId,
+      connectionId: connection.id,
+      slug: 'gitea_owner',
+      capabilities: ['source_control'],
+    });
+  });
+
   it('returns not-found for a missing connection', async () => {
     const app = await createTestApp([sourceProvider()]);
 
@@ -53,6 +93,7 @@ describe('PATCH /integration-connections/:connectionId', () => {
       externalAccountId: 'gitea-owner',
       slug: 'gitea_owner',
       displayName: 'Gitea',
+      capabilities: ['source_control'],
     });
 
     const res = await app.inject({
@@ -75,6 +116,7 @@ describe('PATCH /integration-connections/:connectionId', () => {
       externalAccountId: 'gitea-owner',
       slug: 'gitea_owner',
       displayName: 'Gitea',
+      capabilities: ['source_control'],
     });
 
     const res = await app.inject({
@@ -100,6 +142,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'gitea-owner',
       slug: 'gitea_owner',
       displayName: 'Gitea',
+      capabilities: ['source_control'],
     });
 
     const res = await app.inject({
@@ -133,6 +176,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -183,6 +227,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -234,6 +279,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -264,6 +310,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -297,6 +344,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -328,6 +376,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -356,6 +405,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -396,6 +446,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({
@@ -443,6 +494,7 @@ describe('DELETE /integration-connections/:connectionId', () => {
       externalAccountId: 'T123',
       slug: 'slack_acme',
       displayName: 'Slack Acme',
+      capabilities: [],
     });
 
     const res = await app.inject({

package/src/presentation/routes/manage-connections.ts

@@ -40,16 +40,20 @@ export function createUpdateIntegrationConnectionRoute(registry: IntegrationProv
       }
 
       requireWorkspaceAccess({request, workspaceId: connection.workspaceId});
+      const provider = registry
+        .list()
+        .find((candidate) => candidate.provider === connection.provider);
+      const capabilities = provider?.capabilities ?? [];
       const updated = await updateIntegrationConnectionLifecycleStatus({
         id: connection.id,
         lifecycleStatus: request.body.lifecycle_status,
+        ...(request.body.lifecycle_status === 'active' ? {capabilities} : {}),
       });
       if (!updated) {
         throw new ClientError('Integration connection not found', 'not-found', {status: 404});
       }
 
-      const provider = registry.list().find((candidate) => candidate.provider === updated.provider);
-      return toIntegrationConnectionDto(updated, {capabilities: provider?.capabilities ?? []});
+      return toIntegrationConnectionDto(updated, {capabilities});
     },
   });
 }

package/src/providers/gitea.ts

@@ -1,6 +1,8 @@
 import type {ConnectGiteaConnectionInput} from '@shipfox/api-integration-gitea';
 import type {IntegrationConnection as CoreIntegrationConnection} from '@shipfox/api-integration-spi';
 import {config} from '#config.js';
+import type {IntegrationCapability} from '#core/entities/provider.js';
+import {getIntegrationProviderCapabilities} from '#core/providers/registry.js';
 import {
   getIntegrationConnectionById,
   resolveUniqueConnectionSlug,
@@ -19,6 +21,7 @@ async function loadGiteaModuleParts(): Promise<IntegrationModuleParts> {
     db: giteaDb,
     migrationsPath: giteaMigrationsPath,
   } = await import('@shipfox/api-integration-gitea');
+  let providerCapabilities: IntegrationCapability[] = [];
 
   async function getExistingGiteaConnection(input: {
     org: string;
@@ -53,6 +56,7 @@ async function loadGiteaModuleParts(): Promise<IntegrationModuleParts> {
             slug,
             displayName: input.displayName,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           {tx},
         );
@@ -78,6 +82,7 @@ async function loadGiteaModuleParts(): Promise<IntegrationModuleParts> {
     getIntegrationConnectionById,
     coreDb: db,
   });
+  providerCapabilities = getIntegrationProviderCapabilities(integrationProvider.adapters);
 
   return {
     provider: integrationProvider,

package/src/providers/github.ts

@@ -1,6 +1,8 @@
 import type {ConnectGithubInstallationInput} from '@shipfox/api-integration-github';
 import type {IntegrationConnection as CoreIntegrationConnection} from '@shipfox/api-integration-spi';
 import {config} from '#config.js';
+import type {IntegrationCapability} from '#core/entities/provider.js';
+import {getIntegrationProviderCapabilities} from '#core/providers/registry.js';
 import {
   getIntegrationConnectionById,
   resolveUniqueConnectionSlug,
@@ -55,6 +57,7 @@ async function loadGithubModuleParts(
         }
       : undefined,
   });
+  let providerCapabilities: IntegrationCapability[] = [];
 
   async function getExistingGithubConnection(input: {
     installationId: string;
@@ -91,6 +94,7 @@ async function loadGithubModuleParts(
             slug,
             displayName: input.displayName,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           {tx},
         );
@@ -123,6 +127,7 @@ async function loadGithubModuleParts(
       ? {requireActiveWorkspaceMembership: options.requireActiveWorkspaceMembership}
       : {}),
   });
+  providerCapabilities = getIntegrationProviderCapabilities(integrationProvider.adapters);
 
   return {
     provider: integrationProvider,
@@ -131,7 +136,7 @@ async function loadGithubModuleParts(
       createGithubE2eRoutes({
         getExistingGithubConnection,
         connectGithubInstallation,
-        connectionCapabilities: ['source_control', 'agent_tools'],
+        connectionCapabilities: providerCapabilities,
       }),
     ],
     database: {

package/src/providers/jira.test.ts

@@ -46,6 +46,7 @@ describe('jiraProviderModule', () => {
       externalAccountId: cloudId,
       slug: 'jira_acme',
       displayName: 'Jira Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertJiraInstallation({
       connectionId: connection.id,
@@ -88,6 +89,7 @@ describe('jiraProviderModule', () => {
       externalAccountId: cloudId,
       slug: `jira_${cloudId}`,
       displayName: 'Jira Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertJiraInstallation({
       connectionId: connection.id,
@@ -159,6 +161,7 @@ describe('jiraProviderModule', () => {
       externalAccountId: cloudId,
       slug: 'jira_acme_again',
       displayName: 'Jira Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertJiraInstallation({
       connectionId: replacement.id,

package/src/providers/jira.ts

@@ -5,6 +5,8 @@ import type {
 } from '@shipfox/api-integration-jira';
 import type {IntegrationConnection as CoreIntegrationConnection} from '@shipfox/api-integration-spi';
 import {config} from '#config.js';
+import type {IntegrationCapability} from '#core/entities/provider.js';
+import {getIntegrationProviderCapabilities} from '#core/providers/registry.js';
 import {
   deleteIntegrationConnection,
   getIntegrationConnectionById,
@@ -45,6 +47,7 @@ async function loadJiraModuleParts(
     withJiraWebhookRegistrationLock,
   } = await import('@shipfox/api-integration-jira');
   const jira = createJiraApiClient();
+  let providerCapabilities: IntegrationCapability[] = [];
 
   async function getExistingJiraConnection(input: {
     cloudId: string;
@@ -80,6 +83,7 @@ async function loadJiraModuleParts(
             slug,
             displayName: input.displayName,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           {tx},
         );
@@ -222,6 +226,7 @@ async function loadJiraModuleParts(
           {
             id: connectionId,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           tx === undefined ? {} : {tx: tx as IntegrationTx},
         );
@@ -240,6 +245,7 @@ async function loadJiraModuleParts(
         : {}),
     },
   });
+  providerCapabilities = getIntegrationProviderCapabilities(integrationProvider.adapters);
 
   return {
     provider: integrationProvider,

package/src/providers/linear.test.ts

@@ -36,6 +36,7 @@ describe('linearProviderModule', () => {
       externalAccountId: organizationId,
       slug: 'linear_acme',
       displayName: 'Linear Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertLinearInstallation({
       connectionId: connection.id,
@@ -67,6 +68,7 @@ describe('linearProviderModule', () => {
       externalAccountId: organizationId,
       slug: 'linear_acme_again',
       displayName: 'Linear Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertLinearInstallation({
       connectionId: replacement.id,
@@ -107,6 +109,7 @@ describe('linearProviderModule', () => {
       externalAccountId: organizationId,
       slug: 'linear_acme',
       displayName: 'Linear Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertLinearInstallation({
       connectionId: connection.id,

package/src/providers/linear.ts

@@ -4,6 +4,8 @@ import type {
 } from '@shipfox/api-integration-linear';
 import type {IntegrationConnection as CoreIntegrationConnection} from '@shipfox/api-integration-spi';
 import {config} from '#config.js';
+import type {IntegrationCapability} from '#core/entities/provider.js';
+import {getIntegrationProviderCapabilities} from '#core/providers/registry.js';
 import {
   deleteIntegrationConnection,
   getIntegrationConnectionById,
@@ -36,6 +38,7 @@ async function loadLinearModuleParts(
     migrationsPath: linearMigrationsPath,
     upsertLinearInstallation,
   } = await import('@shipfox/api-integration-linear');
+  let providerCapabilities: IntegrationCapability[] = [];
 
   async function getExistingLinearConnection(input: {
     organizationId: string;
@@ -72,6 +75,7 @@ async function loadLinearModuleParts(
             slug,
             displayName: input.displayName,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           {tx},
         );
@@ -160,6 +164,7 @@ async function loadLinearModuleParts(
         : {}),
     },
   });
+  providerCapabilities = getIntegrationProviderCapabilities(integrationProvider.adapters);
 
   return {
     provider: integrationProvider,
@@ -170,7 +175,7 @@ async function loadLinearModuleParts(
         getExistingLinearConnection,
         connectLinearInstallation,
         disconnectLinearInstallation,
-        connectionCapabilities: ['agent_tools'],
+        connectionCapabilities: providerCapabilities,
       }),
     ],
     database: {

package/src/providers/modules.test.ts

@@ -1,5 +1,13 @@
+import {INTEGRATION_CONNECTION_AVAILABLE} from '@shipfox/api-integration-core-dto';
+import {closeApp, createApp} from '@shipfox/node-fastify';
+import {eq, sql} from 'drizzle-orm';
+import {db} from '#db/db.js';
+import {integrationConnections} from '#db/schema/connections.js';
+import {integrationsOutbox} from '#db/schema/outbox.js';
+
 describe('loadEnabledProviderModules', () => {
-  afterEach(() => {
+  afterEach(async () => {
+    await closeApp();
     vi.unstubAllEnvs();
     vi.resetModules();
   });
@@ -63,4 +71,59 @@ describe('loadEnabledProviderModules', () => {
     expect(parts.map((part) => part.provider.provider)).toEqual(['jira', 'cron', 'webhook']);
     expect(parts[0]?.provider).toMatchObject({provider: 'jira', displayName: 'Jira'});
   });
+
+  it('publishes registry-derived capabilities for a GitHub connect flow', async () => {
+    vi.stubEnv('INTEGRATIONS_ENABLE_GITHUB_PROVIDER', 'true');
+    vi.resetModules();
+
+    const {createPostgresClient} = await import('@shipfox/node-postgres');
+    createPostgresClient();
+    const {loadEnabledProviderModules} = await import('#providers/modules.js');
+    const parts = await loadEnabledProviderModules();
+    const githubPart = parts.find((part) => part.provider.provider === 'github');
+    if (!githubPart?.e2eRoutes) throw new Error('GitHub E2E routes are not configured');
+
+    const workspaceId = crypto.randomUUID();
+    const installationId = Number.parseInt(crypto.randomUUID().replaceAll('-', '').slice(0, 8), 16);
+    try {
+      const app = await createApp({routes: githubPart.e2eRoutes, swagger: false});
+      const response = await app.inject({
+        method: 'POST',
+        url: '/integrations/github-connections',
+        payload: {
+          workspace_id: workspaceId,
+          installation_id: installationId,
+          account_login: 'shipfox-e2e',
+          display_name: 'GitHub E2E',
+          installer_user_id: crypto.randomUUID(),
+        },
+      });
+      const connection = response.json();
+
+      expect(response.statusCode).toBe(201);
+      expect(connection).toMatchObject({id: expect.any(String)});
+
+      const [event] = await db()
+        .select({payload: integrationsOutbox.payload})
+        .from(integrationsOutbox)
+        .where(
+          sql`${integrationsOutbox.eventType} = ${INTEGRATION_CONNECTION_AVAILABLE} AND ${integrationsOutbox.payload}->>'connectionId' = ${connection.id}`,
+        );
+
+      expect(event?.payload).toEqual({
+        provider: 'github',
+        workspaceId,
+        connectionId: connection.id,
+        slug: 'github_shipfox_e2e',
+        capabilities: ['source_control', 'agent_tools'],
+      });
+    } finally {
+      await db()
+        .delete(integrationsOutbox)
+        .where(sql`${integrationsOutbox.payload}->>'workspaceId' = ${workspaceId}`);
+      await db()
+        .delete(integrationConnections)
+        .where(eq(integrationConnections.workspaceId, workspaceId));
+    }
+  });
 });

package/src/providers/sentry.ts

@@ -1,6 +1,7 @@
 import type {ConnectSentryInstallationInput} from '@shipfox/api-integration-sentry';
 import type {IntegrationConnection as CoreIntegrationConnection} from '@shipfox/api-integration-spi';
 import {config} from '#config.js';
+import type {IntegrationCapability} from '#core/entities/provider.js';
 import {
   getIntegrationConnectionById,
   resolveUniqueConnectionSlug,
@@ -22,6 +23,7 @@ async function loadSentryModuleParts(): Promise<IntegrationModuleParts> {
     db: sentryDb,
     migrationsPath: sentryMigrationsPath,
   } = await import('@shipfox/api-integration-sentry');
+  const providerCapabilities: IntegrationCapability[] = [];
 
   async function getConnectionById(
     id: string,
@@ -54,6 +56,7 @@ async function loadSentryModuleParts(): Promise<IntegrationModuleParts> {
             slug,
             displayName: input.displayName,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           {tx},
         );

package/src/providers/slack.test.ts

@@ -38,6 +38,7 @@ describe('slackProviderModule', () => {
       externalAccountId: teamId,
       slug: `slack_${teamId}`,
       displayName: 'Slack Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertSlackInstallation({
       connectionId: connection.id,
@@ -90,6 +91,7 @@ describe('slackProviderModule', () => {
       externalAccountId: teamId,
       slug: `slack_${teamId}`,
       displayName: 'Slack Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertSlackInstallation({
       connectionId: connection.id,
@@ -121,6 +123,7 @@ describe('slackProviderModule', () => {
       externalAccountId: teamId,
       slug: `slack_${teamId}_again`,
       displayName: 'Slack Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertSlackInstallation({
       connectionId: replacement.id,
@@ -167,6 +170,7 @@ describe('slackProviderModule', () => {
       externalAccountId: teamId,
       slug: `slack_${teamId}`,
       displayName: 'Slack Acme',
+      capabilities: ['agent_tools'],
     });
     await upsertSlackInstallation({
       connectionId: connection.id,

package/src/providers/slack.ts

@@ -4,6 +4,8 @@ import type {
 } from '@shipfox/api-integration-slack';
 import type {IntegrationConnection as CoreIntegrationConnection} from '@shipfox/api-integration-spi';
 import {config} from '#config.js';
+import type {IntegrationCapability} from '#core/entities/provider.js';
+import {getIntegrationProviderCapabilities} from '#core/providers/registry.js';
 import {
   deleteIntegrationConnection,
   getIntegrationConnectionById,
@@ -39,6 +41,7 @@ async function loadSlackModuleParts(
     slackSecretsNamespace,
     upsertSlackInstallation,
   } = await import('@shipfox/api-integration-slack');
+  let providerCapabilities: IntegrationCapability[] = [];
 
   async function getExistingSlackConnection(input: {
     teamId: string;
@@ -75,6 +78,7 @@ async function loadSlackModuleParts(
             slug,
             displayName: input.displayName,
             lifecycleStatus: 'active',
+            capabilities: providerCapabilities,
           },
           {tx},
         );
@@ -163,6 +167,7 @@ async function loadSlackModuleParts(
         : {}),
     },
   });
+  providerCapabilities = getIntegrationProviderCapabilities(integrationProvider.adapters);
 
   return {
     provider: integrationProvider,
@@ -173,7 +178,7 @@ async function loadSlackModuleParts(
         getExistingSlackConnection,
         connectSlackInstallation,
         disconnectSlackInstallation,
-        connectionCapabilities: ['agent_tools'],
+        connectionCapabilities: providerCapabilities,
       }),
     ],
     database: {

package/test/route-utils.ts

@@ -69,6 +69,10 @@ export function sourceProvider(overrides: Partial<IntegrationProvider> = {}): In
           throw new Error('not used');
         },
         resolveTriggerReference: () => null,
+        resolveRef: async () => {
+          await Promise.resolve();
+          throw new Error('not used');
+        },
       },
     },
     ...overrides,