[language server] Allow configuration of validation rules.

Closes #1056

Author: alloy

CHANGELOG.md

@@ -20,7 +20,7 @@
 - `apollo-graphql`
   - <First `apollo-graphql` related entry goes here>
 - `apollo-language-server`
-  - <First `apollo-language-server` related entry goes here>
+  - Allow configuration of validation rules [#1288](https://github.com/apollographql/apollo-tooling/pull/1288)
 - `apollo-tools`
   - <First `apollo-tools` related entry goes here>
 - `vscode-apollo`

package-lock.json

@@ -87,7 +87,7 @@ function extractDocumentsWithAST(
         // This currently ignores the anti-pattern of including an interpolated
         // string as anything other than a fragment definition, for example a
         // literal(these cases could be covered during the replacement of
-        // literals in the signature calcluation)
+        // literals in the signature calculation)
         finished.push(
           (path.value.quasi.quasis as Array<{
             value: { cooked: string; raw: string };

packages/apollo-codegen-core/src/loading.ts

@@ -9,6 +9,7 @@ import {
 import URI from "vscode-uri";
 import { WithRequired } from "apollo-env";
 import { getServiceName, parseServiceSpecifier } from "./utils";
+import { ValidationRule } from "graphql/validation/ValidationContext";
 
 export interface EngineStatsWindow {
   to: number;
@@ -72,6 +73,17 @@ export interface ClientConfigFormat extends ConfigBase {
   tagName?: string;
   // stats window config
   statsWindow?: StatsWindowSize;
+
+  /**
+   * Rules that will be applied when validating GraphQL documents.
+   *
+   * If you wish to modify the default list of validation rules, import them from the apollo package:
+   *
+   * ```js
+   * const { defaultValidationRules } = require("apollo/lib/defaultValidationRules");
+   * ```
+   */
+  validationRules?: ValidationRule[];
 }
 
 export const DefaultClientConfig = {

packages/apollo-language-server/src/config/config.ts

@@ -184,7 +184,7 @@ export async function loadConfig({
 
   let { config, filepath } = loadedConfig;
 
-  // selectivly apply defaults when loading the config
+  // selectively apply defaults when loading the config
   // this is just the includes/excludes defaults.
   // These need to go on _all_ configs. That's why this is last.
   if (config.client) config = merge({ client: DefaultClientConfig }, config);

packages/apollo-language-server/src/config/loadConfig.ts

@@ -14,14 +14,16 @@ import { rangeForASTNode } from "./utilities/source";
 
 import { getValidationErrors } from "./errors/validation";
 import { DocumentUri } from "./project/base";
+import { ValidationRule } from "graphql/validation/ValidationContext";
 
 /**
  * Build an array of code diagnostics for all executable definitions in a document.
  */
 export function collectExecutableDefinitionDiagnositics(
   schema: GraphQLSchema,
   queryDocument: GraphQLDocument,
-  fragments: { [fragmentName: string]: FragmentDefinitionNode } = {}
+  fragments: { [fragmentName: string]: FragmentDefinitionNode } = {},
+  rules?: ValidationRule[]
 ): Diagnostic[] {
   const ast = queryDocument.ast;
   if (!ast) return queryDocument.syntaxErrors;
@@ -36,7 +38,8 @@ export function collectExecutableDefinitionDiagnositics(
   for (const error of getValidationErrors(
     schema,
     astWithExecutableDefinitions,
-    fragments
+    fragments,
+    rules
   )) {
     diagnostics.push(
       ...diagnosticsFromError(error, DiagnosticSeverity.Error, "Validation")

packages/apollo-language-server/src/diagnostics.ts

@@ -16,23 +16,22 @@ import {
 } from "graphql";
 
 import { ToolError, logError } from "./logger";
+import { ValidationRule } from "graphql/validation/ValidationContext";
+
+const specifiedRulesToBeRemoved = [NoUnusedFragmentsRule, KnownDirectivesRule];
+
+export const defaultValidationRules: ValidationRule[] = [
+  NoAnonymousQueries,
+  NoTypenameAlias,
+  ...specifiedRules.filter(rule => !specifiedRulesToBeRemoved.includes(rule))
+];
 
 export function getValidationErrors(
   schema: GraphQLSchema,
   document: DocumentNode,
-  fragments?: { [fragmentName: string]: FragmentDefinitionNode }
+  fragments?: { [fragmentName: string]: FragmentDefinitionNode },
+  rules: ValidationRule[] = defaultValidationRules
 ) {
-  const specifiedRulesToBeRemoved = [
-    NoUnusedFragmentsRule,
-    KnownDirectivesRule
-  ];
-
-  const rules = [
-    NoAnonymousQueries,
-    NoTypenameAlias,
-    ...specifiedRules.filter(rule => !specifiedRulesToBeRemoved.includes(rule))
-  ];
-
   const typeInfo = new TypeInfo(schema);
   const context = new ValidationContext(schema, document, typeInfo);
 

packages/apollo-language-server/src/errors/validation.ts

@@ -237,7 +237,8 @@ export class GraphQLClientProject extends GraphQLProject {
           collectExecutableDefinitionDiagnositics(
             this.schema,
             document,
-            fragments
+            fragments,
+            this.config.client.validationRules
           )
         );
       }

packages/apollo-language-server/src/project/client.ts

@@ -0,0 +1,3 @@
+export {
+  defaultValidationRules
+} from "apollo-language-server/lib/errors/validation";