fix(ci): route mac packaging scripts to macos checks · op...

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -1855,7 +1855,7 @@`
`1855`	`1855`	`"test:live:media:music": "node --import tsx scripts/test-live-media.ts music",`
`1856`	`1856`	`"test:live:media:video": "node --import tsx scripts/test-live-media.ts video",`
`1857`	`1857`	`"test:live:models-profiles": "node scripts/test-live.mjs -- src/agents/models.profiles.live.test.ts",`
`1858`		`-"test:macos:ci": "node scripts/test-projects.mjs src/daemon/launchd.test.ts src/daemon/runtime-paths.test.ts src/daemon/runtime-binary.test.ts src/infra/brew.test.ts src/infra/stable-node-path.test.ts test/scripts/vitest-process-group.test.ts test/scripts/package-mac-app.test.ts test/scripts/package-mac-dist.test.ts test/scripts/create-dmg.test.ts",`
	`1858`	`+"test:macos:ci": "node scripts/test-projects.mjs src/daemon/launchd.test.ts src/daemon/runtime-paths.test.ts src/daemon/runtime-binary.test.ts src/infra/brew.test.ts src/infra/stable-node-path.test.ts test/scripts/vitest-process-group.test.ts test/scripts/package-mac-app.test.ts test/scripts/package-mac-dist.test.ts test/scripts/create-dmg.test.ts test/scripts/codesign-mac-app.test.ts test/scripts/notarize-mac-artifact.test.ts",`
`1859`	`1859`	`"test:max": "node scripts/test-projects-max.mjs",`
`1860`	`1860`	`"test:parallels:linux": "bash scripts/e2e/parallels-linux-smoke.sh",`
`1861`	`1861`	`"test:parallels:macos": "bash scripts/e2e/parallels-macos-smoke.sh",`

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,8 @@ const ANDROID_VERSION_SYNC_PATHS = new Set([`
`57`	`57`	`"apps/android/fastlane/metadata/android/en-US/release_notes.txt",`
`58`	`58`	`"apps/android/version.json",`
`59`	`59`	`]);`
`60`		`-const MACOS_APP_CI_PATH_RE = /^(?:apps\/(?:macos\|macos-mlx-tts\|shared\|swabble)\/\|Swabble\/)/u;`
	`60`	`+const MACOS_APP_CI_PATH_RE =`
	`61`	`+/^(?:apps\/(?:macos\|macos-mlx-tts\|shared\|swabble)\/\|Swabble\/\|scripts\/(?:codesign-mac-app\|create-dmg\|notarize-mac-artifact\|package-mac-app\|package-mac-dist)\.sh$\|scripts\/lib\/plistbuddy\.sh$\|test\/scripts\/(?:codesign-mac-app\|create-dmg\|notarize-mac-artifact\|package-mac-app\|package-mac-dist)\.test\.ts$)/u;`
`61`	`62`	`let corepackPnpmShimDir;`
`62`	`63`	`let corepackPnpmShimCleanupRegistered = false;`
`63`	`64`
`@@ -414,7 +415,7 @@ export function createChangedCheckPlan(result, options = {}) {`
`414`	`415`	`} else if (lanes.apps) {`
`415`	`416`	`addLint("lint apps", ["lint:apps"]);`
`416`	`417`	`}`
`417`		`-if (lanes.apps && hasMacosAppCiPath(result.paths)) {`
	`418`	`+if (hasMacosAppCiPath(result.paths)) {`
