@lightdash/cli 2.194.1 → 2.195.0

package/dist/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AAQA,eAAO,MAAM,cAAc,QAAsC,CAAC;AAElE,MAAM,MAAM,MAAM,GAAG;IACjB,IAAI,CAAC,EAAE;QACH,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,gBAAgB,CAAC,EAAE,MAAM,CAAC;KAC7B,CAAC;IACF,OAAO,CAAC,EAAE;QACN,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,OAAO,CAAC,EAAE,MAAM,CAAC;QACjB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB;;;WAGG;QACH,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,kBAAkB,CAAC,EAAE,MAAM,CAAC;QAC5B,cAAc,CAAC,EAAE,MAAM,CAAC;QACxB,WAAW,CAAC,EAAE,MAAM,CAAC;KACxB,CAAC;IACF,OAAO,CAAC,EAAE;QACN,qCAAqC,CAAC,EAAE,OAAO,CAAC;QAChD,gCAAgC,CAAC,EAAE,OAAO,CAAC;KAC9C,CAAC;CACL,CAAC;AAEF,qEAAqE;AACrE,eAAO,MAAM,2BAA2B,aAC1B,MAAM,KACjB,OAAO,CAAC,IAAI,CAgBd,CAAC;AAEF,qEAAqE;AACrE,eAAO,MAAM,iBAAiB,aAChB,MAAM,UACR,MAAM,KACf,OAAO,CAAC,IAAI,CAed,CAAC;AAiCF,eAAO,MAAM,SAAS,QAAa,OAAO,CAAC,MAAM,CAmBhD,CAAC;AAEF,eAAO,MAAM,UAAU,gBAAuB,MAAM,eAAe,MAAM,kBAUxE,CAAC;AAEF,eAAO,MAAM,YAAY,qBAUxB,CAAC;AAEF,eAAO,MAAM,iBAAiB,gBAAuB,MAAM,QAAQ,MAAM,kBAUxE,CAAC;AAEF,eAAO,MAAM,mBAAmB,qBAU/B,CAAC;AAEF,eAAO,MAAM,cAAc,aACb,MAAM,oBACE,MAAM,kBAW3B,CAAC;AAEF,eAAO,MAAM,UAAU,YAAmB,MAAM,CAAC,SAAS,CAAC,kBAM1D,CAAC;AAEF,eAAO,MAAM,SAAS,WAAkB,MAAM,CAAC,SAAS,CAAC,kBAMxD,CAAC"}
+{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AAQA,eAAO,MAAM,cAAc,QAAsC,CAAC;AAElE,MAAM,MAAM,MAAM,GAAG;IACjB,IAAI,CAAC,EAAE;QACH,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,gBAAgB,CAAC,EAAE,MAAM,CAAC;KAC7B,CAAC;IACF,OAAO,CAAC,EAAE;QACN,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,OAAO,CAAC,EAAE,MAAM,CAAC;QACjB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB;;;WAGG;QACH,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,kBAAkB,CAAC,EAAE,MAAM,CAAC;QAC5B,cAAc,CAAC,EAAE,MAAM,CAAC;QACxB,WAAW,CAAC,EAAE,MAAM,CAAC;KACxB,CAAC;IACF,OAAO,CAAC,EAAE;QACN,qCAAqC,CAAC,EAAE,OAAO,CAAC;QAChD,gCAAgC,CAAC,EAAE,OAAO,CAAC;KAC9C,CAAC;CACL,CAAC;AAEF,qEAAqE;AACrE,eAAO,MAAM,2BAA2B,aAC1B,MAAM,KACjB,OAAO,CAAC,IAAI,CAgBd,CAAC;AAEF,qEAAqE;AACrE,eAAO,MAAM,iBAAiB,aAChB,MAAM,UACR,MAAM,KACf,OAAO,CAAC,IAAI,CAed,CAAC;AAiCF,eAAO,MAAM,SAAS,QAAa,OAAO,CAAC,MAAM,CAwBhD,CAAC;AAEF,eAAO,MAAM,UAAU,gBAAuB,MAAM,eAAe,MAAM,kBAUxE,CAAC;AAEF,eAAO,MAAM,YAAY,qBAUxB,CAAC;AAEF,eAAO,MAAM,iBAAiB,gBAAuB,MAAM,QAAQ,MAAM,kBAUxE,CAAC;AAEF,eAAO,MAAM,mBAAmB,qBAU/B,CAAC;AAEF,eAAO,MAAM,cAAc,aACb,MAAM,oBACE,MAAM,kBAW3B,CAAC;AAEF,eAAO,MAAM,UAAU,YAAmB,MAAM,CAAC,SAAS,CAAC,kBAM1D,CAAC;AAEF,eAAO,MAAM,SAAS,WAAkB,MAAM,CAAC,SAAS,CAAC,kBAMxD,CAAC"}

package/dist/config.js

@@ -84,6 +84,10 @@ const getConfig = async () => {
             ...(rawConfig.context || {}),
             apiKey: process.env.LIGHTDASH_API_KEY || rawConfig.context?.apiKey,
             project: process.env.LIGHTDASH_PROJECT || rawConfig.context?.project,
+            projectName: process.env.LIGHTDASH_PROJECT &&
+                process.env.LIGHTDASH_PROJECT !== rawConfig.context?.project
+                ? undefined
+                : rawConfig.context?.projectName,
             serverUrl: process.env.LIGHTDASH_URL || rawConfig.context?.serverUrl,
             proxyAuthorization: process.env.LIGHTDASH_PROXY_AUTHORIZATION ||
                 rawConfig.context?.proxyAuthorization,

package/dist/config.test.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const tslib_1 = require("tslib");
 const fs_1 = require("fs");
+const yaml = tslib_1.__importStar(require("js-yaml"));
 const os = tslib_1.__importStar(require("os"));
 const path = tslib_1.__importStar(require("path"));
 const config_1 = require("./config");
@@ -38,3 +39,34 @@ describePosix('CLI config permissions', () => {
         expect(permissionBits((await fs_1.promises.stat(configPath)).mode)).toBe(0o600);
     });
 });
