@apollo/gateway 2.0.0-alpha.1 → 2.0.0-alpha.2

package/src/__tests__/loadSupergraphSdlFromStorage.test.ts

@@ -8,6 +8,7 @@ import {
   mockOutOfBandReporterUrl,
   mockOutOfBandReportRequestSuccess,
   mockSupergraphSdlRequestSuccess,
+  mockSupergraphSdlRequestIfAfterUnchanged,
 } from './integration/nockMocks';
 import mockedEnv from 'mocked-env';
 
@@ -30,6 +31,8 @@ describe('loadSupergraphSdlFromStorage', () => {
       apiKey,
       endpoint: mockCloudConfigUrl,
       fetcher,
+      compositionId: null,
+
     });
 
     expect(result).toMatchInlineSnapshot(`
@@ -190,12 +193,12 @@ describe('loadSupergraphSdlFromStorage', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        ACCOUNTS @join__graph(name: \\"accounts\\", url: \\"https://accounts.api.com\\")
-        BOOKS @join__graph(name: \\"books\\", url: \\"https://books.api.com\\")
-        DOCUMENTS @join__graph(name: \\"documents\\", url: \\"https://documents.api.com\\")
-        INVENTORY @join__graph(name: \\"inventory\\", url: \\"https://inventory.api.com\\")
-        PRODUCT @join__graph(name: \\"product\\", url: \\"https://product.api.com\\")
-        REVIEWS @join__graph(name: \\"reviews\\", url: \\"https://reviews.api.com\\")
+        ACCOUNTS @join__graph(name: \\"accounts\\", url: \\"https://accounts.api.com.invalid\\")
+        BOOKS @join__graph(name: \\"books\\", url: \\"https://books.api.com.invalid\\")
+        DOCUMENTS @join__graph(name: \\"documents\\", url: \\"https://documents.api.com.invalid\\")
+        INVENTORY @join__graph(name: \\"inventory\\", url: \\"https://inventory.api.com.invalid\\")
+        PRODUCT @join__graph(name: \\"product\\", url: \\"https://product.api.com.invalid\\")
+        REVIEWS @join__graph(name: \\"reviews\\", url: \\"https://reviews.api.com.invalid\\")
       }
 
       scalar JSON
@@ -433,6 +436,8 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
+
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 200 invalid json response body at https://example.cloud-config-url.com/cloudconfig/ reason: Unexpected token I in JSON at position 0"`,
@@ -455,6 +460,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowError(message);
     });
@@ -469,6 +475,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 500 Internal Server Error"`,
@@ -487,6 +494,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 400 invalid json response body at https://example.cloud-config-url.com/cloudconfig/ reason: Unexpected end of JSON input"`,
@@ -508,6 +516,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 400 invalid json response body at https://example.cloud-config-url.com/cloudconfig/ reason: Unexpected end of JSON input"`,
@@ -529,6 +538,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 413 Payload Too Large"`,
@@ -550,6 +560,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 422 Unprocessable Entity"`,
@@ -571,6 +582,7 @@ describe('loadSupergraphSdlFromStorage', () => {
           apiKey,
           endpoint: mockCloudConfigUrl,
           fetcher,
+          compositionId: null,
         }),
       ).rejects.toThrowErrorMatchingInlineSnapshot(
         `"An error occurred while fetching your schema from Apollo: 408 Request Timeout"`,
@@ -593,6 +605,7 @@ describe('loadSupergraphSdlFromStorage', () => {
         apiKey,
         endpoint: mockCloudConfigUrl,
         fetcher,
+        compositionId: null,
       }),
     ).rejects.toThrowErrorMatchingInlineSnapshot(
       `"An error occurred while fetching your schema from Apollo: 504 Gateway Timeout"`,
@@ -614,6 +627,7 @@ describe('loadSupergraphSdlFromStorage', () => {
         apiKey,
         endpoint: mockCloudConfigUrl,
         fetcher,
+        compositionId: null,
       }),
     ).rejects.toThrowErrorMatchingInlineSnapshot(
       `"An error occurred while fetching your schema from Apollo: request to https://example.cloud-config-url.com/cloudconfig/ failed, reason: no response"`,
@@ -635,6 +649,7 @@ describe('loadSupergraphSdlFromStorage', () => {
         apiKey,
         endpoint: mockCloudConfigUrl,
         fetcher,
+        compositionId: null,
       }),
     ).rejects.toThrowErrorMatchingInlineSnapshot(
       `"An error occurred while fetching your schema from Apollo: 502 Bad Gateway"`,
@@ -656,9 +671,24 @@ describe('loadSupergraphSdlFromStorage', () => {
         apiKey,
         endpoint: mockCloudConfigUrl,
         fetcher,
+        compositionId: null,
       }),
     ).rejects.toThrowErrorMatchingInlineSnapshot(
       `"An error occurred while fetching your schema from Apollo: 503 Service Unavailable"`,
     );
   });
