Fix wasm publish (#3003)

apollographql · May 13, 2024 · 35e6065 · 35e6065
1 parent 3914ec2
commit 35e6065
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 12 deletions.
diff --git a/internals-js/src/wasm.ts b/internals-js/src/wasm.ts
@@ -17,9 +17,6 @@ import {aggregateError} from "./error";
 import {ErrorLocation, SourceDirective} from "../wasm/node";
 import {ASTNode, ConstValueNode, SchemaExtensionNode, StringValueNode} from "graphql";
 
-const {validate_connect_directives} =
-    require('../wasm/node') as typeof import('../wasm/node');
-
 /**
  * @throws AggregateGraphQLError
  */
@@ -49,6 +46,8 @@ function hasConnectDirectives(schema: Schema): boolean {
 function validateConnectDirectives(ast: DocumentNode): GraphQLError[] {
     // TODO: pass the _original_ source, not the one with added federation stuff (otherwise positions will be wrong)
     // TODO: unless, of course, there's already infrastructure up above which does translation somehow
+    const {validate_connect_directives} =
+        require('../wasm/node') as typeof import('../wasm/node');
     const source = print(ast)
     return validate_connect_directives(source).map(
         raw => new GraphQLError(raw.message, {

diff --git a/internals-js/wasm/build.sh b/internals-js/wasm/build.sh
diff --git a/internals-js/wasm/node/.npmignore b/internals-js/wasm/node/.npmignore
@@ -0,0 +1,2 @@
+# We don't want to check in build artifacts (thus the .gitignore), but we _do_ need NPM to publish those files
+# .npmignore overrides .gitignore