fix(test): guard platform sync cli values · openclaw/openclaw@8b5b150

Recent Commits to openclaw:main

vincentkoc · 2026-06-20 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,8 @@ import { syncAndroidVersioning } from "./lib/android-version.ts";`
`4`	`4`
`5`	`5`	`type Mode = "check" \| "write";`
`6`	`6`
`7`		`-export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {`
	`7`	`+export function parseArgs(argv: string[]): { help: boolean; mode: Mode; rootDir: string } {`
	`8`	`+let help = false;`
`8`	`9`	`let mode: Mode = "write";`
`9`	`10`	`let rootDir = path.resolve(".");`
`10`	`11`
`@@ -20,39 +21,62 @@ export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {`
`20`	`21`	`break;`
`21`	`22`	`}`
`22`	`23`	`case "--root": {`
`23`		`-const value = argv[index + 1];`
`24`		`-if (!value) {`
`25`		`-throw new Error("Missing value for --root.");`
`26`		`-}`
`27`		`-rootDir = path.resolve(value);`
	`24`	`+rootDir = path.resolve(readOptionValue(argv, index, "--root"));`
`28`	`25`	`index += 1;`
`29`	`26`	`break;`
`30`	`27`	`}`
`31`	`28`	`case "-h":`
`32`	`29`	`case "--help": {`
`33`		`-console.log(`
`34`		`-"Usage: node --import tsx scripts/android-sync-versioning.ts [--write\|--check] [--root dir]",`
`35`		`-);`
`36`		`-process.exit(0);`
	`30`	`+help = true;`
	`31`	`+break;`
`37`	`32`	`}`
`38`	`33`	`default: {`
`39`	`34`	throw new Error(`Unknown argument: ${arg}`);
`40`	`35`	`}`
`41`	`36`	`}`
`42`	`37`	`}`
`43`	`38`
`44`		`-return { mode, rootDir };`
	`39`	`+return { help, mode, rootDir };`
`45`	`40`	`}`
`46`	`41`
`47`		`-const options = parseArgs(process.argv.slice(2));`
`48`		`-const result = syncAndroidVersioning({ mode: options.mode, rootDir: options.rootDir });`
	`42`	`+function readOptionValue(argv: string[], index: number, flag: string): string {`
	`43`	`+const value = argv[index + 1];`
	`44`	`+if (!value \|\| value.startsWith("--")) {`
	`45`	+throw new Error(`Missing value for ${flag}.`);
	`46`	`+}`
	`47`	`+return value;`
	`48`	`+}`
`49`	`49`
`50`		`-if (options.mode === "check") {`
`51`		`-process.stdout.write("Android versioning artifacts are up to date.\n");`
`52`		`-} else if (result.updatedPaths.length === 0) {`
`53`		`-process.stdout.write("Android versioning artifacts already up to date.\n");`
`54`		`-} else {`
	`50`	`+function printUsage(): void {`
`55`	`51`	`process.stdout.write(`
`56`		-`Updated Android versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
	`52`	`+"Usage: node --import tsx scripts/android-sync-versioning.ts [--write\|--check] [--root dir]\n",`
`57`	`53`	`);`
`58`	`54`	`}`
	`55`	`+`
	`56`	`+function main(argv = process.argv.slice(2)): number {`
	`57`	`+const options = parseArgs(argv);`
	`58`	`+if (options.help) {`
	`59`	`+printUsage();`
	`60`	`+return 0;`
	`61`	`+}`
	`62`	`+`
	`63`	`+const result = syncAndroidVersioning({ mode: options.mode, rootDir: options.rootDir });`
	`64`	`+`
	`65`	`+if (options.mode === "check") {`
	`66`	`+process.stdout.write("Android versioning artifacts are up to date.\n");`
	`67`	`+} else if (result.updatedPaths.length === 0) {`
	`68`	`+process.stdout.write("Android versioning artifacts already up to date.\n");`
	`69`	`+} else {`
	`70`	`+process.stdout.write(`
	`71`	+`Updated Android versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
	`72`	`+);`
	`73`	`+}`
	`74`	`+return 0;`
	`75`	`+}`
	`76`	`+`
	`77`	`+try {`
	`78`	`+process.exitCode = main();`
	`79`	`+} catch (error) {`
	`80`	+process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
	`81`	`+process.exitCode = 1;`
	`82`	`+}`

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,8 @@ import { syncIosVersioning } from "./lib/ios-version.ts";`
`4`	`4`
`5`	`5`	`type Mode = "check" \| "write";`
`6`	`6`
`7`		`-export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {`
	`7`	`+export function parseArgs(argv: string[]): { help: boolean; mode: Mode; rootDir: string } {`
	`8`	`+let help = false;`
`8`	`9`	`let mode: Mode = "write";`
`9`	`10`	`let rootDir = path.resolve(".");`
`10`	`11`
`@@ -20,39 +21,62 @@ export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {`
`20`	`21`	`break;`
`21`	`22`	`}`
`22`	`23`	`case "--root": {`
`23`		`-const value = argv[index + 1];`
`24`		`-if (!value) {`
`25`		`-throw new Error("Missing value for --root.");`
`26`		`-}`
`27`		`-rootDir = path.resolve(value);`
	`24`	`+rootDir = path.resolve(readOptionValue(argv, index, "--root"));`
