@lightdash/cli 1.137.0 → 1.138.0

package/dist/handlers/contentAsCode/registry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../../src/handlers/contentAsCode/registry.ts"],"names":[],"mappings":"AAAA,OAAO,EAGH,KAAK,yBAAyB,EACjC,MAAM,mBAAmB,CAAC;AAa3B,MAAM,MAAM,wBAAwB,GAAG;IACnC,IAAI,EAAE,yBAAyB,CAAC;IAChC,YAAY,EAAE,SAAS,yBAAyB,EAAE,CAAC;CACtD,CAAC;AAEF,eAAO,MAAM,2BAA2B,EAAE,SAAS,wBAAwB,EACH,CAAC;AAEzE,eAAO,MAAM,sBAAsB,EAAE,SAAS,wBAAwB,EAwBrE,CAAC;AAEF,eAAO,MAAM,iCAAiC,cAC/B,SAAS,wBAAwB,EAAE,KAC/C,IAeF,CAAC"}
+{"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../../src/handlers/contentAsCode/registry.ts"],"names":[],"mappings":"AAAA,OAAO,EAGH,KAAK,yBAAyB,EACjC,MAAM,mBAAmB,CAAC;AAc3B,MAAM,MAAM,wBAAwB,GAAG;IACnC,IAAI,EAAE,yBAAyB,CAAC;IAChC,YAAY,EAAE,SAAS,yBAAyB,EAAE,CAAC;CACtD,CAAC;AAEF,eAAO,MAAM,2BAA2B,EAAE,SAAS,wBAAwB,EAMtE,CAAC;AAEN,eAAO,MAAM,sBAAsB,EAAE,SAAS,wBAAwB,EAwBrE,CAAC;AAEF,eAAO,MAAM,iCAAiC,cAC/B,SAAS,wBAAwB,EAAE,KAC/C,IAeF,CAAC"}

package/dist/handlers/contentAsCode/registry.js

@@ -4,9 +4,15 @@ exports.assertCodeResourceDependencyOrder = exports.PROJECT_CODE_RESOURCES = exp
 const common_1 = require("@lightdash/common");
 const customRoles_1 = require("../organizationContent/customRoles");
 const groups_1 = require("../organizationContent/groups");
+const themes_1 = require("../organizationContent/themes");
 const users_1 = require("../organizationContent/users");
 const projectResources_1 = require("./projectResources");
