ts-procedures 10.3.1 → 11.0.0

package/src/codegen/__goldens__/ts-no-share-models--models/_client.ts

@@ -59,13 +59,6 @@ export class ClientHttpError extends Error {
   }
 }
 
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
- 
-export const ClientRequestError = ClientHttpError
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
-// eslint-disable-next-line no-redeclare
-export type ClientRequestError = ClientHttpError
-
 export class ClientPathParamError extends Error {
   readonly name = 'ClientPathParamError'
 
@@ -305,7 +298,7 @@ export function resolveBasePath(
  * - `timeoutMs`: the resolved timeout value (may be 0 if per-call explicitly disables)
  * - `combined`: `AbortSignal.any([...])` of all active signals, or undefined if none
  *
- * Per-call `timeout: 0` disables an inherited default timeout (same as `resolveSignal`).
+ * Per-call `timeout: 0` disables an inherited default timeout.
  * Both `defaults.signal` and `options.signal` are included in `combined` when present.
  */
 export interface SignalSources {
@@ -338,25 +331,6 @@ export function resolveSignalSources(
   return { combined, timeoutSignal, userSignal, timeoutMs }
 }
 
-/**
- * Resolves the effective AbortSignal by combining (via `AbortSignal.any`):
- *   - default signal (if any)
- *   - per-call signal (if any)
- *   - timeout signal (if resolved timeout > 0)
- *
- * Returns undefined when none apply. Per-call `timeout: 0` disables an
- * inherited default timeout.
- *
- * @deprecated Prefer `resolveSignalSources` when you need access to individual
- * signal references (e.g. for abort-cause classification).
- */
-export function resolveSignal(
-  defaults: ProcedureCallDefaults | undefined,
-  options: ProcedureCallOptions | undefined,
-): AbortSignal | undefined {
-  return resolveSignalSources(defaults, options).combined
-}
-
 /**
  * Resolves a `ClientHeadersInit` value: a static record passes through, a
  * function is invoked and (if async) awaited — re-evaluated on every call so
@@ -599,19 +573,32 @@ export async function executeCall<TResponse>(config: ExecuteCallConfig): Promise
       procedureName: descriptor.name,
       scope: descriptor.scope,
     })
+    let httpError: Error
     if (typed) {
       // Tag so executeSafeCall can distinguish typed registry errors from plain
       // ClientHttpError without re-inspecting the registry.
       ;(typed as unknown as { __tsProceduresTyped?: boolean }).__tsProceduresTyped = true
-      throw typed
+      httpError = typed
+    } else {
+      httpError = new ClientHttpError({
+        status: response.status,
+        headers: response.headers,
+        body: response.body,
+        procedureName: descriptor.name,
+        scope: descriptor.scope,
+      })
     }
-    throw new ClientHttpError({
-      status: response.status,
-      headers: response.headers,
-      body: response.body,
-      procedureName: descriptor.name,
-      scope: descriptor.scope,
-    })
+    // A non-2xx response is not a thrown adapter error (fetch resolves it), so
+    // it bypasses the catch block above. Fire onError here too so the hook is a
+    // uniform cross-cutting failure signal across adapter throws AND HTTP error
+    // statuses (401/403/5xx). Runs after onAfterResponse, which had its chance
+    // to swallow the error by mutating status back into the 2xx range.
+    await runOnError(
+      { procedureName: descriptor.name, scope: descriptor.scope, request, error: httpError },
+      hooks,
+      options,
+    )
+    throw httpError
   }
 
   // 8. Return the body (or { body, headers } when the route declares res.headers)
@@ -629,7 +616,8 @@ export async function executeCall<TResponse>(config: ExecuteCallConfig): Promise
  *   2. Adapter throw, classified → `kind: 'network' | 'timeout' | 'aborted' | <custom> | 'unknown'`
  *   3. Adapter returns non-2xx → `kind: 'typed'` (registry match) or `kind: 'http'`
  *
- * `onError` hook fires on path 2 and 3 (cross-cutting telemetry); NOT on
+ * `onError` hook fires on path 2 and 3 (cross-cutting telemetry) — including
+ * plain non-2xx `ClientHttpError` and typed registry errors — but NOT on
  * path 1 (usage errors bypass the classifier and onError entirely).
  */
 export async function executeSafeCall<TResponse, ETyped = never>(

package/src/codegen/__goldens__/ts-no-share-models--models/_types.ts

@@ -29,13 +29,6 @@ export class ClientHttpError extends Error {
   }
 }
 
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
- 
-export const ClientRequestError = ClientHttpError
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
-// eslint-disable-next-line no-redeclare
-export type ClientRequestError = ClientHttpError
-
 export class ClientPathParamError extends Error {
   readonly name = 'ClientPathParamError'
 

package/src/codegen/__goldens__/ts-no-share-models--models/index.ts

@@ -7,6 +7,16 @@ import * as _billing from './billing'
 import * as _billingReports from './billing-reports'
 import * as _errorsModule from './_errors'
 
+export {
+  ClientHttpError,
+  ClientPathParamError,
+  ClientStreamError,
+  ClientNetworkError,
+  ClientTimeoutError,
+  ClientAbortError,
+  ClientParseError,
+} from './_client'
+
 export namespace Api {
   export import Orders = _orders.Orders
   export import Billing = _billing.Billing

package/src/codegen/__goldens__/ts-no-share-models--users/_client.ts

@@ -59,13 +59,6 @@ export class ClientHttpError extends Error {
   }
 }
 
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
- 
-export const ClientRequestError = ClientHttpError
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
-// eslint-disable-next-line no-redeclare
-export type ClientRequestError = ClientHttpError
-
 export class ClientPathParamError extends Error {
   readonly name = 'ClientPathParamError'
 
@@ -305,7 +298,7 @@ export function resolveBasePath(
  * - `timeoutMs`: the resolved timeout value (may be 0 if per-call explicitly disables)
  * - `combined`: `AbortSignal.any([...])` of all active signals, or undefined if none
  *
- * Per-call `timeout: 0` disables an inherited default timeout (same as `resolveSignal`).
+ * Per-call `timeout: 0` disables an inherited default timeout.
  * Both `defaults.signal` and `options.signal` are included in `combined` when present.
  */
 export interface SignalSources {
@@ -338,25 +331,6 @@ export function resolveSignalSources(
   return { combined, timeoutSignal, userSignal, timeoutMs }
 }
 
-/**
- * Resolves the effective AbortSignal by combining (via `AbortSignal.any`):
- *   - default signal (if any)
- *   - per-call signal (if any)
- *   - timeout signal (if resolved timeout > 0)
- *
- * Returns undefined when none apply. Per-call `timeout: 0` disables an
- * inherited default timeout.
- *
- * @deprecated Prefer `resolveSignalSources` when you need access to individual
- * signal references (e.g. for abort-cause classification).
- */
-export function resolveSignal(
-  defaults: ProcedureCallDefaults | undefined,
-  options: ProcedureCallOptions | undefined,
-): AbortSignal | undefined {
-  return resolveSignalSources(defaults, options).combined
-}
-
 /**
  * Resolves a `ClientHeadersInit` value: a static record passes through, a
  * function is invoked and (if async) awaited — re-evaluated on every call so
@@ -599,19 +573,32 @@ export async function executeCall<TResponse>(config: ExecuteCallConfig): Promise
       procedureName: descriptor.name,
       scope: descriptor.scope,
     })
+    let httpError: Error
     if (typed) {
       // Tag so executeSafeCall can distinguish typed registry errors from plain
       // ClientHttpError without re-inspecting the registry.
       ;(typed as unknown as { __tsProceduresTyped?: boolean }).__tsProceduresTyped = true
-      throw typed
+      httpError = typed
+    } else {
+      httpError = new ClientHttpError({
+        status: response.status,
+        headers: response.headers,
+        body: response.body,
+        procedureName: descriptor.name,
+        scope: descriptor.scope,
+      })
     }
-    throw new ClientHttpError({
-      status: response.status,
-      headers: response.headers,
-      body: response.body,
-      procedureName: descriptor.name,
-      scope: descriptor.scope,
-    })
+    // A non-2xx response is not a thrown adapter error (fetch resolves it), so
+    // it bypasses the catch block above. Fire onError here too so the hook is a
+    // uniform cross-cutting failure signal across adapter throws AND HTTP error
+    // statuses (401/403/5xx). Runs after onAfterResponse, which had its chance
+    // to swallow the error by mutating status back into the 2xx range.
+    await runOnError(
+      { procedureName: descriptor.name, scope: descriptor.scope, request, error: httpError },
+      hooks,
+      options,
+    )
+    throw httpError
   }
 
   // 8. Return the body (or { body, headers } when the route declares res.headers)
@@ -629,7 +616,8 @@ export async function executeCall<TResponse>(config: ExecuteCallConfig): Promise
  *   2. Adapter throw, classified → `kind: 'network' | 'timeout' | 'aborted' | <custom> | 'unknown'`
  *   3. Adapter returns non-2xx → `kind: 'typed'` (registry match) or `kind: 'http'`
  *
- * `onError` hook fires on path 2 and 3 (cross-cutting telemetry); NOT on
+ * `onError` hook fires on path 2 and 3 (cross-cutting telemetry) — including
+ * plain non-2xx `ClientHttpError` and typed registry errors — but NOT on
  * path 1 (usage errors bypass the classifier and onError entirely).
  */
 export async function executeSafeCall<TResponse, ETyped = never>(

package/src/codegen/__goldens__/ts-no-share-models--users/_types.ts

@@ -29,13 +29,6 @@ export class ClientHttpError extends Error {
   }
 }
 
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
- 
-export const ClientRequestError = ClientHttpError
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
-// eslint-disable-next-line no-redeclare
-export type ClientRequestError = ClientHttpError
-
 export class ClientPathParamError extends Error {
   readonly name = 'ClientPathParamError'
 

package/src/codegen/__goldens__/ts-no-share-models--users/index.ts

@@ -5,6 +5,16 @@ import type { ClientInstance, CreateClientConfig } from './_types'
 import * as _users from './users'
 import * as _errorsModule from './_errors'
 
+export {
+  ClientHttpError,
+  ClientPathParamError,
+  ClientStreamError,
+  ClientNetworkError,
+  ClientTimeoutError,
+  ClientAbortError,
+  ClientParseError,
+} from './_client'
+
 export namespace Api {
   export import Users = _users.Users
   export import Errors = _errorsModule.ApiErrors

package/src/codegen/__goldens__/ts-shared-models-module--models/_client.ts

@@ -59,13 +59,6 @@ export class ClientHttpError extends Error {
   }
 }
 
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
- 
-export const ClientRequestError = ClientHttpError
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
-// eslint-disable-next-line no-redeclare
-export type ClientRequestError = ClientHttpError
-
 export class ClientPathParamError extends Error {
   readonly name = 'ClientPathParamError'
 
@@ -305,7 +298,7 @@ export function resolveBasePath(
  * - `timeoutMs`: the resolved timeout value (may be 0 if per-call explicitly disables)
  * - `combined`: `AbortSignal.any([...])` of all active signals, or undefined if none
  *
- * Per-call `timeout: 0` disables an inherited default timeout (same as `resolveSignal`).
+ * Per-call `timeout: 0` disables an inherited default timeout.
  * Both `defaults.signal` and `options.signal` are included in `combined` when present.
  */
 export interface SignalSources {
@@ -338,25 +331,6 @@ export function resolveSignalSources(
   return { combined, timeoutSignal, userSignal, timeoutMs }
 }
 
-/**
- * Resolves the effective AbortSignal by combining (via `AbortSignal.any`):
- *   - default signal (if any)
- *   - per-call signal (if any)
- *   - timeout signal (if resolved timeout > 0)
- *
- * Returns undefined when none apply. Per-call `timeout: 0` disables an
- * inherited default timeout.
- *
- * @deprecated Prefer `resolveSignalSources` when you need access to individual
- * signal references (e.g. for abort-cause classification).
- */
-export function resolveSignal(
-  defaults: ProcedureCallDefaults | undefined,
-  options: ProcedureCallOptions | undefined,
-): AbortSignal | undefined {
-  return resolveSignalSources(defaults, options).combined
-}
-
 /**
  * Resolves a `ClientHeadersInit` value: a static record passes through, a
  * function is invoked and (if async) awaited — re-evaluated on every call so
@@ -599,19 +573,32 @@ export async function executeCall<TResponse>(config: ExecuteCallConfig): Promise
       procedureName: descriptor.name,
       scope: descriptor.scope,
     })
+    let httpError: Error
     if (typed) {
       // Tag so executeSafeCall can distinguish typed registry errors from plain
       // ClientHttpError without re-inspecting the registry.
       ;(typed as unknown as { __tsProceduresTyped?: boolean }).__tsProceduresTyped = true
-      throw typed
+      httpError = typed
+    } else {
+      httpError = new ClientHttpError({
+        status: response.status,
+        headers: response.headers,
+        body: response.body,
+        procedureName: descriptor.name,
+        scope: descriptor.scope,
+      })
     }
-    throw new ClientHttpError({
-      status: response.status,
-      headers: response.headers,
-      body: response.body,
-      procedureName: descriptor.name,
-      scope: descriptor.scope,
-    })
+    // A non-2xx response is not a thrown adapter error (fetch resolves it), so
+    // it bypasses the catch block above. Fire onError here too so the hook is a
+    // uniform cross-cutting failure signal across adapter throws AND HTTP error
+    // statuses (401/403/5xx). Runs after onAfterResponse, which had its chance
+    // to swallow the error by mutating status back into the 2xx range.
+    await runOnError(
+      { procedureName: descriptor.name, scope: descriptor.scope, request, error: httpError },
+      hooks,
+      options,
+    )
+    throw httpError
   }
 
   // 8. Return the body (or { body, headers } when the route declares res.headers)
@@ -629,7 +616,8 @@ export async function executeCall<TResponse>(config: ExecuteCallConfig): Promise
  *   2. Adapter throw, classified → `kind: 'network' | 'timeout' | 'aborted' | <custom> | 'unknown'`
  *   3. Adapter returns non-2xx → `kind: 'typed'` (registry match) or `kind: 'http'`
  *
- * `onError` hook fires on path 2 and 3 (cross-cutting telemetry); NOT on
+ * `onError` hook fires on path 2 and 3 (cross-cutting telemetry) — including
+ * plain non-2xx `ClientHttpError` and typed registry errors — but NOT on
  * path 1 (usage errors bypass the classifier and onError entirely).
  */
 export async function executeSafeCall<TResponse, ETyped = never>(

package/src/codegen/__goldens__/ts-shared-models-module--models/_types.ts

@@ -29,13 +29,6 @@ export class ClientHttpError extends Error {
   }
 }
 
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
- 
-export const ClientRequestError = ClientHttpError
-/** @deprecated Renamed to `ClientHttpError`. The alias is retained for one minor release after the 7.0.0 major bump and will be removed in a subsequent minor (e.g. 7.1.0). Migrate to `ClientHttpError` now. */
-// eslint-disable-next-line no-redeclare
-export type ClientRequestError = ClientHttpError
-
 export class ClientPathParamError extends Error {
   readonly name = 'ClientPathParamError'
 

package/src/codegen/__goldens__/ts-shared-models-module--models/index.ts

@@ -7,6 +7,16 @@ import * as _billing from './billing'
 import * as _billingReports from './billing-reports'
 import * as _errorsModule from './_errors'
 
+export {
+  ClientHttpError,
+  ClientPathParamError,
+  ClientStreamError,
+  ClientNetworkError,
+  ClientTimeoutError,
+  ClientAbortError,
+  ClientParseError,
+} from './_client'
+
 export namespace Api {
   export import Orders = _orders.Orders
   export import Billing = _billing.Billing

package/src/codegen/e2e.test.ts

@@ -494,7 +494,7 @@ describe('E2E: generateClient full pipeline', () => {
       expect(content).toContain('ProcedureCallOptions')
     })
 
-    it('_client.ts is generated and contains createClient, createFetchAdapter, ClientHttpError (+ deprecated ClientRequestError alias)', async () => {
+    it('_client.ts is generated and contains createClient, createFetchAdapter, ClientHttpError (no ClientRequestError alias)', async () => {
       tmpDir = makeTmpDir()
       await generateClient({ envelope, outDir: tmpDir, selfContained: true })
 
@@ -503,7 +503,7 @@ describe('E2E: generateClient full pipeline', () => {
       expect(content).toContain('createClient')
       expect(content).toContain('createFetchAdapter')
       expect(content).toContain('ClientHttpError')
-      expect(content).toContain('ClientRequestError')
+      expect(content).not.toContain('ClientRequestError')
     })
 
     it('_client.ts imports from ./_types and NOT from ts-procedures/client', async () => {

package/src/codegen/emit-client-runtime.test.ts

@@ -46,14 +46,14 @@ describe('emitClientRuntimeFile', () => {
     expect(result).toMatch(/export function createFetchAdapter/)
   })
 
-  it('contains export class ClientHttpError (renamed from ClientRequestError)', async () => {
+  it('contains export class ClientHttpError', async () => {
     const result = await emitClientRuntimeFile()
     expect(result).toMatch(/export class ClientHttpError/)
   })
 
-  it('contains deprecated ClientRequestError alias', async () => {
+  it('does NOT contain the removed ClientRequestError alias', async () => {
     const result = await emitClientRuntimeFile()
-    expect(result).toContain('ClientRequestError')
+    expect(result).not.toContain('ClientRequestError')
   })
 
   it('contains export class ClientPathParamError', async () => {

package/src/codegen/emit-index.ts

@@ -91,12 +91,31 @@ export function emitIndexFile(groups: ScopeGroup[], options?: EmitIndexOptions):
     })
     .join('\n')
 
+  // Re-export the client error classes so consumers can `instanceof`-narrow
+  // failures (e.g. in an `onError` hook) straight from the package index,
+  // without reaching into `./_client`. Sourced from the runtime import path so
+  // it resolves in both self-contained (`./_client`) and external-runtime
+  // (`ts-procedures/client`) modes. Unconditional: even a service with no typed
+  // errors throws `ClientHttpError`.
+  const errorClassReExport =
+    `export {\n` +
+    `  ClientHttpError,\n` +
+    `  ClientPathParamError,\n` +
+    `  ClientStreamError,\n` +
+    `  ClientNetworkError,\n` +
+    `  ClientTimeoutError,\n` +
+    `  ClientAbortError,\n` +
+    `  ClientParseError,\n` +
+    `} from '${clientRuntimeImportPath}'`
+
   const pieces: string[] = [
     CODEGEN_HEADER,
     `import { createClient } from '${clientRuntimeImportPath}'`,
     `import type { ClientInstance, CreateClientConfig } from '${clientImportPath}'`,
     importsBlock,
     '',
+    errorClassReExport,
+    '',
     namespaceBlock,
     `export function ${factoryName}(client: ClientInstance) {`,
     '  return {',

package/src/codegen/goldens.test.ts

@@ -22,6 +22,24 @@ import { resolveProductionSwiftEmitter } from './targets/swift/ajsc-adapter.js'
  * because the header intentionally records the producing version. Everything
  * else — including the envelope-hash line — must match exactly (both sides
  * hash the same fixture envelopes).
+ *
+ * DOCUMENTED DEVIATIONS from v8.6.0 (intentional bug fixes / major-version
+ * removals; the affected golden files were surgically updated, NOT
+ * full-recaptured):
+ *   1. Bundled client runtime (`_client.ts`, self-contained TS cases only):
+ *      `executeCall` now fires `onError` on non-2xx HTTP responses (401/403/5xx),
+ *      not just on adapter throws. v8.6.0 skipped the hook here despite its
+ *      docstring claiming otherwise. See `src/client/call.ts`.
+ *   2. Bundled client runtime (`_client.ts`, self-contained TS cases only), v11
+ *      major: removed the deprecated `ClientRequestError` alias (`errors.ts`)
+ *      and the deprecated internal `resolveSignal` helper (`resolve-options.ts`,
+ *      superseded by `resolveSignalSources`).
+ *   3. Generated `index.ts` (ALL TS cases, incl. external-runtime), v11 major:
+ *      now re-exports the client error classes (`ClientHttpError`, …) so
+ *      consumers can `instanceof`-narrow failures straight from the package
+ *      index. See `emit-index.ts`.
+ * See the matching notes in `src/client/freeze.test.ts`.
+ * Kotlin/Swift goldens are unaffected by all of the above.
  */
 const here = dirname(fileURLToPath(import.meta.url))
 const fixturesDir = resolve(here, '__fixtures__')

package/src/exports.ts

@@ -59,7 +59,7 @@ export type {
   ValidationOptions,
   TSchemaValidationError,
 } from './schema/compile.js'
-export type { Infer, TSchemaLib, TJSONSchema, Prettify } from './schema/json-schema.js'
+export type { Infer, TJSONSchema, Prettify } from './schema/json-schema.js'
 
 // Doc envelope — offline codegen input
 export { writeDocEnvelope } from './server/doc-envelope.js'

package/src/schema/json-schema.ts

@@ -16,6 +16,3 @@ export type Prettify<TObject> = {
  * provided for TypeBox (other schemas infer as `unknown`).
  */
 export type Infer<SchemaType> = SchemaType extends TSchema ? Static<SchemaType> : unknown
-
-/** v8-compatible alias for {@link Infer}. */
-export type TSchemaLib<SchemaType> = Infer<SchemaType>