Merge pull request #76 from hasura/lyndon/prevendor-by-default-2

Fixing implementation error for configuration defaults: `preVendor`

CHANGELOG.md

@@ -9,6 +9,14 @@ This changelog documents the changes between release versions.
 
 Changes to be included in the next upcoming releaase.
 
+## v0.17
+
+Fixing issue with preVendor default.
+
+PR: https://github.com/hasura/ndc-typescript-deno/pull/76
+
+* Was only applying the default during user-interactive config flows
+
 ## v0.16
 
 Prevendoring by default.

src/connector.ts

@@ -250,17 +250,17 @@ export const connector: sdk.Connector<Configuration, Configuration, State> = {
 
   // TODO: https://github.com/hasura/ndc-typescript-deno/issues/27 Make this add in the defaults
   update_configuration(configuration: Configuration): Promise<Configuration> {
+    return Promise.resolve(configuration);
+  },
+
+  validate_raw_configuration(configuration: Configuration): Promise<Configuration> {
     const defaults = {
       preVendor: true,
     }
     const response = { ...defaults, ...configuration }
     return Promise.resolve(response);
   },
 
-  validate_raw_configuration(configuration: Configuration): Promise<Configuration> {
-    return Promise.resolve(configuration);
-  },
-
   get_schema(config: Configuration): Promise<sdk.SchemaResponse> {
     const result = getInfo(config);
     return Promise.resolve(result.schema);