@lightdash/cli 2.228.1 → 2.229.0

package/dist/handlers/download.test.js

@@ -1,1739 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-const tslib_1 = require("tslib");
-const common_1 = require("@lightdash/common");
-const fs_1 = require("fs");
-const os = tslib_1.__importStar(require("os"));
-const path = tslib_1.__importStar(require("path"));
-const vitest_1 = require("vitest");
-const analytics_1 = require("../analytics/analytics");
-const globalState_1 = tslib_1.__importDefault(require("../globalState"));
-const appsDownload_1 = require("./apps/appsDownload");
-const apiClient_1 = require("./dbt/apiClient");
-const download_1 = require("./download");
-const spacesAsCode_1 = require("./spacesAsCode");
-vitest_1.vi.mock('../analytics/analytics', () => ({
-    LightdashAnalytics: {
-        track: vitest_1.vi.fn(),
-    },
-}));
-vitest_1.vi.mock('../config', () => ({
-    getConfig: vitest_1.vi.fn().mockResolvedValue({
-        user: {
-            userUuid: 'user-uuid',
-            organizationUuid: 'organization-uuid',
-        },
-        context: {
-            apiKey: 'api-key',
-            serverUrl: 'http://lightdash.test',
-            project: 'project-uuid',
-            projectName: 'Test project',
-        },
-        answers: {
-            metadataFileGitignoreNoticeShown: true,
-        },
-    }),
-    setAnswer: vitest_1.vi.fn(),
-}));
-vitest_1.vi.mock('./dbt/apiClient', async (importOriginal) => ({
-    ...(await importOriginal()),
-    getContentAsCodeUploadPermissions: vitest_1.vi.fn(),
-    lightdashApi: vitest_1.vi.fn(),
-}));
-const { assertUniqueSpacePaths, downloadLinkedVirtualViews, downloadSpaces, extractChartTableNames, extractChartTypeRefsFromCharts, getDashboardAppSlugs, getDashboardChartSlugs, getFlatSpaceFileNames, hasContentFilters, isAiAgentsUnavailableError, isExternalConnectionsUnavailableError, isVirtualViewsUnavailableError, countChangeDelta, downloadAiAgents, isFilteredWithNoDashboards, readAiAgentFiles, readSpaceFiles, readSpaceNames, reportOpenDraftsForUpload, sanitizeChartForDownload, shouldFallBackToEmbeddedSpaces, shouldDownloadAiAgents, summarizeUploadChanges, upsertAiAgents, upsertExternalConnections, upsertResources, upsertSpaces, upsertVirtualViews, validateSpaceIdentity, writeSpaceFiles, } = download_1.testHelpers;
-describe('reportOpenDraftsForUpload', () => {
-    beforeEach(() => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        vitest_1.vi.spyOn(globalState_1.default, 'log').mockImplementation(() => undefined);
-    });
-    afterEach(() => {
-        vitest_1.vi.restoreAllMocks();
-    });
-    it('stays quiet when the project has no open drafts', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            openDraftCount: 0,
-        });
-        await reportOpenDraftsForUpload('project-uuid');
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledWith({
-            method: 'GET',
-            url: '/api/v1/projects/project-uuid/code/upload-advisory',
-            body: undefined,
-        });
-        expect(globalState_1.default.log).not.toHaveBeenCalled();
-    });
-    it('warns without blocking when the project has multiple open drafts', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            openDraftCount: 3,
-        });
-        await expect(reportOpenDraftsForUpload('project-uuid')).resolves.toBeUndefined();
-        expect(globalState_1.default.log).toHaveBeenCalledWith(expect.stringContaining('3 open content drafts'));
-        expect(globalState_1.default.log).toHaveBeenCalledWith(expect.stringContaining('Upload will continue'));
-    });
-    it('warns without blocking when the draft count cannot be loaded', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(new Error('draft table unavailable'));
-        await expect(reportOpenDraftsForUpload('project-uuid')).resolves.toBeUndefined();
-        expect(globalState_1.default.log).toHaveBeenCalledWith(expect.stringContaining('Could not check for open content drafts'));
-        expect(globalState_1.default.log).toHaveBeenCalledWith(expect.stringContaining('Upload will continue'));
-    });
-});
-const makeDownloadHandlerOptions = (overrides = {}) => ({
-    verbose: false,
-    charts: [],
-    dashboards: [],
-    alerts: [],
-    agents: [],
-    googleSheets: [],
-    scheduledDeliveries: [],
-    virtualViews: [],
-    externalConnections: [],
-    apps: [],
-    includeAgents: false,
-    includeApps: false,
-    force: false,
-    languageMap: false,
-    skipSpaceCreate: false,
-    public: false,
-    includeCharts: false,
-    nested: false,
-    rootSpaces: false,
-    skipSpaces: true,
-    skipCharts: true,
-    skipDashboards: true,
-    skipAlerts: false,
-    skipAgents: false,
-    skipGoogleSheets: false,
-    skipScheduledDeliveries: false,
-    skipVirtualViews: false,
-    skipExternalConnections: false,
-    includeAlerts: false,
-    includeGoogleSheets: false,
-    includeScheduledDeliveries: false,
-    includeVirtualViews: false,
-    includeExternalConnections: false,
-    includeAll: false,
-    stripPivotSeries: false,
-    concurrency: 1,
-    organization: false,
-    ...overrides,
-});
-const makeContentFilterOptions = (overrides = {}) => ({
-    spacesOnly: false,
-    charts: [],
-    dashboards: [],
-    agents: [],
-    alerts: [],
-    googleSheets: [],
-    scheduledDeliveries: [],
-    virtualViews: [],
-    externalConnections: [],
-    apps: [],
-    ...overrides,
-});
-describe('hasContentFilters', () => {
-    it('treats explicit homepage names as a filtered download and upload', () => {
-        expect(hasContentFilters(makeContentFilterOptions({ homepages: ['Team homepage'] }))).toBe(true);
-    });
-    it('treats explicit apps as a filtered download and upload', () => {
-        expect(hasContentFilters(makeContentFilterOptions({ apps: ['app-reference'] }))).toBe(true);
-    });
-    it('treats explicit external connections as a filtered upload', () => {
-        expect(hasContentFilters(makeContentFilterOptions({ externalConnections: ['stripe'] }))).toBe(true);
-    });
-    it('keeps an unfiltered content operation unfiltered', () => {
-        expect(hasContentFilters(makeContentFilterOptions())).toBe(false);
-    });
-    it('does not treat content selections as filters in spaces-only mode', () => {
-        expect(hasContentFilters(makeContentFilterOptions({
-            spacesOnly: true,
-            apps: ['app-reference'],
-        }))).toBe(false);
-    });
-});
-describe('data app upload loop filter wiring', () => {
-    const makeManifest = (overrides = {}) => ({
-        codeVersion: 1,
-        appUuid: 'app-uuid-unmatched',
-        projectUuid: 'proj-uuid-1',
-        version: 1,
-        name: 'App',
-        description: '',
-        template: null,
-        downloadedAt: '2026-06-25T00:00:00.000Z',
-        ...overrides,
-    });
-    it('skips a folder matching neither slug nor uuid, and uploads one matched by slug', () => {
-        // Mirrors the --apps <appReferences...> wiring in the upload loop:
-        // the filter is built once from the CLI-supplied references, then
-        // matched per folder via uploadFilterMatches.
-        const uploadFilter = (0, appsDownload_1.getDataAppUploadFilter)(['sales-app'], false);
-        const unmatchedFolder = makeManifest({
-            appUuid: 'other-uuid',
-            slug: 'other-slug',
-        });
-        const matchedBySlugFolder = makeManifest({
-            appUuid: 'matched-uuid',
-            slug: 'sales-app',
-        });
-        expect((0, appsDownload_1.uploadFilterMatches)(uploadFilter, unmatchedFolder)).toBe(false);
-        expect((0, appsDownload_1.uploadFilterMatches)(uploadFilter, matchedBySlugFolder)).toBe(true);
-    });
-});
-const makeLooseDashboard = (slug, chartSlugs) => ({
-    slug,
-    name: slug,
-    spaceSlug: 'test-space',
-    version: 1,
-    tiles: chartSlugs.map((chartSlug) => ({
-        properties: chartSlug ? { chartSlug } : {},
-    })),
-    needsUpdating: false,
-});
-const writeFolderDashboard = async (baseDir, slug, chartSlugs) => {
-    const tilesYaml = chartSlugs
-        .map((s) => `  - properties:\n      chartSlug: ${s}\n    type: saved_chart`)
-        .join('\n');
-    const yaml = `contentType: dashboard\nname: ${slug}\nslug: ${slug}\nspaceSlug: test-space\ntiles:\n${tilesYaml}\nversion: 1\n`;
-    await fs_1.promises.writeFile(path.join(baseDir, 'dashboards', `${slug}.yml`), yaml);
-};
-const makeLooseAppDashboard = (slug, appSlugs) => ({
-    slug,
-    name: slug,
-    spaceSlug: 'test-space',
-    version: 1,
-    tiles: appSlugs.map((appSlug) => ({
-        type: 'data_app',
-        properties: { appSlug },
-    })),
-    needsUpdating: false,
-});
-const writeFolderAppDashboard = async (baseDir, slug, appSlugs) => {
-    const tilesYaml = appSlugs
-        .map((s) => `  - properties:\n      appSlug: ${s}\n    type: data_app`)
-        .join('\n');
-    const yaml = `contentType: dashboard\nname: ${slug}\nslug: ${slug}\nspaceSlug: test-space\ntiles:\n${tilesYaml}\nversion: 1\n`;
-    await fs_1.promises.writeFile(path.join(baseDir, 'dashboards', `${slug}.yml`), yaml);
-};
-describe('summarizeUploadChanges', () => {
-    it('reports only phase deltas and warns when resources fail', () => {
-        expect(summarizeUploadChanges({ 'charts created': 1, 'spaces skipped': 3 }, {
-            'charts created': 3,
-            'spaces skipped': 3,
-            'charts skipped': 2,
-            'charts with errors': 1,
-        })).toEqual({
-            detail: '2 created, 2 skipped, 1 with errors',
-            variant: 'warning',
-        });
-    });
-    it('reports an empty phase without inventing a resource count', () => {
-        expect(summarizeUploadChanges({}, {})).toEqual({
-            detail: 'no changes',
-        });
-    });
-});
-const pivotedSeries = (pivotValue, overrides = {}) => ({
-    type: common_1.CartesianSeriesType.BAR,
-    encode: {
-        xRef: { field: 'events_date_day' },
-        yRef: {
-            field: 'orders_count',
-            pivotValues: [{ field: 'orders_status', value: pivotValue }],
-        },
-        x: 'events_date_day',
-        y: `orders_count.orders_status.${pivotValue}`,
-    },
-    ...overrides,
-});
-const makeChart = (series) => ({
-    name: 'pivoted chart',
-    slug: 'pivoted-chart',
-    spaceSlug: 'test-space',
-    tableName: 'orders',
-    version: 1,
-    metricQuery: {
-        additionalMetrics: [],
-        customDimensions: [],
-        dimensionOverrides: {},
-        dimensions: ['orders_status', 'events_date_day'],
-        exploreName: 'orders',
-        filters: {},
-        limit: 500,
-        metricOverrides: {},
-        metrics: ['orders_count'],
-        sorts: [],
-        tableCalculations: [],
-        timezone: 'project_timezone',
-    },
-    chartConfig: {
-        type: common_1.ChartType.CARTESIAN,
-        config: {
-            layout: {
-                xField: 'events_date_day',
-                yField: ['orders_count'],
-            },
-            eChartsConfig: {
-                series,
-            },
-        },
-    },
-    pivotConfig: {
-        columns: ['orders_status'],
-    },
-    dashboardSlug: undefined,
-});
-const writeFolderChart = async (baseDir, slug) => {
-    const yaml = `contentType: chart\nname: ${slug}\nslug: ${slug}\nspaceSlug: test-space\ntableName: orders\nversion: 1\n`;
-    await fs_1.promises.writeFile(path.join(baseDir, 'charts', `${slug}.yml`), yaml);
-};
-describe('upload summary counts', () => {
-    let tmpDir;
-    beforeEach(async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        vitest_1.vi.spyOn(globalState_1.default, 'log').mockImplementation(() => undefined);
-        vitest_1.vi.spyOn(globalState_1.default, 'debug').mockImplementation(() => undefined);
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'download-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'charts'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'dashboards'));
-    });
-    afterEach(async () => {
-        vitest_1.vi.restoreAllMocks();
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('counts only the uploaded type, not the spaces and tiles echoed by the API', async () => {
-        await writeFolderChart(tmpDir, 'chart-a');
-        await writeFolderChart(tmpDir, 'chart-b');
-        await writeFolderDashboard(tmpDir, 'dash', ['chart-a', 'chart-b']);
-        const noChanges = { action: common_1.PromotionAction.NO_CHANGES };
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockImplementation(async ({ url }) => url.includes('/code/charts/')
-            ? {
-                charts: [{ action: common_1.PromotionAction.UPDATE }],
-                spaces: [noChanges],
-                dashboards: [],
-            }
-            : {
-                dashboards: [{ action: common_1.PromotionAction.UPDATE }],
-                charts: [noChanges, noChanges],
-                spaces: [noChanges],
-            });
-        const summarySpy = vitest_1.vi
-            .spyOn(console, 'info')
-            .mockImplementation(() => undefined);
-        const changes = {};
-        const chartsResult = await upsertResources('charts', 'project-uuid', changes, false, [], true, tmpDir);
-        const afterCharts = { ...chartsResult.changes };
-        expect(summarizeUploadChanges({}, afterCharts).detail).toBe('2 updated');
-        const dashboardsResult = await upsertResources('dashboards', 'project-uuid', changes, false, [], true, tmpDir);
-        expect(summarizeUploadChanges(afterCharts, dashboardsResult.changes)
-            .detail).toBe('1 updated');
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledTimes(3);
-        expect(dashboardsResult.changes).toEqual({
-            'charts updated': 2,
-            'dashboards updated': 1,
-        });
-        (0, spacesAsCode_1.logUploadChanges)(dashboardsResult.changes);
-        expect(summarySpy.mock.calls.map(([message]) => message)).toEqual([
-            'Total charts updated: 2 ',
-            'Total dashboards updated: 1 ',
-        ]);
-    });
-});
-describe('getDashboardChartSlugs', () => {
-    let tmpDir;
-    beforeEach(async () => {
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'download-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'dashboards'));
-    });
-    afterEach(async () => {
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('extracts chart slugs from folder dashboards', async () => {
-        await writeFolderDashboard(tmpDir, 'folder-dash', [
-            'chart-a',
-            'chart-b',
-        ]);
-        const slugs = await getDashboardChartSlugs([], tmpDir);
-        expect(slugs.sort()).toEqual(['chart-a', 'chart-b']);
-    });
-    it('extracts chart slugs from loose dashboards (PROD-7869 regression)', async () => {
-        const loose = makeLooseDashboard('loose-dash', [
-            'loose-chart-1',
-            'loose-chart-2',
-        ]);
-        const slugs = await getDashboardChartSlugs([], tmpDir, [loose]);
-        expect(slugs.sort()).toEqual(['loose-chart-1', 'loose-chart-2']);
-    });
-    it('extracts chart slugs from both folder and loose dashboards when unioned', async () => {
-        await writeFolderDashboard(tmpDir, 'folder-dash', ['folder-chart']);
-        const loose = makeLooseDashboard('loose-dash', ['loose-chart']);
-        const slugs = await getDashboardChartSlugs([], tmpDir, [loose]);
-        expect(slugs.sort()).toEqual(['folder-chart', 'loose-chart']);
-    });
-    it('filters by slug across both folder and loose sources', async () => {
-        await writeFolderDashboard(tmpDir, 'folder-dash', ['folder-chart']);
-        const loose = makeLooseDashboard('loose-dash', ['loose-chart']);
-        const onlyFolder = await getDashboardChartSlugs(['folder-dash'], tmpDir, [loose]);
-        expect(onlyFolder).toEqual(['folder-chart']);
-        const onlyLoose = await getDashboardChartSlugs(['loose-dash'], tmpDir, [
-            loose,
-        ]);
-        expect(onlyLoose).toEqual(['loose-chart']);
-    });
-    it('returns empty array when slug filter matches no dashboards', async () => {
-        await writeFolderDashboard(tmpDir, 'folder-dash', ['folder-chart']);
-        const loose = makeLooseDashboard('loose-dash', ['loose-chart']);
-        const slugs = await getDashboardChartSlugs(['nonexistent'], tmpDir, [
-            loose,
-        ]);
-        expect(slugs).toEqual([]);
-    });
-    it('skips tiles without a chartSlug (e.g. markdown, loom)', async () => {
-        const loose = makeLooseDashboard('loose-dash', [
-            'real-chart',
-            undefined,
-            undefined,
-        ]);
-        const slugs = await getDashboardChartSlugs([], tmpDir, [loose]);
-        expect(slugs).toEqual(['real-chart']);
-    });
-});
-describe('extractAppSlugsFromDashboards', () => {
-    const dashboard = (tiles) => ({ slug: 'd', tiles });
-    it('collects app slugs and ignores other tile types', () => {
-        expect(download_1.testHelpers.extractAppSlugsFromDashboards([
-            dashboard([
-                {
-                    type: 'data_app',
-                    properties: { appSlug: 'revenue-explorer' },
-                },
-                {
-                    type: 'saved_chart',
-                    properties: { chartSlug: 'a-chart' },
-                },
-                { type: 'markdown', properties: { content: 'hi' } },
-            ]),
-        ])).toEqual(['revenue-explorer']);
-    });
-    it('dedupes and drops null slugs', () => {
-        expect(download_1.testHelpers.extractAppSlugsFromDashboards([
-            dashboard([
-                { type: 'data_app', properties: { appSlug: 'one' } },
-                { type: 'data_app', properties: { appSlug: 'one' } },
-                { type: 'data_app', properties: { appSlug: null } },
-            ]),
-        ])).toEqual(['one']);
-    });
-});
-describe('extractChartTableNames', () => {
-    const chart = (tableName) => ({
-        slug: 'a-chart',
-        tableName,
-    });
-    it('dedupes table names across charts', () => {
-        expect(extractChartTableNames([
-            chart('orders'),
-            chart('customers'),
-            chart('orders'),
-        ])).toEqual(['orders', 'customers']);
-    });
-    it('drops missing table names (e.g. SQL charts)', () => {
-        expect(extractChartTableNames([chart(undefined), chart(''), chart('one')])).toEqual(['one']);
-    });
-});
-describe('extractChartTypeRefsFromCharts', () => {
-    const vizChart = (config) => ({
-        slug: 'a-chart',
-        chartConfig: { type: 'data_app_viz', config },
-    });
-    it('collects slugs from viz charts, deduped, ignoring other chart types', () => {
-        expect(extractChartTypeRefsFromCharts([
-            vizChart({ dataAppVizSlug: 'heatmap', fieldMapping: {} }),
-            vizChart({ dataAppVizSlug: 'heatmap', fieldMapping: {} }),
-            {
-                slug: 'bar-chart',
-                chartConfig: { type: 'cartesian', config: {} },
-            },
-        ])).toEqual(['heatmap']);
-    });
-    it('falls back to the legacy uuid and skips configless charts', () => {
-        expect(extractChartTypeRefsFromCharts([
-            vizChart({ dataAppVizUuid: 'viz-uuid', fieldMapping: {} }),
-            vizChart(undefined),
-        ])).toEqual(['viz-uuid']);
-    });
-});
-describe('selectVirtualViewCandidates', () => {
-    const chart = (slug, tableName) => ({
-        slug,
-        tableName,
-    });
-    it('collects table names of explicitly selected charts', () => {
-        expect(download_1.testHelpers.selectVirtualViewCandidates({
-            chartItems: [
-                chart('wanted', 'my_virtual_view'),
-                chart('other', 'other_view'),
-            ],
-            chartSlugs: ['wanted'],
-            dashboardItems: [],
-            dashboardSlugs: [],
-        })).toEqual(['my_virtual_view']);
-    });
-    it('collects table names of the selected dashboards charts only', () => {
-        expect(download_1.testHelpers.selectVirtualViewCandidates({
-            chartItems: [
-                chart('dash-chart', 'my_virtual_view'),
-                chart('other-dash-chart', 'other_view'),
-            ],
-            chartSlugs: [],
-            dashboardItems: [
-                makeLooseDashboard('wanted-dash', ['dash-chart']),
-                makeLooseDashboard('other-dash', ['other-dash-chart']),
-            ],
-            dashboardSlugs: ['wanted-dash'],
-        })).toEqual(['my_virtual_view']);
-    });
-    it('ignores dashboards entirely when none are selected', () => {
-        expect(download_1.testHelpers.selectVirtualViewCandidates({
-            chartItems: [chart('dash-chart', 'my_virtual_view')],
-            chartSlugs: [],
-            dashboardItems: [
-                makeLooseDashboard('some-dash', ['dash-chart']),
-            ],
-            dashboardSlugs: [],
-        })).toEqual([]);
-    });
-    it('dedupes across sources and drops charts without a table name', () => {
-        expect(download_1.testHelpers.selectVirtualViewCandidates({
-            chartItems: [
-                chart('a-chart', 'shared_view'),
-                chart('b-chart', 'shared_view'),
-                chart('sql-chart', undefined),
-            ],
-            chartSlugs: ['a-chart', 'sql-chart'],
-            dashboardItems: [makeLooseDashboard('dash', ['b-chart'])],
-            dashboardSlugs: ['dash'],
-        })).toEqual(['shared_view']);
-    });
-});
-describe('getDashboardAppSlugs', () => {
-    let tmpDir;
-    beforeEach(async () => {
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'download-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'dashboards'));
-    });
-    afterEach(async () => {
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('extracts app slugs from folder dashboards', async () => {
-        await writeFolderAppDashboard(tmpDir, 'folder-dash', [
-            'app-a',
-            'app-b',
-        ]);
-        const slugs = await getDashboardAppSlugs([], tmpDir);
-        expect(slugs.sort()).toEqual(['app-a', 'app-b']);
-    });
-    it('filters to the selected dashboards only', async () => {
-        await writeFolderAppDashboard(tmpDir, 'wanted', ['wanted-app']);
-        const loose = makeLooseAppDashboard('other', ['other-app']);
-        const slugs = await getDashboardAppSlugs(['wanted'], tmpDir, [loose]);
-        expect(slugs).toEqual(['wanted-app']);
-    });
-    it('dedupes across dashboards and drops null slugs', async () => {
-        await writeFolderAppDashboard(tmpDir, 'folder-dash', ['shared-app']);
-        const loose = makeLooseAppDashboard('loose-dash', ['shared-app', null]);
-        const slugs = await getDashboardAppSlugs([], tmpDir, [loose]);
-        expect(slugs).toEqual(['shared-app']);
-    });
-    it('returns an empty array when no dashboard has an app tile', async () => {
-        await writeFolderDashboard(tmpDir, 'chart-only', ['chart-a']);
-        const slugs = await getDashboardAppSlugs([], tmpDir);
-        expect(slugs).toEqual([]);
-    });
-});
-describe('isFilteredWithNoDashboards', () => {
-    it('is true for a filtered upload that selects no dashboards (e.g. --charts only)', () => {
-        expect(isFilteredWithNoDashboards(true, [])).toBe(true);
-    });
-    it('is false for an unfiltered upload even with no dashboard slugs', () => {
-        expect(isFilteredWithNoDashboards(false, [])).toBe(false);
-    });
-    it('is false once dashboard slugs are provided', () => {
-        expect(isFilteredWithNoDashboards(true, ['my-dashboard'])).toBe(false);
-    });
-});
-describe('sanitizeChartForDownload', () => {
-    it('preserves per-value pivot series customizations by default', () => {
-        const chart = makeChart([
-            pivotedSeries('completed', {
-                name: 'Completed orders',
-                color: '#1f77b4',
-                isFilteredOut: false,
-            }),
-            pivotedSeries('returned', {
-                name: 'Returned orders',
-                color: '#d62728',
-                isFilteredOut: true,
-            }),
-        ]);
-        const result = sanitizeChartForDownload(chart, false);
-        expect(result).toBe(chart);
-        const series = result.chartConfig.config
-            ?.eChartsConfig.series;
-        expect(series).toHaveLength(2);
-        expect(series?.[1]).toEqual(expect.objectContaining({
-            name: 'Returned orders',
-            color: '#d62728',
-            isFilteredOut: true,
-        }));
-        expect(series?.[1].encode.yRef.pivotValues).toEqual([
-            { field: 'orders_status', value: 'returned' },
-        ]);
-    });
-    it('strips and dedupes per-value pivot series when requested', () => {
-        const chart = makeChart([
-            pivotedSeries('completed', {
-                name: 'Completed orders',
-                color: '#1f77b4',
-            }),
-            pivotedSeries('returned', {
-                name: 'Returned orders',
-                color: '#d62728',
-            }),
-        ]);
-        const result = sanitizeChartForDownload(chart, true);
-        expect(result).not.toBe(chart);
-        const series = result.chartConfig.config
-            ?.eChartsConfig.series;
-        expect(series).toHaveLength(1);
-        expect(series?.[0].name).toBeUndefined();
-        expect(series?.[0].color).toBeUndefined();
-        expect(series?.[0].encode).toEqual({
-            xRef: { field: 'events_date_day' },
-            yRef: { field: 'orders_count' },
-        });
-    });
-});
-describe('downloadLinkedVirtualViews', () => {
-    let tmpDir;
-    const virtualViewDocument = (slug) => ({
-        contentType: common_1.ContentAsCodeType.VIRTUAL_VIEW,
-        version: 1,
-        slug,
-        name: slug,
-        sql: 'SELECT 1 AS order_id',
-        columns: [{ reference: 'order_id', type: 'number' }],
-        parameters: null,
-    });
-    beforeEach(async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'linked-virtual-views-test-'));
-    });
-    afterEach(async () => {
-        vitest_1.vi.restoreAllMocks();
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('writes matched virtual views and stays silent on regular dbt table names', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            virtualViews: [virtualViewDocument('my_virtual_view')],
-            skipped: [],
-            missingSlugs: ['orders', 'customers'],
-        });
-        const count = await downloadLinkedVirtualViews('project-uuid', ['my_virtual_view', 'orders', 'customers'], tmpDir);
-        expect(count).toBe(1);
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledExactlyOnceWith(expect.objectContaining({
-            url: '/api/v1/projects/project-uuid/code/virtualViews?slugs=my_virtual_view&slugs=orders&slugs=customers',
-        }));
-        await expect(fs_1.promises.readFile(path.join(tmpDir, 'virtual-views', 'my_virtual_view.yml'), 'utf-8')).resolves.toContain('slug: my_virtual_view');
-        expect(logSpy).not.toHaveBeenCalled();
-    });
-    it('warns about skipped virtual views', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            virtualViews: [],
-            skipped: [{ slug: 'broken_view', reason: 'malformed' }],
-            missingSlugs: [],
-        });
-        await expect(downloadLinkedVirtualViews('project-uuid', ['broken_view'], tmpDir)).resolves.toBe(0);
-        expect(logSpy).toHaveBeenCalledExactlyOnceWith(expect.stringContaining('Skipped virtual view "broken_view"'));
-    });
-    it('returns null when the server has no virtual views endpoint', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(new common_1.LightdashError({
-            message: 'Not found',
-            name: 'NotFoundError',
-            statusCode: 404,
-            data: {},
-        }));
-        await expect(downloadLinkedVirtualViews('project-uuid', ['orders'], tmpDir)).resolves.toBeNull();
-    });
-    it('rethrows unexpected errors', async () => {
-        const serverError = new common_1.LightdashError({
-            message: 'Boom',
-            name: 'UnexpectedServerError',
-            statusCode: 500,
-            data: {},
-        });
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(serverError);
-        await expect(downloadLinkedVirtualViews('project-uuid', ['orders'], tmpDir)).rejects.toBe(serverError);
-    });
-    it('classifies permission and missing-route errors as unavailable', () => {
-        [403, 404].forEach((statusCode) => {
-            expect(isVirtualViewsUnavailableError(new common_1.LightdashError({
-                message: 'unavailable',
-                name: 'TestError',
-                statusCode,
-                data: {},
-            }))).toBe(true);
-        });
-        expect(isVirtualViewsUnavailableError(new Error('nope'))).toBe(false);
-    });
-});
-describe('upsertVirtualViews', () => {
-    let tmpDir;
-    const virtualView = (slug) => `columns:
-  - name: order_id
-contentType: virtual_view
-name: ${slug}
-parameters: null
-slug: ${slug}
-sql: SELECT 1 AS order_id
-version: 1
-`;
-    beforeEach(async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'virtual-views-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'virtual-views'));
-    });
-    afterEach(async () => {
-        vitest_1.vi.restoreAllMocks();
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('continues uploading after one virtual view is denied', async () => {
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'a-denied.yml'), virtualView('a-denied'));
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'b-allowed.yml'), virtualView('b-allowed'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi)
-            .mockRejectedValueOnce(new Error('Forbidden'))
-            .mockResolvedValueOnce({ action: 'create' });
-        const changes = await upsertVirtualViews('project-uuid', [], {}, false, true, tmpDir);
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledTimes(2);
-        expect(changes).toEqual({
-            'virtual views with errors': 1,
-            'virtual views created': 1,
-        });
-    });
-    it('does not report missing permissions when there are no local virtual views', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        const changes = await upsertVirtualViews('project-uuid', [], {}, false, false, tmpDir);
-        expect(changes).toEqual({});
-        expect(apiClient_1.lightdashApi).not.toHaveBeenCalled();
-        expect(logSpy).not.toHaveBeenCalled();
-    });
-    it('reports one category error without uploading when local virtual views are forbidden', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'a-denied.yml'), virtualView('a-denied'));
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'b-denied.yml'), virtualView('b-denied'));
-        const changes = await upsertVirtualViews('project-uuid', [], {}, false, false, tmpDir);
-        expect(changes).toEqual({});
-        expect(apiClient_1.lightdashApi).not.toHaveBeenCalled();
-        expect(logSpy).toHaveBeenCalledExactlyOnceWith(expect.stringContaining('Error uploading virtual views'));
-    });
-    it('uploads candidate matches only, silent on unmatched candidates', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'my_view.yml'), virtualView('my_view'));
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'unrelated_view.yml'), virtualView('unrelated_view'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            action: 'create',
-        });
-        const changes = await upsertVirtualViews('project-uuid', [], {}, false, true, tmpDir, ['my_view', 'orders', 'customers']);
-        expect(changes).toEqual({ 'virtual views created': 1 });
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledExactlyOnceWith(expect.objectContaining({
-            url: expect.stringContaining('/code/virtualViews/my_view'),
-        }));
-        expect(logSpy).not.toHaveBeenCalled();
-    });
-    it('still warns about explicit slugs missing locally when candidates are present', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'virtual-views', 'my_view.yml'), virtualView('my_view'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            action: 'update',
-        });
-        const changes = await upsertVirtualViews('project-uuid', ['missing_view'], {}, false, true, tmpDir, ['my_view']);
-        expect(changes).toEqual({ 'virtual views updated': 1 });
-        expect(logSpy).toHaveBeenCalledExactlyOnceWith(expect.stringContaining('Virtual view "missing_view" was not found locally'));
-    });
-});
-describe('upsertExternalConnections', () => {
-    let tmpDir;
-    const externalConnectionYaml = (slug, extraLines = []) => [
-        'allowedContentTypes:',
-        '  - application/json',
-        'allowedMethods:',
-        '  - GET',
-        'allowedPathPrefixes: []',
-        'apiKeyLocation: header',
-        'apiKeyName: Authorization',
-        'authType: api_key',
-        'contentType: external_connection',
-        'customHeaders: null',
-        'instructions: null',
-        `name: ${slug}`,
-        'oauthScopes: null',
-        'origin: https://api.example.com',
-        'rateLimitPerMinute: null',
-        'requestMaxBytes: 262144',
-        'responseMaxBytes: 1048576',
-        `slug: ${slug}`,
-        'timeoutMs: 10000',
-        'version: 1',
-        ...extraLines,
-        '',
-    ].join('\n');
-    beforeEach(async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'external-connections-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'external-connections'));
-    });
-    afterEach(async () => {
-        vitest_1.vi.restoreAllMocks();
-        vitest_1.vi.unstubAllEnvs();
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('omits the secret from the request when the env var is unset', async () => {
-        await fs_1.promises.writeFile(path.join(tmpDir, 'external-connections', 'stripe-api.yml'), externalConnectionYaml('stripe-api'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            action: 'no changes',
-        });
-        const changes = await upsertExternalConnections('project-uuid', [], {}, false, true, tmpDir);
-        expect(changes).toEqual({ 'external connections skipped': 1 });
-        const request = vitest_1.vi.mocked(apiClient_1.lightdashApi).mock.calls[0][0];
-        expect(request.url).toContain('/code/externalConnections/stripe-api?force=false');
-        const body = JSON.parse(request.body);
-        expect(body).not.toHaveProperty('secret');
-        expect(body.connection.slug).toBe('stripe-api');
-    });
-    it('resolves the secret from LIGHTDASH_EXTERNAL_CONNECTION_SECRET_<SLUG>', async () => {
-        await fs_1.promises.writeFile(path.join(tmpDir, 'external-connections', 'stripe-api.yml'), externalConnectionYaml('stripe-api'));
-        vitest_1.vi.stubEnv('LIGHTDASH_EXTERNAL_CONNECTION_SECRET_STRIPE_API', 'sk-123');
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            action: 'create',
-        });
-        const changes = await upsertExternalConnections('project-uuid', [], {}, false, true, tmpDir);
-        expect(changes).toEqual({ 'external connections created': 1 });
-        const body = JSON.parse(vitest_1.vi.mocked(apiClient_1.lightdashApi).mock.calls[0][0].body);
-        expect(body.secret).toBe('sk-123');
-        // The secret rides next to the document, never inside it
-        expect(body.connection).not.toHaveProperty('secret');
-    });
-    it('strips a secret authored into the YAML and warns', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'external-connections', 'stripe-api.yml'), externalConnectionYaml('stripe-api', ['secret: leaked-secret']));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            action: 'update',
-        });
-        await upsertExternalConnections('project-uuid', [], {}, false, true, tmpDir);
-        const rawBody = vitest_1.vi.mocked(apiClient_1.lightdashApi).mock.calls[0][0].body;
-        expect(rawBody).not.toContain('leaked-secret');
-        expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('secrets must never be stored in YAML'));
-    });
-    it('reports one category error without uploading when the caller lacks permission', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'external-connections', 'stripe-api.yml'), externalConnectionYaml('stripe-api'));
-        const changes = await upsertExternalConnections('project-uuid', [], {}, false, false, tmpDir);
-        expect(changes).toEqual({});
-        expect(apiClient_1.lightdashApi).not.toHaveBeenCalled();
-        expect(logSpy).toHaveBeenCalledExactlyOnceWith(expect.stringContaining('Error uploading external connections'));
-    });
-    it('appends the env-var hint when a secret-requiring create fails without the env var', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'external-connections', 'stripe-api.yml'), externalConnectionYaml('stripe-api'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(new Error('type "api_key" requires a secret'));
-        const changes = await upsertExternalConnections('project-uuid', [], {}, false, true, tmpDir);
-        expect(changes).toEqual({ 'external connections with errors': 1 });
-        expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('Set LIGHTDASH_EXTERNAL_CONNECTION_SECRET_STRIPE_API'));
-    });
-});
-describe('getExternalConnectionSecretEnvVar', () => {
-    it('upper-snakes the slug', () => {
-        expect((0, download_1.getExternalConnectionSecretEnvVar)('stripe-api-v2')).toBe('LIGHTDASH_EXTERNAL_CONNECTION_SECRET_STRIPE_API_V2');
-    });
-});
-describe('isExternalConnectionsUnavailableError', () => {
-    const lightdashError = (statusCode) => new common_1.LightdashError({
-        message: 'nope',
-        name: 'TestError',
-        statusCode,
-        data: {},
-    });
-    it('classifies permission, missing-route and non-EE errors as unavailable', () => {
-        [403, 404, 422].forEach((statusCode) => {
-            expect(isExternalConnectionsUnavailableError(lightdashError(statusCode))).toBe(true);
-        });
-    });
-    it('keeps real failures fatal', () => {
-        expect(isExternalConnectionsUnavailableError(lightdashError(500))).toBe(false);
-        expect(isExternalConnectionsUnavailableError(new Error('network down'))).toBe(false);
-    });
-});
-describe('AI agent downloads', () => {
-    const unavailableError = new common_1.LightdashError({
-        message: "Unable to initialize service 'aiAgentCoderService' - no factory or provider.",
-        name: 'MissingConfigError',
-        statusCode: 422,
-        data: {},
-    });
-    beforeEach(() => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-    });
-    it('classifies permission, missing-route and non-EE errors as unavailable', () => {
-        [403, 404, 422].forEach((statusCode) => {
-            expect(isAiAgentsUnavailableError(new common_1.LightdashError({
-                message: 'AI agents unavailable',
-                name: 'TestError',
-                statusCode,
-                data: {},
-            }))).toBe(true);
-        });
-    });
-    it('skips unavailable AI agents when selected implicitly', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(unavailableError);
-        await expect(downloadAiAgents('project-uuid', [], true)).resolves.toBe(0);
-    });
-    it('keeps explicit AI agent downloads strict', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(unavailableError);
-        await expect(downloadAiAgents('project-uuid', [], false)).rejects.toBe(unavailableError);
-    });
-});
-describe('countChangeDelta', () => {
-    it('sums positive deltas across change keys', () => {
-        expect(countChangeDelta({ 'AI agents created': 1, 'charts skipped': 2 }, {
-            'AI agents created': 3,
-            'AI agents updated': 1,
-            'charts skipped': 2,
-        })).toBe(3);
-    });
-    it('returns 0 when nothing changed', () => {
-        expect(countChangeDelta({ 'charts created': 1 }, { 'charts created': 1 })).toBe(0);
-    });
-});
-describe('downloadHandler analytics', () => {
-    beforeEach(() => {
-        vitest_1.vi.mocked(analytics_1.LightdashAnalytics.track).mockClear();
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-    });
-    it('records per-type counts on download.completed, including agents', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'agents-download-analytics-'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockImplementation(async ({ url }) => {
-            if (url.startsWith('/api/v1/health')) {
-                return { version: '0.0.0' };
-            }
-            if (url === '/api/v1/projects/project-uuid') {
-                return { name: 'Test project' };
-            }
-            if (url.includes('/code/aiAgents')) {
-                return {
-                    agents: [
-                        {
-                            contentType: common_1.ContentAsCodeType.AI_AGENT,
-                            version: 1,
-                            agentVersion: 2,
-                            slug: 'sales-agent',
-                            name: 'Sales agent',
-                            description: null,
-                            imageUrl: null,
-                            instruction: null,
-                            tags: null,
-                            enableDataAccess: true,
-                            enableSelfImprovement: false,
-                            enableContentTools: false,
-                            enableUserContext: false,
-                            modelConfig: null,
-                        },
-                    ],
-                    missingIds: [],
-                    offset: 1,
-                    total: 1,
-                };
-            }
-            throw new Error(`Unexpected request: ${url}`);
-        });
-        try {
-            await (0, download_1.downloadHandler)(makeDownloadHandlerOptions({
-                includeAgents: true,
-                skipAlerts: true,
-                skipGoogleSheets: true,
-                skipScheduledDeliveries: true,
-                skipVirtualViews: true,
-                skipExternalConnections: true,
-                path: tmpDir,
-            }));
-            expect(analytics_1.LightdashAnalytics.track).toHaveBeenCalledWith(expect.objectContaining({
-                event: 'download.completed',
-                properties: expect.objectContaining({
-                    projectId: 'project-uuid',
-                    agentsNum: 1,
-                }),
-            }));
-            expect(analytics_1.LightdashAnalytics.track).toHaveBeenCalledWith(expect.objectContaining({
-                event: 'download.completed',
-                properties: expect.not.objectContaining({
-                    chartsNum: expect.anything(),
-                    dashboardsNum: expect.anything(),
-                }),
-            }));
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-});
-describe('downloadHandler failures', () => {
-    const unavailableError = new common_1.LightdashError({
-        message: "Unable to initialize service 'aiAgentCoderService' - no factory or provider.",
-        name: 'MissingConfigError',
-        statusCode: 422,
-        data: {},
-    });
-    beforeEach(() => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-    });
-    it('continues to scheduled content after implicitly skipping unavailable AI agents', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'include-all-download-test-'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockImplementation(async ({ url }) => {
-            if (url.startsWith('/api/v1/health')) {
-                return { version: '0.0.0' };
-            }
-            if (url === '/api/v1/projects/project-uuid') {
-                return { name: 'Test project' };
-            }
-            if (url.includes('/code/virtualViews')) {
-                return {
-                    virtualViews: [],
-                    skipped: [],
-                    missingSlugs: [],
-                };
-            }
-            if (url.includes('/code/aiAgents') ||
-                url.includes('/code/homepages')) {
-                throw unavailableError;
-            }
-            if (url.includes('/code/alerts')) {
-                return { alerts: [], skipped: [] };
-            }
-            if (url.includes('/code/scheduledDeliveries')) {
-                return { scheduledDeliveries: [], skipped: [] };
-            }
-            if (url.includes('/code/googleSheets')) {
-                return { googleSheetsSyncs: [], skipped: [] };
-            }
-            if (url.includes('/code/externalConnections')) {
-                throw unavailableError;
-            }
-            if (url.includes('/ee/projects/project-uuid/apps')) {
-                throw unavailableError;
-            }
-            throw new Error(`Unexpected request: ${url}`);
-        });
-        try {
-            await expect((0, download_1.downloadHandler)(makeDownloadHandlerOptions({
-                includeAll: true,
-                path: tmpDir,
-            }))).resolves.toBeUndefined();
-            expect(vitest_1.vi.mocked(apiClient_1.lightdashApi)).toHaveBeenCalledWith(expect.objectContaining({
-                url: expect.stringContaining('/code/scheduledDeliveries'),
-            }));
-            expect(analytics_1.LightdashAnalytics.track).toHaveBeenCalledWith(expect.objectContaining({
-                event: 'download.completed',
-                properties: expect.objectContaining({
-                    agentsNum: 0,
-                    alertsNum: 0,
-                    scheduledDeliveriesNum: 0,
-                    googleSheetsNum: 0,
-                    virtualViewsNum: 0,
-                    externalConnectionsNum: 0,
-                    appsNum: 0,
-                }),
-            }));
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('rethrows fatal project download errors', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockImplementation(async ({ url }) => {
-            if (url.startsWith('/api/v1/health')) {
-                return { version: '0.0.0' };
-            }
-            if (url === '/api/v1/projects/project-uuid') {
-                return { name: 'Test project' };
-            }
-            if (url.includes('/code/aiAgents')) {
-                throw unavailableError;
-            }
-            throw new Error(`Unexpected request: ${url}`);
-        });
-        await expect((0, download_1.downloadHandler)(makeDownloadHandlerOptions({
-            agents: ['sales-agent'],
-        }))).rejects.toBe(unavailableError);
-    });
-});
-describe('uploadHandler failures', () => {
-    beforeEach(() => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        vitest_1.vi.mocked(apiClient_1.getContentAsCodeUploadPermissions).mockReset();
-    });
-    it('rethrows fatal project upload errors', async () => {
-        const fatalError = new Error('Failed to load dbt credentials');
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValue({
-            version: '0.0.0',
-        });
-        vitest_1.vi.mocked(apiClient_1.getContentAsCodeUploadPermissions).mockRejectedValueOnce(fatalError);
-        await expect((0, download_1.uploadHandler)(makeDownloadHandlerOptions({}))).rejects.toBe(fatalError);
-    });
-});
-describe('readAiAgentFiles', () => {
-    let tmpDir;
-    beforeEach(async () => {
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'agent-code-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'ai-agents'));
-    });
-    afterEach(async () => {
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('rejects YAML files with an invalid AI agent content type', async () => {
-        await fs_1.promises.writeFile(path.join(tmpDir, 'ai-agents', 'invalid.yml'), 'contentType: ai_agnet\nslug: invalid-agent\n');
-        await expect(readAiAgentFiles(tmpDir)).rejects.toThrow('Invalid contentType in AI agent file');
-    });
-    it('reads evaluation suites nested in an AI agent file', async () => {
-        await fs_1.promises.writeFile(path.join(tmpDir, 'ai-agents', 'revenue-agent.yml'), [
-            'contentType: ai_agent',
-            'version: 1',
-            'slug: revenue-agent',
-            'evaluations:',
-            '  - title: Core regression suite',
-            '    prompts:',
-            '      - prompt: What was revenue last month?',
-            '        expectedResponse: Uses the certified revenue metric.',
-            '',
-        ].join('\n'));
-        const agents = await readAiAgentFiles(tmpDir);
-        expect(agents[0].evaluations).toEqual([
-            {
-                title: 'Core regression suite',
-                prompts: [
-                    {
-                        prompt: 'What was revenue last month?',
-                        expectedResponse: 'Uses the certified revenue metric.',
-                    },
-                ],
-            },
-        ]);
-    });
-});
-describe('upsertAiAgents', () => {
-    let tmpDir;
-    const unavailableError = new common_1.LightdashError({
-        message: "Unable to initialize service 'aiAgentCoderService' - no factory or provider.",
-        name: 'MissingConfigError',
-        statusCode: 422,
-        data: {},
-    });
-    beforeEach(async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-        vitest_1.vi.spyOn(globalState_1.default, 'log').mockImplementation(() => undefined);
-        tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'agent-upload-test-'));
-        await fs_1.promises.mkdir(path.join(tmpDir, 'ai-agents'));
-        await fs_1.promises.writeFile(path.join(tmpDir, 'ai-agents', 'revenue-agent.yml'), [
-            'contentType: ai_agent',
-            'version: 1',
-            'slug: revenue-agent',
-            '',
-        ].join('\n'));
-    });
-    afterEach(async () => {
-        vitest_1.vi.restoreAllMocks();
-        await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-    });
-    it('skips unavailable AI agents when their upload is implicit', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(unavailableError);
-        await expect(upsertAiAgents('project-uuid', [], {}, false, tmpDir, true)).resolves.toStrictEqual({});
-        expect(globalState_1.default.log).toHaveBeenCalledWith(expect.stringContaining('Skipping AI agents'));
-    });
-    it('fails when unavailable AI agents were selected explicitly', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(unavailableError);
-        await expect(upsertAiAgents('project-uuid', ['revenue-agent'], {}, false, tmpDir, false)).rejects.toBe(unavailableError);
-    });
-});
-describe('shouldDownloadAiAgents', () => {
-    it('does not download AI agents by default', () => {
-        expect(shouldDownloadAiAgents({
-            agents: [],
-            includeAgents: false,
-            includeAll: false,
-        })).toBe(false);
-    });
-    it.each([
-        { agents: [], includeAgents: true, includeAll: false },
-        { agents: ['sales-agent'], includeAgents: false, includeAll: false },
-        { agents: [], includeAgents: false, includeAll: true },
-    ])('downloads AI agents when explicitly selected', (options) => {
-        expect(shouldDownloadAiAgents(options)).toBe(true);
-    });
-    it('does not download AI agents in apps-only mode', () => {
-        expect(shouldDownloadAiAgents({
-            agents: ['sales-agent'],
-            includeAgents: true,
-            includeAll: true,
-            appsOnly: true,
-        })).toBe(false);
-    });
-});
-describe('space files', () => {
-    const accessSpace = (slug) => ({
-        contentType: 'space',
-        version: 1,
-        spaceName: slug,
-        slug,
-        access: {
-            inheritParentPermissions: false,
-            projectMemberAccessRole: null,
-            users: [],
-            groups: [],
-        },
-    });
-    beforeEach(() => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockReset();
-    });
-    const embeddedSpaceResponse = (spaceName) => ({
-        charts: [makeChart([])],
-        languageMap: undefined,
-        missingIds: [],
-        offset: 1,
-        spaces: [
-            {
-                contentType: common_1.ContentAsCodeType.SPACE,
-                spaceName,
-                slug: 'test-space',
-            },
-        ],
-        total: 1,
-    });
-    it('writes new flat space files under spaces by default', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-flat-folder-'));
-        try {
-            await writeSpaceFiles([accessSpace('finance')], 'project', tmpDir, 'flat');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'spaces', 'finance.space.yml'), 'utf-8')).resolves.toContain('slug: finance');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'finance.space.yml'))).rejects.toMatchObject({ code: 'ENOENT' });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('supports the legacy root-spaces flat layout', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-root-layout-'));
-        try {
-            await writeSpaceFiles([accessSpace('finance')], 'project', tmpDir, 'flat', 'root');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'finance.space.yml'), 'utf-8')).resolves.toContain('slug: finance');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces'))).rejects.toMatchObject({ code: 'ENOENT' });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('preserves existing access when filtered metadata updates a space file', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-filtered-policy-'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValue(embeddedSpaceResponse('Updated finance'));
-        const existingFile = path.join(tmpDir, 'finance.space.yml');
-        await fs_1.promises.writeFile(existingFile, `access:
-  groups: []
-  inheritParentPermissions: false
-  projectMemberAccessRole: null
-  users:
-    - email: owner@example.com
-      role: admin
-contentType: space
-slug: test-space
-spaceName: Finance
-version: 1
-`);
-        try {
-            const onProgress = vitest_1.vi.fn();
-            await (0, download_1.downloadContent)(['pivoted-chart'], 'charts', 'project-uuid', 'project', tmpDir, false, false, false, false, false, onProgress);
-            expect(onProgress).toHaveBeenCalledWith('1 of 1 charts downloaded');
-            await expect(readSpaceFiles(tmpDir)).resolves.toEqual([
-                expect.objectContaining({
-                    filePath: existingFile,
-                    space: {
-                        contentType: 'space',
-                        version: 1,
-                        spaceName: 'Updated finance',
-                        slug: 'test-space',
-                        access: {
-                            inheritParentPermissions: false,
-                            projectMemberAccessRole: null,
-                            users: [
-                                {
-                                    email: 'owner@example.com',
-                                    role: common_1.SpaceMemberRole.ADMIN,
-                                },
-                            ],
-                            groups: [],
-                        },
-                    },
-                }),
-            ]);
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces', 'updated-finance.space.yml'))).rejects.toMatchObject({ code: 'ENOENT' });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('writes nested definitions inside each full space path', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-nested-layout-'));
-        try {
-            await writeSpaceFiles([
-                { ...accessSpace('parent'), spaceName: 'Parent' },
-                {
-                    ...accessSpace('parent/child'),
-                    spaceName: 'Child',
-                },
-            ], 'project', tmpDir, 'nested');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'project', 'parent', 'parent.space.yml'), 'utf-8')).resolves.toContain('slug: parent');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'project', 'parent', 'child', 'child.space.yml'), 'utf-8')).resolves.toContain('slug: parent/child');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces'))).rejects.toMatchObject({ code: 'ENOENT' });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('downloads metadata-only spaces when access cannot be exported', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-metadata-fallback-'));
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        await fs_1.promises.writeFile(path.join(tmpDir, 'finance.space.yml'), `access:
-  groups: []
-  inheritParentPermissions: false
-  projectMemberAccessRole: null
-  users: []
-contentType: space
-slug: finance
-spaceName: Finance
-version: 1
-`);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValue({
-            spaces: [
-                {
-                    contentType: common_1.ContentAsCodeType.SPACE,
-                    spaceName: 'Finance',
-                    slug: 'finance',
-                },
-            ],
-            skipped: [
-                {
-                    slug: 'finance',
-                    reason: 'Direct access contains a user without a portable organization identity',
-                },
-            ],
-        });
-        try {
-            await expect(downloadSpaces('project-uuid', 'Project', tmpDir)).resolves.toBe(1);
-            await expect(readSpaceFiles(tmpDir)).resolves.toEqual([
-                expect.objectContaining({
-                    space: {
-                        contentType: 'space',
-                        spaceName: 'Finance',
-                        slug: 'finance',
-                    },
-                }),
-            ]);
-            const downloadedFile = await fs_1.promises.readFile(path.join(tmpDir, 'finance.space.yml'), 'utf-8');
-            expect(downloadedFile).not.toContain('access:');
-            expect(downloadedFile).not.toContain('version:');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces', 'finance.space.yml'))).rejects.toMatchObject({ code: 'ENOENT' });
-            expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('Downloaded space "finance" without access'));
-            expect(logSpy).not.toHaveBeenCalledWith(expect.stringContaining('Skipped space "finance"'));
-        }
-        finally {
-            logSpy.mockRestore();
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('accepts legacy metadata-only files without a version', () => {
-        expect(validateSpaceIdentity({
-            contentType: 'space',
-            spaceName: 'Finance',
-            slug: 'company/finance',
-        }, 'finance.space.yml')).toEqual({
-            contentType: 'space',
-            spaceName: 'Finance',
-            slug: 'company/finance',
-        });
-    });
-    it('reads legacy space names without validating access policy', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-name-test-'));
-        try {
-            await fs_1.promises.writeFile(path.join(tmpDir, 'finance.space.yml'), `access:
-  unsupported: true
-contentType: space
-spaceName: Finance operations
-slug: finance
-version: 99
-`);
-            await fs_1.promises.writeFile(path.join(tmpDir, 'invalid.space.yml'), 'contentType: [invalid\n');
-            await expect(readSpaceFiles(tmpDir)).rejects.toThrow('Invalid version');
-            await expect(readSpaceNames(tmpDir)).resolves.toEqual({
-                finance: 'Finance operations',
-            });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('requires version 1 when access is present', () => {
-        expect(() => validateSpaceIdentity({
-            contentType: 'space',
-            spaceName: 'Finance',
-            slug: 'finance',
-            access: {
-                inheritParentPermissions: false,
-                projectMemberAccessRole: null,
-                users: [],
-                groups: [],
-            },
-        }, 'finance.space.yml')).toThrow('version 1 is required when access is present');
-    });
-    it('normalizes emails and rejects duplicate principals', () => {
-        const parsed = validateSpaceIdentity({
-            contentType: 'space',
-            version: 1,
-            spaceName: 'Finance',
-            slug: 'finance',
-            access: {
-                inheritParentPermissions: false,
-                projectMemberAccessRole: common_1.SpaceMemberRole.VIEWER,
-                users: [
-                    {
-                        email: ' Alice@Example.com ',
-                        role: common_1.SpaceMemberRole.ADMIN,
-                    },
-                ],
-                groups: [
-                    {
-                        name: 'Finance team',
-                        role: common_1.SpaceMemberRole.EDITOR,
-                    },
-                ],
-            },
-        }, 'finance.space.yml');
-        expect(parsed.access?.users).toEqual([
-            { email: 'alice@example.com', role: common_1.SpaceMemberRole.ADMIN },
-        ]);
-        expect(() => validateSpaceIdentity({
-            ...parsed,
-            access: {
-                ...parsed.access,
-                users: [
-                    {
-                        email: 'alice@example.com',
-                        role: common_1.SpaceMemberRole.ADMIN,
-                    },
-                    {
-                        email: 'ALICE@example.com',
-                        role: common_1.SpaceMemberRole.VIEWER,
-                    },
-                ],
-            },
-        }, 'finance.space.yml')).toThrow('Duplicate user email "alice@example.com"');
-        expect(() => validateSpaceIdentity({
-            ...parsed,
-            access: {
-                ...parsed.access,
-                groups: [
-                    {
-                        name: 'Finance team',
-                        role: common_1.SpaceMemberRole.EDITOR,
-                    },
-                    {
-                        name: 'Finance team',
-                        role: common_1.SpaceMemberRole.VIEWER,
-                    },
-                ],
-            },
-        }, 'finance.space.yml')).toThrow('Duplicate group name "Finance team"');
-    });
-    it('reads nested definitions parent-first', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-test-'));
-        try {
-            await fs_1.promises.mkdir(path.join(tmpDir, 'nested'));
-            await fs_1.promises.writeFile(path.join(tmpDir, 'nested', 'child.space.yml'), `contentType: space\nspaceName: Child\nslug: parent/child\n`);
-            await fs_1.promises.writeFile(path.join(tmpDir, 'parent.space.yml'), `contentType: space\nspaceName: Parent\nslug: parent\n`);
-            await expect(readSpaceFiles(tmpDir)).resolves.toEqual([
-                expect.objectContaining({
-                    space: expect.objectContaining({ slug: 'parent' }),
-                }),
-                expect.objectContaining({
-                    space: expect.objectContaining({ slug: 'parent/child' }),
-                }),
-            ]);
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('rejects non-canonical and colliding normalized paths', () => {
-        expect(() => validateSpaceIdentity({
-            contentType: 'space',
-            spaceName: 'Finance',
-            slug: 'Parent_With.Space',
-        }, 'finance.space.yml')).toThrow('canonical lowercase slash-separated path');
-        expect(() => assertUniqueSpacePaths([
-            {
-                filePath: 'one.space.yml',
-                space: accessSpace('parent-child'),
-            },
-            {
-                filePath: 'two.space.yml',
-                space: {
-                    ...accessSpace('parent-child'),
-                    slug: 'parent_child',
-                },
-            },
-        ])).toThrow('resolve to the same normalized path');
-    });
-    it('uses deterministic collision-safe flat filenames', () => {
-        const spaces = [
-            { ...accessSpace('finance-a'), spaceName: 'Finance!' },
-            { ...accessSpace('finance-b'), spaceName: 'Finance?' },
-        ];
-        const first = getFlatSpaceFileNames(spaces);
-        expect(first).toEqual(getFlatSpaceFileNames([...spaces].reverse()).reverse());
-        expect(new Set(first).size).toBe(2);
-        expect(first.every((fileName) => /^finance-[a-f0-9]{8}\.space\.yml$/.test(fileName))).toBe(true);
-    });
-    it('never overwrites an existing filename owned by another slug', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-protected-file-'));
-        const protectedContent = 'contentType: space\nspaceName: Existing\nslug: existing\n';
-        try {
-            await fs_1.promises.writeFile(path.join(tmpDir, 'finance.space.yml'), protectedContent);
-            await writeSpaceFiles([
-                {
-                    ...accessSpace('new-finance'),
-                    spaceName: 'Finance',
-                },
-            ], 'project', tmpDir, 'flat');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'finance.space.yml'), 'utf-8')).resolves.toBe(protectedContent);
-            const newSpaceFile = (await fs_1.promises.readdir(tmpDir)).find((fileName) => fileName !== 'finance.space.yml' &&
-                fileName.endsWith('.space.yml'));
-            expect(newSpaceFile).toBeDefined();
-            expect(await fs_1.promises.readFile(path.join(tmpDir, newSpaceFile), 'utf-8')).toContain('slug: new-finance');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces'))).rejects.toMatchObject({ code: 'ENOENT' });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('uses the legacy root layout for new slugs when a root space file exists', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-mixed-layout-'));
-        try {
-            await fs_1.promises.writeFile(path.join(tmpDir, 'finance.space.yml'), 'contentType: space\nspaceName: Old finance\nslug: finance\n');
-            await writeSpaceFiles([
-                { ...accessSpace('finance'), spaceName: 'New finance' },
-                { ...accessSpace('operations'), spaceName: 'Operations' },
-            ], 'project', tmpDir, 'flat');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'finance.space.yml'), 'utf-8')).resolves.toContain('spaceName: New finance');
-            await expect(fs_1.promises.readFile(path.join(tmpDir, 'operations.space.yml'), 'utf-8')).resolves.toContain('slug: operations');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces'))).rejects.toMatchObject({ code: 'ENOENT' });
-            expect((await readSpaceFiles(tmpDir)).map(({ space }) => space.slug)).toEqual(['finance', 'operations']);
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('rejects duplicate existing slugs across layouts before writing', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-duplicate-layout-'));
-        try {
-            await fs_1.promises.mkdir(path.join(tmpDir, 'spaces'));
-            const duplicate = 'contentType: space\nspaceName: Finance\nslug: finance\n';
-            await fs_1.promises.writeFile(path.join(tmpDir, 'finance.space.yml'), duplicate);
-            await fs_1.promises.writeFile(path.join(tmpDir, 'spaces', 'finance.space.yml'), duplicate);
-            await expect(writeSpaceFiles([accessSpace('operations')], 'project', tmpDir, 'flat')).rejects.toThrow('Multiple existing space files use slug "finance"');
-            await expect(fs_1.promises.access(path.join(tmpDir, 'spaces', 'operations.space.yml'))).rejects.toMatchObject({ code: 'ENOENT' });
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('fails a space download when the space endpoint is unavailable', async () => {
-        const tmpDir = await fs_1.promises.mkdtemp(path.join(os.tmpdir(), 'space-code-api-failure-'));
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValue(new Error('space endpoint unavailable'));
-        try {
-            const error = await downloadSpaces('project-uuid', 'Project', tmpDir).catch((caughtError) => caughtError);
-            expect(error).toMatchObject({
-                name: 'SpaceAsCodeFetchError',
-                message: 'space endpoint unavailable',
-            });
-            expect(shouldFallBackToEmbeddedSpaces(error, false)).toBe(true);
-            expect(shouldFallBackToEmbeddedSpaces(error, true)).toBe(false);
-        }
-        finally {
-            await fs_1.promises.rm(tmpDir, { recursive: true, force: true });
-        }
-    });
-    it('uploads parent-first and aggregates API actions', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi)
-            .mockResolvedValueOnce({
-            action: common_1.SpaceAsCodeAction.CREATE,
-            warnings: ['A direct service-account grant will be removed'],
-        })
-            .mockResolvedValueOnce({
-            action: common_1.SpaceAsCodeAction.UPDATE,
-        });
-        const changes = await upsertSpaces('project-uuid', [
-            { filePath: 'child.space.yml', space: accessSpace('a/b') },
-            { filePath: 'parent.space.yml', space: accessSpace('a') },
-        ], {}, false, false);
-        expect(changes).toEqual({
-            'spaces created': 1,
-            'spaces updated': 1,
-        });
-        expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('A direct service-account grant will be removed'));
-        expect(vitest_1.vi
-            .mocked(apiClient_1.lightdashApi)
-            .mock.calls.map(([request]) => JSON.parse(typeof request.body === 'string'
-            ? request.body
-            : '{}').slug)).toEqual(['a', 'a/b']);
-        logSpy.mockRestore();
-    });
-    it('omits access and warns when skip-space-access is enabled', async () => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockResolvedValueOnce({
-            action: common_1.SpaceAsCodeAction.CREATE,
-        });
-        await upsertSpaces('project-uuid', [{ filePath: 'space.space.yml', space: accessSpace('finance') }], {}, false, true, true);
-        const [request] = vitest_1.vi.mocked(apiClient_1.lightdashApi).mock.calls[0];
-        expect(JSON.parse(String(request.body))).toEqual({
-            contentType: 'space',
-            version: 1,
-            spaceName: 'finance',
-            slug: 'finance',
-        });
-        expect(request.url).toContain('publicSpaceCreate=true');
-        expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('Existing destination access will be preserved'));
-        logSpy.mockRestore();
-    });
-    it.each([
-        'User example@lightdash.com is not a member of this organization',
-        'Group finance does not exist in this organization',
-    ])('suggests skip-space-access when a destination identity is missing: %s', async (errorMessage) => {
-        const logSpy = vitest_1.vi
-            .spyOn(globalState_1.default, 'log')
-            .mockImplementation(() => undefined);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi).mockRejectedValueOnce(new Error(errorMessage));
-        await expect(upsertSpaces('project-uuid', [
-            {
-                filePath: 'space.space.yml',
-                space: accessSpace('finance'),
-            },
-        ], {}, false, false)).rejects.toThrow('content upload was not started');
-        expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('Hint: use --skip-space-access to upload the space without applying its access policy.'));
-        logSpy.mockRestore();
-    });
-    it('treats missing spaces and descendants as nonfatal with skip-space-create', async () => {
-        vitest_1.vi.mocked(apiClient_1.lightdashApi)
-            .mockRejectedValueOnce(new common_1.LightdashError({
-            message: 'Space a does not exist, skipping creation',
-            name: 'NotFoundError',
-            statusCode: 404,
-            data: {},
-        }))
-            .mockResolvedValueOnce({
-            action: common_1.SpaceAsCodeAction.NO_CHANGES,
-        });
-        await expect(upsertSpaces('project-uuid', [
-            { filePath: 'child.space.yml', space: accessSpace('a/b') },
-            { filePath: 'other.space.yml', space: accessSpace('z') },
-            { filePath: 'parent.space.yml', space: accessSpace('a') },
-        ], {}, true, false)).resolves.toEqual({
-            'spaces skipped': 2,
-            'spaces unchanged': 1,
-        });
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledTimes(2);
-        expect(vitest_1.vi
-            .mocked(apiClient_1.lightdashApi)
-            .mock.calls.map(([request]) => JSON.parse(typeof request.body === 'string'
-            ? request.body
-            : '{}').slug)).toEqual(['a', 'z']);
-    });
-    it('skips descendants after a parent failure and blocks content', async () => {
-        const summarySpy = vitest_1.vi
-            .spyOn(console, 'info')
-            .mockImplementation(() => undefined);
-        vitest_1.vi.mocked(apiClient_1.lightdashApi)
-            .mockRejectedValueOnce(new Error('parent rejected'))
-            .mockResolvedValueOnce({
-            action: common_1.SpaceAsCodeAction.CREATE,
-        });
-        const changes = {};
-        await expect(upsertSpaces('project-uuid', [
-            { filePath: 'child.space.yml', space: accessSpace('a/b') },
-            { filePath: 'other.space.yml', space: accessSpace('z') },
-            { filePath: 'parent.space.yml', space: accessSpace('a') },
-        ], changes, false, false)).rejects.toThrow('content upload was not started');
-        expect(apiClient_1.lightdashApi).toHaveBeenCalledTimes(2);
-        expect(changes).toEqual({
-            'spaces with errors': 1,
-            'spaces created': 1,
-            'spaces dependency skipped': 1,
-        });
-        expect(summarySpy.mock.calls.map(([message]) => message)).toEqual([
-            'Total spaces with errors: 1 ',
-            'Total spaces created: 1 ',
-            'Total spaces dependency skipped: 1 ',
-        ]);
-        summarySpy.mockRestore();
-    });
-});
-describe('parseContentFilters', () => {
-    const { parseContentFilters } = download_1.testHelpers;
-    it('passes slugs and uuids through untouched', () => {
-        expect(parseContentFilters([
-            'sales-overview',
-            '00000000-0000-0000-0000-000000000001',
-        ])).toBe('?ids=sales-overview&ids=00000000-0000-0000-0000-000000000001');
-    });
-    it('extracts uuids from legacy app urls', () => {
-        expect(parseContentFilters([
-            'https://app.lightdash.cloud/projects/00000000-0000-0000-0000-0000000000aa/dashboards/00000000-0000-0000-0000-000000000001/view',
-            'https://app.lightdash.cloud/projects/00000000-0000-0000-0000-0000000000aa/saved/00000000-0000-0000-0000-000000000002',
-        ])).toBe('?ids=00000000-0000-0000-0000-000000000001&ids=00000000-0000-0000-0000-000000000002');
-    });
-    it('extracts slugs from app urls that use project and content slugs', () => {
-        expect(parseContentFilters([
-            'https://app.lightdash.cloud/projects/jaffle-shop/dashboards/daily-sales/view',
-            'https://app.lightdash.cloud/projects/jaffle-shop/saved/monthly-revenue/view?tab=1',
-            'https://app.lightdash.cloud/projects/jaffle-shop/dashboards/sales-overview/view/tabs/some-tab',
-        ])).toBe('?ids=daily-sales&ids=monthly-revenue&ids=sales-overview');
-    });
-});