@sqaoss/flowy 1.3.0 → 1.3.1

package/src/util/format.test.ts

@@ -45,4 +45,69 @@ describe('outputError', () => {
 
     expect(exitSpy).toHaveBeenCalledWith(1)
   })
+
+  test('exits 1 for VALIDATION_ERROR (usage/validation class)', () => {
+    vi.spyOn(console, 'error').mockImplementation(() => {})
+    const exitSpy = vi
+      .spyOn(process, 'exit')
+      .mockImplementation(() => undefined as never)
+
+    outputError(
+      Object.assign(new Error('bad input'), { code: 'VALIDATION_ERROR' }),
+    )
+
+    expect(exitSpy).toHaveBeenCalledWith(1)
+  })
+
+  test('exits 1 for CONFLICT (usage/validation class)', () => {
+    vi.spyOn(console, 'error').mockImplementation(() => {})
+    const exitSpy = vi
+      .spyOn(process, 'exit')
+      .mockImplementation(() => undefined as never)
+
+    outputError(Object.assign(new Error('conflict'), { code: 'CONFLICT' }))
+
+    expect(exitSpy).toHaveBeenCalledWith(1)
+  })
+
+  test('exits 2 for NOT_FOUND', () => {
+    vi.spyOn(console, 'error').mockImplementation(() => {})
+    const exitSpy = vi
+      .spyOn(process, 'exit')
+      .mockImplementation(() => undefined as never)
+
+    outputError(
+      Object.assign(new Error('Node bad-id not found'), { code: 'NOT_FOUND' }),
+    )
+
+    expect(exitSpy).toHaveBeenCalledWith(2)
+  })
+
+  test('exits 3 for SERVER_ERROR', () => {
+    vi.spyOn(console, 'error').mockImplementation(() => {})
+    const exitSpy = vi
+      .spyOn(process, 'exit')
+      .mockImplementation(() => undefined as never)
+
+    outputError(
+      Object.assign(new Error('Server returned HTTP 502'), {
+        code: 'SERVER_ERROR',
+      }),
+    )
+
+    expect(exitSpy).toHaveBeenCalledWith(3)
+  })
+
+  test('exits 4 for NETWORK_ERROR', () => {
+    vi.spyOn(console, 'error').mockImplementation(() => {})
+    const exitSpy = vi
+      .spyOn(process, 'exit')
+      .mockImplementation(() => undefined as never)
+
+    outputError(
+      Object.assign(new Error('Request timed out'), { code: 'NETWORK_ERROR' }),
+    )
+
+    expect(exitSpy).toHaveBeenCalledWith(4)
+  })
 })

package/src/util/format.ts

@@ -2,6 +2,27 @@ export function output(data: unknown): void {
   console.log(JSON.stringify(data, null, 2))
 }
 
+/**
+ * Map a GraphQL `extensions.code` (or transport code) to a distinct process
+ * exit code so callers can branch on failure class:
+ *   1 — usage / validation / conflict (default)
+ *   2 — not found
+ *   3 — server error (non-2xx, masked, non-JSON)
+ *   4 — network / transport (timeout, connection failure)
+ */
+function exitCodeFor(code: string | undefined): number {
+  switch (code) {
+    case 'NOT_FOUND':
+      return 2
+    case 'SERVER_ERROR':
+      return 3
+    case 'NETWORK_ERROR':
+      return 4
+    default:
+      return 1
+  }
+}
+
 export function outputError(error: unknown): void {
   const message = error instanceof Error ? error.message : String(error)
   const rawCode =
@@ -10,5 +31,5 @@ export function outputError(error: unknown): void {
   console.error(
     JSON.stringify(code ? { error: message, code } : { error: message }),
   )
-  process.exit(1)
+  process.exit(exitCodeFor(code))
 }