feat(perf): remove imports that bloat bundle (#1215)

Author: jasonkuhrt

.gitignore

@@ -9,3 +9,4 @@ website/.vitepress/.temp
 website/.vitepress/dist
 website/.vitepress/cache
 legacy
+project/

eslint.config.js

@@ -3,6 +3,7 @@ import tsEslint from 'typescript-eslint'
 
 export default tsEslint.config({
   ignores: [
+    'project/**/*',
     'examples/35_custom-scalar/custom-scalar.ts',
     'eslint.config.js',
     'vite.config.ts',

examples/__outputs__/20_output/output_envelope.output.test.txt

@@ -10,8 +10,6 @@
       { name: 'Weedle' }
     ]
   },
-  errors: undefined,
-  extensions: undefined,
   response: Response {
     status: 200,
     statusText: 'OK',

examples/__outputs__/20_output/output_return-error_return-error-execution__return-error-execution.output.txt

@@ -8,7 +8,7 @@ ContextualAggregateError: One or more errors in the execution result.
   context: {},
   cause: undefined,
   errors: [
-    GraphQLError: [
+    ContextualError: [
       {
         "code": "too_small",
         "minimum": 1,
@@ -21,15 +21,15 @@ ContextualAggregateError: One or more errors in the execution result.
         ]
       }
     ]
-        at <anonymous> (/some/path/to/http.ts:XX:XX:47)
+        at <anonymous> (/some/path/to/handleOutput.ts:XX:XX:16)
         at Array.map (<anonymous>)
-        at parseExecutionResult (/some/path/to/http.ts:XX:XX:28)
-        at Object.unpack (/some/path/to/core.ts:XX:XX:26)
+        at handleOutput (/some/path/to/handleOutput.ts:XX:XX:21)
+        at executeDocument (/some/path/to/requestMethods.ts:XX:XX:10)
         at process.processTicksAndRejections (node:internal/process/task_queues:XX:XX)
-        at async runHook (/some/path/to/runHook.ts:XX:XX:16) {
-      path: [ 'addPokemon' ],
-      locations: undefined,
-      extensions: [Object: null prototype] {}
+        at async executeRootField (/some/path/to/requestMethods.ts:XX:XX:18)
+        at async <anonymous> (/some/path/to/output_return-error_return-error-execution__return-error-execution.ts:XX:XX:16) {
+      context: { locations: [ { line: 2, column: 3 } ], path: [ 'addPokemon' ] },
+      cause: undefined
     }
   ]
 }

package.json

@@ -84,6 +84,7 @@
     "check:publint": "publint run --strict",
     "prepublishOnly": "pnpm build",
     "build:docs": "doctoc README.md --notitle && dprint fmt README.md",
+    "build:emit": "pnpm clean && pnpm tsc --noCheck --project tsconfig.build.json && chmod +x ./build/generator/cli/generate.js",
     "build": "pnpm clean && pnpm tsc --project tsconfig.build.json && chmod +x ./build/generator/cli/generate.js",
     "clean": "tsc --build --clean && rm -rf build",
     "test:unit": "vitest --exclude tests/examples",

src/documentBuilder/Select/document.ts

@@ -80,7 +80,7 @@ export const createDocumentNormalizedFromRootTypeSelection = (
     operations: {
       [operationName ?? defaultOperationName]: {
         name: operationName ?? null,
-        type: Grafaid.RootTypeNameToOperationName[rootTypeName],
+        type: Grafaid.Document.RootTypeToOperationType[rootTypeName],
         rootType: rootTypeName,
         selectionSet,
       },
@@ -100,15 +100,15 @@ export const normalizeOrThrow = (document: DocumentObject): DocumentNormalized =
     [name, selectionSet],
   ): [name: string, OperationNormalized] => [name, {
     name,
-    type: Grafaid.RootTypeNameToOperationName[Grafaid.Schema.RootTypeName.Query],
+    type: Grafaid.Document.RootTypeToOperationType[Grafaid.Schema.RootTypeName.Query],
     rootType: Grafaid.Schema.RootTypeName.Query,
     selectionSet,
   }])
   const mutationOperations = Object.entries(document.mutation ?? {}).map((
     [name, selectionSet],
   ): [name: string, OperationNormalized] => [name, {
     name,
-    type: Grafaid.RootTypeNameToOperationName[Grafaid.Schema.RootTypeName.Mutation],
+    type: Grafaid.Document.RootTypeToOperationType[Grafaid.Schema.RootTypeName.Mutation],
     rootType: Grafaid.Schema.RootTypeName.Mutation,
     selectionSet,
   }])

src/generator/generators/MethodsRoot.ts

@@ -30,8 +30,8 @@ export const ModuleGeneratorMethodsRoot = createModuleGenerator(
       export interface BuilderMethodsRoot<$Context extends ${identifiers.$$Utilities}.ClientContext> {
         ${
       config.schema.kindMap.Root.map(node => {
-        const operationName =
-          Grafaid.RootTypeNameToOperationName[node.name as keyof typeof Grafaid.RootTypeNameToOperationName]
+        const operationName = Grafaid.Document
+          .RootTypeToOperationType[node.name as keyof typeof Grafaid.Document.RootTypeToOperationType]
         return `${operationName}: ${node.name}Methods<$Context>`
       }).join(`\n`)
     }

src/layers/5_request/core.ts

@@ -1,10 +1,8 @@
-import { type ExecutionResult } from 'graphql'
 import { SelectionSetGraphqlMapper } from '../../documentBuilder/SelectGraphQLMapper/__.js'
 import { Anyware } from '../../lib/anyware/__.js'
-import type { Grafaid } from '../../lib/grafaid/__.js'
+import { Grafaid } from '../../lib/grafaid/__.js'
 import { getOperationDefinition, OperationTypeToAccessKind, print } from '../../lib/grafaid/document.js'
-import { execute } from '../../lib/grafaid/execute.js'
-import { operationTypeToRootType } from '../../lib/grafaid/graphql.js'
+import { execute } from '../../lib/grafaid/execute.js' // todo
 import {
   getRequestEncodeSearchParameters,
   getRequestHeadersRec,
@@ -44,15 +42,16 @@ export const graffleMappedResultToRequest = (
   if (!operation_) throw new Error(`Unknown operation named "${String(operationName)}".`)
 
   return {
-    rootType: operationTypeToRootType[operation_.operation],
+    rootType: Grafaid.Document.OperationTypeToRootType[operation_.operation],
     operationName,
     operation: operation_,
     query: document,
     variables: variables_,
   }
 }
 
-export const anyware = Anyware.create<HookSequence, HookMap, ExecutionResult>({
+// todo execution result only if transport is memory
+export const anyware = Anyware.create<HookSequence, HookMap, Grafaid.FormattedExecutionResult>({
   // If core errors caused by an abort error then raise it as a direct error.
   // This is an expected possible error. Possible when user cancels a request.
   passthroughErrorWith: (signal) => {

src/layers/5_request/hooks.ts

@@ -1,4 +1,4 @@
-import type { ExecutionResult, GraphQLSchema } from 'graphql'
+import type { FormattedExecutionResult, GraphQLSchema } from 'graphql'
 import type { Select } from '../../documentBuilder/Select/__.js'
 import type { Grafaid } from '../../lib/grafaid/__.js'
 import type { getRequestEncodeSearchParameters, postRequestEncodeBody } from '../../lib/grafaid/http/http.js'
@@ -96,7 +96,7 @@ export type HookDefUnpack<$Config extends Config> = {
     & TransportInput<
       $Config,
       { response: Response },
-      { result: ExecutionResult }
+      { result: FormattedExecutionResult }
     >
 }
 
@@ -108,7 +108,7 @@ export type HookDefDecode<$Config extends Config> = {
       $Config,
       { response: Response }
     >
-    & { result: ExecutionResult }
+    & { result: FormattedExecutionResult }
 }
 
 export type HookMap<$Config extends Config = Config> = {

src/layers/6_client/client.ts

@@ -1,4 +1,4 @@
-import { CustomScalars } from '../../extensions/CustomScalars/CustomScalars.js'
+import { CustomScalars } from '../../extensions/CustomScalars/CustomScalars.js' // todo
 import type { ConfigManager } from '../../lib/config-manager/__.js'
 import type { Fluent } from '../../lib/fluent/__.js'
 import { proxyGet } from '../../lib/prelude.js'
@@ -11,7 +11,7 @@ import type { FnInternal } from './properties/internal.js'
 import { type FnRetry, retryProperties } from './properties/retry.js'
 import { type ScalarFn, scalarProperties } from './properties/scalar.js'
 import { type FnWith, withProperties } from './properties/with.js'
-import { type FnRequestMethods, requestMethodsProperties } from './requestMethods/requestMethods.js'
+import { type FnRequestMethods, requestMethodsProperties } from './requestMethods/requestMethods.js' // todo
 import { type InputStatic } from './Settings/Input.js'
 import { type NormalizeInput } from './Settings/InputToConfig.js'