@hot-updater/cli-tools 1.0.0-rc.43 → 1.0.0-rc.45

package/dist/index.d.mts

@@ -79,7 +79,7 @@ declare const printAppSetup: ({ baseURL, ...setup }: AppSetup & {
 //#region src/assembleServer.d.ts
 /**
  * The server hot-updater.config.ts describes, as the CLI runs it: core and
- * the plugins over the config's database and storage.
+ * the plugins over the config's database.
  */
 interface AssembledServer {
   /** The config's database: the server's own, or `standaloneRepository(...)`. */
@@ -91,8 +91,6 @@ interface AssembledServer {
    * `standaloneRepository`, they go through the server's admin API.
    */
   readonly core: HotUpdaterCoreApi;
-  /** Where the CLI uploads bundles; undefined when the config names none. */
-  readonly storage: StorageAdapter | undefined;
   /** The plugins as configured, as a frozen copy. */
   readonly plugins: readonly AnyHotUpdaterPlugin[];
   /** Each plugin's API by id; undefined over standaloneRepository, whose plugins run on the server. */
@@ -109,6 +107,11 @@ interface AssembledServer {
 /** What `assembleServer` runs: hot-updater.config.ts's `database`, `storage`, and `plugins`. */
 interface AssembleServerOptions {
   readonly database: ConfiguredDatabase;
+  /**
+   * The config's storage, through which core resolves file URLs and the
+   * plugins read stored files. Tooling that does neither, such as init
+   * provisioning a credential, leaves it out.
+   */
   readonly storage?: StorageAdapter;
   readonly plugins?: readonly AnyHotUpdaterPlugin[];
 }
@@ -194,7 +197,7 @@ declare class ConfigBuilder {
   setStorage(storageConfig: ProviderConfig): this;
   /** Sets the server's database and its imports. */
   setDatabase(databaseConfig: ProviderConfig): this;
-  /** Sets the plugins the server runs, such as a provider package's `plugins`. */
+  /** Sets the server's plugin factory array and its imports. */
   setPlugins(pluginsConfig: ProviderConfig): this;
   /** Sets the code between the environment loading and the config, such as a credentials helper. */
   setIntermediateCode(code: string): this;
@@ -343,7 +346,7 @@ type HotUpdaterConfigScaffold = {
     initializer: string;
     callee: string;
   };
-  /** The plugins the server runs, such as a provider package's `plugins`. */
+  /** The server's plugin factory array. */
   plugins: {
     initializer: string;
   };
@@ -367,7 +370,8 @@ declare const writeHotUpdaterConfig: (scaffold: HotUpdaterConfigScaffold, filePa
  * Writes hot-updater.config.ts and says what it did. `settings` names the
  * provider in messages, such as "Supabase". A config it cannot merge is
  * kept, with what to add to it. The plugins file an older init wrote is
- * removed; one the project wrote is named, since nothing reads it.
+ * removed only after a successful config write and when the config no longer
+ * references it. A file the project wrote is kept with a cleanup notice.
  */
 declare const writeHotUpdaterFiles: (scaffold: HotUpdaterConfigScaffold, { cwd, settings }: {
   cwd?: string;
@@ -573,8 +577,6 @@ interface ServerClientEndpoint {
 interface ServerDefinition {
   /** The database as configured, with the tooling `hot-updater db` runs. */
   readonly database: ToolingDatabase;
-  /** The storage as configured, in order. */
-  readonly storage: readonly StorageAdapter[];
   readonly plugins: readonly AnyHotUpdaterPlugin[];
   /** The client plugins an app adds for the plugins: what init prints. */
   readonly clientPlugins: readonly PluginClientPlugin[];

package/dist/index.mjs

@@ -16,7 +16,7 @@ import l__default from "node:readline";
 import tty, { ReadStream } from "node:tty";
 import Ei, { appendFileSync, createReadStream, createWriteStream, existsSync, lstatSync, readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import path$1, { basename, dirname as dirname$1, join, posix, win32 } from "node:path";
-import { createBundleStorageKey, createMemoryAdapter, createStorageRootUriWithPath, getContentType, getManifestAssetDownloadPath, getManifestAssetStoragePath, isContentAddressedAssetFileHash, isRemoteDatabase, resolveManifestAssetStorageUri, toolingTargetOf } from "@hot-updater/plugin-core";
+import { createBundleStorageKey, createMemoryAdapter, createStorageAdapter, createStorageRootUriWithPath, getContentType, getManifestAssetDownloadPath, getManifestAssetStoragePath, isContentAddressedAssetFileHash, isRemoteDatabase, resolveManifestAssetStorageUri, toolingTargetOf } from "@hot-updater/plugin-core";
 import { createHotUpdater } from "@hot-updater/server";
 import os, { constants } from "node:os";
 import crypto$1, { createHash, randomBytes, randomUUID } from "node:crypto";
@@ -22970,6 +22970,11 @@ const printAppSetup = ({ baseURL, ...setup }) => {
 };
 //#endregion
 //#region src/assembleServer.ts
+/** The storage of an assembly that reads no stored file: it owns no URI. */
+const noStorage = createStorageAdapter({
+	name: "none",
+	protocol: "none"
+});
 /** `createHotUpdater`, with public client routes unless a plugin provides clientAuth. */
 const createServer = (database, storage, plugins) => createHotUpdater({
 	database,
@@ -22977,27 +22982,25 @@ const createServer = (database, storage, plugins) => createHotUpdater({
 	plugins,
 	...plugins.some((plugin) => plugin.provides?.clientAuth) ? {} : { clientAccess: "public" }
 });
-function assembleServer({ database, storage, plugins = [] }) {
+function assembleServer({ database, storage = noStorage, plugins = [] }) {
 	if (isRemoteDatabase(database)) {
 		const shape = createServer({
 			name: "memory",
 			adapter: createMemoryAdapter()
-		}, [], plugins);
+		}, storage, plugins);
 		return {
 			database,
 			core: database.core,
-			storage,
 			plugins: shape.plugins,
 			api: void 0,
 			clientPlugins: shape.clientPlugins,
 			...shape.clientAuth === void 0 ? {} : { clientAuth: shape.clientAuth }
 		};
 	}
-	const server = createServer(database, storage === void 0 ? [] : [storage], plugins);
+	const server = createServer(database, storage, plugins);
 	return {
 		database,
 		core: server.core,
-		storage,
 		plugins: server.plugins,
 		api: server.api,
 		clientPlugins: server.clientPlugins,
@@ -25184,7 +25187,7 @@ var ConfigBuilder = class {
 		this.imports.push(...databaseConfig.imports);
 		return this;
 	}
-	/** Sets the plugins the server runs, such as a provider package's `plugins`. */
+	/** Sets the server's plugin factory array and its imports. */
 	setPlugins(pluginsConfig) {
 		this.pluginsInfo = pluginsConfig;
 		this.imports.push(...pluginsConfig.imports);
@@ -44123,6 +44126,7 @@ const MANAGED_IMPORT_PACKAGES = /* @__PURE__ */ new Set([
 	"firebase-admin",
 	"firebase-admin/app",
 	"hot-updater",
+	"hot-updater/plugins",
 	"@aws-sdk/credential-provider-sso",
 	"@aws-sdk/credential-providers",
 	"@hot-updater/aws",
@@ -44131,6 +44135,7 @@ const MANAGED_IMPORT_PACKAGES = /* @__PURE__ */ new Set([
 	"@hot-updater/expo",
 	"@hot-updater/firebase",
 	"@hot-updater/rock",
+	"@hot-updater/server/plugins",
 	"@hot-updater/supabase"
 ]);
 const MANAGED_HELPER_NAMES = /* @__PURE__ */ new Set([
@@ -44149,7 +44154,7 @@ const BUILD_IMPORT_PACKAGES = new Set([...KNOWN_BUILD_CALLEES].map((callee) => `
 const importDeclarationsOf = (source) => source.program.body.filter((statement) => statement.type === "ImportDeclaration");
 const parseConfigSource = (text) => {
 	const result = parseSync(CONFIG_FILE_NAME, text, {
-		astType: "js",
+		astType: "ts",
 		lang: "ts",
 		preserveParens: false,
 		sourceType: "module",
@@ -44316,21 +44321,78 @@ const mergeObjectLiteralText = (existingObject, newObject, replaceIncompatiblePr
 	}).map((property) => getPropertyBlockText(newObject.source, property));
 	return applyTextEdits(existingText, [...edits, ...appendPropertiesEdits(existingObject.source, existingObject.objectExpression, missingPropertyTexts)]);
 };
-const buildMergedCallInitializer = (existing, next) => {
-	const [existingArgument] = existing.callExpression.arguments;
+/**
+* The edit that deletes `property` from its object: with its comma, and with
+* the rest of its line when the property starts one.
+*/
+const removePropertyEdit = (text, property) => {
+	const comma = /^[ \t]*,/.exec(text.slice(property.end));
+	const end = property.end + (comma?.[0].length ?? 0);
+	if (startsLine(text, property)) {
+		const rest = /^[ \t]*(?:\/\/[^\n]*)?\n/.exec(text.slice(end));
+		return {
+			start: lineStartOf(text, property.start),
+			end: end + (rest?.[0].length ?? 0),
+			text: ""
+		};
+	}
+	const space = /^[ \t]*/.exec(text.slice(end));
+	return {
+		start: property.start,
+		end: end + (space?.[0].length ?? 0),
+		text: ""
+	};
+};
+const parseCall = (text) => {
+	const source = parseConfigSource(`(${text});`);
+	const statement = source?.program.body[0];
+	return source && statement?.type === "ExpressionStatement" && statement.expression.type === "CallExpression" ? {
+		callExpression: statement.expression,
+		source
+	} : null;
+};
+/**
+* The project's call without its spreads of the managed helpers the rebuilt
+* config no longer declares.
+*/
+const withoutDroppedHelpers = (call, droppedHelpers) => {
+	const [argument] = call.callExpression.arguments;
+	if (call.callExpression.arguments.length !== 1 || argument?.type !== "ObjectExpression") return call;
+	const callStart = call.callExpression.start;
+	const edits = argument.properties.filter((property) => property.type === "SpreadElement" && property.argument.type === "Identifier" && droppedHelpers.has(property.argument.name)).map((property) => {
+		const edit = removePropertyEdit(call.source.text, property);
+		return {
+			...edit,
+			start: edit.start - callStart,
+			end: edit.end - callStart
+		};
+	});
+	if (edits.length === 0) return call;
+	return parseCall(applyTextEdits(getNodeText(call.source, call.callExpression), edits));
+};
+/** Whether `text` uses one of `names`; text that doesn't parse counts as a use. */
+const usesAny = (text, names) => {
+	if (names.size === 0) return false;
+	const used = usedIdentifiers(`(${text});`);
+	return used === null || [...names].some((name) => used.has(name));
+};
+const buildMergedCallInitializer = (existing, next, droppedHelpers) => {
+	const project = withoutDroppedHelpers(existing, droppedHelpers);
+	if (!project) return null;
+	const [existingArgument] = project.callExpression.arguments;
 	const [nextArgument] = next.callExpression.arguments;
-	if (existing.callExpression.arguments.length === 1 && next.callExpression.arguments.length === 1 && existingArgument?.type === "ObjectExpression" && nextArgument?.type === "ObjectExpression") {
+	if (project.callExpression.arguments.length === 1 && next.callExpression.arguments.length === 1 && existingArgument?.type === "ObjectExpression" && nextArgument?.type === "ObjectExpression") {
 		const mergedObjectLiteral = mergeObjectLiteralText({
 			objectExpression: existingArgument,
-			source: existing.source
+			source: project.source
 		}, {
 			objectExpression: nextArgument,
 			source: next.source
 		});
 		if (!mergedObjectLiteral) return null;
-		return `${getNodeText(existing.source, existing.callExpression.callee)}(${mergedObjectLiteral})`;
+		return `${getNodeText(project.source, project.callExpression.callee)}(${mergedObjectLiteral})`;
 	}
-	return getNodeText(existing.source, existing.callExpression);
+	return getNodeText(project.source, project.callExpression);
 };
 const findManagedProperty = (objectExpression, propertyName) => {
 	const property = objectExpression.properties.find((candidate) => candidate.type === "Property" && candidate.kind === "init" && !candidate.method && !candidate.shorthand && getObjectPropertyName(candidate) === propertyName);
@@ -44369,10 +44431,12 @@ const mergeHelperStatement = (existingStatementText, helper) => {
 * `plugins`: a call to the same adapter keeps the project's arguments and
 * gains the scaffold's missing ones, and `plugins` is the scaffold's. A build
 * that is not a plain build adapter, such as `withSentry(bare())`, stays the
-* project's (`keptBuild`). Null when `build`, `storage`, or `database` is no
-* call.
+* project's (`keptBuild`). The merged call loses its spreads of the managed
+* helpers the rebuilt config no longer declares (`droppedHelpers`), and a
+* call that still uses one takes the scaffold's. Null when `build`,
+* `storage`, or `database` is no call.
 */
-const updateManagedObject = (existing, next) => {
+const updateManagedObject = (existing, next, droppedHelpers) => {
 	const objectStart = existing.objectExpression.start;
 	const objectText = getNodeText(existing.source, existing.objectExpression);
 	const propertyEdits = [];
@@ -44408,9 +44472,9 @@ const updateManagedObject = (existing, next) => {
 			}, {
 				callExpression: nextProperty.value,
 				source: next.source
-			});
+			}, droppedHelpers);
 			if (!mergedInitializer) return null;
-			nextInitializerText = mergedInitializer;
+			if (!usesAny(mergedInitializer, droppedHelpers)) nextInitializerText = mergedInitializer;
 		}
 		propertyEdits.push({
 			start: existingProperty.value.start - objectStart,
@@ -44440,8 +44504,25 @@ const getManagedHelperName = (statement) => {
 	if (declaration?.id.type !== "Identifier") return null;
 	return declaration.id.name;
 };
-/** Whether `text` refers to `name` as an identifier. */
-const usesIdentifier = (text, name) => new RegExp(`(?<![\\w$])${name.replace(/\$/g, "\\$")}(?![\\w$])`).test(text);
+/** Identifier uses, excluding static property names, strings, and comments. */
+const usedIdentifiers = (text) => {
+	const source = parseConfigSource(text);
+	if (!source) return null;
+	const names = /* @__PURE__ */ new Set();
+	const propertyNames = /* @__PURE__ */ new Set();
+	new Visitor({
+		Property(property) {
+			if (!property.computed && !property.shorthand) propertyNames.add(property.key.start);
+		},
+		MemberExpression(member) {
+			if (!member.computed) propertyNames.add(member.property.start);
+		},
+		Identifier(identifier) {
+			if (!propertyNames.has(identifier.start)) names.add(identifier.name);
+		}
+	}).visit(source.program);
+	return names;
+};
 /**
 * What a managed package's existing imports bring that the rebuilt config
 * still uses and the scaffold doesn't import, such as a project's own `cert`
@@ -44453,6 +44534,7 @@ const keptManagedImports = (declarations, scaffoldImports, usedText) => {
 	const bound = new Set(scaffoldImports.flatMap((info) => [...(info.named ?? []).map((name) => name.split(/\s+as\s+/).at(-1)), ...info.defaultOrNamespace === void 0 ? [] : [info.defaultOrNamespace.replace(/^\*\s+as\s+/, "")]]));
 	const imports = [];
 	const texts = [];
+	const references = usedIdentifiers(usedText);
 	for (const declaration of declarations) {
 		const pkg = declaration.source.value;
 		const named = [];
@@ -44460,7 +44542,7 @@ const keptManagedImports = (declarations, scaffoldImports, usedText) => {
 		let namespaceName;
 		for (const specifier of declaration.specifiers) {
 			const local = specifier.local.name;
-			if (bound.has(local) || !usesIdentifier(usedText, local)) continue;
+			if (bound.has(local) || references && !references.has(local)) continue;
 			if (specifier.type === "ImportDefaultSpecifier") defaultName = local;
 			else if (specifier.type === "ImportNamespaceSpecifier") namespaceName = local;
 			else {
@@ -44471,7 +44553,9 @@ const keptManagedImports = (declarations, scaffoldImports, usedText) => {
 		}
 		if (named.length === 0 && !defaultName && !namespaceName) continue;
 		if (declaration.importKind === "type") {
-			texts.push(`import type { ${named.join(", ")} } from "${pkg}";`);
+			if (named.length > 0) texts.push(`import type { ${named.join(", ")} } from "${pkg}";`);
+			if (namespaceName) texts.push(`import type * as ${namespaceName} from "${pkg}";`);
+			if (defaultName) texts.push(`import type ${defaultName} from "${pkg}";`);
 			continue;
 		}
 		if (named.length > 0) imports.push({
@@ -44553,28 +44637,26 @@ const mergeHotUpdaterConfigText = (existingText, scaffold) => {
 	const existingConfig = findDefineConfigObject(existingSource);
 	const nextConfig = findDefineConfigObject(nextSource);
 	if (!existingConfig || !nextConfig) return { reason: "Existing config is not a supported `export default defineConfig({ ... })` shape." };
+	const existingPlugins = findPluginsProperty(existingConfig.objectExpression);
+	if (existingPlugins && existingConfig.objectExpression.properties.some((property) => property.start > existingPlugins.start && (property.type === "SpreadElement" || property.computed || getObjectPropertyName(property) === "plugins"))) return { reason: "A later spread or property can overwrite plugins. Move plugins after it before running init again." };
+	const scaffoldHelpers = new Set(scaffold.helperStatements.map((helper) => helper.name));
+	const droppedHelpers = new Set(existingSource.program.body.flatMap((statement) => {
+		const name = statement.start < existingConfig.exportDeclaration.start ? getManagedHelperName(statement) : null;
+		return name !== null && MANAGED_HELPER_NAMES.has(name) && !scaffoldHelpers.has(name) ? [name] : [];
+	}));
 	const nextObject = updateManagedObject({
 		objectExpression: existingConfig.objectExpression,
 		source: existingSource
 	}, {
 		objectExpression: nextConfig.objectExpression,
 		source: nextSource
-	});
+	}, droppedHelpers);
 	if (!nextObject) return { reason: "Existing config uses dynamic build/storage/database expressions that cannot be merged safely." };
 	const exportFullStart = getTopLevelFullStart(existingSource, existingConfig.exportDeclaration);
 	const firstTriviaContent = existingText.slice(exportFullStart, existingConfig.exportDeclaration.start).search(/\S/);
 	const managedBodyEnd = firstTriviaContent === -1 ? existingConfig.exportDeclaration.start : exportFullStart + firstTriviaContent;
 	const bodyEdit = rebuildManagedBody(existingSource, managedBodyEnd, scaffold);
 	if (!bodyEdit) return { reason: "Existing helper declarations could not be merged safely." };
-	const scaffoldNames = new Map(scaffold.imports.flatMap((info) => (info.named ?? []).map((name) => [name.split(/\s+as\s+/).at(-1), info.pkg])));
-	for (const declaration of importDeclarationsOf(existingSource)) {
-		const pkg = declaration.source.value;
-		if (MANAGED_IMPORT_PACKAGES.has(pkg) && !(nextObject.keptBuild && BUILD_IMPORT_PACKAGES.has(pkg))) continue;
-		for (const specifier of declaration.specifiers) {
-			const scaffoldPackage = scaffoldNames.get(specifier.local.name);
-			if (scaffoldPackage !== void 0 && scaffoldPackage !== pkg) return { reason: `The import of ${specifier.local.name} from "${pkg}" takes the name init imports from "${scaffoldPackage}".` };
-		}
-	}
 	const objectEdit = {
 		start: existingConfig.objectExpression.start,
 		end: existingConfig.objectExpression.end,
@@ -44589,6 +44671,43 @@ const mergeHotUpdaterConfigText = (existingText, scaffold) => {
 			text: ""
 		}))
 	]);
+	const usedSource = parseConfigSource(usedText);
+	const usedConfig = usedSource && findDefineConfigObject(usedSource);
+	const plugins = usedConfig && findPluginsProperty(usedConfig.objectExpression);
+	const keptReferences = usedIdentifiers(plugins ? applyTextEdits(usedText, [{
+		start: plugins.start,
+		end: plugins.end,
+		text: "plugins: []"
+	}]) : usedText);
+	const scaffoldNames = new Map(scaffold.imports.flatMap((info) => (info.named ?? []).map((name) => {
+		const parts = name.split(/\s+as\s+/);
+		return [parts.at(-1), {
+			pkg: info.pkg,
+			imported: parts[0]
+		}];
+	})));
+	const bindings = [];
+	for (const statement of usedSource?.program.body ?? []) {
+		const declaration = statement.type === "ExportNamedDeclaration" ? statement.declaration : statement;
+		if (declaration?.type === "VariableDeclaration") bindings.push(...declaration.declarations.map(({ id }) => id));
+		else if ((declaration?.type === "FunctionDeclaration" || declaration?.type === "ClassDeclaration") && declaration.id) bindings.push(declaration.id);
+	}
+	for (const binding of bindings) {
+		if (binding.type === "Identifier" && scaffoldNames.has(binding.name)) return { reason: `The declaration of ${binding.name} conflicts with an import init needs.` };
+		if (binding.type === "ObjectPattern") bindings.push(...binding.properties.map((property) => property.type === "RestElement" ? property.argument : property.value));
+		else if (binding.type === "ArrayPattern") bindings.push(...binding.elements.filter((element) => element !== null));
+		else if (binding.type === "AssignmentPattern") bindings.push(binding.left);
+		else if (binding.type === "RestElement") bindings.push(binding.argument);
+	}
+	for (const declaration of importDeclarationsOf(existingSource)) {
+		const pkg = declaration.source.value;
+		for (const specifier of declaration.specifiers) {
+			if (MANAGED_IMPORT_PACKAGES.has(pkg) && !(nextObject.keptBuild && BUILD_IMPORT_PACKAGES.has(pkg)) && keptReferences && !keptReferences.has(specifier.local.name)) continue;
+			const scaffoldImport = scaffoldNames.get(specifier.local.name);
+			const imported = specifier.type === "ImportSpecifier" && specifier.imported.type === "Identifier" ? specifier.imported.name : null;
+			if (scaffoldImport !== void 0 && (scaffoldImport.imported !== imported || scaffoldImport.pkg !== pkg && !(scaffoldImport.pkg === "hot-updater/plugins" && pkg === "@hot-updater/server/plugins"))) return { reason: `The import of ${specifier.local.name} from "${pkg}" takes the name init imports from "${scaffoldImport.pkg}".` };
+		}
+	}
 	return { text: applyTextEdits(existingText, [
 		objectEdit,
 		bodyEdit,
@@ -44686,14 +44805,16 @@ const renderServerSettings = (scaffold) => {
 const PLUGINS_FILE_PATH = "hotUpdater.plugins.ts";
 /**
 * Whether `text` is the plugins file an older init wrote: comments and a
-* re-export of a provider package's `plugins`, which the config imports now.
+* re-export of a provider package's former `plugins`, which the config's
+* `plugins` list replaces.
 */
 const isGeneratedPluginsFile = (text) => /^export\{plugins\}from(["'])@hot-updater\/[\w-]+\1;?$/u.test(text.replace(/^\s*\/\/.*$/gmu, "").replace(/\s+/gu, ""));
 /**
 * Writes hot-updater.config.ts and says what it did. `settings` names the
 * provider in messages, such as "Supabase". A config it cannot merge is
 * kept, with what to add to it. The plugins file an older init wrote is
-* removed; one the project wrote is named, since nothing reads it.
+* removed only after a successful config write and when the config no longer
+* references it. A file the project wrote is kept with a cleanup notice.
 */
 const writeHotUpdaterFiles = async (scaffold, { cwd = process.cwd(), settings }) => {
 	const config = await writeHotUpdaterConfig(scaffold, path.join(cwd, HOT_UPDATER_CONFIG_PATH));
@@ -44708,6 +44829,17 @@ const writeHotUpdaterFiles = async (scaffold, { cwd = process.cwd(), settings })
 	const pluginsPath = path.join(cwd, PLUGINS_FILE_PATH);
 	const pluginsText = await readTextFile(pluginsPath);
 	if (pluginsText === null) return { config };
+	if (config.status === "skipped") return {
+		config,
+		pluginsFile: "kept"
+	};
+	if ((await fs$1.readFile(config.path, "utf-8")).includes("./hotUpdater.plugins")) {
+		p.log.warn(`Kept '${PLUGINS_FILE_PATH}': '${HOT_UPDATER_CONFIG_PATH}' still references it.`);
+		return {
+			config,
+			pluginsFile: "kept"
+		};
+	}
 	if (isGeneratedPluginsFile(pluginsText)) {
 		await fs$1.rm(pluginsPath);
 		p.log.success(`Removed '${PLUGINS_FILE_PATH}': \`plugins\` in '${HOT_UPDATER_CONFIG_PATH}' lists the plugins the server runs.`);
@@ -45126,8 +45258,6 @@ const loadPlatformConfigs = async (platforms, { channel }) => {
 	return configs;
 };
 const resolveConfig = (config) => {
-	for (const key of ["authorityId", "catalogId"]) if (config && Object.hasOwn(config, key)) throw new Error(`Remove ${key} from hot-updater.config. Catalog identity is managed internally.`);
-	if (config && Object.hasOwn(config, "compressStrategy")) throw new Error("Remove compressStrategy from hot-updater.config. OTA artifacts use manifest files with per-file Brotli compression.");
 	const mergedConfig = mergeConfigSources(config, getDefaultConfig());
 	const signing = normalizeSigningConfig(mergedConfig.signing);
 	return {
@@ -45143,7 +45273,7 @@ var ServerDefinitionError = class extends Error {
 };
 const isRecord = (value) => typeof value === "object" && value !== null;
 /** Whether `value` has the public properties tooling reads from a server definition. */
-const isServerDefinition = (value) => isRecord(value) && isRecord(value.database) && typeof value.database.name === "string" && Array.isArray(value.storage) && Array.isArray(value.plugins) && Array.isArray(value.clientPlugins) && Array.isArray(value.clientEndpoints) && isRecord(value.core) && isRecord(value.api);
+const isServerDefinition = (value) => isRecord(value) && isRecord(value.database) && typeof value.database.name === "string" && Array.isArray(value.plugins) && Array.isArray(value.clientPlugins) && Array.isArray(value.clientEndpoints) && isRecord(value.core) && isRecord(value.api);
 /**
 * `value` as a server definition, or a `ServerDefinitionError` that says
 * what `source`, the module it came from, must export instead.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hot-updater/cli-tools",
-  "version": "1.0.0-rc.43",
+  "version": "1.0.0-rc.45",
   "type": "module",
   "engines": {
     "node": ">=20.19.0"
@@ -49,9 +49,9 @@
     "oxc-parser": "0.141.0",
     "oxc-transform": "0.141.0",
     "unconfig": "7.5.0",
-    "@hot-updater/protocol": "1.0.0-rc.43",
-    "@hot-updater/server": "1.0.0-rc.43",
-    "@hot-updater/plugin-core": "1.0.0-rc.43"
+    "@hot-updater/protocol": "1.0.0-rc.45",
+    "@hot-updater/plugin-core": "1.0.0-rc.45",
+    "@hot-updater/server": "1.0.0-rc.45"
   },
   "devDependencies": {
     "@clack/prompts": "1.7.0",
@@ -65,7 +65,7 @@
     "tar": "^7.5.16",
     "verkit": "0.4.0",
     "workspace-tools": "^0.41.7",
-    "@hot-updater/test-utils": "1.0.0-rc.43"
+    "@hot-updater/test-utils": "1.0.0-rc.45"
   },
   "inlinedDependencies": {
     "@babel/code-frame": "7.29.0",