-exports.ORGANIZATION_CODE_RESOURCES = [customRoles_1.CUSTOM_ROLE_CODE_RESOURCE, users_1.USER_CODE_RESOURCE, groups_1.GROUP_CODE_RESOURCE];
+exports.ORGANIZATION_CODE_RESOURCES = [
+    customRoles_1.CUSTOM_ROLE_CODE_RESOURCE,
+    users_1.USER_CODE_RESOURCE,
+    groups_1.GROUP_CODE_RESOURCE,
+    themes_1.THEME_CODE_RESOURCE,
+];
 exports.PROJECT_CODE_RESOURCES = [
     { kind: common_1.ContentAsCodeType.SPACE, dependencies: [] },
     projectResources_1.VIRTUAL_VIEW_CODE_RESOURCE,

package/dist/handlers/dbt/apiClient.d.ts

@@ -1,11 +1,15 @@
 import { ApiResponse, ProjectType, type LightdashUserWithAbilityRules } from '@lightdash/common';
-import { BodyInit } from 'node-fetch';
+import { BodyInit, type Response } from 'node-fetch';
 type LightdashApiProps = {
     method: 'GET' | 'POST' | 'PATCH' | 'DELETE' | 'PUT';
     url: string;
     body: BodyInit | undefined;
 };
+type LightdashRawApiProps = LightdashApiProps & {
+    headers?: Record<string, string>;
+};
 export declare const setGzipEnabled: (enabled: boolean) => void;
+export declare const lightdashRawApi: ({ method, url, body, headers: requestHeaders, }: LightdashRawApiProps) => Promise<Response>;
 export declare const lightdashApi: <T extends ApiResponse['results']>({ method, url, body, }: LightdashApiProps) => Promise<T>;
 export declare const getUserContext: () => Promise<LightdashUserWithAbilityRules>;
 export type ContentAsCodeUploadPermissions = {

package/dist/handlers/dbt/apiClient.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"apiClient.d.ts","sourceRoot":"","sources":["../../../src/handlers/dbt/apiClient.ts"],"names":[],"mappings":"AACA,OAAO,EAGH,WAAW,EAIX,WAAW,EAEX,KAAK,6BAA6B,EAGrC,MAAM,mBAAmB,CAAC;AAC3B,OAAc,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAS7C,KAAK,iBAAiB,GAAG;IACrB,MAAM,EAAE,KAAK,GAAG,MAAM,GAAG,OAAO,GAAG,QAAQ,GAAG,KAAK,CAAC;IACpD,GAAG,EAAE,MAAM,CAAC;IACZ,IAAI,EAAE,QAAQ,GAAG,SAAS,CAAC;CAC9B,CAAC;AAKF,eAAO,MAAM,cAAc,YAAa,OAAO,SAE9C,CAAC;AAEF,eAAO,MAAM,YAAY,GAAU,CAAC,SAAS,WAAW,CAAC,SAAS,CAAC,0BAIhE,iBAAiB,KAAG,OAAO,CAAC,CAAC,CAqE/B,CAAC;AAEF,eAAO,MAAM,cAAc,QACb,OAAO,CAAC,6BAA6B,CAKzC,CAAC;AAEX,MAAM,MAAM,8BAA8B,GAAG;IACzC,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,MAAM,EAAE,OAAO,CAAC;IAChB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,OAAO,CAAC;IAClB,mBAAmB,EAAE,OAAO,CAAC;CAChC,CAAC;AAEF,eAAO,MAAM,iCAAiC,gBAC7B,MAAM,KACpB,OAAO,CAAC,8BAA8B,CA0FxC,CAAC;AAEF,eAAO,MAAM,8BAA8B,wBAClB,MAAM,GAAG,SAAS,eAC1B,WAAW,KACzB,OAAO,CAAC,IAAI,CA0Gd,CAAC;AAEF,eAAO,MAAM,qBAAqB,QAAa,OAAO,CAAC,IAAI,CA2B1D,CAAC"}
+{"version":3,"file":"apiClient.d.ts","sourceRoot":"","sources":["../../../src/handlers/dbt/apiClient.ts"],"names":[],"mappings":"AACA,OAAO,EAGH,WAAW,EAIX,WAAW,EAEX,KAAK,6BAA6B,EAGrC,MAAM,mBAAmB,CAAC;AAC3B,OAAc,EAAE,QAAQ,EAAE,KAAK,QAAQ,EAAE,MAAM,YAAY,CAAC;AAS5D,KAAK,iBAAiB,GAAG;IACrB,MAAM,EAAE,KAAK,GAAG,MAAM,GAAG,OAAO,GAAG,QAAQ,GAAG,KAAK,CAAC;IACpD,GAAG,EAAE,MAAM,CAAC;IACZ,IAAI,EAAE,QAAQ,GAAG,SAAS,CAAC;CAC9B,CAAC;AAEF,KAAK,oBAAoB,GAAG,iBAAiB,GAAG;IAC5C,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CACpC,CAAC;AAKF,eAAO,MAAM,cAAc,YAAa,OAAO,SAE9C,CAAC;AAEF,eAAO,MAAM,eAAe,oDAKzB,oBAAoB,KAAG,OAAO,CAAC,QAAQ,CA8BzC,CAAC;AAEF,eAAO,MAAM,YAAY,GAAU,CAAC,SAAS,WAAW,CAAC,SAAS,CAAC,0BAIhE,iBAAiB,KAAG,OAAO,CAAC,CAAC,CAgD/B,CAAC;AAEF,eAAO,MAAM,cAAc,QACb,OAAO,CAAC,6BAA6B,CAKzC,CAAC;AAEX,MAAM,MAAM,8BAA8B,GAAG;IACzC,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,MAAM,EAAE,OAAO,CAAC;IAChB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,OAAO,CAAC;IAClB,mBAAmB,EAAE,OAAO,CAAC;CAChC,CAAC;AAEF,eAAO,MAAM,iCAAiC,gBAC7B,MAAM,KACpB,OAAO,CAAC,8BAA8B,CA0FxC,CAAC;AAEF,eAAO,MAAM,8BAA8B,wBAClB,MAAM,GAAG,SAAS,eAC1B,WAAW,KACzB,OAAO,CAAC,IAAI,CA0Gd,CAAC;AAEF,eAAO,MAAM,qBAAqB,QAAa,OAAO,CAAC,IAAI,CA2B1D,CAAC"}

package/dist/handlers/dbt/apiClient.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.checkLightdashVersion = exports.checkProjectCreationPermission = exports.getContentAsCodeUploadPermissions = exports.getUserContext = exports.lightdashApi = exports.setGzipEnabled = void 0;
+exports.checkLightdashVersion = exports.checkProjectCreationPermission = exports.getContentAsCodeUploadPermissions = exports.getUserContext = exports.lightdashApi = exports.lightdashRawApi = exports.setGzipEnabled = void 0;
 const tslib_1 = require("tslib");
 const ability_1 = require("@casl/ability");
 const common_1 = require("@lightdash/common");
@@ -18,14 +18,33 @@ const setGzipEnabled = (enabled) => {
     gzipEnabled = enabled;
 };
 exports.setGzipEnabled = setGzipEnabled;
-const lightdashApi = async ({ method, url, body, }) => {
+const lightdashRawApi = async ({ method, url, body, headers: requestHeaders = {}, }) => {
     const config = await (0, config_1.getConfig)();
     if (!(config.context?.apiKey && config.context.serverUrl)) {
         throw new common_1.AuthorizationError(`Not logged in. Run 'lightdash login --help'`);
     }
-    const headers = (0, utils_1.buildRequestHeaders)(config.context.apiKey);
+    const headers = {
+        ...(0, utils_1.buildRequestHeaders)(config.context.apiKey),
+        ...requestHeaders,
+    };
     const fullUrl = new url_1.URL(url, config.context.serverUrl).href;
     globalState_1.default.debug(`> Making HTTP ${method} request to: ${fullUrl}`);
+    const response = await (0, node_fetch_1.default)(fullUrl, { method, headers, body });
+    globalState_1.default.debug(`> HTTP request returned status: ${response.status}`);
+    if (!response.ok) {
+        const contentType = response.headers.get('content-type');
+        if (contentType?.includes('application/json')) {
+            const data = await response.json();
+            throw new common_1.LightdashError(data.error);
+        }
+        const responseText = await response.text();
+        throw new Error(`Received non-JSON response from server (status ${response.status}): ${responseText}`);
+    }
+    return response;
+};
+exports.lightdashRawApi = lightdashRawApi;
+const lightdashApi = async ({ method, url, body, }) => {
+    const headers = {};
     let requestBody = body;
     if (gzipEnabled &&
         body &&
@@ -42,21 +61,11 @@ const lightdashApi = async ({ method, url, body, }) => {
             globalState_1.default.debug(`> Gzip compression failed, sending uncompressed: ${err}`);
         }
     }
-    return (0, node_fetch_1.default)(fullUrl, { method, headers, body: requestBody })
-        .then((r) => {
-        globalState_1.default.debug(`> HTTP request returned status: ${r.status}`);
-        if (!r.ok) {
-            const contentType = r.headers.get('content-type');
-            if (contentType && contentType.includes('application/json')) {
-                return r.json().then((d) => {
-                    throw new common_1.LightdashError(d.error);
-                });
-            }
-            return r.text().then((text) => {
-                throw new Error(`Received non-JSON response from server (status ${r.status}): ${text}`);
-            });
-        }
-        return r;
+    return (0, exports.lightdashRawApi)({
+        method,
+        url,
+        body: requestBody,
+        headers,
     })
         .then((r) => r.json())
         .then((d) => {

package/dist/handlers/organizationContent/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/handlers/organizationContent/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,KAAK,MAAM,EAAE,MAAM,cAAc,CAAC;AA0B3C,KAAK,0BAA0B,GAAG;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB,CAAC;AAkBF,eAAO,MAAM,4BAA4B,gBAAiB,MAAM,KAAG,MAClC,CAAC;AAElC,eAAO,MAAM,2BAA2B,4BAGrC,0BAA0B,KAAG,OAAO,CAAC,IAAI,CA2F3C,CAAC;AAEF,eAAO,MAAM,yBAAyB,yCAInC,0BAA0B,KAAG,OAAO,CAAC,IAAI,CAoJ3C,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/handlers/organizationContent/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,KAAK,MAAM,EAAE,MAAM,cAAc,CAAC;AAgC3C,KAAK,0BAA0B,GAAG;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB,CAAC;AAqBF,eAAO,MAAM,4BAA4B,gBAAiB,MAAM,KAAG,MAClC,CAAC;AAElC,eAAO,MAAM,2BAA2B,4BAGrC,0BAA0B,KAAG,OAAO,CAAC,IAAI,CAiG3C,CAAC;AAEF,eAAO,MAAM,yBAAyB,yCAInC,0BAA0B,KAAG,OAAO,CAAC,IAAI,CA+K3C,CAAC"}

package/dist/handlers/organizationContent/index.js

@@ -12,10 +12,12 @@ const resource_1 = require("../contentAsCode/resource");
 const contentAsCodePaths_1 = require("../contentAsCodePaths");
 const customRoles_1 = require("./customRoles");
 const groups_1 = require("./groups");
+const themes_1 = require("./themes");
 const users_1 = require("./users");
 const createCustomRolePartialUploadError = (message) => new resource_1.CodeResourcePhaseError('custom_role', message);
 const createUserPartialUploadError = (message) => new resource_1.CodeResourcePhaseError('user', message);
 const createGroupPartialUploadError = (message) => new resource_1.CodeResourcePhaseError('group', message);
+const createThemePartialUploadError = (message) => new resource_1.CodeResourcePhaseError('theme', message);
 const isPartialUploadError = resource_1.isCodeResourcePhaseError;
 const isGroupServiceDisabledError = (error) => error instanceof common_1.LightdashError &&
     error.statusCode === 403 &&
@@ -65,6 +67,11 @@ const downloadOrganizationContent = async ({ customPath, config, }) => {
             output.completeItem('service unavailable', 'warning');
             globalState_1.default.debug('> Warning: groups were not downloaded because the group service is not enabled');
         }
+        const themesTotal = await output.runItem({
+            label: 'Themes',
+            action: () => (0, themes_1.downloadThemes)(organizationContentPath),
+            detail: (total) => `${total} downloaded`,
+        });
         const renderedSummary = output.complete(organizationContentPath, (Date.now() - start) / 1000);
         if (!renderedSummary) {
             globalState_1.default.log(styles.success(`Downloaded organization content saved to ${organizationContentPath}`));
@@ -78,6 +85,7 @@ const downloadOrganizationContent = async ({ customPath, config, }) => {
                 customRolesNum: customRolesTotal,
                 usersNum: usersTotal,
                 groupsNum: groupsTotal,
+                themesNum: themesTotal,
                 timeToCompleted: (Date.now() - start) / 1000,
             },
         });
@@ -118,6 +126,7 @@ const uploadOrganizationContent = async ({ customPath, config, sendInvites = fal
         scope: 'organization',
     });
     try {
+        const preparedThemes = await (0, themes_1.prepareThemeUploads)(organizationContentPath);
         output.startItem('Custom roles');
         const summary = await (0, customRoles_1.uploadCustomRoles)(organizationUuid, organizationContentPath);
         const summaryMessage = (0, customRoles_1.formatCustomRoleUploadSummary)(summary);
@@ -152,6 +161,18 @@ const uploadOrganizationContent = async ({ customPath, config, sendInvites = fal
             throw createGroupPartialUploadError(`Processed groups: ${groupSummaryMessage}`);
         }
         output.completeItem(groupSummaryMessage);
+        output.startItem('Themes');
+        const themeSummary = await (0, themes_1.uploadThemes)(preparedThemes);
+        const themeSummaryMessage = (0, themes_1.formatThemeUploadSummary)(themeSummary);
+        if (themeSummary.failed > 0) {
+            themeSummary.failures.forEach(({ message }) => globalState_1.default.log(styles.error(message)));
+            if (themeSummary.completedSlugs.length > 0) {
+                globalState_1.default.log(styles.warning(`Processed themes before failure: ${themeSummary.completedSlugs.join(', ')}`));
+            }
+            output.prepareForFailureDetails();
+            throw createThemePartialUploadError(`Processed themes: ${themeSummaryMessage}`);
+        }
+        output.completeItem(themeSummaryMessage);
         const renderedSummary = output.complete(organizationContentPath, (Date.now() - start) / 1000);
         if (!renderedSummary) {
             globalState_1.default.log(styles.success(`Uploaded organization content from ${organizationContentPath}`));
@@ -173,6 +194,10 @@ const uploadOrganizationContent = async ({ customPath, config, sendInvites = fal
                 groupsCreated: groupSummary.created,
                 groupsUpdated: groupSummary.updated,
                 groupsUnchanged: groupSummary.unchanged,
+                themesCreated: themeSummary.created,
+                themesUpdated: themeSummary.updated,
+                themesUnchanged: themeSummary.unchanged,
+                themesUploaded: themeSummary.created + themeSummary.updated,
                 timeToCompleted: (Date.now() - start) / 1000,
             },
         });

package/dist/handlers/organizationContent/index.test.js

@@ -7,6 +7,7 @@ const globalState_1 = tslib_1.__importDefault(require("../../globalState"));
 const customRoles_1 = require("./customRoles");
 const groups_1 = require("./groups");
 const index_1 = require("./index");
+const themes_1 = require("./themes");
 const users_1 = require("./users");
 vi.mock('./customRoles', async (importOriginal) => ({
     ...(await importOriginal()),
@@ -24,6 +25,12 @@ vi.mock('./groups', async (importOriginal) => ({
     uploadGroups: vi.fn(),
     countDependencySkippedGroups: vi.fn(),
 }));
+vi.mock('./themes', async (importOriginal) => ({
+    ...(await importOriginal()),
+    downloadThemes: vi.fn(),
+    prepareThemeUploads: vi.fn(),
+    uploadThemes: vi.fn(),
+}));
 describe('organization content download', () => {
     const spinner = {
         start: vi.fn(),
@@ -47,6 +54,7 @@ describe('organization content download', () => {
         vi.mocked(customRoles_1.downloadCustomRoles).mockResolvedValue(2);
         vi.mocked(users_1.downloadUsers).mockResolvedValue(3);
         vi.mocked(groups_1.downloadGroups).mockResolvedValue(4);
+        vi.mocked(themes_1.downloadThemes).mockResolvedValue(5);
     });
     it('skips groups when the group service is disabled', async () => {
         vi.mocked(groups_1.downloadGroups).mockRejectedValue(new common_1.LightdashError({
@@ -62,7 +70,7 @@ describe('organization content download', () => {
     });
     it('reports a duration for every downloaded resource', async () => {
         await (0, index_1.downloadOrganizationContent)({ config });
-        expect(spinner.succeed).toHaveBeenCalledTimes(3);
+        expect(spinner.succeed).toHaveBeenCalledTimes(4);
         spinner.succeed.mock.calls.forEach(([message]) => expect(message).toMatch(/\(\d+ms\)$/));
     });
     it('still fails for other group download errors', async () => {
@@ -112,6 +120,14 @@ describe('organization content upload sequencing', () => {
         dependencySkipped: 0,
         failures: failed > 0 ? [{ message: 'group failure' }] : [],
     });
+    const themeSummary = (failed = 0) => ({
+        created: failed > 0 ? 1 : 0,
+        updated: 0,
+        unchanged: 0,
+        failed,
+        completedSlugs: failed > 0 ? ['processed-theme'] : [],
+        failures: failed > 0 ? [{ message: 'theme failure' }] : [],
+    });
     beforeEach(() => {
         vi.clearAllMocks();
         vi.spyOn(globalState_1.default, 'startSpinner').mockReturnValue(spinner);
@@ -120,10 +136,16 @@ describe('organization content upload sequencing', () => {
         vi.mocked(customRoles_1.uploadCustomRoles).mockResolvedValue(customRoleSummary());
         vi.mocked(users_1.uploadUsers).mockResolvedValue(userSummary());
         vi.mocked(groups_1.uploadGroups).mockResolvedValue(groupSummary());
+        vi.mocked(themes_1.prepareThemeUploads).mockResolvedValue([]);
+        vi.mocked(themes_1.uploadThemes).mockResolvedValue(themeSummary());
         vi.mocked(groups_1.countDependencySkippedGroups).mockResolvedValue(2);
     });
-    it('awaits custom roles, then users, then groups', async () => {
+    it('preflights themes, then awaits organization resource uploads in order', async () => {
         const phaseOrder = [];
+        vi.mocked(themes_1.prepareThemeUploads).mockImplementation(async () => {
+            phaseOrder.push('themes:preflight:start', 'themes:preflight:end');
+            return [];
+        });
         vi.mocked(customRoles_1.uploadCustomRoles).mockImplementation(async () => {
             phaseOrder.push('roles:start', 'roles:end');
             return customRoleSummary();
@@ -136,19 +158,27 @@ describe('organization content upload sequencing', () => {
             phaseOrder.push('groups:start', 'groups:end');
             return groupSummary();
         });
+        vi.mocked(themes_1.uploadThemes).mockImplementation(async () => {
+            phaseOrder.push('themes:start', 'themes:end');
+            return themeSummary();
+        });
         await (0, index_1.uploadOrganizationContent)({ config });
         expect(phaseOrder).toStrictEqual([
+            'themes:preflight:start',
+            'themes:preflight:end',
             'roles:start',
             'roles:end',
             'users:start',
             'users:end',
             'groups:start',
             'groups:end',
+            'themes:start',
+            'themes:end',
         ]);
     });
     it('reports a duration for every uploaded resource', async () => {
         await (0, index_1.uploadOrganizationContent)({ config });
-        expect(spinner.succeed).toHaveBeenCalledTimes(3);
+        expect(spinner.succeed).toHaveBeenCalledTimes(4);
         spinner.succeed.mock.calls.forEach(([message]) => expect(message).toMatch(/\(\d+ms\)$/));
     });
     it('does not start users or groups when custom roles fail', async () => {
@@ -156,12 +186,27 @@ describe('organization content upload sequencing', () => {
         await expect((0, index_1.uploadOrganizationContent)({ config })).rejects.toThrow('Processed custom roles');
         expect(users_1.uploadUsers).not.toHaveBeenCalled();
         expect(groups_1.uploadGroups).not.toHaveBeenCalled();
+        expect(themes_1.uploadThemes).not.toHaveBeenCalled();
         expect(groups_1.countDependencySkippedGroups).toHaveBeenCalledOnce();
     });
     it('does not start groups when users fail', async () => {
         vi.mocked(users_1.uploadUsers).mockResolvedValue(userSummary(1));
         await expect((0, index_1.uploadOrganizationContent)({ config })).rejects.toThrow('Processed users');
         expect(groups_1.uploadGroups).not.toHaveBeenCalled();
+        expect(themes_1.uploadThemes).not.toHaveBeenCalled();
         expect(groups_1.countDependencySkippedGroups).toHaveBeenCalledOnce();
     });
+    it('does not mutate organization resources when theme preflight fails', async () => {
+        vi.mocked(themes_1.prepareThemeUploads).mockRejectedValue(new Error('Invalid theme'));
+        await expect((0, index_1.uploadOrganizationContent)({ config })).rejects.toThrow('Invalid theme');
+        expect(customRoles_1.uploadCustomRoles).not.toHaveBeenCalled();
+        expect(users_1.uploadUsers).not.toHaveBeenCalled();
+        expect(groups_1.uploadGroups).not.toHaveBeenCalled();
+        expect(themes_1.uploadThemes).not.toHaveBeenCalled();
+    });
+    it('fails after reporting a partial theme upload', async () => {
+        vi.mocked(themes_1.uploadThemes).mockResolvedValue(themeSummary(1));
+        await expect((0, index_1.uploadOrganizationContent)({ config })).rejects.toThrow('Processed themes: 1 created, 0 updated, 0 unchanged, 1 failed');
+        expect(globalState_1.default.log).toHaveBeenCalledWith(expect.stringContaining('processed-theme'));
+    });
 });

package/dist/handlers/organizationContent/themePackage.d.ts

@@ -0,0 +1,22 @@
+import { type OrganizationDesignFileKind, type OrganizationDesignPackageManifest } from '@lightdash/common';
+type ThemePackageFile = {
+    kind: OrganizationDesignFileKind;
+    filename: string;
+    contentType: string;
+    body: Buffer;
+};
+export type ParsedThemePackage = {
+    manifest: OrganizationDesignPackageManifest;
+    files: ThemePackageFile[];
+};
+export type PreparedThemeUpload = {
+    directory: string;
+    manifest: OrganizationDesignPackageManifest;
+    archive: Buffer;
+};
+export declare const getThemesFolder: (organizationContentPath: string) => string;
+export declare const prepareThemeUploads: (organizationContentPath: string) => Promise<PreparedThemeUpload[]>;
+export declare const parseThemeArchive: (archive: Buffer) => Promise<ParsedThemePackage>;
+export declare const writeThemePackage: (organizationContentPath: string, parsed: ParsedThemePackage) => Promise<void>;
+export {};
+//# sourceMappingURL=themePackage.d.ts.map

package/dist/handlers/organizationContent/themePackage.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"themePackage.d.ts","sourceRoot":"","sources":["../../../src/handlers/organizationContent/themePackage.ts"],"names":[],"mappings":"AACA,OAAO,EAkBH,KAAK,0BAA0B,EAC/B,KAAK,iCAAiC,EACzC,MAAM,mBAAmB,CAAC;AAgB3B,KAAK,gBAAgB,GAAG;IACpB,IAAI,EAAE,0BAA0B,CAAC;IACjC,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,MAAM,CAAC;CAChB,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAAG;IAC7B,QAAQ,EAAE,iCAAiC,CAAC;IAC5C,KAAK,EAAE,gBAAgB,EAAE,CAAC;CAC7B,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,iCAAiC,CAAC;IAC5C,OAAO,EAAE,MAAM,CAAC;CACnB,CAAC;AAEF,eAAO,MAAM,eAAe,4BAA6B,MAAM,KAAG,MAClB,CAAC;AAiRjD,eAAO,MAAM,mBAAmB,4BACH,MAAM,KAChC,OAAO,CAAC,mBAAmB,EAAE,CA4D/B,CAAC;AAEF,eAAO,MAAM,iBAAiB,YACjB,MAAM,KAChB,OAAO,CAAC,kBAAkB,CAwJvB,CAAC;AA6BP,eAAO,MAAM,iBAAiB,4BACD,MAAM,UACvB,kBAAkB,KAC3B,OAAO,CAAC,IAAI,CA2Cd,CAAC"}