Replace c8 ignore comments with v8 ignore for vitest compatibility

Author: jdalton

src/file-upload.ts

@@ -89,7 +89,7 @@ export async function createUploadRequest(
     const requestBody = [
       ...requestBodyNoBoundaries.flatMap(part => [
         boundarySep,
-        /* c8 ignore next - Array.isArray branch for part is defensive coding for edge cases. */
+        /* v8 ignore next - Array.isArray branch for part is defensive coding for edge cases. */
         ...(Array.isArray(part) ? part : [part]),
       ]),
       finalBoundary,
@@ -117,12 +117,12 @@ export async function createUploadRequest(
 
     try {
       for (const part of requestBody) {
-        /* c8 ignore next 3 - aborted state is difficult to test reliably */
+        /* v8 ignore next 3 - aborted state is difficult to test reliably */
         if (aborted) {
           break
         }
         if (typeof part === 'string') {
-          /* c8 ignore next 5 - backpressure handling requires specific stream conditions */
+          /* v8 ignore next 5 - backpressure handling requires specific stream conditions */
           if (!req.write(part)) {
             // Wait for 'drain' if backpressure is signaled.
             // eslint-disable-next-line no-await-in-loop
@@ -133,17 +133,17 @@ export async function createUploadRequest(
           const stream = part as Readable
           // eslint-disable-next-line no-await-in-loop
           for await (const chunk of stream) {
-            /* c8 ignore next 3 - aborted state during streaming is difficult to test reliably */
+            /* v8 ignore next 3 - aborted state during streaming is difficult to test reliably */
             if (aborted) {
               break
             }
-            /* c8 ignore next 3 - backpressure handling requires specific stream conditions */
+            /* v8 ignore next 3 - backpressure handling requires specific stream conditions */
             if (!req.write(chunk)) {
               await events.once(req, 'drain')
             }
           }
           // Ensure trailing CRLF after file part.
-          /* c8 ignore next 4 - trailing CRLF backpressure handling is edge case */
+          /* v8 ignore next 4 - trailing CRLF backpressure handling is edge case */
           if (!aborted && !req.write('\r\n')) {
             // eslint-disable-next-line no-await-in-loop
             await events.once(req, 'drain')
@@ -152,7 +152,7 @@ export async function createUploadRequest(
           if (typeof part.destroy === 'function') {
             part.destroy()
           }
-          /* c8 ignore next 3 - defensive check for non-string/stream types */
+          /* v8 ignore next 3 - defensive check for non-string/stream types */
         } else {
           throw new TypeError('Expected "string" or "stream" type')
         }

src/http-client.ts

@@ -29,11 +29,11 @@ export class ResponseError extends Error {
    * Automatically formats error message with status code and message.
    */
   constructor(response: IncomingMessage, message = '') {
-    /* c8 ignore next 2 - statusCode and statusMessage may be undefined in edge cases */
+    /* v8 ignore next 2 - statusCode and statusMessage may be undefined in edge cases */
     const statusCode = response.statusCode ?? 'unknown'
     const statusMessage = response.statusMessage ?? 'No status message'
     super(
-      /* c8 ignore next - fallback empty message if not provided */
+      /* v8 ignore next - fallback empty message if not provided */
       `Socket API ${message || 'Request failed'} (${statusCode}): ${statusMessage}`,
     )
     this.name = 'ResponseError'
@@ -126,7 +126,7 @@ export async function getErrorResponseBody(
     response.setEncoding('utf8')
     response.on('data', (chunk: string) => (body += chunk))
     response.on('end', () => resolve(body))
-    /* c8 ignore next - Extremely rare network or stream error during error response reading. */
+    /* v8 ignore next - Extremely rare network or stream error during error response reading. */
     response.on('error', e => reject(e))
   })
 }
@@ -151,7 +151,7 @@ export async function getResponse(
   return await new Promise((resolve, reject) => {
     let timedOut = false
     req.on('response', (response: IncomingMessage) => {
-      /* c8 ignore next 3 - Race condition where response arrives after timeout. */
+      /* v8 ignore next 3 - Race condition where response arrives after timeout. */
       if (timedOut) {
         return
       }
@@ -162,13 +162,13 @@ export async function getResponse(
       req.destroy()
       reject(new Error('Request timed out'))
     })
-    /* c8 ignore start - Network error handling during request, difficult to test reliably. */
+    /* v8 ignore start - Network error handling during request, difficult to test reliably. */
     req.on('error', e => {
       if (!timedOut) {
         reject(e)
       }
     })
-    /* c8 ignore stop */
+    /* v8 ignore stop */
   })
 }
 
@@ -215,7 +215,7 @@ export async function getResponseJson(
       Object.setPrototypeOf(enhancedError, SyntaxError.prototype)
       throw enhancedError
     }
-    /* c8 ignore start - Error instanceof check and unknown error handling for JSON parsing edge cases. */
+    /* v8 ignore start - Error instanceof check and unknown error handling for JSON parsing edge cases. */
     if (e instanceof Error) {
       throw e
     }
@@ -229,7 +229,7 @@ export async function getResponseJson(
     unknownError.originalResponse = responseBody
     Object.setPrototypeOf(unknownError, SyntaxError.prototype)
     throw unknownError
-    /* c8 ignore stop */
+    /* v8 ignore stop */
   }
 }
 
@@ -239,7 +239,7 @@ export async function getResponseJson(
  */
 export function isResponseOk(response: IncomingMessage): boolean {
   const { statusCode } = response
-  /* c8 ignore next - Defensive fallback for edge cases where statusCode might be undefined. */
+  /* v8 ignore next - Defensive fallback for edge cases where statusCode might be undefined. */
   return statusCode ? statusCode >= 200 && statusCode < 300 : false
 }
 
@@ -250,7 +250,7 @@ export function isResponseOk(response: IncomingMessage): boolean {
 export function reshapeArtifactForPublicPolicy<
   T extends Record<string, unknown>,
 >(data: T, isAuthenticated: boolean, actions?: string | undefined): T {
-  /* c8 ignore start - Public policy artifact reshaping for unauthenticated users, difficult to test edge cases. */
+  /* v8 ignore start - Public policy artifact reshaping for unauthenticated users, difficult to test edge cases. */
   // If user is not authenticated, provide a different response structure
   // optimized for the public free-tier experience.
   if (!isAuthenticated) {
@@ -310,7 +310,7 @@ export function reshapeArtifactForPublicPolicy<
     }
   }
   return data
-  /* c8 ignore stop */
+  /* v8 ignore stop */
 }
 
 /**

src/promise-queue.ts

@@ -54,7 +54,7 @@ export class PromiseQueue {
     }
 
     const task = this.queue.shift()
-    /* c8 ignore next 3 - Defensive check; unreachable since we verify queue.length above */
+    /* v8 ignore next 3 - Defensive check; unreachable since we verify queue.length above */
     if (!task) {
       return
     }