+describe('CLI project environment override', () => {
+    afterEach(() => {
+        vi.restoreAllMocks();
+        vi.unstubAllEnvs();
+    });
+    it.each([
+        ['other-project', undefined],
+        ['stored-project', 'Stored project'],
+        ['', 'Stored project'],
+        [undefined, 'Stored project'],
+    ])('only retains the cached name for its own project (override: %s)', async (projectOverride, expectedName) => {
+        vi.stubEnv('LIGHTDASH_PROJECT', projectOverride);
+        vi.spyOn(fs_1.promises, 'chmod').mockResolvedValue(undefined);
+        vi.spyOn(fs_1.promises, 'readFile').mockResolvedValue(yaml.dump({
+            user: { anonymousUuid: 'anonymous-user' },
+            context: {
+                project: 'stored-project',
+                projectName: 'Stored project',
+                previewProject: 'preview-project',
+                previewName: 'Preview project',
+            },
+        }));
+        const config = await (0, config_1.getConfig)();
+        expect(config.context).toMatchObject({
+            project: projectOverride || 'stored-project',
+            projectName: expectedName,
+            previewProject: 'preview-project',
+            previewName: 'Preview project',
+        });
+    });
+});

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

@@ -1 +1 @@
-{"version":3,"file":"renameHandler.d.ts","sourceRoot":"","sources":["../../src/handlers/renameHandler.ts"],"names":[],"mappings":"AAAA,OAAO,EAMH,UAAU,EAEb,MAAM,mBAAmB,CAAC;AAmB3B,KAAK,oBAAoB,GAAG;IACxB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,UAAU,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,OAAO,CAAC;IAChB,SAAS,EAAE,OAAO,CAAC;IACnB,IAAI,EAAE,OAAO,CAAC;IACd,QAAQ,EAAE,OAAO,CAAC;CACrB,CAAC;AAwCF,eAAO,MAAM,aAAa,YAAmB,oBAAoB,kBAmNhE,CAAC"}
+{"version":3,"file":"renameHandler.d.ts","sourceRoot":"","sources":["../../src/handlers/renameHandler.ts"],"names":[],"mappings":"AAAA,OAAO,EAMH,UAAU,EAEb,MAAM,mBAAmB,CAAC;AAmB3B,KAAK,oBAAoB,GAAG;IACxB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,UAAU,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,OAAO,CAAC;IAChB,SAAS,EAAE,OAAO,CAAC;IACnB,IAAI,EAAE,OAAO,CAAC;IACd,QAAQ,EAAE,OAAO,CAAC;CACrB,CAAC;AAwCF,eAAO,MAAM,aAAa,YAAmB,oBAAoB,kBAwNhE,CAAC"}

