dilim/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-08-09 03:22:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(test): guard platform sync cli values

Browse Source
This commit is contained in:
Vincent Koc
2026-06-20 03:27:15 +02:00
parent 4db7d6a90a
commit 8b5b150e02
4 changed files with 114 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files

62
scripts/android-sync-versioning.ts
View File

@@ -4,7 +4,8 @@ import { syncAndroidVersioning } from "./lib/android-version.ts";
type Mode = "check" | "write";
export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {
export function parseArgs(argv: string[]): { help: boolean; mode: Mode; rootDir: string } {
let help = false;
let mode: Mode = "write";
let rootDir = path.resolve(".");
@@ -20,20 +21,14 @@ export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {
break;
}
case "--root": {
const value = argv[index + 1];
if (!value) {
throw new Error("Missing value for --root.");
}
rootDir = path.resolve(value);
rootDir = path.resolve(readOptionValue(argv, index, "--root"));
index += 1;
break;
}
case "-h":
case "--help": {
console.log(
"Usage: node --import tsx scripts/android-sync-versioning.ts [--write|--check] [--root dir]",
);
process.exit(0);
help = true;
break;
}
default: {
throw new Error(`Unknown argument: ${arg}`);
@@ -41,18 +36,47 @@ export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {
}
}
return { mode, rootDir };
return { help, mode, rootDir };
}
const options = parseArgs(process.argv.slice(2));
const result = syncAndroidVersioning({ mode: options.mode, rootDir: options.rootDir });
function readOptionValue(argv: string[], index: number, flag: string): string {
const value = argv[index + 1];
if (!value || value.startsWith("--")) {
throw new Error(`Missing value for ${flag}.`);
}
return value;
}
if (options.mode === "check") {
process.stdout.write("Android versioning artifacts are up to date.\n");
} else if (result.updatedPaths.length === 0) {
process.stdout.write("Android versioning artifacts already up to date.\n");
} else {
function printUsage(): void {
process.stdout.write(
`Updated Android versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
"Usage: node --import tsx scripts/android-sync-versioning.ts [--write|--check] [--root dir]\n",
);
}
function main(argv = process.argv.slice(2)): number {
const options = parseArgs(argv);
if (options.help) {
printUsage();
return 0;
}
const result = syncAndroidVersioning({ mode: options.mode, rootDir: options.rootDir });
if (options.mode === "check") {
process.stdout.write("Android versioning artifacts are up to date.\n");
} else if (result.updatedPaths.length === 0) {
process.stdout.write("Android versioning artifacts already up to date.\n");
} else {
process.stdout.write(
`Updated Android versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
);
}
return 0;
}
try {
process.exitCode = main();
} catch (error) {
process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
process.exitCode = 1;
}

62
scripts/ios-sync-versioning.ts
View File

@@ -4,7 +4,8 @@ import { syncIosVersioning } from "./lib/ios-version.ts";
type Mode = "check" | "write";
export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {
export function parseArgs(argv: string[]): { help: boolean; mode: Mode; rootDir: string } {
let help = false;
let mode: Mode = "write";
let rootDir = path.resolve(".");
@@ -20,20 +21,14 @@ export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {
break;
}
case "--root": {
const value = argv[index + 1];
if (!value) {
throw new Error("Missing value for --root.");
}
rootDir = path.resolve(value);
rootDir = path.resolve(readOptionValue(argv, index, "--root"));
index += 1;
break;
}
case "-h":
case "--help": {
console.log(
"Usage: node --import tsx scripts/ios-sync-versioning.ts [--write|--check] [--root dir]",
);
process.exit(0);
help = true;
break;
}
default: {
throw new Error(`Unknown argument: ${arg}`);
@@ -41,18 +36,47 @@ export function parseArgs(argv: string[]): { mode: Mode; rootDir: string } {
}
}
return { mode, rootDir };
return { help, mode, rootDir };
}
const options = parseArgs(process.argv.slice(2));
const result = syncIosVersioning({ mode: options.mode, rootDir: options.rootDir });
function readOptionValue(argv: string[], index: number, flag: string): string {
const value = argv[index + 1];
if (!value || value.startsWith("--")) {
throw new Error(`Missing value for ${flag}.`);
}
return value;
}
if (options.mode === "check") {
process.stdout.write("iOS versioning artifacts are up to date.\n");
} else if (result.updatedPaths.length === 0) {
process.stdout.write("iOS versioning artifacts already up to date.\n");
} else {
function printUsage(): void {
process.stdout.write(
`Updated iOS versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
"Usage: node --import tsx scripts/ios-sync-versioning.ts [--write|--check] [--root dir]\n",
);
}
function main(argv = process.argv.slice(2)): number {
const options = parseArgs(argv);
if (options.help) {
printUsage();
return 0;
}
const result = syncIosVersioning({ mode: options.mode, rootDir: options.rootDir });
if (options.mode === "check") {
process.stdout.write("iOS versioning artifacts are up to date.\n");
} else if (result.updatedPaths.length === 0) {
process.stdout.write("iOS versioning artifacts already up to date.\n");
} else {
process.stdout.write(
`Updated iOS versioning artifacts:\n- ${result.updatedPaths.map((filePath) => path.relative(process.cwd(), filePath)).join("\n- ")}\n`,
);
}
return 0;
}
try {
process.exitCode = main();
} catch (error) {
process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
process.exitCode = 1;
}

14
test/scripts/android-version.test.ts
View File

@@ -62,6 +62,20 @@ describe("resolveAndroidVersion", () => {
expect(result.stderr).toBe("");
});
it("rejects missing Android sync CLI root values before reading version files", () => {
const result = spawnSync(
process.execPath,
["--import", "tsx", "scripts/android-sync-versioning.ts", "--root", "--check"],
{
cwd: process.cwd(),
encoding: "utf8",
},
);
expect(result.status).toBe(1);
expect(result.stderr).toBe("Missing value for --root.\n");
});
it("parses pinned release versions and Android version codes", () => {
const rootDir = writeAndroidFixture({
version: "2026.6.2",

14
test/scripts/ios-version.test.ts
View File

@@ -58,6 +58,20 @@ describe("resolveIosVersion", () => {
expect(result.stderr).toBe("");
});
it("rejects missing iOS sync CLI root values before reading version files", () => {
const result = spawnSync(
process.execPath,
["--import", "tsx", "scripts/ios-sync-versioning.ts", "--root", "--check"],
{
cwd: process.cwd(),
encoding: "utf8",
},
);
expect(result.status).toBe(1);
expect(result.stderr).toBe("Missing value for --root.\n");
});
it("parses pinned release versions and derives Apple marketing fields", () => {
const rootDir = writeIosFixture({
version: "2026.4.6",