@lightdash/cli 1.94.0 → 1.95.1

package/dist/handlers/apps/validate.d.ts

@@ -15,6 +15,11 @@ export type AppsValidationIssue = {
 export type AppsValidationCoverage = DataAppDataReferences['stats'] & {
     unanalyzed: number;
 };
+export type AppsValidationUnanalyzedReference = {
+    kind: DataAppDataReferences['references'][number]['kind'];
+    unresolved: string[];
+    location: DataReferenceLocation;
+};
 export type AppValidationResult = {
     path: string;
     name: string | null;
@@ -23,6 +28,7 @@ export type AppValidationResult = {
     errors: AppsValidationIssue[];
     warnings: AppsValidationIssue[];
     coverage: AppsValidationCoverage;
+    unanalyzedReferences: AppsValidationUnanalyzedReference[];
 };
 export type AppsValidationReport = {
     build: boolean;
@@ -64,7 +70,8 @@ export declare const validateDataAppBuild: (args: {
 export declare const fetchLiveDataAppExploreIndex: (projectUuid: string) => Promise<DataAppExploreIndex>;
 export declare const validateLocalDataApp: (appPath: string, options: ValidateAppOptions) => Promise<AppValidationResult>;
 export declare const buildAppsValidationReport: (apps: AppValidationResult[], live: boolean, build?: boolean) => AppsValidationReport;
-export declare const renderAppsValidationHuman: (report: AppsValidationReport) => string;
+export declare const renderAppsValidationHuman: (report: AppsValidationReport, verbose?: boolean) => string;
+export declare const renderAppsValidationJson: (report: AppsValidationReport) => string;
 export declare const appsValidateHandler: (pathArgs: string[] | undefined, options: AppsValidateOptions) => Promise<void>;
 export {};
 //# sourceMappingURL=validate.d.ts.map

package/dist/handlers/apps/validate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../../../src/handlers/apps/validate.ts"],"names":[],"mappings":"AAAA,OAAO,EAeH,KAAK,WAAW,EAChB,KAAK,qBAAqB,EAC1B,KAAK,mBAAmB,EAGxB,KAAK,qBAAqB,EAC7B,MAAM,mBAAmB,CAAC;AAsB3B,MAAM,MAAM,kBAAkB,GAAG,OAAO,GAAG,MAAM,CAAC;AAElD,MAAM,MAAM,mBAAmB,GAAG;IAC9B,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,kBAAkB,CAAC;IAC3B,IAAI,EAAE,OAAO,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;CACpB,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAC7B,OAAO,GACP,QAAQ,GACR,cAAc,GACd,qBAAqB,GACrB,UAAU,GACV,gBAAgB,GAChB,oBAAoB,GACpB,cAAc,CAAC;AAErB,MAAM,MAAM,mBAAmB,GAAG;IAC9B,IAAI,EAAE,uBAAuB,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,qBAAqB,GAAG,IAAI,CAAC;CAC1C,CAAC;AAEF,MAAM,MAAM,sBAAsB,GAAG,qBAAqB,CAAC,OAAO,CAAC,GAAG;IAClE,UAAU,EAAE,MAAM,CAAC;CACtB,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,mBAAmB,EAAE,CAAC;IAC9B,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,QAAQ,EAAE,sBAAsB,CAAC;CACpC,CAAC;AAEF,MAAM,MAAM,oBAAoB,GAAG;IAC/B,KAAK,EAAE,OAAO,CAAC;IACf,KAAK,EAAE,OAAO,CAAC;IACf,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,OAAO,EAAE;QACL,IAAI,EAAE,MAAM,CAAC;QACb,MAAM,EAAE,MAAM,CAAC;QACf,QAAQ,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,MAAM,CAAC;QAClB,mBAAmB,EAAE,MAAM,CAAC;KAC/B,CAAC;IACF,IAAI,EAAE,mBAAmB,EAAE,CAAC;CAC/B,CAAC;AAEF,KAAK,kBAAkB,GAAG;IACtB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,IAAI,EAAE,OAAO,CAAC;IACd,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,aAAa,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC,mBAAmB,CAAC,CAAC;IACrE,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE;QACd,MAAM,EAAE,MAAM,CAAC;QACf,MAAM,EAAE,WAAW,CAAC;KACvB,KAAK,OAAO,CAAC,mBAAmB,EAAE,CAAC,CAAC;CACxC,CAAC;AAmLF,KAAK,mBAAmB,GAAG;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,EAAE,CAAC;IACf,GAAG,EAAE,MAAM,CAAC;IACZ,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;IACxB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF,MAAM,MAAM,sBAAsB,GAAG,CACjC,OAAO,EAAE,mBAAmB,KAC3B,OAAO,CAAC,IAAI,CAAC,CAAC;AAmEnB,eAAO,MAAM,oBAAoB,SAAgB;IAC7C,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,CAAC;IACpB,UAAU,CAAC,EAAE,sBAAsB,CAAC;CACvC,KAAG,OAAO,CAAC,mBAAmB,EAAE,CA+HhC,CAAC;AAUF,eAAO,MAAM,4BAA4B,gBACxB,MAAM,KACpB,OAAO,CAAC,mBAAmB,CAqB7B,CAAC;AAEF,eAAO,MAAM,oBAAoB,YACpB,MAAM,WACN,kBAAkB,KAC5B,OAAO,CAAC,mBAAmB,CAsI7B,CAAC;AAEF,eAAO,MAAM,yBAAyB,SAC5B,mBAAmB,EAAE,QACrB,OAAO,sBAEd,oBAyBF,CAAC;AAmBF,eAAO,MAAM,yBAAyB,WAC1B,oBAAoB,KAC7B,MA4CF,CAAC;AAEF,eAAO,MAAM,mBAAmB,aAClB,MAAM,EAAE,GAAG,SAAS,WACrB,mBAAmB,KAC7B,OAAO,CAAC,IAAI,CA4Cd,CAAC"}
+{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../../../src/handlers/apps/validate.ts"],"names":[],"mappings":"AAAA,OAAO,EAeH,KAAK,WAAW,EAChB,KAAK,qBAAqB,EAC1B,KAAK,mBAAmB,EAGxB,KAAK,qBAAqB,EAC7B,MAAM,mBAAmB,CAAC;AAsB3B,MAAM,MAAM,kBAAkB,GAAG,OAAO,GAAG,MAAM,CAAC;AAElD,MAAM,MAAM,mBAAmB,GAAG;IAC9B,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,kBAAkB,CAAC;IAC3B,IAAI,EAAE,OAAO,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;CACpB,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAC7B,OAAO,GACP,QAAQ,GACR,cAAc,GACd,qBAAqB,GACrB,UAAU,GACV,gBAAgB,GAChB,oBAAoB,GACpB,cAAc,CAAC;AAErB,MAAM,MAAM,mBAAmB,GAAG;IAC9B,IAAI,EAAE,uBAAuB,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,qBAAqB,GAAG,IAAI,CAAC;CAC1C,CAAC;AAEF,MAAM,MAAM,sBAAsB,GAAG,qBAAqB,CAAC,OAAO,CAAC,GAAG;IAClE,UAAU,EAAE,MAAM,CAAC;CACtB,CAAC;AAEF,MAAM,MAAM,iCAAiC,GAAG;IAC5C,IAAI,EAAE,qBAAqB,CAAC,YAAY,CAAC,CAAC,MAAM,CAAC,CAAC,MAAM,CAAC,CAAC;IAC1D,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,QAAQ,EAAE,qBAAqB,CAAC;CACnC,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,mBAAmB,EAAE,CAAC;IAC9B,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,QAAQ,EAAE,sBAAsB,CAAC;IACjC,oBAAoB,EAAE,iCAAiC,EAAE,CAAC;CAC7D,CAAC;AAEF,MAAM,MAAM,oBAAoB,GAAG;IAC/B,KAAK,EAAE,OAAO,CAAC;IACf,KAAK,EAAE,OAAO,CAAC;IACf,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,OAAO,EAAE;QACL,IAAI,EAAE,MAAM,CAAC;QACb,MAAM,EAAE,MAAM,CAAC;QACf,QAAQ,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,MAAM,CAAC;QAClB,mBAAmB,EAAE,MAAM,CAAC;KAC/B,CAAC;IACF,IAAI,EAAE,mBAAmB,EAAE,CAAC;CAC/B,CAAC;AAEF,KAAK,kBAAkB,GAAG;IACtB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,IAAI,EAAE,OAAO,CAAC;IACd,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,aAAa,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC,mBAAmB,CAAC,CAAC;IACrE,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE;QACd,MAAM,EAAE,MAAM,CAAC;QACf,MAAM,EAAE,WAAW,CAAC;KACvB,KAAK,OAAO,CAAC,mBAAmB,EAAE,CAAC,CAAC;CACxC,CAAC;AAmLF,KAAK,mBAAmB,GAAG;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,EAAE,CAAC;IACf,GAAG,EAAE,MAAM,CAAC;IACZ,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;IACxB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF,MAAM,MAAM,sBAAsB,GAAG,CACjC,OAAO,EAAE,mBAAmB,KAC3B,OAAO,CAAC,IAAI,CAAC,CAAC;AAmEnB,eAAO,MAAM,oBAAoB,SAAgB;IAC7C,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,CAAC;IACpB,UAAU,CAAC,EAAE,sBAAsB,CAAC;CACvC,KAAG,OAAO,CAAC,mBAAmB,EAAE,CA+HhC,CAAC;AAUF,eAAO,MAAM,4BAA4B,gBACxB,MAAM,KACpB,OAAO,CAAC,mBAAmB,CAqB7B,CAAC;AAEF,eAAO,MAAM,oBAAoB,YACpB,MAAM,WACN,kBAAkB,KAC5B,OAAO,CAAC,mBAAmB,CAgJ7B,CAAC;AAEF,eAAO,MAAM,yBAAyB,SAC5B,mBAAmB,EAAE,QACrB,OAAO,sBAEd,oBAyBF,CAAC;AAkCF,eAAO,MAAM,yBAAyB,WAC1B,oBAAoB,wBAE7B,MAkDF,CAAC;AAEF,eAAO,MAAM,wBAAwB,WACzB,oBAAoB,KAC7B,MAKM,CAAC;AAEV,eAAO,MAAM,mBAAmB,aAClB,MAAM,EAAE,GAAG,SAAS,WACrB,mBAAmB,KAC7B,OAAO,CAAC,IAAI,CA4Cd,CAAC"}

package/dist/handlers/apps/validate.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.appsValidateHandler = exports.renderAppsValidationHuman = exports.buildAppsValidationReport = exports.validateLocalDataApp = exports.fetchLiveDataAppExploreIndex = exports.validateDataAppBuild = void 0;
+exports.appsValidateHandler = exports.renderAppsValidationJson = exports.renderAppsValidationHuman = exports.buildAppsValidationReport = exports.validateLocalDataApp = exports.fetchLiveDataAppExploreIndex = exports.validateDataAppBuild = void 0;
 const tslib_1 = require("tslib");
 const common_1 = require("@lightdash/common");
 const execa_1 = tslib_1.__importDefault(require("execa"));
@@ -293,6 +293,7 @@ const validateLocalDataApp = async (appPath, options) => {
             errors: [issue('bundle', (0, common_1.getErrorMessage)(error))],
             warnings,
             coverage: emptyCoverage(),
+            unanalyzedReferences: [],
         };
     }
     const { manifest } = bundle;
@@ -306,6 +307,13 @@ const validateLocalDataApp = async (appPath, options) => {
     }
     const extracted = (0, common_1.extractDataAppDataReferences)(sourceFiles);
     const coverage = toCoverage(extracted.stats);
+    const unanalyzedReferences = extracted.references
+        .filter((reference) => reference.unresolved.length > 0)
+        .map((reference) => ({
+        kind: reference.kind,
+        unresolved: [...reference.unresolved],
+        location: reference.location,
+    }));
     warnings.push(...extracted.parseErrors.map((parseError) => issue('source_parse', `Could not statically analyze ${parseError.path}: ${parseError.message}`)));
     for (const ref of extracted.references) {
         if (ref.kind === 'externalFetch' &&
@@ -370,6 +378,7 @@ const validateLocalDataApp = async (appPath, options) => {
         errors,
         warnings,
         coverage,
+        unanalyzedReferences,
     };
 };
 exports.validateLocalDataApp = validateLocalDataApp;
@@ -406,7 +415,14 @@ const formatIssue = (entry) => {
         : '';
     return `${prefix}${entry.message}`;
 };
-const renderAppsValidationHuman = (report) => {
+const referenceKindLabels = {
+    query: 'query',
+    savedChart: 'saved chart',
+    externalFetch: 'external fetch',
+    globalFilter: 'global filter',
+};
+const formatUnanalyzedReference = (reference) => `${reference.location.path}:${reference.location.line}:${reference.location.column} — ${referenceKindLabels[reference.kind]} (unresolved: ${reference.unresolved.join(', ')})`;
+const renderAppsValidationHuman = (report, verbose = false) => {
     const lines = [
         `Validating ${report.summary.apps} data app(s) using ${report.mode === 'live'
             ? 'the live project semantic layer'
@@ -420,6 +436,12 @@ const renderAppsValidationHuman = (report) => {
         const label = app.name ? `${app.name} (${app.path})` : app.path;
         lines.push(`${app.valid ? styles.success('✓') : styles.error('✗')} ${label}`);
         lines.push(`  ${formatCoverage(app.coverage)}`);
+        if (verbose && app.unanalyzedReferences.length > 0) {
+            lines.push('  Static analysis gaps:');
+            for (const reference of app.unanalyzedReferences) {
+                lines.push(`    ${formatUnanalyzedReference(reference)}`);
+            }
+        }
         for (const warning of app.warnings) {
             lines.push(`  ${styles.warning('warning')} ${formatIssue(warning)}`);
         }
@@ -434,6 +456,8 @@ const renderAppsValidationHuman = (report) => {
     return `${lines.join('\n')}\n`;
 };
 exports.renderAppsValidationHuman = renderAppsValidationHuman;
+const renderAppsValidationJson = (report) => `${JSON.stringify(report, (key, value) => (key === 'unanalyzedReferences' ? undefined : value), 2)}\n`;
+exports.renderAppsValidationJson = renderAppsValidationJson;
 const appsValidateHandler = async (pathArgs, options) => {
     globalState_1.default.setVerbose(options.verbose);
     const paths = pathArgs && pathArgs.length > 0 ? pathArgs : ['.'];
@@ -460,8 +484,8 @@ const appsValidateHandler = async (pathArgs, options) => {
     })));
     const report = (0, exports.buildAppsValidationReport)(apps, options.live, options.build);
     process.stdout.write(options.format === 'json'
-        ? `${JSON.stringify(report, null, 2)}\n`
-        : (0, exports.renderAppsValidationHuman)(report));
+        ? (0, exports.renderAppsValidationJson)(report)
+        : (0, exports.renderAppsValidationHuman)(report, options.verbose));
     if (!report.valid) {
         throw new common_1.ParameterError(`Data app validation failed with ${report.summary.errors} error(s).`);
     }

package/dist/handlers/apps/validate.test.js

@@ -392,29 +392,69 @@ describe('validateDataAppBuild', () => {
     });
 });
 describe('renderAppsValidationHuman', () => {
-    it('makes partial green coverage explicit', () => {
-        const report = (0, validate_1.buildAppsValidationReport)([
-            {
-                path: '/tmp/orders-app',
-                name: 'Orders app',
-                projectUuid: 'project-uuid',
-                valid: true,
-                errors: [],
-                warnings: [],
-                coverage: {
-                    callSites: 14,
-                    fullyResolved: 11,
-                    partiallyResolved: 2,
-                    unresolved: 1,
-                    unanalyzed: 3,
-                },
+    const report = (0, validate_1.buildAppsValidationReport)([
+        {
+            path: '/tmp/orders-app',
+            name: 'Orders app',
+            projectUuid: 'project-uuid',
+            valid: true,
+            errors: [],
+            warnings: [],
+            coverage: {
+                callSites: 14,
+                fullyResolved: 11,
+                partiallyResolved: 2,
+                unresolved: 1,
+                unanalyzed: 3,
             },
-        ], false, true);
+            unanalyzedReferences: [
+                {
+                    kind: 'query',
+                    unresolved: ['explore', 'dimensions'],
+                    location: {
+                        path: 'src/DynamicQuery.tsx',
+                        line: 8,
+                        column: 17,
+                    },
+                },
+                {
+                    kind: 'globalFilter',
+                    unresolved: ['field'],
+                    location: {
+                        path: 'src/ResultsTable.tsx',
+                        line: 42,
+                        column: 21,
+                    },
+                },
+                {
+                    kind: 'externalFetch',
+                    unresolved: ['alias'],
+                    location: {
+                        path: 'src/ExternalData.tsx',
+                        line: 12,
+                        column: 9,
+                    },
+                },
+            ],
+        },
+    ], false, true);
+    it('makes partial green coverage explicit', () => {
         const output = (0, validate_1.renderAppsValidationHuman)(report);
         expect(output).toContain("3 of 14 data-reference call site(s) couldn't be fully analyzed");
         expect(output).toContain('unresolved values were skipped');
         expect(output).toContain('Validation passed');
         expect(output).toContain('Offline snapshots may be stale');
         expect(output).toContain('running Vite production builds');
+        expect(output).not.toContain('Static analysis gaps');
+    });
+    it('lists unresolved call sites and parts in verbose output', () => {
+        const output = (0, validate_1.renderAppsValidationHuman)(report, true);
+        expect(output).toContain('Static analysis gaps:');
+        expect(output).toContain('src/DynamicQuery.tsx:8:17 — query (unresolved: explore, dimensions)');
+        expect(output).toContain('src/ResultsTable.tsx:42:21 — global filter (unresolved: field)');
+    });
+    it('keeps verbose-only reference details out of JSON output', () => {
+        const output = (0, validate_1.renderAppsValidationJson)(report);
+        expect(JSON.parse(output)).not.toHaveProperty('apps.0.unanalyzedReferences');
     });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightdash/cli",
-  "version": "1.94.0",
+  "version": "1.95.1",
   "description": "Lightdash CLI tool",
   "license": "MIT",
   "repository": {
@@ -41,8 +41,8 @@
     "unique-names-generator": "4.7.1",
     "uuid": "11.1.1",
     "yaml": "2.8.3",
-    "@lightdash/common": "1.94.0",
-    "@lightdash/warehouses": "1.94.0"
+    "@lightdash/common": "1.95.1",
+    "@lightdash/warehouses": "1.95.1"
   },
   "devDependencies": {
     "@types/inquirer": "8.2.4",