@lightdash/cli 0.3466.0 → 0.3468.0

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

@@ -1,6 +1,7 @@
 import { type DataAppCodeDownload } from '@lightdash/common';
 export declare const DEFAULT_APPS_LIMIT = 50;
 export declare const getDataAppUuidFromReference: (reference: string) => string;
+export declare const getDataAppUploadFilter: (references: string[], includeApps: boolean) => Set<string> | null;
 /**
  * Resolves the --apps-limit flag. Commander passes the raw string (or
  * undefined when the flag was not given, so an explicit flag is

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

@@ -1 +1 @@
-{"version":3,"file":"appsDownload.d.ts","sourceRoot":"","sources":["../../../src/handlers/apps/appsDownload.ts"],"names":[],"mappings":"AAAA,OAAO,EAIH,KAAK,mBAAmB,EAC3B,MAAM,mBAAmB,CAAC;AAG3B,eAAO,MAAM,kBAAkB,KAAK,CAAC;AAErC,eAAO,MAAM,2BAA2B,GAAI,WAAW,MAAM,KAAG,MAoB/D,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,gBAAgB,GACzB,UAAU,MAAM,GAAG,SAAS,EAC5B,aAAa,OAAO,KACrB;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,eAAe,EAAE,MAAM,GAAG,IAAI,CAAA;CAejD,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,kCAAkC,GAAI,KAAK,OAAO,KAAG,OACP,CAAC;AAE5D,MAAM,MAAM,qBAAqB,GAC3B;IAAE,IAAI,EAAE,MAAM,CAAA;CAAE,GAChB;IAAE,IAAI,EAAE,UAAU,CAAC;IAAC,QAAQ,EAAE,MAAM,EAAE,CAAA;CAAE,GACxC;IAAE,IAAI,EAAE,UAAU,CAAC;IAAC,aAAa,EAAE,MAAM,EAAE,CAAA;CAAE,CAAC;AAEpD;;;;GAIG;AACH,eAAO,MAAM,oBAAoB,GAAI,SAAS;IAC1C,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB,KAAG,qBASH,CAAC;AAEF,eAAO,MAAM,aAAa,GACtB,aAAa,MAAM,EAAE,EACrB,OAAO,MAAM,KACd;IAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;IAAC,cAAc,EAAE,MAAM,CAAA;CAG7C,CAAC;AAEH;;GAEG;AACH,eAAO,MAAM,0BAA0B,GACnC,SAAS,MAAM,EACf,MAAM,mBAAmB,KAC1B,mBAOF,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAAG;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AAEtE,MAAM,MAAM,uBAAuB,GAC7B;IAAE,IAAI,EAAE,gBAAgB,CAAA;CAAE,GAC1B;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AAExC;;;;;GAKG;AACH,eAAO,MAAM,wBAAwB,GACjC,KAAK,OAAO,KACb,uBAoBF,CAAC;AAEF;;;GAGG;AACH,eAAO,MAAM,oBAAoB,GAAI,MAAM;IACvC,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;CACvB,KAAG,MAC0O,CAAC;AAE/O;;;;GAIG;AACH,eAAO,MAAM,oBAAoB,GAAI,SAAS,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,KAAG,MAGpB,CAAC;AAEpD;;;GAGG;AACH,eAAO,MAAM,oBAAoB,GAC7B,SAAS,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,KAChC,OAKF,CAAC;AAEF;;;;;;;GAOG;AACH,eAAO,MAAM,iBAAiB,GAC1B,qBAAqB,MAAM,EAC3B,mBAAmB,MAAM,EACzB,WAAW,OAAO,KACnB,SAAS,GAAG,oBAKd,CAAC;AAEF,eAAO,MAAM,mBAAmB,GAC5B,cAAc,MAAM,EACpB,OAAO,MAAM,EACb,UAAU,kBAAkB,EAAE,EAC9B,SAAS,MAAM,EACf,sBAAsB,MAAM,KAC7B;IAAE,EAAE,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAA;CAiBxD,CAAC"}
+{"version":3,"file":"appsDownload.d.ts","sourceRoot":"","sources":["../../../src/handlers/apps/appsDownload.ts"],"names":[],"mappings":"AAAA,OAAO,EAIH,KAAK,mBAAmB,EAC3B,MAAM,mBAAmB,CAAC;AAG3B,eAAO,MAAM,kBAAkB,KAAK,CAAC;AAErC,eAAO,MAAM,2BAA2B,GAAI,WAAW,MAAM,KAAG,MAoB/D,CAAC;AAEF,eAAO,MAAM,sBAAsB,GAC/B,YAAY,MAAM,EAAE,EACpB,aAAa,OAAO,KACrB,GAAG,CAAC,MAAM,CAAC,GAAG,IAC4D,CAAC;AAE9E;;;;;GAKG;AACH,eAAO,MAAM,gBAAgB,GACzB,UAAU,MAAM,GAAG,SAAS,EAC5B,aAAa,OAAO,KACrB;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,eAAe,EAAE,MAAM,GAAG,IAAI,CAAA;CAejD,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,kCAAkC,GAAI,KAAK,OAAO,KAAG,OACP,CAAC;AAE5D,MAAM,MAAM,qBAAqB,GAC3B;IAAE,IAAI,EAAE,MAAM,CAAA;CAAE,GAChB;IAAE,IAAI,EAAE,UAAU,CAAC;IAAC,QAAQ,EAAE,MAAM,EAAE,CAAA;CAAE,GACxC;IAAE,IAAI,EAAE,UAAU,CAAC;IAAC,aAAa,EAAE,MAAM,EAAE,CAAA;CAAE,CAAC;AAEpD;;;;GAIG;AACH,eAAO,MAAM,oBAAoB,GAAI,SAAS;IAC1C,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB,KAAG,qBASH,CAAC;AAEF,eAAO,MAAM,aAAa,GACtB,aAAa,MAAM,EAAE,EACrB,OAAO,MAAM,KACd;IAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;IAAC,cAAc,EAAE,MAAM,CAAA;CAG7C,CAAC;AAEH;;GAEG;AACH,eAAO,MAAM,0BAA0B,GACnC,SAAS,MAAM,EACf,MAAM,mBAAmB,KAC1B,mBAOF,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAAG;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AAEtE,MAAM,MAAM,uBAAuB,GAC7B;IAAE,IAAI,EAAE,gBAAgB,CAAA;CAAE,GAC1B;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AAExC;;;;;GAKG;AACH,eAAO,MAAM,wBAAwB,GACjC,KAAK,OAAO,KACb,uBAoBF,CAAC;AAEF;;;GAGG;AACH,eAAO,MAAM,oBAAoB,GAAI,MAAM;IACvC,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;CACvB,KAAG,MAC0O,CAAC;AAE/O;;;;GAIG;AACH,eAAO,MAAM,oBAAoB,GAAI,SAAS,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,KAAG,MAGpB,CAAC;AAEpD;;;GAGG;AACH,eAAO,MAAM,oBAAoB,GAC7B,SAAS,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,KAChC,OAKF,CAAC;AAEF;;;;;;;GAOG;AACH,eAAO,MAAM,iBAAiB,GAC1B,qBAAqB,MAAM,EAC3B,mBAAmB,MAAM,EACzB,WAAW,OAAO,KACnB,SAAS,GAAG,oBAKd,CAAC;AAEF,eAAO,MAAM,mBAAmB,GAC5B,cAAc,MAAM,EACpB,OAAO,MAAM,EACb,UAAU,kBAAkB,EAAE,EAC9B,SAAS,MAAM,EACf,sBAAsB,MAAM,KAC7B;IAAE,EAAE,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAA;CAiBxD,CAAC"}

package/dist/handlers/apps/appsDownload.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.appsDownloadSummary = exports.classifyAppUpload = exports.shouldWarnAllSkipped = exports.computeUpsertedTotal = exports.manifestRetargetHint = exports.classifyAppDownloadError = exports.ensureDownloadedAppContext = exports.capListedApps = exports.selectAppsToDownload = exports.shouldFallBackToSpaceScopedListing = exports.resolveAppsLimit = exports.getDataAppUuidFromReference = exports.DEFAULT_APPS_LIMIT = void 0;
+exports.appsDownloadSummary = exports.classifyAppUpload = exports.shouldWarnAllSkipped = exports.computeUpsertedTotal = exports.manifestRetargetHint = exports.classifyAppDownloadError = exports.ensureDownloadedAppContext = exports.capListedApps = exports.selectAppsToDownload = exports.shouldFallBackToSpaceScopedListing = exports.resolveAppsLimit = exports.getDataAppUploadFilter = exports.getDataAppUuidFromReference = exports.DEFAULT_APPS_LIMIT = void 0;
 const common_1 = require("@lightdash/common");
 const uuid_1 = require("uuid");
 exports.DEFAULT_APPS_LIMIT = 50;
@@ -21,6 +21,8 @@ const getDataAppUuidFromReference = (reference) => {
     return appSegmentIndex >= 0 ? pathSegments[appSegmentIndex + 1] : reference;
 };
 exports.getDataAppUuidFromReference = getDataAppUuidFromReference;
+const getDataAppUploadFilter = (references, includeApps) => includeApps ? null : new Set(references.map(exports.getDataAppUuidFromReference));
+exports.getDataAppUploadFilter = getDataAppUploadFilter;
 /**
  * Resolves the --apps-limit flag. Commander passes the raw string (or
  * undefined when the flag was not given, so an explicit flag is
@@ -38,7 +40,7 @@ const resolveAppsLimit = (rawLimit, includeApps) => {
         limit: parseInt(rawLimit, 10),
         noEffectWarning: includeApps
             ? null
-            : '--apps-limit only applies to --include-apps; explicit --apps UUIDs are never capped.',
+            : '--apps-limit only applies to --include-apps; explicit --apps references are never capped.',
     };
 };
 exports.resolveAppsLimit = resolveAppsLimit;

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

@@ -25,6 +25,17 @@ describe('getDataAppUuidFromReference', () => {
         expect((0, appsDownload_1.getDataAppUuidFromReference)(reference)).toBe(reference);
     });
 });
+describe('getDataAppUploadFilter', () => {
+    const appUuid = 'd3afc44c-6f0f-4d9f-a267-fb739efa31dd';
+    it('matches app manifests by UUID when a URL is passed', () => {
+        expect((0, appsDownload_1.getDataAppUploadFilter)([
+            `https://app.lightdash.cloud/projects/project-uuid/apps/${appUuid}/view`,
+        ], false)).toEqual(new Set([appUuid]));
+    });
+    it('does not filter app folders when --include-apps is passed', () => {
+        expect((0, appsDownload_1.getDataAppUploadFilter)([appUuid], true)).toBeNull();
+    });
+});
 const makeDownload = () => ({
     manifest: {
         codeVersion: 1,

package/dist/handlers/compile.d.ts

@@ -12,6 +12,8 @@ export type CompileHandlerOptions = DbtCompileOptions & {
     disableTimestampConversion?: boolean;
     validateWarehouseColumns?: boolean;
 };
+export declare const hasBlockingCompileError: (explore: Explore | ExploreError) => boolean;
+export declare const stripWarehouseColumnErrors: (explore: Explore | ExploreError) => Explore | ExploreError;
 export declare const compile: (options: CompileHandlerOptions) => Promise<(Explore | ExploreError)[]>;
 export declare const compileHandler: (originalOptions: CompileHandlerOptions) => Promise<void>;
 //# sourceMappingURL=compile.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"compile.d.ts","sourceRoot":"","sources":["../../src/handlers/compile.ts"],"names":[],"mappings":"AAAA,OAAO,EAMH,OAAO,EACP,YAAY,EAgBf,MAAM,mBAAmB,CAAC;AAoB3B,OAAO,EAAE,iBAAiB,EAA8B,MAAM,eAAe,CAAC;AAI9E,MAAM,MAAM,qBAAqB,GAAG,iBAAiB,GAAG;IACpD,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,0BAA0B,CAAC,EAAE,OAAO,CAAC;IACrC,wBAAwB,CAAC,EAAE,OAAO,CAAC;CACtC,CAAC;AAwNF,eAAO,MAAM,OAAO,GAAU,SAAS,qBAAqB,wCAgU3D,CAAC;AAEF,eAAO,MAAM,cAAc,GACvB,iBAAiB,qBAAqB,kBAmCzC,CAAC"}
+{"version":3,"file":"compile.d.ts","sourceRoot":"","sources":["../../src/handlers/compile.ts"],"names":[],"mappings":"AAAA,OAAO,EAMH,OAAO,EACP,YAAY,EAgBf,MAAM,mBAAmB,CAAC;AAoB3B,OAAO,EAAE,iBAAiB,EAA8B,MAAM,eAAe,CAAC;AAI9E,MAAM,MAAM,qBAAqB,GAAG,iBAAiB,GAAG;IACpD,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,0BAA0B,CAAC,EAAE,OAAO,CAAC;IACrC,wBAAwB,CAAC,EAAE,OAAO,CAAC;CACtC,CAAC;AAEF,eAAO,MAAM,uBAAuB,GAChC,SAAS,OAAO,GAAG,YAAY,KAChC,OAIW,CAAC;AAEf,eAAO,MAAM,0BAA0B,GACnC,SAAS,OAAO,GAAG,YAAY,KAChC,OAAO,GAAG,YAiBZ,CAAC;AAwNF,eAAO,MAAM,OAAO,GAAU,SAAS,qBAAqB,wCAgU3D,CAAC;AAEF,eAAO,MAAM,cAAc,GACvB,iBAAiB,qBAAqB,kBAmCzC,CAAC"}

package/dist/handlers/compile.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.compileHandler = exports.compile = void 0;
+exports.compileHandler = exports.compile = exports.stripWarehouseColumnErrors = exports.hasBlockingCompileError = void 0;
 const tslib_1 = require("tslib");
 const common_1 = require("@lightdash/common");
 const warehouses_1 = require("@lightdash/warehouses");
@@ -18,6 +18,23 @@ const styles = tslib_1.__importStar(require("../styles"));
 const compile_1 = require("./dbt/compile");
 const getDbtVersion_1 = require("./dbt/getDbtVersion");
 const getWarehouseClient_1 = tslib_1.__importDefault(require("./dbt/getWarehouseClient"));
+const hasBlockingCompileError = (explore) => (0, common_1.isExploreError)(explore) ||
+    explore.warnings?.some((warning) => warning.type === common_1.InlineErrorType.WAREHOUSE_COLUMN_ERROR) === true;
+exports.hasBlockingCompileError = hasBlockingCompileError;
+const stripWarehouseColumnErrors = (explore) => {
+    if ((0, common_1.isExploreError)(explore) || explore.warnings === undefined) {
+        return explore;
+    }
+    const { warnings: currentWarnings, ...exploreWithoutWarnings } = explore;
+    const warnings = currentWarnings.filter((warning) => warning.type !== common_1.InlineErrorType.WAREHOUSE_COLUMN_ERROR);
+    if (warnings.length === currentWarnings.length) {
+        return explore;
+    }
+    return warnings.length > 0
+        ? { ...exploreWithoutWarnings, warnings }
+        : exploreWithoutWarnings;
+};
+exports.stripWarehouseColumnErrors = stripWarehouseColumnErrors;
 const getDisplayableDiagnostics = (explore) => {
     const diagnostics = explore.warnings ?? [];
     const errors = diagnostics.filter((diagnostic) => diagnostic.type === common_1.InlineErrorType.WAREHOUSE_COLUMN_ERROR);
@@ -253,7 +270,7 @@ const compile = async (options) => {
         else {
             globalState_1.default.debug('> Skipping warehouse catalog');
             if (options.validateWarehouseColumns === true) {
-                console.error(styles.warning('> Skipping warehouse column validation because --skip-warehouse-catalog was supplied'));
+                console.error(styles.warning('> Skipping warehouse column validation because the warehouse catalog is skipped'));
             }
         }
         const validModelsWithTypes = applyMetricFlowMetrics((0, common_1.attachTypesToModels)(validModels, catalog, false, 
@@ -373,7 +390,7 @@ const compileHandler = async (originalOptions) => {
     options.skipWarehouseCatalog = projectTypeConfig.skipWarehouseCatalog;
     globalState_1.default.setVerbose(options.verbose);
     const explores = await (0, exports.compile)(options);
-    const errorsCount = explores.filter((e) => (0, common_1.isExploreError)(e)).length;
+    const errorsCount = explores.filter(exports.hasBlockingCompileError).length;
     console.error('');
     if (errorsCount > 0) {
         console.error(styles.error(`Failed to compile project. Found ${errorsCount} error${errorsCount > 1 ? 's' : ''}`));

package/dist/handlers/compileWarehouseColumnValidation.test.js

@@ -11,9 +11,11 @@ const manifest_1 = require("../dbt/manifest");
 const validation_1 = require("../dbt/validation");
 const loader_1 = require("../lightdash/loader");
 const compile_1 = require("./compile");
+const apiClient_1 = require("./dbt/apiClient");
 const compile_2 = require("./dbt/compile");
 const getDbtVersion_1 = require("./dbt/getDbtVersion");
 const getWarehouseClient_1 = tslib_1.__importDefault(require("./dbt/getWarehouseClient"));
+const deploy_1 = require("./deploy");
 vi.mock('../analytics/analytics');
 vi.mock('../config', () => ({
     getConfig: vi.fn().mockResolvedValue({ user: null, context: null }),
@@ -23,6 +25,10 @@ vi.mock('../dbt/manifest');
 vi.mock('../dbt/validation');
 vi.mock('../lightdash/loader');
 vi.mock('./dbt/compile');
+vi.mock('./dbt/apiClient', async (importOriginal) => {
+    const original = await importOriginal();
+    return { ...original, lightdashApi: vi.fn() };
+});
 vi.mock('./dbt/getDbtVersion');
 vi.mock('./dbt/getWarehouseClient');
 vi.mock('@lightdash/warehouses', async (importOriginal) => {
@@ -116,7 +122,7 @@ const baseOptions = (projectDir) => ({
     warehouseCredentials: false,
     disableTimestampConversion: false,
 });
-const SKIP_WARNING = 'Skipping warehouse column validation because --skip-warehouse-catalog was supplied';
+const SKIP_WARNING = 'Skipping warehouse column validation because the warehouse catalog is skipped';
 const YML_SKIP_WARNING = 'Skipping warehouse column validation because it is not supported for Lightdash YAML projects';
 const DBT_CLOUD_SKIP_WARNING = 'Skipping warehouse column validation because dbt Cloud CLI cannot run warehouse queries';
 const setupWarehouseClient = () => {
@@ -268,6 +274,49 @@ describe('compile warehouse column validation', () => {
             expect(warningTypes).toEqual([
                 common_1.InlineErrorType.WAREHOUSE_COLUMN_ERROR,
             ]);
+            expect(result.filter(compile_1.hasBlockingCompileError)).toHaveLength(1);
+            expect(result
+                .map(compile_1.stripWarehouseColumnErrors)
+                .flatMap((explore) => (0, common_1.isExploreError)(explore) ? [] : (explore.warnings ?? []))).toEqual([]);
+        });
+        test('blocks deploy when the warehouse rejects a column', async () => {
+            setupWarehouseClient();
+            mockProberToAppendWarning(WAREHOUSE_WARNING);
+            const explores = await (0, compile_1.compile)({
+                ...baseOptions(tempDir),
+                validateWarehouseColumns: true,
+            });
+            vi.spyOn(process, 'exit').mockImplementation(() => {
+                throw new Error('process.exit');
+            });
+            await expect((0, deploy_1.deploy)(explores, {
+                ...baseOptions(tempDir),
+                projectUuid: 'projectUuid',
+                ignoreErrors: false,
+                validateWarehouseColumns: true,
+            })).rejects.toThrow('process.exit');
+            expect(process.exit).toHaveBeenCalledWith(1);
+        });
+        test('does not persist warehouse diagnostics when deploy errors are ignored', async () => {
+            setupWarehouseClient();
+            mockProberToAppendWarning(WAREHOUSE_WARNING);
+            const explores = await (0, compile_1.compile)({
+                ...baseOptions(tempDir),
+                validateWarehouseColumns: true,
+            });
+            await (0, deploy_1.deploy)(explores, {
+                ...baseOptions(tempDir),
+                projectUuid: 'projectUuid',
+                ignoreErrors: true,
+                validateWarehouseColumns: true,
+            });
+            const deployRequest = vi
+                .mocked(apiClient_1.lightdashApi)
+                .mock.calls.find(([request]) => request.url === '/api/v1/projects/projectUuid/explores');
+            if (deployRequest === undefined) {
+                throw new Error('Expected explores deploy request');
+            }
+            expect(deployRequest[0].body).not.toContain(common_1.InlineErrorType.WAREHOUSE_COLUMN_ERROR);
         });
         test('keeps generic warnings hidden when partial compilation is disabled', async () => {
             vi.stubEnv('PARTIAL_COMPILATION_ENABLED', 'false');
@@ -317,7 +366,7 @@ describe('compile warehouse column validation', () => {
             });
             setupWarehouseClient();
             mockProberToAppendWarning(WAREHOUSE_WARNING);
-            await (0, compile_1.compile)({
+            const result = await (0, compile_1.compile)({
                 ...baseOptions(tempDir),
                 validateWarehouseColumns: true,
             });
@@ -326,6 +375,11 @@ describe('compile warehouse column validation', () => {
                 expect.stringContaining('Skipped metric "id"'),
                 expect.stringContaining('Compiled 1 explores, SUCCESS=0 ERRORS=1'),
             ]));
+            expect(result
+                .map(compile_1.stripWarehouseColumnErrors)
+                .flatMap((explore) => (0, common_1.isExploreError)(explore)
+                ? []
+                : (explore.warnings ?? []).map((warning) => warning.type))).toEqual([common_1.InlineErrorType.DUPLICATE_FIELD_NAME]);
         });
         test('renders only the warehouse error when partial compilation is disabled and generic warnings exist', async () => {
             vi.stubEnv('PARTIAL_COMPILATION_ENABLED', 'false');

package/dist/handlers/deploy.d.ts

@@ -18,6 +18,7 @@ type DeployHandlerOptions = DbtCompileOptions & {
     parallelBatches?: string;
     gzip?: boolean;
     disableTimestampConversion?: boolean;
+    validateWarehouseColumns: boolean;
 };
 type DeployArgs = DeployHandlerOptions & {
     projectUuid: string;

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

@@ -1 +1 @@
-{"version":3,"file":"deploy.d.ts","sourceRoot":"","sources":["../../src/handlers/deploy.ts"],"names":[],"mappings":"AAAA,OAAO,EACH,wBAAwB,EAGxB,OAAO,EACP,YAAY,EAUf,MAAM,mBAAmB,CAAC;AAuB3B,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAC;AASlD,KAAK,oBAAoB,GAAG,iBAAiB,GAAG;IAC5C,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,OAAO,GAAG,MAAM,CAAC;IAC1B,OAAO,EAAE,OAAO,CAAC;IACjB,YAAY,EAAE,OAAO,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,uBAAuB,CAAC,EAAE,MAAM,CAAC;IACjC,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,0BAA0B,CAAC,EAAE,OAAO,CAAC;CACxC,CAAC;AAEF,KAAK,UAAU,GAAG,oBAAoB,GAAG;IACrC,WAAW,EAAE,MAAM,CAAC;CACvB,CAAC;AAEF,eAAO,MAAM,iBAAiB,GAC1B,gBAAgB,wBAAwB,CAAC,UAAU,CAAC,GAAG,SAAS,SA0BnE,CAAC;AAiPF,eAAO,MAAM,MAAM,GACf,UAAU,CAAC,OAAO,GAAG,YAAY,CAAC,EAAE,EACpC,SAAS,UAAU,KACpB,OAAO,CAAC,IAAI,CA2Id,CAAC;AAiGF,eAAO,MAAM,aAAa,GAAU,iBAAiB,oBAAoB,kBAmKxE,CAAC"}
+{"version":3,"file":"deploy.d.ts","sourceRoot":"","sources":["../../src/handlers/deploy.ts"],"names":[],"mappings":"AAAA,OAAO,EACH,wBAAwB,EAGxB,OAAO,EACP,YAAY,EASf,MAAM,mBAAmB,CAAC;AA2B3B,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAC;AASlD,KAAK,oBAAoB,GAAG,iBAAiB,GAAG;IAC5C,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,OAAO,GAAG,MAAM,CAAC;IAC1B,OAAO,EAAE,OAAO,CAAC;IACjB,YAAY,EAAE,OAAO,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,uBAAuB,CAAC,EAAE,MAAM,CAAC;IACjC,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,0BAA0B,CAAC,EAAE,OAAO,CAAC;IACrC,wBAAwB,EAAE,OAAO,CAAC;CACrC,CAAC;AAEF,KAAK,UAAU,GAAG,oBAAoB,GAAG;IACrC,WAAW,EAAE,MAAM,CAAC;CACvB,CAAC;AAEF,eAAO,MAAM,iBAAiB,GAC1B,gBAAgB,wBAAwB,CAAC,UAAU,CAAC,GAAG,SAAS,SA0BnE,CAAC;AAiPF,eAAO,MAAM,MAAM,GACf,UAAU,CAAC,OAAO,GAAG,YAAY,CAAC,EAAE,EACpC,SAAS,UAAU,KACpB,OAAO,CAAC,IAAI,CA6Id,CAAC;AAiGF,eAAO,MAAM,aAAa,GAAU,iBAAiB,oBAAoB,kBAmKxE,CAAC"}

package/dist/handlers/deploy.js

@@ -191,7 +191,7 @@ const deploy = async (explores, options) => {
         globalState_1.default.log(styles.warning('No explores found'));
         process.exit(1);
     }
-    const errors = explores.filter((e) => (0, common_1.isExploreError)(e)).length;
+    const errors = explores.filter(compile_1.hasBlockingCompileError).length;
     if (errors > 0) {
         if (options.ignoreErrors) {
             console.error(styles.warning(`\nDeploying project with ${errors} errors\n`));
@@ -201,6 +201,7 @@ const deploy = async (explores, options) => {
             process.exit(1);
         }
     }
+    const deployableExplores = explores.map(compile_1.stripWarehouseColumnErrors);
     const lightdashProjectConfig = await (0, lightdash_config_1.readAndLoadLightdashProjectConfig)(path_1.default.resolve(options.projectDir), options.projectUuid);
     // These two methods are not critical to the deployment process, so we can ignore errors and show warnings instead
     try {
@@ -229,11 +230,11 @@ const deploy = async (explores, options) => {
     }
     // Use batched deploy if enabled
     if (options.useBatchedDeploy) {
-        await deployBatched(explores, options);
+        await deployBatched(deployableExplores, options);
     }
     else {
         const deployStartTime = Date.now();
-        const deployPayload = JSON.stringify(explores);
+        const deployPayload = JSON.stringify(deployableExplores);
         try {
             const deployResponse = await (0, apiClient_1.lightdashApi)({
                 method: 'PUT',

package/dist/handlers/download.d.ts

@@ -62,6 +62,7 @@ export declare const testHelpers: {
     getDashboardChartSlugs: (dashboardSlugs: string[], customPath?: string, looseDashboards?: (DashboardAsCode & {
         needsUpdating: boolean;
     })[]) => Promise<string[]>;
+    hasUploadFilters: ({ spacesOnly, charts, dashboards, agents, alerts, googleSheets, scheduledDeliveries, virtualViews, apps, }: Pick<DownloadHandlerOptions, "spacesOnly" | "charts" | "dashboards" | "agents" | "alerts" | "googleSheets" | "scheduledDeliveries" | "virtualViews" | "apps">) => boolean;
     readAiAgentFiles: (customPath?: string) => Promise<AgentAsCode[]>;
     readSpaceFiles: (customPath?: string) => Promise<SpaceCodeFile[]>;
     readSpaceNames: (customPath?: string) => Promise<Record<string, string>>;

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

@@ -1 +1 @@
-{"version":3,"file":"download.d.ts","sourceRoot":"","sources":["../../src/handlers/download.ts"],"names":[],"mappings":"AAEA,OAAO,EACH,WAAW,EAuBX,WAAW,EAGX,eAAe,EAef,KAAK,WAAW,EACnB,MAAM,mBAAmB,CAAC;AAY3B,OAAO,EAIH,KAAK,0BAA0B,EAClC,MAAM,iCAAiC,CAAC;AA8DzC,OAAO,EAmBH,KAAK,aAAa,EACrB,MAAM,gBAAgB,CAAC;AAExB,MAAM,MAAM,sBAAsB,GAAG;IACjC,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,KAAK,EAAE,OAAO,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,OAAO,CAAC;IACrB,eAAe,EAAE,OAAO,CAAC;IACzB,MAAM,EAAE,OAAO,CAAC;IAChB,aAAa,EAAE,OAAO,CAAC;IACvB,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,EAAE,OAAO,CAAC;IACpB,cAAc,EAAE,OAAO,CAAC;IACxB,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;IACpB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,uBAAuB,EAAE,OAAO,CAAC;IACjC,gBAAgB,EAAE,OAAO,CAAC;IAC1B,aAAa,EAAE,OAAO,CAAC;IACvB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,0BAA0B,EAAE,OAAO,CAAC;IACpC,mBAAmB,EAAE,OAAO,CAAC;IAC7B,UAAU,EAAE,OAAO,CAAC;IACpB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,YAAY,EAAE,OAAO,CAAC;IACtB,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB,CAAC;AA8FF,KAAK,aAAa,GAAG;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,QAAQ,GAAG,YAAY,CAAC;IAC9B,YAAY,EAAE,MAAM,CAAC;CACxB,CAAC;AA6XF,KAAK,mBAAmB,GAAG,QAAQ,GAAG,YAAY,GAAG,WAAW,CAAC;AAkDjE,eAAO,MAAM,eAAe,GACxB,KAAK,MAAM,EAAE,EACb,MAAM,mBAAmB,EACzB,WAAW,MAAM,EACjB,aAAa,MAAM,EACnB,aAAa,MAAM,EACnB,cAAa,OAAe,EAC5B,SAAQ,OAAe,EACvB,aAAY,OAAe,EAC3B,mBAAkB,OAAe,EACjC,aAAY,OAAe,EAC3B,aAAa,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,KACtC,OAAO,CAAC,CAAC,MAAM,EAAE,MAAM,EAAE,EAAE,aAAa,EAAE,EAAE,WAAW,EAAE,CAAC,CAsI5D,CAAC;AA2cF,eAAO,MAAM,eAAe,GACxB,SAAS,sBAAsB,KAChC,OAAO,CAAC,IAAI,CA+nBd,CAAC;AAudF,eAAO,MAAM,aAAa,GACtB,SAAS,sBAAsB,KAChC,OAAO,CAAC,IAAI,CA8tBd,CAAC;AAEF,eAAO,MAAM,WAAW;;;;6CAnwBJ,MAAM,EAAE,eACX,MAAM,oBACF,CAAC,eAAe,GAAG;QAAE,aAAa,EAAE,OAAO,CAAA;KAAE,CAAC,EAAE;oCAryCpD,MAAM,KACpB,OAAO,CAAC,WAAW,EAAE,CAAC;;;sCA9xBd,WAAW,oBACA,OAAO,KAC1B,WAAW;;+EA9FX,IAAI,CACH,sBAAsB,EACtB,YAAY,GAAG,eAAe,GAAG,QAAQ,GAAG,UAAU,CACzD,KAAG,OAAO;;qCAqxDC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,SACvB,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,KAC9B;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,0BAA0B,CAAA;KAAE;;oCAjkC5C,MAAM,SACV,MAAM,EAAE,WACN,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,SACxB,OAAO,aACH,OAAO,eACL,MAAM,KACpB,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;;;CAitEjC,CAAC"}
+{"version":3,"file":"download.d.ts","sourceRoot":"","sources":["../../src/handlers/download.ts"],"names":[],"mappings":"AAEA,OAAO,EACH,WAAW,EAuBX,WAAW,EAGX,eAAe,EAef,KAAK,WAAW,EACnB,MAAM,mBAAmB,CAAC;AAY3B,OAAO,EAIH,KAAK,0BAA0B,EAClC,MAAM,iCAAiC,CAAC;AA+DzC,OAAO,EAmBH,KAAK,aAAa,EACrB,MAAM,gBAAgB,CAAC;AAExB,MAAM,MAAM,sBAAsB,GAAG;IACjC,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,KAAK,EAAE,OAAO,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,OAAO,CAAC;IACrB,eAAe,EAAE,OAAO,CAAC;IACzB,MAAM,EAAE,OAAO,CAAC;IAChB,aAAa,EAAE,OAAO,CAAC;IACvB,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,EAAE,OAAO,CAAC;IACpB,cAAc,EAAE,OAAO,CAAC;IACxB,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;IACpB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,uBAAuB,EAAE,OAAO,CAAC;IACjC,gBAAgB,EAAE,OAAO,CAAC;IAC1B,aAAa,EAAE,OAAO,CAAC;IACvB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,0BAA0B,EAAE,OAAO,CAAC;IACpC,mBAAmB,EAAE,OAAO,CAAC;IAC7B,UAAU,EAAE,OAAO,CAAC;IACpB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,YAAY,EAAE,OAAO,CAAC;IACtB,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB,CAAC;AAgIF,KAAK,aAAa,GAAG;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,QAAQ,GAAG,YAAY,CAAC;IAC9B,YAAY,EAAE,MAAM,CAAC;CACxB,CAAC;AA6XF,KAAK,mBAAmB,GAAG,QAAQ,GAAG,YAAY,GAAG,WAAW,CAAC;AAkDjE,eAAO,MAAM,eAAe,GACxB,KAAK,MAAM,EAAE,EACb,MAAM,mBAAmB,EACzB,WAAW,MAAM,EACjB,aAAa,MAAM,EACnB,aAAa,MAAM,EACnB,cAAa,OAAe,EAC5B,SAAQ,OAAe,EACvB,aAAY,OAAe,EAC3B,mBAAkB,OAAe,EACjC,aAAY,OAAe,EAC3B,aAAa,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,KACtC,OAAO,CAAC,CAAC,MAAM,EAAE,MAAM,EAAE,EAAE,aAAa,EAAE,EAAE,WAAW,EAAE,CAAC,CAsI5D,CAAC;AA2cF,eAAO,MAAM,eAAe,GACxB,SAAS,sBAAsB,KAChC,OAAO,CAAC,IAAI,CAunBd,CAAC;AAudF,eAAO,MAAM,aAAa,GACtB,SAAS,sBAAsB,KAChC,OAAO,CAAC,IAAI,CAguBd,CAAC;AAEF,eAAO,MAAM,WAAW;;;;6CArwBJ,MAAM,EAAE,eACX,MAAM,oBACF,CAAC,eAAe,GAAG;QAAE,aAAa,EAAE,OAAO,CAAA;KAAE,CAAC,EAAE;mIAvqElE,IAAI,CACH,sBAAsB,EACpB,YAAY,GACZ,QAAQ,GACR,YAAY,GACZ,QAAQ,GACR,QAAQ,GACR,cAAc,GACd,qBAAqB,GACrB,cAAc,GACd,MAAM,CACX,KAAG,OAAO;oCA+3BM,MAAM,KACpB,OAAO,CAAC,WAAW,EAAE,CAAC;;;sCA9xBd,WAAW,oBACA,OAAO,KAC1B,WAAW;;+EAhIX,IAAI,CACH,sBAAsB,EACtB,YAAY,GAAG,eAAe,GAAG,QAAQ,GAAG,UAAU,CACzD,KAAG,OAAO;;qCA+yDC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,SACvB,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,KAC9B;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,0BAA0B,CAAA;KAAE;;oCAzjC5C,MAAM,SACV,MAAM,EAAE,WACN,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,SACxB,OAAO,aACH,OAAO,eACL,MAAM,KACpB,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;;;CA4sEjC,CAAC"}

package/dist/handlers/download.js

@@ -30,6 +30,17 @@ const selectProject_1 = require("./selectProject");
 const spacesAsCode_1 = require("./spacesAsCode");
 const shouldDownloadAiAgents = ({ includeAll, includeAgents, agents, appsOnly, }) => appsOnly !== true &&
     (includeAll === true || includeAgents === true || agents.length > 0);
+const hasUploadFilters = ({ spacesOnly, charts, dashboards, agents, alerts, googleSheets, scheduledDeliveries, virtualViews, apps, }) => !spacesOnly &&
+    [
+        charts,
+        dashboards,
+        agents,
+        alerts,
+        googleSheets,
+        scheduledDeliveries,
+        virtualViews,
+        apps ?? [],
+    ].some((filters) => filters.length > 0);
 /*
     This function is used to parse the content filters.
     It can be slugs, uuids or urls
@@ -745,14 +756,7 @@ const downloadHandler = async (options) => {
     if (options.rootSpaces && options.nested) {
         throw new common_1.ParameterError('--root-spaces cannot be combined with --nested');
     }
-    const hasFilters = !options.spacesOnly &&
-        (options.charts.length > 0 ||
-            options.dashboards.length > 0 ||
-            options.agents.length > 0 ||
-            options.alerts.length > 0 ||
-            options.googleSheets.length > 0 ||
-            options.scheduledDeliveries.length > 0 ||
-            options.virtualViews.length > 0);
+    const hasFilters = hasUploadFilters(options);
     const shouldDownloadSpaces = !isOrganizationDownload && !options.skipSpaces && !hasFilters;
     let skipEmbeddedSpaces = !hasFilters || options.skipSpaces;
     if (shouldDownloadSpaces) {
@@ -1599,12 +1603,12 @@ const uploadHandler = async (options) => {
                 });
             }
         }
-        // Upload data apps (enterprise, opt-in via --apps <uuids...> or
+        // Upload data apps (enterprise, opt-in via --apps <references...> or
         // --include-apps, fire-and-forget)
-        const explicitAppUuids = Array.isArray(options.apps)
+        const explicitAppReferences = Array.isArray(options.apps)
             ? options.apps
             : [];
-        const shouldUploadApps = options.includeApps === true || explicitAppUuids.length > 0;
+        const shouldUploadApps = options.includeApps === true || explicitAppReferences.length > 0;
         let appsCreated = 0;
         let appsUpdated = 0;
         let appsFailed = 0;
@@ -1617,9 +1621,9 @@ const uploadHandler = async (options) => {
         }
         else if (shouldUploadApps) {
             output.startItem('Data apps');
-            // --include-apps uploads every folder on disk; explicit UUIDs
+            // --include-apps uploads every folder on disk; explicit references
             // filter folders by their manifest appUuid
-            const filterUuids = options.includeApps === true ? null : new Set(explicitAppUuids);
+            const filterUuids = (0, appsDownload_1.getDataAppUploadFilter)(explicitAppReferences, options.includeApps === true);
             const baseDir = (0, contentAsCodePaths_1.getDownloadFolder)(options.path);
             const appsDir = path.join(baseDir, 'apps');
             let appFolderEntries;
@@ -1856,6 +1860,7 @@ exports.testHelpers = {
     downloadSpaces: spacesAsCode_1.downloadSpaces,
     getFlatSpaceFileNames: spacesAsCode_1.getFlatSpaceFileNames,
     getDashboardChartSlugs,
+    hasUploadFilters,
     readAiAgentFiles,
     readSpaceFiles: spacesAsCode_1.readSpaceFiles,
     readSpaceNames: spacesAsCode_1.readSpaceNames,

package/dist/handlers/download.test.js

@@ -13,7 +13,33 @@ vitest_1.vi.mock('./dbt/apiClient', async (importOriginal) => ({
     ...(await importOriginal()),
     lightdashApi: vitest_1.vi.fn(),
 }));
-const { assertUniqueSpacePaths, downloadSpaces, getDashboardChartSlugs, getFlatSpaceFileNames, readAiAgentFiles, readSpaceFiles, readSpaceNames, sanitizeChartForDownload, shouldFallBackToEmbeddedSpaces, shouldDownloadAiAgents, summarizeUploadChanges, upsertSpaces, upsertVirtualViews, validateSpaceIdentity, writeSpaceFiles, } = download_1.testHelpers;
+const { assertUniqueSpacePaths, downloadSpaces, getDashboardChartSlugs, getFlatSpaceFileNames, hasUploadFilters, readAiAgentFiles, readSpaceFiles, readSpaceNames, sanitizeChartForDownload, shouldFallBackToEmbeddedSpaces, shouldDownloadAiAgents, summarizeUploadChanges, upsertSpaces, upsertVirtualViews, validateSpaceIdentity, writeSpaceFiles, } = download_1.testHelpers;
+const makeUploadFilterOptions = (overrides = {}) => ({
+    spacesOnly: false,
+    charts: [],
+    dashboards: [],
+    agents: [],
+    alerts: [],
+    googleSheets: [],
+    scheduledDeliveries: [],
+    virtualViews: [],
+    apps: [],
+    ...overrides,
+});
+describe('hasUploadFilters', () => {
+    it('treats explicit apps as a filtered upload', () => {
+        expect(hasUploadFilters(makeUploadFilterOptions({ apps: ['app-reference'] }))).toBe(true);
+    });
+    it('keeps an unfiltered upload unfiltered', () => {
+        expect(hasUploadFilters(makeUploadFilterOptions())).toBe(false);
+    });
+    it('does not treat content selections as filters in spaces-only mode', () => {
+        expect(hasUploadFilters(makeUploadFilterOptions({
+            spacesOnly: true,
+            apps: ['app-reference'],
+        }))).toBe(false);
+    });
+});
 const makeLooseDashboard = (slug, chartSlugs) => ({
     slug,
     name: slug,

package/dist/handlers/preview.d.ts

@@ -19,6 +19,7 @@ type PreviewHandlerOptions = DbtCompileOptions & {
     parallelBatches?: string;
     expiresIn?: string;
     disableTimestampConversion?: boolean;
+    validateWarehouseColumns: boolean;
 };
 type StopPreviewHandlerOptions = {
     name: string;

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

@@ -1 +1 @@
-{"version":3,"file":"preview.d.ts","sourceRoot":"","sources":["../../src/handlers/preview.ts"],"names":[],"mappings":"AACA,OAAO,EACH,+BAA+B,EAKlC,MAAM,mBAAmB,CAAC;AAgB3B,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAC;AAOlD,KAAK,qBAAqB,GAAG,iBAAiB,GAAG;IAC7C,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,OAAO,CAAC;IACtB,kBAAkB,EAAE,+BAA+B,CAAC;IACpD,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,uBAAuB,CAAC,EAAE,MAAM,CAAC;IACjC,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,0BAA0B,CAAC,EAAE,OAAO,CAAC;CACxC,CAAC;AAEF,KAAK,yBAAyB,GAAG;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;CACpB,CAAC;AA2GF,eAAO,MAAM,cAAc,GACvB,iBAAiB,qBAAqB,KACvC,OAAO,CAAC,IAAI,CAmSd,CAAC;AAEF,eAAO,MAAM,mBAAmB,GAC5B,iBAAiB,qBAAqB,KACvC,OAAO,CAAC,IAAI,CA8Kd,CAAC;AAEF,eAAO,MAAM,kBAAkB,GAC3B,SAAS,yBAAyB,KACnC,OAAO,CAAC,IAAI,CA0Cd,CAAC"}
+{"version":3,"file":"preview.d.ts","sourceRoot":"","sources":["../../src/handlers/preview.ts"],"names":[],"mappings":"AACA,OAAO,EACH,+BAA+B,EAKlC,MAAM,mBAAmB,CAAC;AAgB3B,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAC;AAOlD,KAAK,qBAAqB,GAAG,iBAAiB,GAAG;IAC7C,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,OAAO,CAAC;IACtB,kBAAkB,EAAE,+BAA+B,CAAC;IACpD,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,uBAAuB,CAAC,EAAE,MAAM,CAAC;IACjC,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,0BAA0B,CAAC,EAAE,OAAO,CAAC;IACrC,wBAAwB,EAAE,OAAO,CAAC;CACrC,CAAC;AAEF,KAAK,yBAAyB,GAAG;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;CACpB,CAAC;AA2GF,eAAO,MAAM,cAAc,GACvB,iBAAiB,qBAAqB,KACvC,OAAO,CAAC,IAAI,CAmSd,CAAC;AAEF,eAAO,MAAM,mBAAmB,GAC5B,iBAAiB,qBAAqB,KACvC,OAAO,CAAC,IAAI,CA8Kd,CAAC;AAEF,eAAO,MAAM,kBAAkB,GAC3B,SAAS,yBAAyB,KACnC,OAAO,CAAC,IAAI,CA0Cd,CAAC"}

package/dist/index.js

@@ -63,6 +63,7 @@ function parseDisableTimestampConversionOption(value) {
     }
     return value.toLowerCase() === 'true';
 }
+const validateWarehouseColumnsDescription = `Check physical \${TABLE}.column references in dimensions and metrics by executing zero-row queries against the warehouse. Requires warehouse credentials.`;
 function parseProjectArgument(value) {
     if (value === undefined) {
         throw new commander_1.InvalidArgumentError('No project argument provided.');
@@ -231,6 +232,7 @@ commander_1.program
     .option('--no-warehouse-credentials', 'Compile without any warehouse credentials. Skips dbt compile + warehouse catalog')
     .option('--use-dbt-list [true|false]', 'Use `dbt list` instead of `dbt compile` to generate dbt manifest.json', parseUseDbtListOption, true)
     .option('--disable-timestamp-conversion [true|false]', 'Disable timestamp conversion to UTC for Snowflake warehouses. Only use this if your timestamp values are already in UTC.', parseDisableTimestampConversionOption)
+    .option('--validate-warehouse-columns', validateWarehouseColumnsDescription, false)
     .action(compile_1.compileHandler);
 commander_1.program
     .command('preview')
@@ -269,6 +271,7 @@ commander_1.program
     .option('--batch-size <number>', 'Number of explores to deploy in each batch (default: 50)', '50')
     .option('--parallel-batches <number>', 'Number of batches to send in parallel (default: 1, use higher values with caution)', '1')
     .option('--expires-in <hours>', 'Number of hours until the preview project auto-expires (default: 720, i.e. 30 days)')
+    .option('--validate-warehouse-columns', validateWarehouseColumnsDescription, false)
     .action(preview_1.previewHandler);
 commander_1.program
     .command('start-preview')
@@ -307,6 +310,7 @@ commander_1.program
     .option('--batch-size <number>', 'Number of explores to deploy in each batch (default: 50)', '50')
     .option('--parallel-batches <number>', 'Number of batches to send in parallel (default: 1, use higher values with caution)', '1')
     .option('--expires-in <hours>', 'Number of hours until the preview project auto-expires (default: 720, i.e. 30 days)')
+    .option('--validate-warehouse-columns', validateWarehouseColumnsDescription, false)
     .action(preview_1.startPreviewHandler);
 commander_1.program
     .command('stop-preview')
@@ -398,7 +402,7 @@ const uploadCommand = commander_1.program
     .option('--include-charts', 'Include charts updates when uploading dashboards', false)
     .option('--validate', 'Validate charts and dashboards after upload', false)
     .option('--gzip', 'Enable gzip compression for request bodies', false)
-    .option('--apps <appUuids...>', 'Upload specific data apps by UUID (enterprise).')
+    .option('--apps <appReferences...>', 'Upload specific data apps by UUID or URL (enterprise).')
     .option('--include-apps', 'Upload all app folders on disk (enterprise).', false)
     .option('--create-new', 'Always create a new app from the uploaded code instead of updating the app referenced by lightdash-app.yml.', false)
     .option('--organization', 'upload all organization-scoped resources without selecting a project', false)
@@ -441,6 +445,7 @@ commander_1.program
     .option('--batch-size <number>', 'Number of explores per batch (default: 50)', '50')
     .option('--parallel-batches <number>', 'Number of batches to send in parallel (default: 1, use higher values with caution)', '1')
     .option('--gzip', 'Enable gzip compression for request bodies', false)
+    .option('--validate-warehouse-columns', validateWarehouseColumnsDescription, false)
     .action(deploy_1.deployHandler);
 commander_1.program
     .command('refresh')
@@ -481,7 +486,7 @@ commander_1.program
     .addOption(new commander_1.Option('--only <elems...>', 'Specify project elements to validate')
     .choices(Object.values(common_1.ValidationTarget))
     .default(Object.values(common_1.ValidationTarget)))
-    .option('--validate-warehouse-columns', `Check physical \${TABLE}.column references in dimensions and metrics by executing zero-row queries against the warehouse. Requires warehouse credentials and only applies when tables are validated.`, false)
+    .option('--validate-warehouse-columns', `${validateWarehouseColumnsDescription} Only applies when tables are validated.`, false)
     .option('--include-spaces <spaceSlugs...>', 'Only report validation errors for charts and dashboards in these spaces (and their sub-spaces). Spaces must be visible to your credentials')
     .option('--exclude-spaces <spaceSlugs...>', 'Skip validation errors for charts and dashboards in these spaces (and their sub-spaces). Spaces must be visible to your credentials')
     .action(validate_1.validateHandler);

package/dist/vendor/template/references/chart-references.md

@@ -22,6 +22,15 @@ Each file contains:
 - `metricQuery.sorts` — sort order
 - `metricQuery.limit` — row limit
 - `metricQuery.tableCalculations` — computed columns (may be empty)
+- `chartConfig` — the saved visualization config. `chartConfig.type` tells you
+  what to render: `big_number` (KPI stat), `cartesian` (bar/line/area/scatter —
+  the series type is inside `chartConfig.config.eChartsConfig.series`), `table`,
+  `pie`, `funnel`, `treemap`, `gauge`, `map`, `sankey`, or `custom`. Reproduce
+  the chart type instead of guessing one from the data shape; deeper styling
+  details inside the config are optional inspiration, not a spec.
+- `pivotConfig` — pivot dimension field IDs when the saved chart pivots its
+  results (`null` otherwise). A pivoted table/bar means one series per value of
+  the pivot dimension.
 
 **How to use these:**
 1. Read the JSON file(s) to understand what data the user wants in their app

package/dist/vendor/template/references/dashboard-blueprint.md

@@ -0,0 +1,100 @@
+# Dashboard blueprint (dashboard attached to the prompt)
+
+> Read this when the prompt announces an attached dashboard with a blueprint at
+> `/tmp/dashboard/blueprint.json`.
+
+## Contents
+
+- What the blueprint JSON contains
+- How to map tabs, the tile grid, and tile types to app structure
+- How to wire dashboard filters as interactive controls
+- What to skip and when to deviate
+
+The user attached a Lightdash dashboard. The blueprint is its full structural
+definition — use it as the layout spec and recreate the dashboard's design,
+instead of inventing a layout from scratch. The chart queries themselves live in
+`/tmp/metric-queries/*.json` (see `chart-references.md`); the blueprint tells you
+how those charts were arranged and filtered.
+
+## Blueprint shape
+
+```jsonc
+{
+  "dashboardUuid": "…",
+  "name": "Sales overview",          // use as the app title unless the user names it
+  "description": "…",                // may be null
+  "tabs": [                          // sorted by order; [] when the dashboard has no tabs
+    { "uuid": "tab-1", "name": "Summary", "order": 0, "hidden": false }
+  ],
+  "tiles": [ /* see below */ ],
+  "filters": {                       // dashboard-level filters
+    "dimensions": [ /* DashboardFilterRule */ ],
+    "metrics": [],
+    "tableCalculations": []
+  },
+  "parameters": { /* default parameter values, or null */ },
+  "config": { /* date-zoom settings etc., or null */ }
+}
+```
+
+### Tiles
+
+Every tile has: `uuid`, `type`, `x`, `y`, `w`, `h`, `tabUuid`, `properties`.
+
+**Grid geometry:** positions are on a **36-column grid** — `x`/`w` are column
+units (a full-width tile is `w: 36`, a half-width tile `w: 18`), `y`/`h` are row
+units of roughly 55px. Translate proportionally into your layout (e.g. CSS grid
+with 12 columns: divide `x`/`w` by 3). Preserve the relative arrangement — what
+sits side by side, what spans full width, the top-to-bottom order — rather than
+the pixel-exact math. `tabUuid` says which tab the tile belongs to (`null` when
+the dashboard has no tabs).
+
+Tile types and their `properties`:
+
+- `saved_chart` — `savedChartUuid`, `chartName`, optional `title` (falls back to
+  the chart name) and `hideTitle`. **Match it to its metric-query file via
+  `savedChartUuid` = the `chartUuid` field inside each
+  `/tmp/metric-queries/*.json`.** Build the tile's visualization from that
+  file's `metricQuery` + `chartConfig` (or `savedChart(uuid)` when the file says
+  LINKED).
+- `markdown` — `title`, `content` (markdown source). Recreate as a text/prose
+  section.
+- `heading` — `text`, `showDivider`. Recreate as a section header.
+- `loom` — `title`, `url`. Recreate as a link out to the video (do not embed
+  arbitrary iframes).
+- `sql_chart` — `savedSqlUuid`, `chartName`. Its query is NOT available in
+  `/tmp/metric-queries/`. Keep its place in the layout: render a clearly
+  labelled placeholder card with the tile's title, or rebuild it from catalog
+  fields if the prompt makes the intent obvious.
+- `data_app` — an embedded data app. Skip it; note it in a comment.
+
+### Tabs
+
+Recreate tabs as top-level navigation (tab bar / segmented control), one view
+per tab, tiles assigned by `tabUuid`, ordered by `order`. Skip tabs with
+`hidden: true`. Single implicit tab (empty `tabs`) → a single page, no tab
+chrome.
+
+### Dashboard filters
+
+Each rule in `filters.dimensions` is a `DashboardFilterRule`: `target`
+(`fieldId`, `tableName`), `operator`, `values`, `label`, and `tileTargets`.
+Recreate these as the app's interactive filter controls (see the global-filters
+guidance in `skill.md`):
+
+- One control per rule; `label` (or the target field's label) is the control's
+  caption; `values` are the default selection; `disabled: true` means "no
+  default value" — start unfiltered.
+- `tileTargets` maps tile `uuid` → the field to filter on for that tile, or
+  `false` when the rule must NOT apply to that tile. A missing entry means
+  "apply with the rule's own target field". Respect exclusions — that a chart is
+  exempt from a filter is deliberate dashboard design.
+- Apply selections via `.filters()` — inline convention for `query(...)`,
+  qualified ids for `savedChart(...)` (rules in `chart-references.md`).
+
+## When to deviate
+
+The blueprint wins on structure (tabs, arrangement, which filters exist) unless
+the user's prompt asks for something different — the prompt always wins. You may
+modernize presentation (spacing, typography, card styling per the design skill)
+while keeping the structure recognizably the same dashboard.

package/dist/vendor/template/skill.md

@@ -148,6 +148,10 @@ These metrics are **not additive over time**. A balance on Monday plus the balan
 
 If the prompt lists referenced charts (files under `/tmp/metric-queries/*.json`), read `/app/references/chart-references.md` before writing any query code — it defines the JSON shape, linked-vs-copied chart semantics (`savedChart`), and the field-id mapping rules.
 
+## Attached dashboard blueprint
+
+If the prompt announces an attached dashboard (blueprint at `/tmp/dashboard/blueprint.json`), read `/app/references/dashboard-blueprint.md` before designing any layout — the blueprint defines the dashboard's tabs, tile grid, and filters, and it is the layout spec to recreate unless the user asks for a different design.
+
 ## Linked external connections
 
 If the prompt lists external connections (files under `/tmp/external-data/`), read `/app/references/external-apis.md` before calling any external API — it documents each connection file and the `externalFetch` rules.