os initialize: Deprecate the --type parameter

Change-type: patch

docs/balena-cli.md

@@ -3048,6 +3048,7 @@ because we need to access the raw devices directly.
 
 Examples:
 
+	$ balena os initialize ../path/rpi.img
 	$ balena os initialize ../path/rpi.img --type raspberry-pi
 
 ### Arguments

src/commands/internal/osinit.ts

@@ -38,9 +38,6 @@ export default class OsinitCmd extends Command {
 		image: Args.string({
 			required: true,
 		}),
-		type: Args.string({
-			required: true,
-		}),
 		config: Args.string({
 			required: true,
 		}),
@@ -58,7 +55,7 @@ export default class OsinitCmd extends Command {
 		const { getManifest, osProgressHandler } = await import(
 			'../../utils/helpers'
 		);
-		const manifest = await getManifest(params.image, params.type);
+		const manifest = await getManifest(params.image);
 
 		const { initialize } = await import('balena-device-init');
 		const initializeEmitter = await initialize(params.image, manifest, config);

src/commands/os/configure.ts

@@ -197,6 +197,7 @@ export default class OsConfigureCmd extends Command {
 
 		const deviceTypeManifest = await helpers.getManifest(
 			params.image,
+			// TODO: Drop this in the next major in favor of calling checkManifestMatchesDeviceType() directly
 			deviceTypeSlug,
 		);
 

src/commands/os/initialize.ts

@@ -37,6 +37,7 @@ export default class OsInitializeCmd extends Command {
 	`;
 
 	public static examples = [
+		'$ balena os initialize ../path/rpi.img',
 		'$ balena os initialize ../path/rpi.img --type raspberry-pi',
 	];
 
@@ -63,6 +64,11 @@ export default class OsInitializeCmd extends Command {
 		console.info(`Initializing device ${INIT_WARNING_MESSAGE}`);
 
 		const manifest = await getManifest(params.image, options.type);
+		// the "type" arg will be removed on the next major - warn user
+		if (options.type != null) {
+			const { deviceTypeArgDeprecation } = await import('../../utils/messages');
+			console.log(deviceTypeArgDeprecation('type'));
+		}
 
 		const answers = await getCliForm().run(manifest.initialization?.options, {
 			override: {
@@ -81,13 +87,7 @@ export default class OsInitializeCmd extends Command {
 			await safeUmount(answers.drive);
 		}
 
-		await sudo([
+		await sudo(['internal', 'osinit', params.image, JSON.stringify(answers)]);
-			'internal',
-			'osinit',
-			params.image,
-			options.type,
-			JSON.stringify(answers),
-		]);
 
 		if (answers.drive != null) {
 			const { safeUmount } = await import('../../utils/umount');

src/utils/common-flags.ts

@@ -94,7 +94,6 @@ export const deviceType = Flags.string({
 	description:
 		'device type (Check available types with `balena device-type list`)',
 	char: 't',
-	required: true,
 });
 
 export const json = Flags.boolean({

src/utils/helpers.ts

@@ -103,14 +103,47 @@ export async function runCommand<T>(commandArgs: string[]): Promise<T> {
 	return run(commandArgs) as Promise<T>;
 }
 
+export async function checkManifestMatchesDeviceType(
+	manifest: BalenaSdk.DeviceTypeJson.DeviceType,
+	deviceType: string,
+) {
+	if (
+		manifest.slug !== deviceType &&
+		manifest.slug !==
+			(await getBalenaSdk().models.deviceType.get(deviceType)).slug
+	) {
+		const { ExpectedError } = await import('../errors');
+		throw new ExpectedError(
+			`The device type of the provided OS image ${manifest.slug}, does not match the expected device type ${deviceType}`,
+		);
+	}
+}
+
 export async function getManifest(
 	image: string,
-	deviceType: string,
+	// TODO: Drop this parameter in the next major
+	fallbackDeviceType?: string,
 ): Promise<BalenaSdk.DeviceTypeJson.DeviceType> {
 	const init = await import('balena-device-init');
-	const sdk = getBalenaSdk();
 	const manifest = await init.getImageManifest(image);
-	if (manifest != null) {
+	if (manifest == null) {
+		// TODO: Change this in the next major to throw when not being able to find the manifest
+		// on the provided image, after confirming that this works for all supported OS versions.
+		if (fallbackDeviceType != null) {
+			console.error(
+				`[warn] Error while finding a device-type.json on the provided image path. Attempting to fetch from the API.`,
+			);
+			const sdk = getBalenaSdk();
+			return await sdk.models.config.getDeviceTypeManifestBySlug(
+				fallbackDeviceType,
+			);
+		}
+		const { ExpectedError } = await import('../errors');
+		throw new ExpectedError(
+			`Could not find the manifest on the provided image path '${image}'. Please double check that this is a valid balenaOS image.`,
+		);
+	}
+
 	const config = manifest.configuration?.config;
 	if (config?.partition != null) {
 		const { getBootPartition } = await import('balena-config-json');
@@ -125,26 +158,12 @@ export async function getManifest(
 		}
 		// TODO: Add handling for when we no longer include a `config.partition` at all.
 	}
-	} else {
+	// TODO: Drop this in the next major along with the fallbackDeviceType argument
-		// TODO: Change this in the next major to throw, after confirming that this works for all supported OS versions.
+	// and move the checkManifestMatchesDeviceType() to the caller side when still necessary.
-		console.error(
+	if (fallbackDeviceType != null) {
-			`[warn] Error while finding a device-type.json on the provided image path. Attempting to fetch from the API.`,
+		await checkManifestMatchesDeviceType(manifest, fallbackDeviceType);
-		);
 	}
-	if (
+	return manifest;
-		manifest != null &&
-		manifest.slug !== deviceType &&
-		manifest.slug !== (await sdk.models.deviceType.get(deviceType)).slug
-	) {
-		const { ExpectedError } = await import('../errors');
-		throw new ExpectedError(
-			`The device type of the provided OS image ${manifest.slug}, does not match the expected device type ${deviceType}`,
-		);
-	}
-	return (
-		manifest ??
-		(await sdk.models.config.getDeviceTypeManifestBySlug(deviceType))
-	);
 }
 
 export const areDeviceTypesCompatible = async (

src/utils/messages.ts

@@ -189,6 +189,12 @@ If you have a particular use for buildArg, which is not satisfied by build-time
 secrets, please contact us via support or the forums: https://forums.balena.io/
 \n`;
 
+export const deviceTypeArgDeprecation = (paramName: string) => `\
+WARNING: You have specified a '--${paramName}' option, which is now deprecated, and
+may be removed in the future. The balena-cli will now try to auto-detect
+the device type of the provided balenaOS image.
+\n`;
+
 export function getNodeEngineVersionWarn(
 	version: string,
 	validVersions: string,