+
+  it('successfully responds to SDL unchanged by returning null', async () => {
+    mockSupergraphSdlRequestIfAfterUnchanged("id-1234");
+
+    const fetcher = getDefaultFetcher();
+    const result = await loadSupergraphSdlFromStorage({
+        graphRef,
+        apiKey,
+        endpoint: mockCloudConfigUrl,
+        fetcher,
+        compositionId: "id-1234",
+    });
+    expect(result).toBeNull();
+  });
 });

package/src/config.ts

@@ -2,11 +2,11 @@ import { GraphQLError, GraphQLSchema } from "graphql";
 import { HeadersInit } from "node-fetch";
 import { fetch } from 'apollo-server-env';
 import { GraphQLRequestContextExecutionDidStart, Logger } from "apollo-server-types";
-import { ServiceDefinition } from "@apollo/federation";
 import { GraphQLDataSource } from './datasources/types';
 import { QueryPlan } from '@apollo/query-planner';
 import { OperationContext } from './operationContext';
 import { ServiceMap } from './executeQueryPlan';
+import { ServiceDefinition } from "@apollo/federation-internals";
 
 export type ServiceEndpointDefinition = Pick<ServiceDefinition, 'name' | 'url'>;
 
@@ -135,31 +135,14 @@ export interface RemoteGatewayConfig extends GatewayConfigBase {
     | ((service: ServiceEndpointDefinition) => Promise<HeadersInit> | HeadersInit);
 }
 
