@shipfox/api-logs 11.0.0 → 12.0.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/api-logs",
   "license": "MIT",
-  "version": "11.0.0",
+  "version": "12.0.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -25,18 +25,18 @@
     "@temporalio/workflow": "1.18.1",
     "drizzle-orm": "^0.45.2",
     "zod": "^4.4.3",
-    "@shipfox/api-auth-context": "11.0.0",
-    "@shipfox/api-logs-dto": "10.2.0",
-    "@shipfox/api-workflows-dto": "10.0.0",
-    "@shipfox/workflow-document": "2.1.3",
+    "@shipfox/api-auth-context": "12.0.0",
+    "@shipfox/api-logs-dto": "12.0.0",
+    "@shipfox/workflow-document": "3.0.0",
+    "@shipfox/api-workflows-dto": "12.0.0",
     "@shipfox/config": "1.2.4",
-    "@shipfox/node-drizzle": "0.3.4",
+    "@shipfox/node-drizzle": "0.3.5",
     "@shipfox/node-error-monitoring": "0.3.0",
-    "@shipfox/node-fastify": "0.4.0",
-    "@shipfox/node-module": "1.0.4",
+    "@shipfox/node-module": "1.0.5",
+    "@shipfox/node-fastify": "0.4.1",
     "@shipfox/node-opentelemetry": "0.6.3",
     "@shipfox/node-outbox": "0.2.6",