`418`	`419`	`add("macOS app CI tests", ["test:macos:ci"], baseEnv);`
`419`	`420`	`}`
`420`	`421`

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,8 @@ const INSTALL_SMOKE_WORKFLOW_SCOPE_RE = /^\.github\/workflows\/install-smoke\.ym`
`34`	`34`	`const NATIVE_PROTOCOL_GEN_RE = /^apps\/shared\/OpenClawKit\/Sources\/OpenClawProtocol\//;`
`35`	`35`	`const MACOS_NATIVE_RE =`
`36`	`36`	`/^(apps\/macos\/\|apps\/macos-mlx-tts\/\|apps\/ios\/\|apps\/shared\/\|apps\/swabble\/\|Swabble\/)/;`
`37`		`-const MACOS_SCRIPT_SCOPE_RE = /^(scripts\/create-dmg\.sh\|test\/scripts\/create-dmg\.test\.ts)$/;`
	`37`	`+const MACOS_SCRIPT_SCOPE_RE =`
	`38`	`+/^(?:scripts\/(?:codesign-mac-app\|create-dmg\|notarize-mac-artifact\|package-mac-app\|package-mac-dist)\.sh\|scripts\/lib\/plistbuddy\.sh\|test\/scripts\/(?:codesign-mac-app\|create-dmg\|notarize-mac-artifact\|package-mac-app\|package-mac-dist)\.test\.ts)$/;`
`38`	`39`	`const ANDROID_NATIVE_RE = /^(apps\/android\/\|apps\/shared\/)/;`
`39`	`40`	`const NODE_SCOPE_RE =`
`40`	`41`	`/^(src\/\|test\/\|extensions\/\|packages\/\|scripts\/\|ui\/\|\.github\/\|openclaw\.mjs$\|package\.json$\|pnpm-lock\.yaml$\|pnpm-workspace\.yaml$\|tsconfig.\.json$\|vitest.\.ts$\|tsdown\.config\.ts$\|\.oxlintrc\.json$\|\.oxfmtrc\.jsonc$)/;`

Original file line number	Diff line number	Diff line change
`@@ -296,27 +296,44 @@ describe("detectChangedScope", () => {`
`296`	`296`	`});`
`297`	`297`
`298`	`298`	`it("runs macOS CI for macOS packaging scripts with Darwin-only tests", () => {`
`299`		`-expect(detectChangedScope(["scripts/create-dmg.sh"])).toEqual({`
`300`		`-runNode: true,`
`301`		`-runMacos: true,`
`302`		`-runAndroid: false,`
`303`		`-runWindows: false,`
`304`		`-runSkillsPython: false,`
`305`		`-runChangedSmoke: false,`
`306`		`-runControlUiI18n: false,`
`307`		`-});`
	`299`	`+for (const changedPath of [`
	`300`	`+"scripts/codesign-mac-app.sh",`
	`301`	`+"scripts/create-dmg.sh",`
	`302`	`+"scripts/lib/plistbuddy.sh",`
	`303`	`+"scripts/notarize-mac-artifact.sh",`
	`304`	`+"scripts/package-mac-app.sh",`
	`305`	`+"scripts/package-mac-dist.sh",`
	`306`	`+]) {`
	`307`	`+expect(detectChangedScope([changedPath])).toEqual({`
	`308`	`+runNode: true,`
	`309`	`+runMacos: true,`
	`310`	`+runAndroid: false,`
	`311`	`+runWindows: false,`
	`312`	`+runSkillsPython: false,`
	`313`	`+runChangedSmoke: false,`
	`314`	`+runControlUiI18n: false,`
	`315`	`+});`
	`316`	`+}`
`308`	`317`	`});`
`309`	`318`
`310`		`-it("runs macOS CI for the Darwin-only DMG owner test", () => {`
`311`		`-expect(detectChangedScope(["test/scripts/create-dmg.test.ts"])).toEqual({`
`312`		`-runNode: true,`
`313`		`-runMacos: true,`
`314`		`-runAndroid: false,`
`315`		`-runWindows: false,`
`316`		`-runSkillsPython: false,`
`317`		`-runChangedSmoke: false,`
`318`		`-runControlUiI18n: false,`
`319`		`-});`
	`319`	`+it("runs macOS CI for Darwin-only mac packaging owner tests", () => {`
	`320`	`+for (const changedPath of [`
	`321`	`+"test/scripts/codesign-mac-app.test.ts",`
	`322`	`+"test/scripts/create-dmg.test.ts",`
	`323`	`+"test/scripts/notarize-mac-artifact.test.ts",`
	`324`	`+"test/scripts/package-mac-app.test.ts",`
	`325`	`+"test/scripts/package-mac-dist.test.ts",`
	`326`	`+]) {`
	`327`	`+expect(detectChangedScope([changedPath])).toEqual({`
	`328`	`+runNode: true,`
	`329`	`+runMacos: true,`
	`330`	`+runAndroid: false,`
	`331`	`+runWindows: false,`
	`332`	`+runSkillsPython: false,`
	`333`	`+runChangedSmoke: false,`
	`334`	`+runControlUiI18n: false,`
	`335`	`+});`
	`336`	`+}`