`28`	`25`	`index += 1;`
`29`	`26`	`break;`
`30`	`27`	`}`
`31`	`28`	`case "-h":`
`32`	`29`	`case "--help": {`
`33`		`-console.log(`
`34`		`-"Usage: node --import tsx scripts/ios-sync-versioning.ts [--write\|--check] [--root dir]",`
`35`		`-);`
`36`		`-process.exit(0);`
	`30`	`+help = true;`
	`31`	`+break;`
`37`	`32`	`}`
`38`	`33`	`default: {`
`39`	`34`	throw new Error(`Unknown argument: ${arg}`);
`40`	`35`	`}`
`41`	`36`	`}`
`42`	`37`	`}`
`43`	`38`
`44`		`-return { mode, rootDir };`
	`39`	`+return { help, mode, rootDir };`
`45`	`40`	`}`
`46`	`41`
`47`		`-const options = parseArgs(process.argv.slice(2));`
`48`		`-const result = syncIosVersioning({ mode: options.mode, rootDir: options.rootDir });`
	`42`	`+function readOptionValue(argv: string[], index: number, flag: string): string {`
	`43`	`+const value = argv[index + 1];`
	`44`	`+if (!value \|\| value.startsWith("--")) {`
	`45`	+throw new Error(`Missing value for ${flag}.`);
	`46`	`+}`
	`47`	`+return value;`
	`48`	`+}`
`49`	`49`
`50`		`-if (options.mode === "check") {`
`51`		`-process.stdout.write("iOS versioning artifacts are up to date.\n");`
`52`		`-} else if (result.updatedPaths.length === 0) {`
`53`		`-process.stdout.write("iOS versioning artifacts already up to date.\n");`
`54`		`-} else {`
	`50`	`+function printUsage(): void {`
`55`	`51`	`process.stdout.write(`
`56`		-`Updated iOS versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
	`52`	`+"Usage: node --import tsx scripts/ios-sync-versioning.ts [--write\|--check] [--root dir]\n",`
`57`	`53`	`);`
`58`	`54`	`}`
	`55`	`+`
	`56`	`+function main(argv = process.argv.slice(2)): number {`
	`57`	`+const options = parseArgs(argv);`
	`58`	`+if (options.help) {`
	`59`	`+printUsage();`
	`60`	`+return 0;`
	`61`	`+}`
	`62`	`+`
	`63`	`+const result = syncIosVersioning({ mode: options.mode, rootDir: options.rootDir });`
	`64`	`+`
	`65`	`+if (options.mode === "check") {`
	`66`	`+process.stdout.write("iOS versioning artifacts are up to date.\n");`
	`67`	`+} else if (result.updatedPaths.length === 0) {`
	`68`	`+process.stdout.write("iOS versioning artifacts already up to date.\n");`
	`69`	`+} else {`
	`70`	`+process.stdout.write(`
	`71`	+`Updated iOS versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
	`72`	`+);`
	`73`	`+}`
	`74`	`+return 0;`
	`75`	`+}`
	`76`	`+`
	`77`	`+try {`
	`78`	`+process.exitCode = main();`
	`79`	`+} catch (error) {`
	`80`	+process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
	`81`	`+process.exitCode = 1;`
	`82`	`+}`

Original file line number	Diff line number	Diff line change
`@@ -62,6 +62,20 @@ describe("resolveAndroidVersion", () => {`
`62`	`62`	`expect(result.stderr).toBe("");`
`63`	`63`	`});`
`64`	`64`
	`65`	`+it("rejects missing Android sync CLI root values before reading version files", () => {`
	`66`	`+const result = spawnSync(`
	`67`	`+process.execPath,`
	`68`	`+["--import", "tsx", "scripts/android-sync-versioning.ts", "--root", "--check"],`
	`69`	`+{`
	`70`	`+cwd: process.cwd(),`
	`71`	`+encoding: "utf8",`
	`72`	`+},`
	`73`	`+);`
	`74`	`+`
	`75`	`+expect(result.status).toBe(1);`
	`76`	`+expect(result.stderr).toBe("Missing value for --root.\n");`
	`77`	`+});`
	`78`	`+`
`65`	`79`	`it("parses pinned release versions and Android version codes", () => {`
`66`	`80`	`const rootDir = writeAndroidFixture({`
`67`	`81`	`version: "2026.6.2",`

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,20 @@ describe("resolveIosVersion", () => {`
`58`	`58`	`expect(result.stderr).toBe("");`
`59`	`59`	`});`
`60`	`60`
	`61`	`+it("rejects missing iOS sync CLI root values before reading version files", () => {`
	`62`	`+const result = spawnSync(`
	`63`	`+process.execPath,`
	`64`	`+["--import", "tsx", "scripts/ios-sync-versioning.ts", "--root", "--check"],`
	`65`	`+{`
	`66`	`+cwd: process.cwd(),`
	`67`	`+encoding: "utf8",`
	`68`	`+},`
	`69`	`+);`
	`70`	`+`
	`71`	`+expect(result.status).toBe(1);`
	`72`	`+expect(result.stderr).toBe("Missing value for --root.\n");`
	`73`	`+});`
	`74`	`+`
`61`	`75`	`it("parses pinned release versions and derives Apple marketing fields", () => {`
`62`	`76`	`const rootDir = writeIosFixture({`
`63`	`77`	`version: "2026.4.6",`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main