-    "@shipfox/node-postgres": "0.4.4",
+    "@shipfox/node-postgres": "0.5.0",
     "@shipfox/node-temporal": "0.4.4"
   },
   "imports": {

package/src/config.ts

@@ -97,7 +97,7 @@ if (config.LOG_READ_INLINE_MAX_BYTES < 1) {
 }
 
 // A whole agent_session line is one record in one body. If the body limit is below the
-// line cap, a legitimate large session line could never fit in one request body — Fastify
+// line cap, a legitimate large session line could never fit in one request body: Fastify
 // would 413 it before the per-line validator runs. Fail fast at startup instead.
 if (config.LOG_APPEND_BODY_LIMIT_BYTES < config.LOG_MAX_SESSION_LINE_BYTES) {
   throw new Error(

package/src/core/append-logs.test.ts

@@ -193,6 +193,7 @@ describe('appendLogs', () => {
       const workflows = createFakeInterModuleClients({
         workflows: defineInterModulePresentation(workflowsInterModuleContract, {
           startRunFromTrigger: vi.fn(),
+          resolveWorkflowRunTriggerReference: vi.fn(),
           deliverEventToJobListener: vi.fn(),
           getStepLogContext: () => ({harness: 'claude' as const}),
           getLeasedAgentToolContext: vi.fn(),
@@ -238,6 +239,7 @@ describe('appendLogs', () => {
       const workflows = createFakeInterModuleClients({
         workflows: defineInterModulePresentation(workflowsInterModuleContract, {
           startRunFromTrigger: vi.fn(),
+          resolveWorkflowRunTriggerReference: vi.fn(),
           deliverEventToJobListener: vi.fn(),
           getStepLogContext: () => ({harness: 'claude' as const}),
           getLeasedAgentToolContext: vi.fn(),
@@ -318,6 +320,7 @@ describe('appendLogs', () => {
       const workflows = createFakeInterModuleClients({
         workflows: defineInterModulePresentation(workflowsInterModuleContract, {
           startRunFromTrigger: vi.fn(),
+          resolveWorkflowRunTriggerReference: vi.fn(),
           deliverEventToJobListener: vi.fn(),
           getStepLogContext: () => ({harness: 'claude' as const}),
           getLeasedAgentToolContext: vi.fn(),
@@ -375,6 +378,7 @@ describe('appendLogs', () => {
       const workflows = createFakeInterModuleClients({
         workflows: defineInterModulePresentation(workflowsInterModuleContract, {
           startRunFromTrigger: vi.fn(),
+          resolveWorkflowRunTriggerReference: vi.fn(),
           deliverEventToJobListener: vi.fn(),
           getStepLogContext: () => ({harness: 'claude' as const}),
           getLeasedAgentToolContext: vi.fn(),

package/src/core/append-logs.ts

@@ -124,7 +124,7 @@ function parseAppendBody(body: Buffer): ParsedBody {
 
 /**
  * A line that fails the raw write union but is a valid record under the read union
- * can only be a server-only `capped`/`runner_lost` tombstone — i.e. a forgery
+ * can only be a server-only `capped`/`runner_lost` tombstone: i.e. a forgery
  * attempt. Returns its type for the audit warn, or undefined for plain garbage.
  */
 function detectForgedType(line: string): string | undefined {

package/src/core/budget.ts

@@ -7,7 +7,7 @@ export interface AllowedBudgetParams {
 
 /**
  * Accrual budget in stored bytes: `base + rate * elapsedMinutes`, floored to an
- * integer. No hard ceiling — job duration limits bound total volume.
+ * integer. No hard ceiling: job duration limits bound total volume.
  */
 export function allowedBudget({
   baseBytes,

package/src/core/errors.ts

@@ -8,10 +8,10 @@ export class OffsetGapError extends Error {
 
 /**
  * The append body is not whole, newline-terminated records of the raw log
- * record contract. `forgedType` is set only for the detectable forgery case — a
+ * record contract. `forgedType` is set only for the detectable forgery case: a
  * line that is a valid server-only record (`capped`/`runner_lost`) under the read
- * union but is not valid on the raw write path — so the append path can emit a narrowed audit warn
- * without logging the payload.
+ * union but is not valid on the raw write path. The append path can emit a
+ * narrowed audit warning without logging the payload.
  */
 export class MalformedLogChunkError extends Error {
   constructor(
@@ -27,7 +27,7 @@ export class MalformedLogChunkError extends Error {
  * The lease's `(workspaceId, projectId, workflowRunAttemptId)` does not match the values
  * stamped on the existing stream row. Since these are functionally determined
  * by `jobId` via workflows FKs, a mismatch implies a forged token or a
- * cross-job lease confusion — never a legitimate request.
+ * cross-job lease confusion. It is never a legitimate request.
  */
 export class LeaseStreamMismatchError extends Error {
   constructor() {

package/src/core/finalize-attempt-stream.test.ts

@@ -96,6 +96,7 @@ describe('finalizeAttemptLogStream', () => {
     const workflows = createFakeInterModuleClients({
       workflows: defineInterModulePresentation(workflowsInterModuleContract, {
         startRunFromTrigger: vi.fn(),
+        resolveWorkflowRunTriggerReference: vi.fn(),
         deliverEventToJobListener: vi.fn(),
         getStepLogContext: () => ({harness: 'claude' as const}),
         getLeasedAgentToolContext: vi.fn(),

package/src/db/schema/attempt-streams.ts

@@ -17,7 +17,7 @@ import {pgTable} from './common.js';
 
 /**
  * One stream per (job, step, attempt). Identity is scoped by `job_id` (from the
- * lease), so a lease can only ever reach its own job's streams — cross-job log
+ * lease), so a lease can only ever reach its own job's streams; cross-job log
  * injection is structurally impossible. `workspace_id`, `project_id`, and `workflow_run_attempt_id`
  * are stamped from the lease at create time; they are functionally determined by
  * `job_id` via workflows FKs, so they are denormalized here for self-contained
@@ -32,7 +32,7 @@ import {pgTable} from './common.js';
  * Per-row `committed_length` and `declared_total_bytes` are bounded by the
  * per-job budget, so `mode: 'number'` is safe on the hot path. Any cross-row
  * aggregate (workspace or system-wide totals) MUST read as bigint at the
- * query site — the global sum is unbounded and would silently lose precision
+ * query site: the global sum is unbounded and would silently lose precision
  * past 2^53 as a JS number.
  */
 export const attemptStreams = pgTable(

package/src/db/schema/job-accounting.ts

@@ -11,7 +11,7 @@ import {pgTable} from './common.js';
  *
  * Per-row `stored_bytes_used` is bounded by the per-job budget, so `mode:
  * 'number'` is safe on the hot path. Any cross-row aggregate (workspace or
- * system-wide totals) MUST read as bigint at the query site — the global sum
+ * system-wide totals) MUST read as bigint at the query site: the global sum
  * is unbounded and would silently lose precision past 2^53 as a JS number.
  */
 export const jobAccounting = pgTable('job_accounting', {

package/test/fixtures/workflows-client.ts

@@ -9,6 +9,7 @@ export function createTestWorkflowsClient(): WorkflowsModuleClient {
   return createFakeInterModuleClients({
     workflows: defineInterModulePresentation(workflowsInterModuleContract, {
       startRunFromTrigger: vi.fn(),
+      resolveWorkflowRunTriggerReference: vi.fn(),
       deliverEventToJobListener: vi.fn(),
       getStepLogContext: () => ({harness: 'pi' as const}),
       getLeasedAgentToolContext: vi.fn(),

package/test/globalSetup.ts

@@ -11,7 +11,7 @@ const BUCKET_READY_POLL_INTERVAL_MS = 500;
 
 // `garage-init` exits 0 once it has posted the key/bucket ACLs, but Garage does not guarantee
 // those are immediately usable for S3 uploads. Poll HeadBucket so the suite never starts before
-// the store will accept the configured credentials — without this, the compaction tests race the
+// the store will accept the configured credentials: without this, the compaction tests race the
 // ACL propagation and fail with `AccessDenied: No such key`.
 async function waitForBucketReachable(timeoutMs: number): Promise<void> {
   const deadline = Date.now() + timeoutMs;