@shipfox/api-logs 10.0.0 → 10.1.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/api-logs",
   "license": "MIT",
-  "version": "10.0.0",
+  "version": "10.1.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -25,7 +25,7 @@
     "@temporalio/workflow": "1.18.1",
     "drizzle-orm": "^0.45.2",
     "zod": "^4.4.3",
-    "@shipfox/api-auth-context": "10.0.0",
+    "@shipfox/api-auth-context": "10.1.0",
     "@shipfox/api-logs-dto": "9.0.2",
     "@shipfox/api-workflows-dto": "10.0.0",
     "@shipfox/workflow-document": "2.1.3",

package/src/core/session/claude/rows.ts

@@ -13,6 +13,7 @@ import {
   formatNumber,
   isMeta,
   metaItem,
+  numberField,
   stringField,
   stringifyValue,
   toJson,
@@ -34,6 +35,101 @@ export const PURE_PROGRESS_CLAUDE_SYSTEM_SUBTYPES = new Set<string>([
 ]);
 const OUTPUT_REPROMPT_PREFIX = 'The previous turn ended without setting required workflow outputs:';
 
+type SystemEventMapping = {
+  label: string | ((message: Record<string, unknown>) => string);
+  tone:
+    | SessionViewLifecycleRow['tone']
+    | ((message: Record<string, unknown>) => SessionViewLifecycleRow['tone']);
+  detail?: (message: Record<string, unknown>) => string | null;
+  meta?: (message: Record<string, unknown>) => readonly SessionViewRowMeta[];
+};
+
+const SYSTEM_EVENT_MAPPINGS: Record<string, SystemEventMapping> = {
+  permission_denied: {
+    label: 'Permission denied',
+    tone: 'error',
+    detail: (message) => stringField(message, 'tool_name') ?? null,
+    meta: (message) =>
+      [
+        metaItem(
+          'reason',
+          stringField(message, 'decision_reason') ?? stringField(message, 'message'),
+        ),
+      ].filter(isMeta),
+  },
+  api_retry: {
+    label: 'API retry',
+    tone: 'warning',
+    detail: (message) => {
+      const attempt = numberField(message, 'attempt');
+      const maxRetries = numberField(message, 'max_retries');
+      return attempt == null || maxRetries == null
+        ? null
+        : `attempt ${formatNumber(attempt)}/${formatNumber(maxRetries)}`;
+    },
+    meta: (message) =>
+      [
+        numberMeta(message, 'error_status', 'status'),
+        numberMeta(message, 'retry_delay_ms', 'retry delay', 'ms'),
+      ].filter(isMeta),
+  },
+  informational: {
+    label: (message) => titleCase(stringField(message, 'level') ?? 'informational'),
+    tone: (message) => (stringField(message, 'level') === 'warning' ? 'warning' : 'default'),
+    detail: (message) => stringField(message, 'content') ?? null,
+  },
+  compact_boundary: {
+    label: 'Context compacted',
+    tone: 'default',
+    meta: (message) => {
+      const compactMetadata = asLooseObject(field(message, 'compact_metadata')) ?? {};
+      return [
+        metaItem('trigger', stringField(compactMetadata, 'trigger')),
+        numberMeta(compactMetadata, 'pre_tokens', 'tokens before', 'tokens'),
+        numberMeta(compactMetadata, 'post_tokens', 'tokens after', 'tokens'),
+        numberMeta(compactMetadata, 'duration_ms', 'duration', 'ms'),
+      ].filter(isMeta);
+    },
+  },
+  model_refusal_fallback: {
+    label: 'Model refused, fell back',
+    tone: 'warning',
+  },
+  model_refusal_no_fallback: {
+    label: 'Model refused',
+    tone: 'error',
+  },
+  mirror_error: {
+    label: 'Session mirror failed',
+    tone: 'error',
+    detail: (message) => stringField(message, 'error') ?? null,
+  },
+  worker_shutting_down: {
+    label: 'Worker shutting down',
+    tone: 'warning',
+  },
+  elicitation_complete: {
+    label: 'Elicitation complete',
+    tone: 'default',
+  },
+  notification: {
+    label: 'Notification',
+    tone: 'default',
+  },
+  task_started: {
+    label: 'Task started',
+    tone: 'default',
+  },
+  task_updated: {
+    label: 'Task updated',
+    tone: 'default',
+  },
+  task_notification: {
+    label: 'Task notification',
+    tone: 'default',
+  },
+};
+
 export function systemRow(
   timestamp: number,
   message: Record<string, unknown>,
@@ -51,8 +147,25 @@ export function systemRow(
     ),
   ].filter(isMeta);
 
-  if (!isInit)
-    return lifecycleRow(timestamp, 'Session event', sessionId ?? null, 'default', false, baseMeta);
+  if (!isInit) {
+    const mapping = subtype === undefined ? undefined : SYSTEM_EVENT_MAPPINGS[subtype];
+    const label =
+      mapping === undefined
+        ? `Session event (${subtype ?? 'unknown'})`
+        : typeof mapping.label === 'function'
+          ? mapping.label(message)
+          : mapping.label;
+    const tone =
+      mapping === undefined
+        ? 'default'
+        : typeof mapping.tone === 'function'
+          ? mapping.tone(message)
+          : mapping.tone;
+    const detail = mapping?.detail?.(message) ?? null;
+    const meta = [...baseMeta, ...(mapping?.meta?.(message) ?? [])];
+
+    return lifecycleRow(timestamp, label, detail, tone, false, meta);
+  }
 
   const isNewSession =
     !context.hasInit || sessionId === undefined || sessionId !== context.sessionId;
@@ -61,11 +174,17 @@ export function systemRow(
   context.turn = isNewSession ? 1 : context.turn + 1;
 
   const label = isNewSession ? 'Session started' : `Turn ${context.turn} started`;
-  const meta = [metaItem('turn', isNewSession ? null : String(context.turn)), ...baseMeta].filter(
-    isMeta,
-  );
+  const meta = [
+    metaItem('session', isNewSession ? sessionId : null),
+    metaItem('turn', isNewSession ? null : String(context.turn)),
+    ...baseMeta,
+  ].filter(isMeta);
+
+  return lifecycleRow(timestamp, label, null, 'default', false, meta);
+}
 
-  return lifecycleRow(timestamp, label, sessionId ?? null, 'default', false, meta);
+function titleCase(value: string): string {
+  return value.length === 0 ? value : `${value[0]?.toUpperCase()}${value.slice(1)}`;
 }
 
 export function assistantRows(

package/src/core/session/claude-parser.test.ts

@@ -5,6 +5,148 @@ const record = (data: unknown, ts = 1) => ({
   ts,
   data: typeof data === 'string' ? data : JSON.stringify(data),
 });
+const meta = (label: string, value: string, inline?: boolean) =>
+  inline == null ? {label, value} : {label, value, inline};
+
+const namedSystemEvents = [
+  {
+    subtype: 'permission_denied',
+    data: {tool_name: 'mcp__shipfox_outputs__set_output', decision_reason: 'Permission mode'},
+    expected: {
+      label: 'Permission denied',
+      detail: 'mcp__shipfox_outputs__set_output',
+      meta: [meta('reason', 'Permission mode')],
+      tone: 'error',
+    },
+  },
+  {
+    subtype: 'api_retry',
+    data: {attempt: 2, max_retries: 3, error_status: 429, retry_delay_ms: 1_500},
+    expected: {
+      label: 'API retry',
+      detail: 'attempt 2/3',
+      meta: [meta('status', '429'), meta('retry delay', '1,500 ms')],
+      tone: 'warning',
+    },
+  },
+  {
+    subtype: 'informational',
+    data: {level: 'warning', content: 'The hook blocked continuation.'},
+    expected: {
+      label: 'Warning',
+      detail: 'The hook blocked continuation.',
+      meta: [],
+      tone: 'warning',
+    },
+  },
+  {
+    subtype: 'compact_boundary',
+    data: {
+      compact_metadata: {trigger: 'auto', pre_tokens: 12_000, post_tokens: 4_000, duration_ms: 250},
+    },
+    expected: {
+      label: 'Context compacted',
+      detail: null,
+      meta: [
+        meta('trigger', 'auto'),
+        meta('tokens before', '12K tokens'),
+        meta('tokens after', '4,000 tokens'),
+        meta('duration', '250 ms'),
+      ],
+      tone: 'default',
+    },
+  },
+  {
+    subtype: 'model_refusal_fallback',
+    data: {},
+    expected: {
+      label: 'Model refused, fell back',
+      detail: null,
+      meta: [],
+      tone: 'warning',
+    },
+  },
+  {
+    subtype: 'model_refusal_no_fallback',
+    data: {},
+    expected: {
+      label: 'Model refused',
+      detail: null,
+      meta: [],
+      tone: 'error',
+    },
+  },
+  {
+    subtype: 'mirror_error',
+    data: {error: 'Session store unavailable'},
+    expected: {
+      label: 'Session mirror failed',
+      detail: 'Session store unavailable',
+      meta: [],
+      tone: 'error',
+    },
+  },
+  {
+    subtype: 'worker_shutting_down',
+    data: {},
+    expected: {
+      label: 'Worker shutting down',
+      detail: null,
+      meta: [],
+      tone: 'warning',
+    },
+  },
+  {
+    subtype: 'elicitation_complete',
+    data: {},
+    expected: {
+      label: 'Elicitation complete',
+      detail: null,
+      meta: [],
+      tone: 'default',
+    },
+  },
+  {
+    subtype: 'notification',
+    data: {},
+    expected: {
+      label: 'Notification',
+      detail: null,
+      meta: [],
+      tone: 'default',
+    },
+  },
+  {
+    subtype: 'task_started',
+    data: {},
+    expected: {
+      label: 'Task started',
+      detail: null,
+      meta: [],
+      tone: 'default',
+    },
+  },
+  {
+    subtype: 'task_updated',
+    data: {},
+    expected: {
+      label: 'Task updated',
+      detail: null,
+      meta: [],
+      tone: 'default',
+    },
+  },
+  {
+    subtype: 'task_notification',
+    data: {},
+    expected: {
+      label: 'Task notification',
+      detail: null,
+      meta: [],
+      tone: 'default',
+    },
+  },
+] as const;
 
 describe('parseClaudeSessionRecord', () => {
   it('returns a raw row for malformed JSON', () => {
@@ -44,8 +186,9 @@ describe('parseClaudeSessionRecord', () => {
         kind: 'lifecycle',
         timestamp: 1,
         label: 'Session started',
-        detail: 'session-1',
+        detail: null,
         meta: [
+          {label: 'session', value: 'session-1'},
           {label: 'cwd', value: '/workspace', inline: false},
           {label: 'model', value: 'claude-opus-4-8'},
         ],
@@ -63,6 +206,23 @@ describe('parseClaudeSessionRecord', () => {
     expect(rows).toEqual([]);
   });
 
+  it.each(namedSystemEvents)('names and tones $subtype system events', ({
+    subtype,
+    data,
+    expected,
+  }) => {
+    const rows = parseClaudeSessionRecord(record({type: 'system', subtype, ...data}));
+
+    expect(rows).toEqual([
+      {
+        kind: 'lifecycle',
+        timestamp: 1,
+        ...expected,
+        terminalFailure: false,
+      },
+    ]);
+  });
+
   it('reports repeated init and result records as turns within one session', () => {
     const context = createClaudeParseContext();
     const rows = [
@@ -90,8 +250,8 @@ describe('parseClaudeSessionRecord', () => {
         kind: 'lifecycle',
         timestamp: 1,
         label: 'Session started',
-        detail: 'session-1',
-        meta: [],
+        detail: null,
+        meta: [meta('session', 'session-1')],
         tone: 'default',
         terminalFailure: false,
       },
@@ -108,7 +268,7 @@ describe('parseClaudeSessionRecord', () => {
         kind: 'lifecycle',
         timestamp: 1,
         label: 'Turn 2 started',
-        detail: 'session-1',
+        detail: null,
         meta: [{label: 'turn', value: '2'}],
         tone: 'default',
         terminalFailure: false,
@@ -125,7 +285,7 @@ describe('parseClaudeSessionRecord', () => {
     ]);
   });
 
-  it('keeps non-init system messages as session events', () => {
+  it('qualifies unmapped system events instead of using a bare generic label', () => {
     const rows = parseClaudeSessionRecord(
       record({type: 'system', subtype: 'custom_event', session_id: 'session-1'}),
     );
@@ -134,8 +294,8 @@ describe('parseClaudeSessionRecord', () => {
       {
         kind: 'lifecycle',
         timestamp: 1,
-        label: 'Session event',
-        detail: 'session-1',
+        label: 'Session event (custom_event)',
+        detail: null,
         meta: [],
         tone: 'default',
         terminalFailure: false,