`320`	`337`	`});`
`321`	`338`
`322`	`339`	`it("runs Windows only for Windows-relevant changes", () => {`

Original file line number	Diff line number	Diff line change
`@@ -1502,6 +1502,40 @@ describe("scripts/changed-lanes", () => {`
`1502`	`1502`	`}`
`1503`	`1503`	`});`
`1504`	`1504`
	`1505`	`+it("runs macOS app CI tests for macOS packaging scripts and owner tests", () => {`
	`1506`	`+for (const changedPath of [`
	`1507`	`+"scripts/codesign-mac-app.sh",`
	`1508`	`+"scripts/create-dmg.sh",`
	`1509`	`+"scripts/lib/plistbuddy.sh",`
	`1510`	`+"scripts/notarize-mac-artifact.sh",`
	`1511`	`+"scripts/package-mac-app.sh",`
	`1512`	`+"scripts/package-mac-dist.sh",`
	`1513`	`+"test/scripts/codesign-mac-app.test.ts",`
	`1514`	`+"test/scripts/create-dmg.test.ts",`
	`1515`	`+"test/scripts/notarize-mac-artifact.test.ts",`
	`1516`	`+"test/scripts/package-mac-app.test.ts",`
	`1517`	`+"test/scripts/package-mac-dist.test.ts",`
	`1518`	`+]) {`
	`1519`	`+const result = detectChangedLanes([changedPath]);`
	`1520`	`+const plan = createChangedCheckPlan(result, {`
	`1521`	`+env: { PATH: "/usr/bin" },`
	`1522`	`+platform: "linux",`
	`1523`	`+swiftlintAvailable: false,`
	`1524`	`+});`
	`1525`	`+`
	`1526`	`+expectLanes(result.lanes, {`
	`1527`	`+tooling: true,`
	`1528`	`+});`
	`1529`	`+expect(plan.commands.map((command) => command.args[0])).not.toContain("lint:apps");`
	`1530`	`+expect(plan.commands).toContainEqual(`
	`1531`	`+expect.objectContaining({`
	`1532`	`+name: "macOS app CI tests",`
	`1533`	`+args: ["test:macos:ci"],`
	`1534`	`+}),`
	`1535`	`+);`
	`1536`	`+}`
	`1537`	`+});`
	`1538`	`+`
`1505`	`1539`	`it("routes appcast changes to appcast owner tests", () => {`
`1506`	`1540`	`const result = detectChangedLanes(["appcast.xml"]);`
`1507`	`1541`	`const plan = createChangedCheckPlan(result);`

Original file line number	Diff line number	Diff line change
`@@ -377,6 +377,8 @@ describe("package-mac-app plist stamping", () => {`
`377`	`377`	`expect(macosCi).toContain("test/scripts/package-mac-app.test.ts");`
`378`	`378`	`expect(macosCi).toContain("test/scripts/package-mac-dist.test.ts");`
`379`	`379`	`expect(macosCi).toContain("test/scripts/create-dmg.test.ts");`
	`380`	`+expect(macosCi).toContain("test/scripts/codesign-mac-app.test.ts");`
	`381`	`+expect(macosCi).toContain("test/scripts/notarize-mac-artifact.test.ts");`
`380`	`382`	`});`
`381`	`383`
`382`	`384`	`it("fails closed when required Swift resources are missing", () => {`

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

推荐订阅源

Recent Commits to openclaw:main