@shipfox/api-definitions 6.0.0 → 7.1.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/api-definitions",
   "license": "MIT",
-  "version": "6.0.0",
+  "version": "7.1.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -31,7 +31,7 @@
     "yaml": "^2.8.3",
     "zod": "^4.4.3",
     "@shipfox/api-agent-dto": "6.0.0",
-    "@shipfox/api-auth-context": "6.0.0",
+    "@shipfox/api-auth-context": "7.1.0",
     "@shipfox/api-definitions-dto": "6.0.0",
     "@shipfox/api-integration-core-dto": "6.0.0",
     "@shipfox/api-projects-dto": "6.0.0",
@@ -40,34 +40,16 @@
     "@shipfox/expression": "1.1.3",
     "@shipfox/inter-module": "0.2.0",
     "@shipfox/node-drizzle": "0.3.2",
-    "@shipfox/node-fastify": "0.2.4",
-    "@shipfox/node-module": "0.4.0",
-    "@shipfox/node-opentelemetry": "0.5.2",
+    "@shipfox/node-fastify": "0.3.0",
+    "@shipfox/node-error-monitoring": "0.2.0",
+    "@shipfox/node-module": "0.5.0",
+    "@shipfox/node-opentelemetry": "0.6.0",
     "@shipfox/node-outbox": "0.2.4",
     "@shipfox/node-postgres": "0.4.2",
-    "@shipfox/node-temporal": "0.3.2",
+    "@shipfox/node-temporal": "0.4.0",
     "@shipfox/runner-labels": "0.1.1",
     "@shipfox/workflow-document": "2.1.1"
   },
-  "devDependencies": {
-    "@temporalio/client": "1.18.1",
-    "@temporalio/testing": "1.18.1",
-    "@temporalio/worker": "1.18.1",
-    "@types/js-yaml": "^4.0.9",
-    "@types/pg": "^8.15.5",
-    "drizzle-kit": "^0.31.10",
-    "fastify": "^5.3.3",
-    "fastify-type-provider-zod": "^6.0.0",
-    "fishery": "^2.4.0",
-    "@shipfox/api-integration-core": "6.0.0",
-    "@shipfox/biome": "1.8.2",
-    "@shipfox/depcruise": "1.0.2",
-    "@shipfox/regex": "0.2.2",
-    "@shipfox/swc": "1.2.6",
-    "@shipfox/ts-config": "1.3.8",
-    "@shipfox/typescript": "1.1.7",
-    "@shipfox/vitest": "1.2.3"
-  },
   "scripts": {
     "build": "shipfox-swc && shipfox-temporal-bundle dist/temporal/workflows/index.js",
     "check": "shipfox-biome-check",

package/src/temporal/activities/sync-activities.test.ts

@@ -1,6 +1,7 @@
 import type {IntegrationSourceControlService} from '@shipfox/api-integration-core';
 import {integrationsInterModuleContract} from '@shipfox/api-integration-core-dto';
 import {createInterModuleKnownError} from '@shipfox/inter-module';
+import {isErrorReported} from '@shipfox/node-error-monitoring';
 import {ApplicationFailure} from '@temporalio/common';
 import {sql} from 'drizzle-orm';
 import {db, definitionSyncStates} from '#db/index.js';
@@ -141,6 +142,8 @@ describe('definition sync activities', () => {
         type: 'unknown',
         message: 'temporary outage',
       });
+      const error = await result.catch((error: unknown) => error);
+      expect(isErrorReported(error)).toBe(false);
     });
 
     it('preserves retryable provider error codes for workflow-level failure persistence', async () => {
@@ -170,6 +173,8 @@ describe('definition sync activities', () => {
         type: 'provider-timeout',
         message: 'integrations.resolveSourceRepository: provider-failure',
       });
+      const error = await result.catch((error: unknown) => error);
+      expect(isErrorReported(error)).toBe(true);
     });
   });
 

package/src/temporal/activities/sync-activities.ts

@@ -1,4 +1,5 @@
 import type {IntegrationsModuleClient} from '@shipfox/api-integration-core-dto';
+import {markErrorReported} from '@shipfox/node-error-monitoring';
 import {Context} from '@temporalio/activity';
 import {ApplicationFailure} from '@temporalio/common';
 import type {DefinitionSyncErrorCode} from '#core/entities/sync-state.js';
@@ -183,9 +184,10 @@ async function runWithPermanentTranslation<T>(operation: () => Promise<T>): Prom
       throw error;
     }
     const failure = classifySyncFailure(error);
-    if (!failure.retryable) {
-      throw ApplicationFailure.nonRetryable(failure.message, failure.code);
-    }
-    throw ApplicationFailure.retryable(failure.message, failure.code);
+    const translatedError = failure.retryable
+      ? ApplicationFailure.retryable(failure.message, failure.code)
+      : ApplicationFailure.nonRetryable(failure.message, failure.code);
+    if (failure.code !== 'unknown') markErrorReported(translatedError);
+    throw translatedError;
   }
 }