@lightdash/cli 2.198.5 → 2.199.0

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

@@ -1 +1 @@
-{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../../src/handlers/validate.ts"],"names":[],"mappings":"AAAA,OAAO,EAKH,OAAO,EACP,YAAY,EASZ,kBAAkB,EAElB,gBAAgB,EACnB,MAAM,mBAAmB,CAAC;AAO3B,OAAO,EAAW,qBAAqB,EAAE,MAAM,WAAW,CAAC;AAU3D,eAAO,MAAM,iBAAiB,gBACb,MAAM,YACT,CAAC,OAAO,GAAG,YAAY,CAAC,EAAE,qBACjB,gBAAgB,EAAE;;EAMnC,CAAC;AAEP,eAAO,MAAM,WAAW,YAAmB,MAAM;;;EAK3C,CAAC;AAEP,eAAO,MAAM,aAAa,gBAAuB,MAAM,SAAS,MAAM,8DAKhE,CAAC;AAmCP,eAAO,MAAM,qBAAqB,YAAI,OAAO,EAAE,SAAS,CAAU,CAAC;AACnE,MAAM,MAAM,kBAAkB,GAAG,CAAC,OAAO,qBAAqB,CAAC,CAAC,MAAM,CAAC,CAAC;AAExE,eAAO,MAAM,6CAA6C,2FACkC,CAAC;AAE7F,KAAK,sBAAsB,GAAG,qBAAqB,GAAG;IAClD,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,gBAAgB,EAAE,CAAC;IACzB,wBAAwB,EAAE,OAAO,CAAC;IAClC,QAAQ,CAAC,EAAE,kBAAkB,CAAC;IAC9B,8BAA8B,CAAC,EAAE,OAAO,CAAC;IACzC,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;IACzB,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;CAC5B,CAAC;AAEF,wBAAgB,uBAAuB,CACnC,OAAO,EAAE,IAAI,CACT,sBAAsB,EACtB,UAAU,GAAG,gCAAgC,CAChD,GACF,kBAAkB,CAKpB;AAED,KAAK,wBAAwB,GAAG;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,WAAW,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,KAAK,CAAC;CAC5C,CAAC;AAEF,eAAO,MAAM,iBAAiB,iDAI3B,wBAAwB,KAAG,OAAO,CAAC,MAAM,CAY3C,CAAC;AAEF,eAAO,MAAM,eAAe,oBACP,sBAAsB,kBAuY1C,CAAC"}
+{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../../src/handlers/validate.ts"],"names":[],"mappings":"AAAA,OAAO,EAKH,OAAO,EACP,YAAY,EASZ,kBAAkB,EAElB,gBAAgB,EACnB,MAAM,mBAAmB,CAAC;AAO3B,OAAO,EAAW,qBAAqB,EAAE,MAAM,WAAW,CAAC;AAU3D,eAAO,MAAM,iBAAiB,gBACb,MAAM,YACT,CAAC,OAAO,GAAG,YAAY,CAAC,EAAE,qBACjB,gBAAgB,EAAE;;EAMnC,CAAC;AAEP,eAAO,MAAM,WAAW,YAAmB,MAAM;;;EAK3C,CAAC;AAEP,eAAO,MAAM,aAAa,gBAAuB,MAAM,SAAS,MAAM,8DAKhE,CAAC;AAmCP,eAAO,MAAM,qBAAqB,YAAI,OAAO,EAAE,SAAS,CAAU,CAAC;AACnE,MAAM,MAAM,kBAAkB,GAAG,CAAC,OAAO,qBAAqB,CAAC,CAAC,MAAM,CAAC,CAAC;AAExE,eAAO,MAAM,6CAA6C,2FACkC,CAAC;AAE7F,KAAK,sBAAsB,GAAG,qBAAqB,GAAG;IAClD,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,gBAAgB,EAAE,CAAC;IACzB,wBAAwB,EAAE,OAAO,CAAC;IAClC,QAAQ,CAAC,EAAE,kBAAkB,CAAC;IAC9B,8BAA8B,CAAC,EAAE,OAAO,CAAC;IACzC,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;IACzB,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;CAC5B,CAAC;AAEF,wBAAgB,uBAAuB,CACnC,OAAO,EAAE,IAAI,CACT,sBAAsB,EACtB,UAAU,GAAG,gCAAgC,CAChD,GACF,kBAAkB,CAKpB;AAED,KAAK,wBAAwB,GAAG;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,WAAW,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,KAAK,CAAC;CAC5C,CAAC;AAEF,eAAO,MAAM,iBAAiB,iDAI3B,wBAAwB,KAAG,OAAO,CAAC,MAAM,CAY3C,CAAC;AAEF,eAAO,MAAM,eAAe,oBACP,sBAAsB,kBAoW1C,CAAC"}

package/dist/handlers/validate.js

@@ -114,28 +114,7 @@ const validateHandler = async (originalOptions) => {
     }
     options.disableTimestampConversion =
         await (0, timestampConversion_1.getProjectDisableTimestampConversion)(options.disableTimestampConversion, projectUuid);
-    let validationTargets = options.only ? [...options.only] : [];
-    const hasPartialModelSelection = Boolean(options.select?.length) ||
-        Boolean(options.models?.length) ||
-        Boolean(options.exclude?.length) ||
-        Boolean(options.selector);
-    const includesDataAppValidation = validationTargets.length === 0 ||
-        validationTargets.includes(common_1.ValidationTarget.APPS);
-    if (hasPartialModelSelection && includesDataAppValidation) {
-        if (validationTargets.length === 1 &&
-            validationTargets[0] === common_1.ValidationTarget.APPS) {
-            throw new common_1.ParameterError('Data app validation requires a full project compile. Remove the dbt model selection flags to use --only apps.');
-        }
-        validationTargets =
-            validationTargets.length === 0
-                ? [
-                    common_1.ValidationTarget.TABLES,
-                    common_1.ValidationTarget.CHARTS,
-                    common_1.ValidationTarget.DASHBOARDS,
-                ]
-                : validationTargets.filter((target) => target !== common_1.ValidationTarget.APPS);
-        console.error(styles.warning('> Skipping data app validation because dbt model selection flags produce a partial semantic layer'));
-    }
+    const validationTargets = options.only ? [...options.only] : [];
     await analytics_1.LightdashAnalytics.track({
         event: 'validate.started',
         properties: {

package/dist/handlers/validate.test.js

@@ -73,7 +73,7 @@ const brokenChartError = {
     lastUpdatedBy: 'Ada Lovelace',
     lastUpdatedAt: new Date('2026-08-06T15:30:00Z'),
 };
-describe('validateHandler warehouse column validation', () => {
+describe('validateHandler', () => {
     let errorOutput;
     let validationResults = [];
     const skipWarnings = () => errorOutput.filter((line) => line.includes('Skipping warehouse column validation'));
@@ -165,31 +165,46 @@ describe('validateHandler warehouse column validation', () => {
         expect(compile_1.compile).toHaveBeenCalledWith(expect.objectContaining({ validateWarehouseColumns: false }));
         expect(skipWarnings()).toEqual([]);
     });
-    test('skips data app validation when dbt selection produces a partial semantic layer', async () => {
+    test.each([
+        { select: ['orders'] },
+        { models: ['orders'] },
+        { exclude: ['customers'] },
+        { selector: 'marts' },
+    ])('validates data apps with model selection %j', async (selection) => {
+        const only = [common_1.ValidationTarget.CHARTS, common_1.ValidationTarget.APPS];
         await (0, validate_1.validateHandler)({
             ...baseOptions,
-            only: [common_1.ValidationTarget.CHARTS, common_1.ValidationTarget.APPS],
-            select: ['orders'],
+            ...selection,
+            only,
         });
-        const validationRequest = vi
-            .mocked(apiClient_1.lightdashApi)
-            .mock.calls.find(([request]) => request.method === 'POST' &&
-            request.url.endsWith('/validate'));
-        expect(validationRequest).toBeDefined();
-        expect(JSON.parse(String(validationRequest[0].body))).toEqual(expect.objectContaining({
-            validationTargets: [common_1.ValidationTarget.CHARTS],
+        expect(compile_1.compile).toHaveBeenCalledTimes(1);
+        expect(compile_1.compile).toHaveBeenCalledWith(expect.objectContaining(selection));
+        expect(apiClient_1.lightdashApi).toHaveBeenCalledWith(expect.objectContaining({
+            method: 'POST',
+            url: `/api/v1/projects/${baseOptions.project}/validate`,
+            body: JSON.stringify({ explores: [], validationTargets: only }),
         }));
-        expect(errorOutput.some((line) => line.includes('Skipping data app validation'))).toBe(true);
     });
-    test('rejects apps-only validation against a partial semantic layer', async () => {
-        await expect((0, validate_1.validateHandler)({
+    test.each([
+        { only: [] },
+        { only: Object.values(common_1.ValidationTarget) },
+        { only: [common_1.ValidationTarget.APPS] },
+    ])('preserves validation targets %j with model selection', async ({ only }) => {
+        await (0, validate_1.validateHandler)({
             ...baseOptions,
-            only: [common_1.ValidationTarget.APPS],
+            only,
             select: ['orders'],
-        })).rejects.toThrow('Data app validation requires a full project compile');
-        expect(compile_1.compile).not.toHaveBeenCalled();
+        });
+        expect(apiClient_1.lightdashApi).toHaveBeenCalledWith(expect.objectContaining({
+            method: 'POST',
+            url: `/api/v1/projects/${baseOptions.project}/validate`,
+            body: JSON.stringify({
+                explores: [],
+                validationTargets: only,
+            }),
+        }));
     });
-    test('prints the latest data app version author and date', async () => {
+    test('fails selected apps-only validation and prints the latest version author and date', async () => {
         vi.spyOn(process, 'exit').mockImplementation(() => undefined);
         vi.mocked(apiClient_1.lightdashApi).mockImplementation(async ({ method, url }) => {
             if (method === 'POST' && url.endsWith('/validate')) {
@@ -223,8 +238,11 @@ describe('validateHandler warehouse column validation', () => {
         await (0, validate_1.validateHandler)({
             ...baseOptions,
             only: [common_1.ValidationTarget.APPS],
+            select: ['orders'],
         });
+        expect(process.exit).toHaveBeenCalledWith(1);
         const output = errorOutput.join('\n');
+        expect(output).toContain('Dimension does not exist');
         expect(output).toContain('Ada Lovelace');
         expect(output).toContain('2026-08-06');
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightdash/cli",
-  "version": "2.198.5",
+  "version": "2.199.0",
   "description": "Lightdash CLI tool",
   "license": "MIT",
   "repository": {
@@ -22,8 +22,8 @@
     "@actions/core": "1.11.1",
     "@aws-sdk/credential-providers": "3.995.0",
     "@casl/ability": "6.8.0",
-    "@lightdash/common": "2.198.5",
-    "@lightdash/warehouses": "2.198.5",
+    "@lightdash/common": "2.199.0",
+    "@lightdash/warehouses": "2.199.0",
     "@sidvind/better-ajv-errors": "7.0.0",
     "@types/columnify": "1.5.1",
     "ajv": "8.18.0",