-// TODO(trevor:cloudconfig): This type goes away
-export interface LegacyManagedGatewayConfig extends GatewayConfigBase {
-  federationVersion?: number;
-  /**
-   * Setting this to null will cause the gateway to use the old mechanism for
-   * managed federation via GCS + composition.
-   */
-  schemaConfigDeliveryEndpoint: null;
-}
-
-// TODO(trevor:cloudconfig): This type becomes the only managed config
-export interface PrecomposedManagedGatewayConfig extends GatewayConfigBase {
+export interface ManagedGatewayConfig extends GatewayConfigBase {
   /**
    * This configuration option shouldn't be used unless by recommendation from
-   * Apollo staff. This can also be set to `null` (see above) in order to revert
-   * to the previous mechanism for managed federation.
+   * Apollo staff.
    */
   schemaConfigDeliveryEndpoint?: string;
 }
 
-// TODO(trevor:cloudconfig): This union is no longer needed
-export type ManagedGatewayConfig =
-  | LegacyManagedGatewayConfig
-  | PrecomposedManagedGatewayConfig;
-
 interface ManuallyManagedServiceDefsGatewayConfig extends GatewayConfigBase {
   experimental_updateServiceDefinitions: Experimental_UpdateServiceDefinitions;
 }
@@ -216,30 +199,12 @@ export function isManuallyManagedConfig(
 export function isManagedConfig(
   config: GatewayConfig,
 ): config is ManagedGatewayConfig {
-  return isPrecomposedManagedConfig(config) || isLegacyManagedConfig(config);
-}
-
-// TODO(trevor:cloudconfig): This merges with `isManagedConfig`
-export function isPrecomposedManagedConfig(
-  config: GatewayConfig,
-): config is PrecomposedManagedGatewayConfig {
-  return (
-    !isLegacyManagedConfig(config) &&
-    (('schemaConfigDeliveryEndpoint' in config &&
-      typeof config.schemaConfigDeliveryEndpoint === 'string') ||
-      (!isRemoteConfig(config) &&
-        !isLocalConfig(config) &&
-        !isSupergraphSdlConfig(config) &&
-        !isManuallyManagedConfig(config)))
-  );
-}
-
-export function isLegacyManagedConfig(
-  config: GatewayConfig,
-): config is LegacyManagedGatewayConfig {
   return (
-    'schemaConfigDeliveryEndpoint' in config &&
-    config.schemaConfigDeliveryEndpoint === null
+    'schemaConfigDeliveryEndpoint' in config ||
+    (!isRemoteConfig(config) &&
+      !isLocalConfig(config) &&
+      !isSupergraphSdlConfig(config) &&
+      !isManuallyManagedConfig(config))
   );
 }
 

package/src/core/__tests__/core.test.ts

@@ -41,7 +41,7 @@ describe('core v0.1', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        WORLD @join__graph(name: "world", url: "https://world.api.com")
+        WORLD @join__graph(name: "world", url: "https://world.api.com.invalid")
       }
 
       type Query {
@@ -97,7 +97,7 @@ describe('core v0.1', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        WORLD @join__graph(name: "world", url: "https://world.api.com")
+        WORLD @join__graph(name: "world", url: "https://world.api.com.invalid")
       }
 
       type Query {
@@ -173,7 +173,7 @@ describe('core v0.2', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        WORLD @join__graph(name: "world", url: "https://world.api.com")
+        WORLD @join__graph(name: "world", url: "https://world.api.com.invalid")
       }
 
       type Query {
@@ -243,7 +243,7 @@ describe('core v0.2', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        WORLD @join__graph(name: "world", url: "https://world.api.com")
+        WORLD @join__graph(name: "world", url: "https://world.api.com.invalid")
       }
 
       type Query {
@@ -315,7 +315,7 @@ describe('core v0.2', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        WORLD @join__graph(name: "world", url: "https://world.api.com")
+        WORLD @join__graph(name: "world", url: "https://world.api.com.invalid")
       }
 
       type Query {
@@ -391,7 +391,7 @@ describe('core v0.2', () => {
       scalar join__FieldSet
 
       enum join__Graph {
-        WORLD @join__graph(name: "world", url: "https://world.api.com")
+        WORLD @join__graph(name: "world", url: "https://world.api.com.invalid")
       }
 
       type Query {

package/src/datasources/types.ts

@@ -47,6 +47,6 @@ export type GraphQLDataSourceProcessOptions<
       /**
        * Mostly provided for historical reasons.
        */
-      context: {};
+      context: Record<string, any>;
     }
 );

package/src/executeQueryPlan.ts

@@ -14,7 +14,6 @@ import {
   GraphQLSchema,
 } from 'graphql';
 import { Trace, google } from 'apollo-reporting-protobuf';
-import { defaultRootOperationNameLookup } from '@apollo/federation';
 import { GraphQLDataSource, GraphQLDataSourceRequestKind } from './datasources/types';
 import { OperationContext } from './operationContext';
 import {
@@ -30,6 +29,7 @@ import { deepMerge } from './utilities/deepMerge';
 import { isNotNullOrUndefined } from './utilities/array';
 import { SpanStatusCode } from "@opentelemetry/api";
 import { OpenTelemetrySpanNames, tracer } from "./utilities/opentelemetry";
+import { defaultRootName } from '@apollo/federation-internals';
 
 export type ServiceMap = {
   [serviceName: string]: GraphQLDataSource;
@@ -85,7 +85,7 @@ export async function executeQueryPlan<TContext>(
         }
       }
 
-      let result = await tracer.startActiveSpan(OpenTelemetrySpanNames.POST_PROCESSING, async (span) => {
+      const result = await tracer.startActiveSpan(OpenTelemetrySpanNames.POST_PROCESSING, async (span) => {
 
         // FIXME: Re-executing the query is a pretty heavy handed way of making sure
         // only explicitly requested fields are included and field ordering follows
@@ -278,7 +278,7 @@ async function executeFetch<TContext>(
 
       if (entities.length < 1) return;
 
-      let variables = Object.create(null);
+      const variables = Object.create(null);
       if (fetch.variableUsages) {
         for (const variableName of fetch.variableUsages) {
           const providedVariables = context.requestContext.request.variables;
@@ -450,10 +450,7 @@ async function executeFetch<TContext>(
           // to have the default names (Query, Mutation, Subscription) even
           // if the implementing services choose different names, so we override
           // whatever the implementing service reported here.
-          const rootTypeName =
-            defaultRootOperationNameLookup[
-              context.operationContext.operation.operation
-            ];
+          const rootTypeName = defaultRootName(context.operationContext.operation.operation);
           traceNode.trace.root?.child?.forEach((child) => {
             child.parentType = rootTypeName;
           });

package/src/index.ts

@@ -12,9 +12,6 @@ import {
   GraphQLSchema,
   VariableDefinitionNode,
 } from 'graphql';
-import {
-  ServiceDefinition,
-} from '@apollo/federation';
 import loglevel from 'loglevel';
 
 import { buildOperationContext, OperationContext } from './operationContext';
@@ -59,11 +56,8 @@ import {
   ServiceDefinitionUpdate,
   SupergraphSdlUpdate,
   CompositionUpdate,
-  isPrecomposedManagedConfig,
-  isLegacyManagedConfig,
 } from './config';
 import { loadSupergraphSdlFromStorage } from './loadSupergraphSdlFromStorage';
-import { getServiceDefinitionsFromStorage } from './legacyLoadServicesFromStorage';
 import { SpanStatusCode } from '@opentelemetry/api';
 import { OpenTelemetrySpanNames, tracer } from './utilities/opentelemetry';
 
@@ -71,6 +65,7 @@ import {
   buildSupergraphSchema,
   operationFromDocument,
   Schema,
+  ServiceDefinition,
 } from '@apollo/federation-internals';
 import { composeServices } from '@apollo/composition'
 
@@ -87,6 +82,8 @@ type WarnedStates = {
   remoteWithLocalConfig?: boolean;
 };
 
+export { ServiceDefinition } from '@apollo/federation-internals';
+
 export function getDefaultFetcher() {
   const { name, version } = require('../package.json');
   return fetcher.defaults({
@@ -111,20 +108,6 @@ export function getDefaultFetcher() {
   });
 }
 
-/**
- * TODO(trevor:cloudconfig): Stop exporting this
- * @deprecated This will be removed in a future version of @apollo/gateway
- */
-export const getDefaultGcsFetcher = deprecate(
-  getDefaultFetcher,
-  `'getDefaultGcsFetcher' is deprecated. Use 'getDefaultFetcher' instead.`,
-);
-/**
- * TODO(trevor:cloudconfig): Stop exporting this
- * @deprecated This will be removed in a future version of @apollo/gateway
- */
-export const GCS_RETRY_COUNT = 5;
-
 export const HEALTH_CHECK_QUERY =
   'query __ApolloServiceHealthCheck__ { __typename }';
 export const SERVICE_DEFINITION_QUERY =
@@ -220,10 +203,8 @@ export class ApolloGateway implements GraphQLService {
   private experimental_pollInterval?: number;
   // Configure the endpoint by which gateway will access its precomposed schema.
   // * `string` means use that endpoint
-  // * `null` will revert the gateway to legacy mode (polling GCS and composing the schema itself).
   // * `undefined` means the gateway is not using managed federation
-  // TODO(trevor:cloudconfig): `null` should be disallowed in the future.
-  private schemaConfigDeliveryEndpoint?: string | null;
+  private schemaConfigDeliveryEndpoint?: string;
 
   constructor(config?: GatewayConfig) {
     this.config = {
@@ -251,19 +232,14 @@ export class ApolloGateway implements GraphQLService {
     this.experimental_pollInterval = config?.experimental_pollInterval;
 
     // 1. If config is set to a `string`, use it
-    // 2. If config is explicitly set to `null`, fallback to GCS
-    // 3. If the env var is set, use that
-    // 4. If config is `undefined`, use the default uplink URL
-
-    // This if case unobviously handles 1, 2, and 4.
-    if (isPrecomposedManagedConfig(this.config)) {
+    // 2. If the env var is set, use that
+    // 3. If config is `undefined`, use the default uplink URL
+    if (isManagedConfig(this.config)) {
       const envEndpoint = process.env.APOLLO_SCHEMA_CONFIG_DELIVERY_ENDPOINT;
       this.schemaConfigDeliveryEndpoint =
         this.config.schemaConfigDeliveryEndpoint ??
         envEndpoint ??
         'https://uplink.api.apollographql.com/';
-    } else if (isLegacyManagedConfig(this.config)) {
-      this.schemaConfigDeliveryEndpoint = null;
     }
 
     if (isManuallyManagedConfig(this.config)) {
@@ -617,7 +593,7 @@ export class ApolloGateway implements GraphQLService {
     legacyDontNotifyOnSchemaChangeListeners: boolean = false,
   ): void {
     if (this.queryPlanStore) this.queryPlanStore.flush();
-    this.apiSchema = coreSchema.toAPISchema(); 
+    this.apiSchema = coreSchema.toAPISchema();
     this.schema = wrapSchemaWithAliasResolver(this.apiSchema.toGraphQLJSSchema());
     this.queryPlanner = new QueryPlanner(coreSchema);
 
@@ -940,24 +916,20 @@ export class ApolloGateway implements GraphQLService {
       );
     }
 
-    // TODO(trevor:cloudconfig): This condition goes away completely
-    if (isPrecomposedManagedConfig(config)) {
-      return loadSupergraphSdlFromStorage({
-        graphRef: this.apolloConfig!.graphRef!,
-        apiKey: this.apolloConfig!.key!,
-        endpoint: this.schemaConfigDeliveryEndpoint!,
-        fetcher: this.fetcher,
-      });
-    } else if (isLegacyManagedConfig(config)) {
-      return getServiceDefinitionsFromStorage({
-        graphRef: this.apolloConfig!.graphRef!,
-        apiKeyHash: this.apolloConfig!.keyHash!,
-        federationVersion: config.federationVersion || 1,
-        fetcher: this.fetcher,
-      });
-    } else {
-      throw new Error('Programming error: unhandled configuration');
-    }
+    const result = await loadSupergraphSdlFromStorage({
+      graphRef: this.apolloConfig!.graphRef!,
+      apiKey: this.apolloConfig!.key!,
+      endpoint: this.schemaConfigDeliveryEndpoint!,
+      fetcher: this.fetcher,
+      compositionId: this.compositionId ?? null,
+    });
+
+    return (
+      result ?? {
+        id: this.compositionId!,
+        supergraphSdl: this.supergraphSdl!,
+      }
+    );
   }
 
   private maybeWarnOnConflictingConfig() {

package/src/loadServicesFromRemoteEndpoint.ts

@@ -4,7 +4,7 @@ import { Headers, HeadersInit } from 'node-fetch';
 import { GraphQLDataSource, GraphQLDataSourceRequestKind } from './datasources/types';
 import { SERVICE_DEFINITION_QUERY } from './';
 import { CompositionUpdate, ServiceEndpointDefinition } from './config';
-import { ServiceDefinition } from '@apollo/federation';
+import { ServiceDefinition } from '@apollo/federation-internals';
 
 type Service = ServiceEndpointDefinition & {
   dataSource: GraphQLDataSource;

package/src/loadSupergraphSdlFromStorage.ts

@@ -5,8 +5,8 @@ import { SupergraphSdlQuery } from './__generated__/graphqlTypes';
 
 // Magic /* GraphQL */ comment below is for codegen, do not remove
 export const SUPERGRAPH_SDL_QUERY = /* GraphQL */`#graphql
-  query SupergraphSdl($apiKey: String!, $ref: String!) {
-    routerConfig(ref: $ref, apiKey: $apiKey) {
+  query SupergraphSdl($apiKey: String!, $ref: String!, $ifAfterId: ID) {
+    routerConfig(ref: $ref, apiKey: $apiKey, ifAfterId: $ifAfterId) {
       __typename
       ... on RouterConfigResult {
         id
@@ -43,11 +43,13 @@ export async function loadSupergraphSdlFromStorage({
   apiKey,
   endpoint,
   fetcher,
+  compositionId,
 }: {
   graphRef: string;
   apiKey: string;
   endpoint: string;
   fetcher: typeof fetch;
+  compositionId: string | null;
 }) {
   let result: Response;
   const requestDetails = {
@@ -57,6 +59,7 @@ export async function loadSupergraphSdlFromStorage({
       variables: {
         ref: graphRef,
         apiKey,
+        ifAfterId: compositionId,
       },
     }),
     headers: {
@@ -124,13 +127,13 @@ export async function loadSupergraphSdlFromStorage({
       supergraphSdl,
       // messages,
     } = routerConfig;
-
-    // `supergraphSdl` should not be nullable in the schema, but it currently is
     return { id, supergraphSdl: supergraphSdl! };
   } else if (routerConfig.__typename === 'FetchError') {
     // FetchError case
     const { code, message } = routerConfig;
     throw new Error(`${code}: ${message}`);
+  } else if (routerConfig.__typename === 'Unchanged') {
+    return null;
   } else {
     throw new Error('Programming error: unhandled response failure');
   }

package/src/make-fetch-happen.d.ts

@@ -36,7 +36,7 @@ declare module 'make-fetch-happen' {
   }
 
   export interface CacheManager {
-    delete(req: Request): Promise<Boolean>;
+    delete(req: Request): Promise<boolean>;
     put(req: Request, res: Response): Promise<Response>;
     match(req: Request): Promise<Response | undefined>;
   }

package/src/operationContext.ts

@@ -20,7 +20,7 @@ interface BuildOperationContextOptions {
   schema: GraphQLSchema;
   operationDocument: DocumentNode;
   operationName?: string;
-};
+}
 
 export function buildOperationContext({
   schema,
@@ -28,7 +28,7 @@ export function buildOperationContext({
   operationName,
 }: BuildOperationContextOptions): OperationContext {
   let operation: OperationDefinitionNode | undefined;
-  let operationCount: number = 0;
+  let operationCount = 0;
   const fragments: {
     [fragmentName: string]: FragmentDefinitionNode;
   } = Object.create(null);

package/src/utilities/__tests__/cleanErrorOfInaccessibleElements.test.ts

@@ -56,7 +56,7 @@ describe('cleanErrorOfInaccessibleNames', () => {
       },
     });
 
-    const cleaned = cleanErrorOfInaccessibleNames(schema, result.errors?.[0]!);
+    const cleaned = cleanErrorOfInaccessibleNames(schema, result.errors![0]!);
     expect(cleaned.message).toMatchInlineSnapshot(
       `"Abstract type \\"Foo\\" was resolve to a type [inaccessible type] that does not exist inside schema."`,
     );

package/src/utilities/array.ts

@@ -27,7 +27,7 @@ export function findAndExtract<T>(
   const index = array.findIndex(predicate);
   if (index === -1) return [undefined, array];
 
-  let remaining = array.slice(0, index);
+  const remaining = array.slice(0, index);
   if (index < array.length - 1) {
     remaining.push(...array.slice(index + 1));
   }