Remove dead retry exports

- Delete NON_RETRYABLE_STATUS_CODES: it was defined and publicly
  re-exported but never used in any retry logic (the condition uses
  RETRYABLE_STATUS_CODES). Drop its definition, the index.ts re-export,
  and the tests that only asserted its value.
- Make DEFAULT_RETRY_METHODS module-private; it's only used in-module by
  DEFAULT_RETRY_CONFIG and imported nowhere.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: zeevmoney

src/index.ts

@@ -27,12 +27,7 @@ export { PermitConnectionError, PermitError, PermitPDPStatusError } from './enfo
 export { Context, ContextTransform } from './utils/context';
 export { ApiContext, PermitContextError, ApiKeyLevel } from './api/context';
 export { PermitApiError } from './api/base';
-export {
-  IRetryConfig,
-  RetryConditionFn,
-  RETRYABLE_STATUS_CODES,
-  NON_RETRYABLE_STATUS_CODES,
-} from './utils/retry';
+export { IRetryConfig, RetryConditionFn, RETRYABLE_STATUS_CODES } from './utils/retry';
 
 export interface IPermitClient extends IEnforcer {
   /**

src/tests/unit/retry.spec.ts

@@ -6,7 +6,6 @@ import {
   DEFAULT_RETRY_CONFIG,
   defaultRetryCondition,
   IRetryConfig,
-  NON_RETRYABLE_STATUS_CODES,
   parseRetryAfter,
   resolveRetryConfig,
   RETRYABLE_STATUS_CODES,
@@ -40,10 +39,6 @@ test('RETRYABLE_STATUS_CODES contains expected status codes', (t) => {
   t.deepEqual(RETRYABLE_STATUS_CODES, [408, 429, 500, 502, 503, 504]);
 });
 
-test('NON_RETRYABLE_STATUS_CODES contains expected status codes', (t) => {
-  t.deepEqual(NON_RETRYABLE_STATUS_CODES, [400, 401, 403, 404, 422]);
-});
-
 // ============================================
 // Tests for defaultRetryCondition
 // ============================================
@@ -335,7 +330,6 @@ test('retry types are exported from SDK', async (t) => {
   const sdk = await import('../../index');
 
   t.truthy(sdk.RETRYABLE_STATUS_CODES);
-  t.truthy(sdk.NON_RETRYABLE_STATUS_CODES);
   t.deepEqual(sdk.RETRYABLE_STATUS_CODES, [408, 429, 500, 502, 503, 504]);
 });
 

src/utils/retry.ts

@@ -5,15 +5,10 @@ import { AxiosError } from 'axios';
  */
 export const RETRYABLE_STATUS_CODES = [408, 429, 500, 502, 503, 504];
 
-/**
- * HTTP status codes that should NOT trigger a retry
- */
-export const NON_RETRYABLE_STATUS_CODES = [400, 401, 403, 404, 422];
-
 /**
  * Default HTTP methods that are safe to retry
  */
-export const DEFAULT_RETRY_METHODS = ['GET', 'HEAD', 'OPTIONS', 'PUT', 'DELETE'];
+const DEFAULT_RETRY_METHODS = ['GET', 'HEAD', 'OPTIONS', 'PUT', 'DELETE'];
 
 /**
  * Function type for custom retry condition evaluation