package/dist/handlers/renameHandler.js

@@ -13,7 +13,7 @@ const globalState_1 = tslib_1.__importDefault(require("../globalState"));
 const styles = tslib_1.__importStar(require("../styles"));
 const apiClient_1 = require("./dbt/apiClient");
 const refresh_1 = require("./dbt/refresh");
-const resolveProjectFlag_1 = require("./resolveProjectFlag");
+const selectProject_1 = require("./selectProject");
 const validate_1 = require("./validate");
 const REFETCH_JOB_INTERVAL = 2000;
 const waitUntilRenameFinished = (jobUuid) => (0, validate_1.waitUntilFinished)({
@@ -45,9 +45,10 @@ const renameHandler = async (options) => {
     if (!config.context?.apiKey || !config.context.serverUrl) {
         throw new common_1.AuthorizationError(`Not logged in. Run 'lightdash login --help'`);
     }
-    const projectUuid = options.project
-        ? await (0, resolveProjectFlag_1.resolveProjectFlag)(options.project)
-        : config.context.previewProject || config.context.project;
+    const overrideProjectUuid = await (0, selectProject_1.resolveProjectOverride)(config, options.project);
+    const projectUuid = overrideProjectUuid ||
+        config.context.previewProject ||
+        config.context.project;
     if (!projectUuid) {
         throw new common_1.LightdashError({
             message: 'No project selected. Run lightdash config set-project',
@@ -57,7 +58,7 @@ const renameHandler = async (options) => {
         });
     }
     // Log current project info
-    if (options.project) {
+    if (overrideProjectUuid) {
         console.error(`\n${styles.success('Renaming in project:')} ${projectUuid}\n`);
     }
     else {

package/dist/handlers/renameHandler.test.js

@@ -1,8 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const tslib_1 = require("tslib");
 const common_1 = require("@lightdash/common");
 const analytics_1 = require("../analytics/analytics");
 const config_1 = require("../config");
+const globalState_1 = tslib_1.__importDefault(require("../globalState"));
 const apiClient_1 = require("./dbt/apiClient");
 const refresh_1 = require("./dbt/refresh");
 const renameHandler_1 = require("./renameHandler");
@@ -29,6 +31,7 @@ const emptyResults = {
 };
 const RENAME_JOB_ID = 'rename-job-id';
 const VALIDATION_JOB_ID = 'validation-job-id';
+const PREVIEW_PROJECT = '33333333-3333-4333-8333-333333333333';
 describe('renameHandler follow-up validation', () => {
     let errorOutput;
     const mockApi = (failingJobId) => {
@@ -54,11 +57,19 @@ describe('renameHandler follow-up validation', () => {
             if (url.includes('/validate?jobId=')) {
                 return [];
             }
+            if (method === 'GET' &&
+                url === `/api/v1/projects/${PREVIEW_PROJECT}`) {
+                return {
+                    projectUuid: PREVIEW_PROJECT,
+                    type: common_1.ProjectType.PREVIEW,
+                };
+            }
             throw new Error(`Unexpected API call: ${method} ${url}`);
         });
     };
     beforeEach(() => {
         vi.clearAllMocks();
+        vi.stubEnv('LIGHTDASH_PROJECT', undefined);
         vi.mocked(apiClient_1.checkLightdashVersion).mockResolvedValue(undefined);
         vi.mocked(config_1.getConfig).mockResolvedValue({
             context: {
@@ -78,11 +89,41 @@ describe('renameHandler follow-up validation', () => {
     });
     afterEach(() => {
         vi.restoreAllMocks();
+        vi.unstubAllEnvs();
     });
     const trackedEvents = () => vi.mocked(analytics_1.LightdashAnalytics.track).mock.calls.map(([payload]) => ({
         event: payload.event,
         properties: payload.properties,
     }));
+    test.each([undefined, baseOptions.project])('targets the environment project unless --project overrides it (%s)', async (explicitProject) => {
+        const envProject = '22222222-2222-4222-8222-222222222222';
+        vi.stubEnv('LIGHTDASH_PROJECT', envProject);
+        vi.spyOn(globalState_1.default, 'isNonInteractive').mockReturnValue(true);
+        vi.mocked(config_1.getConfig).mockResolvedValue({
+            context: {
+                apiKey: 'test-key',
+                serverUrl: 'http://localhost',
+                project: envProject,
+                previewProject: PREVIEW_PROJECT,
+                previewName: 'Stale preview',
+            },
+        });
+        mockApi(null);
+        await (0, renameHandler_1.renameHandler)({ ...baseOptions, project: explicitProject });
+        const expectedProject = explicitProject || envProject;
+        expect(apiClient_1.lightdashApi).toHaveBeenCalledWith(expect.objectContaining({
+            method: 'POST',
+            url: `/api/v1/projects/${expectedProject}/rename`,
+        }));
+        const output = errorOutput.join('\n');
+        expect(output).toContain(`Renaming in project: ${expectedProject}`);
+        if (explicitProject) {
+            expect(output).not.toContain('Stale preview');
+        }
+        else {
+            expect(output).toContain('active preview "Stale preview" ignored');
+        }
+    });
     test('reports a failed validation job as a validation failure, not a rename failure', async () => {
         mockApi(VALIDATION_JOB_ID);
         await (0, renameHandler_1.renameHandler)({ ...baseOptions });

package/dist/handlers/selectProject.d.ts

@@ -3,14 +3,21 @@ export type ProjectSelection = {
     projectUuid: string;
     isPreview: boolean;
 };
+/**
+ * Resolves an explicit project override: `--project` wins over
+ * LIGHTDASH_PROJECT, and either one bypasses an active preview.
+ * Returns undefined when neither is set.
+ */
+export declare const resolveProjectOverride: (config: Config, explicitProject: string | undefined) => Promise<string | undefined>;
 /**
  * Resolves which project to use, prompting the user if there's an active preview.
  *
  * Priority:
  * 1. If `explicitProject` is provided (via --project flag), use it
- * 2. If there's an active preview and we're in interactive mode, ask the user
- * 3. In non-interactive mode with active preview, use preview project
- * 4. Fall back to the main project
+ * 2. If LIGHTDASH_PROJECT is set, use it (warns when a preview is ignored)
+ * 3. If there's an active preview and we're in interactive mode, ask the user
+ * 4. In non-interactive mode with active preview, use preview project
+ * 5. Fall back to the main project
  */
 export declare const selectProject: (config: Config, explicitProject?: string) => Promise<ProjectSelection | undefined>;
 /**

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

@@ -1 +1 @@
-{"version":3,"file":"selectProject.d.ts","sourceRoot":"","sources":["../../src/handlers/selectProject.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,MAAM,EAAuB,MAAM,WAAW,CAAC;AAMxD,MAAM,MAAM,gBAAgB,GAAG;IAC3B,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,OAAO,CAAC;CACtB,CAAC;AAmCF;;;;;;;;GAQG;AACH,eAAO,MAAM,aAAa,WACd,MAAM,oBACI,MAAM,KACzB,OAAO,CAAC,gBAAgB,GAAG,SAAS,CAmFtC,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,kBAAkB,cAChB,gBAAgB,UACnB,MAAM,UACN,MAAM,KACf,IAsBF,CAAC"}
+{"version":3,"file":"selectProject.d.ts","sourceRoot":"","sources":["../../src/handlers/selectProject.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,MAAM,EAAuB,MAAM,WAAW,CAAC;AAMxD,MAAM,MAAM,gBAAgB,GAAG;IAC3B,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,OAAO,CAAC;CACtB,CAAC;AAsDF;;;;GAIG;AACH,eAAO,MAAM,sBAAsB,WACvB,MAAM,mBACG,MAAM,GAAG,SAAS,KACpC,OAAO,CAAC,MAAM,GAAG,SAAS,CAW5B,CAAC;AAEF;;;;;;;;;GASG;AACH,eAAO,MAAM,aAAa,WACd,MAAM,oBACI,MAAM,KACzB,OAAO,CAAC,gBAAgB,GAAG,SAAS,CAmFtC,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,kBAAkB,cAChB,gBAAgB,UACnB,MAAM,UACN,MAAM,KACf,IAsBF,CAAC"}

package/dist/handlers/selectProject.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.logSelectedProject = exports.selectProject = void 0;
+exports.logSelectedProject = exports.selectProject = exports.resolveProjectOverride = void 0;
 const tslib_1 = require("tslib");
 const common_1 = require("@lightdash/common");
 const inquirer_1 = tslib_1.__importDefault(require("inquirer"));
@@ -34,22 +34,47 @@ const validatePreviewProject = async (previewProjectUuid) => {
         return previewProjectUuid;
     }
 };
+const warnIgnoredPreview = async (config, projectUuid) => {
+    const previewProject = await validatePreviewProject(config.context?.previewProject);
+    if (!previewProject || previewProject === projectUuid) {
+        return;
+    }
+    const previewName = config.context?.previewName;
+    const previewLabel = previewName ? `"${previewName}"` : previewProject;
+    globalState_1.default.log(styles.warning(`\nUsing project ${projectUuid} from LIGHTDASH_PROJECT; active preview ${previewLabel} ignored.\nPass --project ${previewProject} or unset LIGHTDASH_PROJECT to target the preview.\n`));
+};
+/**
+ * Resolves an explicit project override: `--project` wins over
+ * LIGHTDASH_PROJECT, and either one bypasses an active preview.
+ * Returns undefined when neither is set.
+ */
+const resolveProjectOverride = async (config, explicitProject) => {
+    if (explicitProject) {
+        return (0, resolveProjectFlag_1.resolveProjectFlag)(explicitProject);
+    }
+    const envProject = process.env.LIGHTDASH_PROJECT;
+    if (!envProject) {
+        return undefined;
+    }
+    const projectUuid = await (0, resolveProjectFlag_1.resolveProjectFlag)(envProject);
+    await warnIgnoredPreview(config, projectUuid);
+    return projectUuid;
+};
+exports.resolveProjectOverride = resolveProjectOverride;
 /**
  * Resolves which project to use, prompting the user if there's an active preview.
  *
  * Priority:
  * 1. If `explicitProject` is provided (via --project flag), use it
- * 2. If there's an active preview and we're in interactive mode, ask the user
- * 3. In non-interactive mode with active preview, use preview project
- * 4. Fall back to the main project
+ * 2. If LIGHTDASH_PROJECT is set, use it (warns when a preview is ignored)
+ * 3. If there's an active preview and we're in interactive mode, ask the user
+ * 4. In non-interactive mode with active preview, use preview project
+ * 5. Fall back to the main project
  */
 const selectProject = async (config, explicitProject) => {
-    // If explicit project provided via --project flag, use it
-    if (explicitProject) {
-        return {
-            projectUuid: await (0, resolveProjectFlag_1.resolveProjectFlag)(explicitProject),
-            isPreview: false,
-        };
+    const overrideProjectUuid = await (0, exports.resolveProjectOverride)(config, explicitProject);
+    if (overrideProjectUuid) {
+        return { projectUuid: overrideProjectUuid, isPreview: false };
     }
     const mainProject = config.context?.project;
     const previewProject = await validatePreviewProject(config.context?.previewProject);

package/dist/handlers/selectProject.test.js

@@ -1,6 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const tslib_1 = require("tslib");
 const common_1 = require("@lightdash/common");
+const inquirer_1 = tslib_1.__importDefault(require("inquirer"));
+const config_1 = require("../config");
+const globalState_1 = tslib_1.__importDefault(require("../globalState"));
 const apiClient_1 = require("./dbt/apiClient");
 const selectProject_1 = require("./selectProject");
 vi.mock('inquirer');
@@ -22,8 +26,70 @@ const mockPreviewProjectResponse = (uuid) => {
     });
 };
 describe('selectProject', () => {
+    beforeEach(() => {
+        vi.stubEnv('LIGHTDASH_PROJECT', undefined);
+    });
     afterEach(() => {
-        vi.clearAllMocks();
+        vi.resetAllMocks();
+        vi.restoreAllMocks();
+        vi.unstubAllEnvs();
+    });
+    it.each([true, false])('uses LIGHTDASH_PROJECT instead of an active preview and warns (non-interactive: %s)', async (nonInteractive) => {
+        vi.stubEnv('LIGHTDASH_PROJECT', MAIN_UUID);
+        vi.spyOn(globalState_1.default, 'isNonInteractive').mockReturnValue(nonInteractive);
+        const log = vi
+            .spyOn(globalState_1.default, 'log')
+            .mockImplementation(() => { });
+        mockPreviewProjectResponse(PREVIEW_UUID);
+        const config = {
+            context: {
+                project: MAIN_UUID,
+                previewProject: PREVIEW_UUID,
+                previewName: 'My preview',
+            },
+        };
+        expect(await (0, selectProject_1.selectProject)(config)).toEqual({
+            projectUuid: MAIN_UUID,
+            isPreview: false,
+        });
+        expect(inquirer_1.default.prompt).not.toHaveBeenCalled();
+        expect(log).toHaveBeenCalledTimes(1);
+        const warning = String(log.mock.calls[0][0]);
+        expect(warning).toContain(`Using project ${MAIN_UUID} from LIGHTDASH_PROJECT`);
+        expect(warning).toContain('active preview "My preview" ignored');
+        expect(warning).toContain(`--project ${PREVIEW_UUID}`);
+    });
+    it('clears a deleted preview instead of warning when LIGHTDASH_PROJECT is set', async () => {
+        vi.stubEnv('LIGHTDASH_PROJECT', MAIN_UUID);
+        const log = vi.spyOn(globalState_1.default, 'log').mockImplementation(() => { });
+        mockLightdashApi.mockRejectedValueOnce(new common_1.LightdashError({
+            message: 'Not found',
+            name: 'NotFoundError',
+            statusCode: 404,
+            data: {},
+        }));
+        expect(await (0, selectProject_1.selectProject)({
+            context: { project: MAIN_UUID, previewProject: PREVIEW_UUID },
+        })).toEqual({ projectUuid: MAIN_UUID, isPreview: false });
+        expect(config_1.unsetPreviewProject).toHaveBeenCalledTimes(1);
+        expect(log).not.toHaveBeenCalled();
+    });
+    it.each([
+        ['no preview is active', { project: MAIN_UUID }],
+        [
+            'LIGHTDASH_PROJECT points at the preview itself',
+            { project: MAIN_UUID, previewProject: PREVIEW_UUID },
+        ],
+    ])('does not warn when %s', async (_label, context) => {
+        const envProject = context.previewProject ?? MAIN_UUID;
+        vi.stubEnv('LIGHTDASH_PROJECT', envProject);
+        const log = vi.spyOn(globalState_1.default, 'log').mockImplementation(() => { });
+        mockPreviewProjectResponse(PREVIEW_UUID);
+        expect(await (0, selectProject_1.selectProject)({ context })).toEqual({
+            projectUuid: envProject,
+            isPreview: false,
+        });
+        expect(log).not.toHaveBeenCalled();
     });
     it('returns the preview project (without prompting) when the stored main and preview UUIDs are the same', async () => {
         // This is the misconfigured state produced by an older `set-project`
@@ -45,7 +111,8 @@ describe('selectProject', () => {
             isPreview: true,
         });
     });
-    it('uses an explicit --project UUID as-is, without any API call, even when a preview is active', async () => {
+    it.each([undefined, MAIN_UUID])('uses an explicit --project UUID ahead of the environment (%s) and active preview', async (envProject) => {
+        vi.stubEnv('LIGHTDASH_PROJECT', envProject);
         const config = {
             context: {
                 project: MAIN_UUID,
@@ -60,7 +127,9 @@ describe('selectProject', () => {
         });
         expect(mockLightdashApi).not.toHaveBeenCalled();
     });
-    it('resolves an explicit --project slug through the org projects list', async () => {
+    it.each(['flag', 'env'])('resolves a project slug from the %s through the org projects list', async (source) => {
+        if (source === 'env')
+            vi.stubEnv('LIGHTDASH_PROJECT', 'jaffle-shop');
         const config = {
             context: { project: MAIN_UUID },
         };
@@ -68,8 +137,23 @@ describe('selectProject', () => {
             { projectUuid: PREVIEW_UUID, slug: 'other', name: 'Other' },
             { projectUuid: MAIN_UUID, slug: 'jaffle-shop', name: 'Jaffle' },
         ]);
-        const selection = await (0, selectProject_1.selectProject)(config, 'jaffle-shop');
-        expect(selection).toEqual({ projectUuid: MAIN_UUID, isPreview: false });
+        const selection = await (0, selectProject_1.selectProject)(config, source === 'flag' ? 'jaffle-shop' : undefined);
+        expect(selection).toEqual({
+            projectUuid: MAIN_UUID,
+            isPreview: false,
+        });
+    });
+    it.each([undefined, ''])('keeps the active preview default without an environment override (%s)', async (envProject) => {
+        vi.stubEnv('LIGHTDASH_PROJECT', envProject);
+        vi.spyOn(globalState_1.default, 'isNonInteractive').mockReturnValue(true);
+        mockPreviewProjectResponse(PREVIEW_UUID);
+        expect(await (0, selectProject_1.selectProject)({
+            context: {
+                project: MAIN_UUID,
+                previewProject: PREVIEW_UUID,
+            },
+        })).toEqual({ projectUuid: PREVIEW_UUID, isPreview: true });
+        expect(inquirer_1.default.prompt).not.toHaveBeenCalled();
     });
     it('returns the main project when only the main project is configured', async () => {
         const config = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightdash/cli",
-  "version": "2.194.1",
+  "version": "2.195.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.194.1",
-    "@lightdash/warehouses": "2.194.1",
+    "@lightdash/common": "2.195.0",
+    "@lightdash/warehouses": "2.195.0",
     "@sidvind/better-ajv-errors": "7.0.0",
     "@types/columnify": "1.5.1",
     "ajv": "8.18.0",