wgc 0.44.2 → 0.45.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wgc",
-  "version": "0.44.2",
+  "version": "0.45.1",
   "description": "The official CLI tool to manage the GraphQL Federation Platform Cosmo",
   "type": "module",
   "main": "dist/index.js",
@@ -33,9 +33,9 @@
     "@bufbuild/protobuf": "^1.4.1",
     "@connectrpc/connect": "^1.1.3",
     "@connectrpc/connect-node": "^1.1.3",
-    "@wundergraph/composition": "0.18.5",
-    "@wundergraph/cosmo-connect": "0.59.1",
-    "@wundergraph/cosmo-shared": "0.20.16",
+    "@wundergraph/composition": "0.20.0",
+    "@wundergraph/cosmo-connect": "0.61.0",
+    "@wundergraph/cosmo-shared": "0.21.0",
     "boxen": "^7.1.1",
     "cli-progress": "^3.12.0",
     "cli-table3": "^0.6.3",
@@ -70,5 +70,5 @@
     "typescript": "^5.2.2",
     "vitest": "^0.34.1"
   },
-  "gitHead": "7ef74c61ea8185cc54791be56a9630e50360c057"
+  "gitHead": "1ad9a19ca38b657f1e8230e7d6b5ccf8e0f84bac"
 }

package/src/commands/subgraph/commands/check.ts

@@ -86,7 +86,6 @@ export default (opts: BaseCommandOptions) => {
 
     const changesTable = new Table({
       head: [pc.bold(pc.white('CHANGE')), pc.bold(pc.white('TYPE')), pc.bold(pc.white('DESCRIPTION'))],
-      colWidths: [15, 30, 80],
       wordWrap: true,
     });
 
@@ -100,12 +99,22 @@ export default (opts: BaseCommandOptions) => {
       wordWrap: true,
     });
 
+    const lintIssuesTable = new Table({
+      head: [pc.bold(pc.white('LINT_RULE')), pc.bold(pc.white('ERROR_MESSAGE')), pc.bold(pc.white('LINE NUMBER'))],
+      colAligns: ['left', 'left', 'center'],
+      wordWrap: true,
+    });
+
     let success = false;
     let finalStatement = '';
 
     let studioCheckDestination = '';
     if (resp.checkId && resp.checkedFederatedGraphs.length > 0) {
-      studioCheckDestination = `Open in studio: ${config.webURL}/${resp.checkedFederatedGraphs[0].organizationSlug}/${resp.checkedFederatedGraphs[0].namespace}/graph/${resp.checkedFederatedGraphs[0].name}/checks/${resp.checkId}`;
+      studioCheckDestination = `${pc.bold('Open in studio')}: ${config.webURL}/${
+        resp.checkedFederatedGraphs[0].organizationSlug
+      }/${resp.checkedFederatedGraphs[0].namespace}/graph/${resp.checkedFederatedGraphs[0].name}/checks/${
+        resp.checkId
+      }`;
     }
 
     switch (resp.response?.code) {
@@ -113,9 +122,11 @@ export default (opts: BaseCommandOptions) => {
         if (
           resp.nonBreakingChanges.length === 0 &&
           resp.breakingChanges.length === 0 &&
-          resp.compositionErrors.length === 0
+          resp.compositionErrors.length === 0 &&
+          resp.lintErrors.length === 0 &&
+          resp.lintWarnings.length === 0
         ) {
-          console.log(`\nDetected no changes.\n${studioCheckDestination}\n`);
+          console.log(`\nDetected no changes.\nDetected no lint issues.\n\n${studioCheckDestination}\n`);
 
           success = true;
 
@@ -167,13 +178,21 @@ export default (opts: BaseCommandOptions) => {
 
           if (resp.breakingChanges.length > 0) {
             for (const breakingChange of resp.breakingChanges) {
-              changesTable.push([pc.red('BREAKING'), breakingChange.changeType, breakingChange.message]);
+              changesTable.push([
+                `${logSymbols.error} ${pc.red('BREAKING')}`,
+                breakingChange.changeType,
+                breakingChange.message,
+              ]);
             }
           }
 
           if (resp.nonBreakingChanges.length > 0) {
             for (const nonBreakingChange of resp.nonBreakingChanges) {
-              changesTable.push(['NON-BREAKING', nonBreakingChange.changeType, nonBreakingChange.message]);
+              changesTable.push([
+                `${logSymbols.success} NON-BREAKING`,
+                nonBreakingChange.changeType,
+                nonBreakingChange.message,
+              ]);
             }
           }
 
@@ -192,12 +211,32 @@ export default (opts: BaseCommandOptions) => {
           console.log(compositionErrorsTable.toString());
         }
 
+        if (resp.lintErrors.length > 0 || resp.lintWarnings.length > 0) {
+          success = resp.lintErrors.length === 0;
+          console.log('\nDetected lint issues:');
+          for (const error of resp.lintErrors) {
+            lintIssuesTable.push([
+              `${logSymbols.error} ${pc.red(error.lintRuleType)}`,
+              error.message,
+              error.issueLocation?.line,
+            ]);
+          }
+          for (const warning of resp.lintWarnings) {
+            lintIssuesTable.push([
+              `${logSymbols.warning} ${pc.yellow(warning.lintRuleType)}`,
+              warning.message,
+              warning.issueLocation?.line,
+            ]);
+          }
+          console.log(lintIssuesTable.toString());
+        }
+
         if (success) {
           console.log(
             '\n' +
               logSymbols.success +
               pc.green(` Schema check passed. ${finalStatement}`) +
-              '\n' +
+              '\n\n' +
               studioCheckDestination +
               '\n',
           );