@invarn/cibuild 2.8.2 → 2.8.4

package/dist/src/commands/a-properties-stand-in-declares-the-keys-the-build-reads.test.js

@@ -351,4 +351,32 @@ describe("a build that reads the properties file directly", () => {
         expect(await keys()).toEqual(["api.key"]);
     });
 });
+/**
+ * A scan that answers well on a small repository and thins out on a large one
+ * is the worse of the two failures, because the large one is the one someone
+ * is paid to build.
+ *
+ * Manifests and source shared a file budget at first, and source was walked
+ * first. Measured on a real application — 1743 source files, 47 manifests —
+ * the source walk spent the whole budget and the manifests were never opened,
+ * so every key referenced only as `${KEY}` in a manifest came back missing.
+ * Nine of that application's twenty declared keys were lost that way, and the
+ * scan reported the other eleven with no sign anything was wrong.
+ *
+ * This is the only test here that builds a tree big enough to exhaust a
+ * budget, because that is the only way to express the defect. It costs about
+ * half a second.
+ */
+describe("a repository larger than the source budget", () => {
+    test("still reads its manifests", async () => {
+        projectApplyingTheSecretsPlugin();
+        write("app/src/main/AndroidManifest.xml", '<manifest android:value="${MANIFEST_ONLY_KEY}" />\n');
+        // Past SOURCE_FILE_CAP, spread over directories so no single readdir is
+        // unrepresentative.
+        for (let i = 0; i < 8100; i++) {
+            write(`app/src/main/java/p${i % 40}/F${i}.kt`, `val x${i} = 1\n`);
+        }
+        expect(await keys()).toContain("MANIFEST_ONLY_KEY");
+    }, 60_000);
+});
 //# sourceMappingURL=a-properties-stand-in-declares-the-keys-the-build-reads.test.js.map

package/dist/src/commands/android-scanner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"android-scanner.d.ts","sourceRoot":"","sources":["../../../src/commands/android-scanner.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,eAAe,GAAG,SAAS,GAAG,MAAM,CAAC;AAEjD,MAAM,MAAM,eAAe,GACvB,cAAc,GACd,SAAS,GACT,iBAAiB,GACjB,gBAAgB,GAChB,cAAc,GACd,UAAU,GACV,gBAAgB,CAAC;AAErB,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,eAAe,CAAC;IAC1B,QAAQ,EAAE,eAAe,CAAC;IAC1B,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,UAAU;IACzB,QAAQ,EAAE,YAAY,EAAE,CAAC;IACzB,+EAA+E;IAC/E,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B;;;;;;;;OAQG;IACH,yBAAyB,CAAC,EAAE,MAAM,CAAC;IACnC,uFAAuF;IACvF,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,2FAA2F;IAC3F,aAAa,EAAE,aAAa,CAAC;IAC7B;;;;;;;;;;;;;;;;;;;;OAoBG;IACH,mBAAmB,EAAE,MAAM,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,aAAa;IAC5B,gGAAgG;IAChG,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,wFAAwF;IACxF,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,uFAAuF;IACvF,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAikBD;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,2BAA2B,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAcnE;AAyKD,kFAAkF;AAClF,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAO7D;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,uBAAuB,CACrC,IAAI,EAAE,MAAM,GACX;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,GAAG,SAAS,CA8CjD;AA2BD;;;;;;;;;;;GAWG;AACH,wBAAgB,yBAAyB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAG7E;AAQD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,aAAa,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,GAAG,SAAS,CAsBvF;AAED,oEAAoE;AACpE,wBAAgB,eAAe,CAAC,aAAa,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,GAAG,SAAS,CAIrF;AAED;;;;;;;;;;;;;;;GAeG;AACH,eAAO,MAAM,uBAAuB,UAAsB,CAAC;AAE3D;;;;;;;;;;GAUG;AACH,eAAO,MAAM,6BAA6B,QAAuC,CAAC;AAElF,yEAAyE;AACzE,MAAM,MAAM,eAAe,GACvB,SAAS,GACT,WAAW,GACX,mBAAmB,GACnB,eAAe,GACf,uBAAuB,CAAC;AAE5B,+DAA+D;AAC/D,MAAM,WAAW,iBAAiB;IAChC,6DAA6D;IAC7D,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,EAAE,eAAe,CAAC;IACtB,0EAA0E;IAC1E,QAAQ,EAAE,MAAM,CAAC;IACjB,2EAA2E;IAC3E,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,SAAS,EAAE,MAAM,EAAE,CAAC;CACrB;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAuCG;AACH,wBAAgB,iBAAiB,CAC/B,mBAAmB,EAAE,MAAM,GAAG,SAAS,EACvC,aAAa,CAAC,EAAE,MAAM,GACrB,iBAAiB,CAkCnB;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,mBAAmB,CACjC,mBAAmB,EAAE,MAAM,GAAG,SAAS,EACvC,aAAa,CAAC,EAAE,MAAM,GACrB,MAAM,GAAG,SAAS,CAEpB;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,kBAAkB,CAChC,mBAAmB,EAAE,MAAM,GAAG,SAAS,EACvC,aAAa,CAAC,EAAE,MAAM,EACtB,MAAM,CAAC,EAAE,MAAM,GACd,MAAM,GAAG,SAAS,CAwBpB;AAwLD;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,aAAa,CAsCxE;AAMD,wBAAsB,kBAAkB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC,CA8RjF;AAgBD,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,UAAU,GAAG,MAAM,CAqE3D"}
+{"version":3,"file":"android-scanner.d.ts","sourceRoot":"","sources":["../../../src/commands/android-scanner.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,eAAe,GAAG,SAAS,GAAG,MAAM,CAAC;AAEjD,MAAM,MAAM,eAAe,GACvB,cAAc,GACd,SAAS,GACT,iBAAiB,GACjB,gBAAgB,GAChB,cAAc,GACd,UAAU,GACV,gBAAgB,CAAC;AAErB,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,eAAe,CAAC;IAC1B,QAAQ,EAAE,eAAe,CAAC;IAC1B,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,UAAU;IACzB,QAAQ,EAAE,YAAY,EAAE,CAAC;IACzB,+EAA+E;IAC/E,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B;;;;;;;;OAQG;IACH,yBAAyB,CAAC,EAAE,MAAM,CAAC;IACnC,uFAAuF;IACvF,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,2FAA2F;IAC3F,aAAa,EAAE,aAAa,CAAC;IAC7B;;;;;;;;;;;;;;;;;;;;OAoBG;IACH,mBAAmB,EAAE,MAAM,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,aAAa;IAC5B,gGAAgG;IAChG,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,wFAAwF;IACxF,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,uFAAuF;IACvF,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAolBD;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,2BAA2B,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAcnE;AAyKD,kFAAkF;AAClF,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAO7D;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,uBAAuB,CACrC,IAAI,EAAE,MAAM,GACX;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,GAAG,SAAS,CA8CjD;AA2BD;;;;;;;;;;;GAWG;AACH,wBAAgB,yBAAyB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAG7E;AAQD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,aAAa,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,GAAG,SAAS,CAsBvF;AAED,oEAAoE;AACpE,wBAAgB,eAAe,CAAC,aAAa,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,GAAG,SAAS,CAIrF;AAED;;;;;;;;;;;;;;;GAeG;AACH,eAAO,MAAM,uBAAuB,UAAsB,CAAC;AAE3D;;;;;;;;;;GAUG;AACH,eAAO,MAAM,6BAA6B,QAAuC,CAAC;AAElF,yEAAyE;AACzE,MAAM,MAAM,eAAe,GACvB,SAAS,GACT,WAAW,GACX,mBAAmB,GACnB,eAAe,GACf,uBAAuB,CAAC;AAE5B,+DAA+D;AAC/D,MAAM,WAAW,iBAAiB;IAChC,6DAA6D;IAC7D,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,EAAE,eAAe,CAAC;IACtB,0EAA0E;IAC1E,QAAQ,EAAE,MAAM,CAAC;IACjB,2EAA2E;IAC3E,OAAO,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,SAAS,EAAE,MAAM,EAAE,CAAC;CACrB;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAuCG;AACH,wBAAgB,iBAAiB,CAC/B,mBAAmB,EAAE,MAAM,GAAG,SAAS,EACvC,aAAa,CAAC,EAAE,MAAM,GACrB,iBAAiB,CAkCnB;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,mBAAmB,CACjC,mBAAmB,EAAE,MAAM,GAAG,SAAS,EACvC,aAAa,CAAC,EAAE,MAAM,GACrB,MAAM,GAAG,SAAS,CAEpB;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,kBAAkB,CAChC,mBAAmB,EAAE,MAAM,GAAG,SAAS,EACvC,aAAa,CAAC,EAAE,MAAM,EACtB,MAAM,CAAC,EAAE,MAAM,GACd,MAAM,GAAG,SAAS,CAwBpB;AAwLD;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,aAAa,CAsCxE;AAMD,wBAAsB,kBAAkB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC,CA8RjF;AAgBD,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,UAAU,GAAG,MAAM,CAqE3D"}

package/dist/src/commands/android-scanner.js

@@ -433,7 +433,23 @@ const MACHINE_LOCAL_KEYS = new Set(["sdk.dir", "ndk.dir", "cmake.dir"]);
 // runs on every scan of one.
 const SOURCE_EXTENSIONS = [".kt", ".kts", ".java"];
 const SOURCE_SKIP_DIRS = new Set(["build", "node_modules"]);
-const SOURCE_FILE_CAP = 1500;
+/**
+ * **Manifests get their own budget, and a source walk cannot spend it.**
+ *
+ * They shared one at first, with source walked first, and on a real application
+ * that meant the manifests were never reached at all: 1743 source files against
+ * a 1500-file cap left nothing, and the keys referenced only as `${KEY}` in a
+ * manifest — which is most of them — came back missing. A scan that answers
+ * well on a small repository and silently thins out on a large one is the worse
+ * of the two failures, because the large one is the one someone is paid to
+ * build.
+ *
+ * There are few manifests and many source files — 47 against 1743 in that same
+ * application — so the two are not competing for a scarce resource, they were
+ * just sharing the wrong one.
+ */
+const SOURCE_FILE_CAP = 8000;
+const MANIFEST_FILE_CAP = 500;
 const SOURCE_DEPTH_CAP = 12;
 /** Source files under `dir`, up to the shared file budget. */
 function findSourceFiles(dir, budget, depth = 0) {
@@ -536,14 +552,17 @@ function collectPropertyKeys(gradleFiles) {
     }
     if (moduleDirs.size === 0)
         return [];
-    const budget = { left: SOURCE_FILE_CAP };
+    const sourceBudget = { left: SOURCE_FILE_CAP };
+    const manifestBudget = { left: MANIFEST_FILE_CAP };
     for (const dir of outermost([...moduleDirs])) {
-        for (const file of findSourceFiles(dir, budget)) {
-            for (const name of extractBuildConfigFieldRefs(safeRead(file)))
+        // Manifests first. They are the likeliest place for a key and the cheapest
+        // to read, so if anything is ever going to run short it must not be these.
+        for (const manifest of findManifests(dir, manifestBudget)) {
+            for (const name of extractManifestXmlPlaceholders(safeRead(manifest)))
                 keys.add(name);
         }
-        for (const manifest of findManifests(dir, budget)) {
-            for (const name of extractManifestXmlPlaceholders(safeRead(manifest)))
+        for (const file of findSourceFiles(dir, sourceBudget)) {
+            for (const name of extractBuildConfigFieldRefs(safeRead(file)))
                 keys.add(name);
         }
     }

package/dist/src/yaml/steps/android-variant-resolution.test.js

@@ -15,7 +15,11 @@
  * not re-implemented.
  */
 import { execFileSync } from "node:child_process";
-import { PICK_VARIANT_TASK_FUNCTION, variantTaskCommands } from "./android.js";
+import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { AndroidBuildForUITestingStepExecutor, AndroidLintStepExecutor, AndroidUnitTestStepExecutor, GradleBuildStepExecutor, PICK_VARIANT_TASK_FUNCTION, gradleProjectDir, gradleTaskPath, variantTaskCommands, } from "./android.js";
+import { testConfig } from "./test-config.js";
 /** Run the shipped shell function over some Gradle output. */
 function pick(gradleOutput, prefix, suffix, buildType) {
     const script = `${PICK_VARIANT_TASK_FUNCTION}\ncibuild_pick_variant_task "${prefix}" "${suffix}" "${buildType}"`;
@@ -114,4 +118,143 @@ describe("the step that uses it", () => {
         expect(script).toContain('exit "$VARIANT_STATUS"');
     });
 });
+/**
+ * An app whose root project is the application: no `include`, the Android
+ * application plugin applied in the root build file, sources in `src/`.
+ * Gradle's path for that project is `:`, so a pipeline says `module: ":"`.
+ */
+describe("a module that names the root project", () => {
+    const escape = (value) => value;
+    test("runs the root project's own task, with no empty module segment", () => {
+        const script = variantTaskCommands({
+            module: ":",
+            variant: "debug",
+            prefix: "test",
+            suffix: "UnitTest",
+            escape,
+        }).join("\n");
+        expect(script).toContain("$GRADLE_CMD :testDebugUnitTest");
+        expect(script).not.toContain("::");
+    });
+    test("retries the resolved task on the root project too", () => {
+        const script = variantTaskCommands({
+            module: ":",
+            variant: "debug",
+            prefix: "lint",
+            suffix: "",
+            escape,
+        }).join("\n");
+        expect(script).toContain("$GRADLE_CMD :lintDebug");
+        expect(script).toContain("$GRADLE_CMD :$RESOLVED_TASK");
+        expect(script).toContain('echo "Using :$RESOLVED_TASK');
+        expect(script).not.toContain("::");
+    });
+    // An explicit empty string reaches the step as is; it must not become
+    // `::testDebugUnitTest`.
+    test("treats an empty module as the root", () => {
+        const script = variantTaskCommands({
+            module: "",
+            variant: "debug",
+            prefix: "test",
+            suffix: "UnitTest",
+            escape,
+        }).join("\n");
+        expect(script).toContain("$GRADLE_CMD :testDebugUnitTest");
+        expect(script).not.toContain("::");
+    });
+    test("leaves a named module exactly as it was", () => {
+        const options = { variant: "debug", prefix: "test", suffix: "UnitTest", escape };
+        const script = variantTaskCommands({ module: "app", ...options }).join("\n");
+        expect(script).toContain("$GRADLE_CMD :app:testDebugUnitTest 2>&1");
+        expect(script).toContain("  echo \"Using :app:$RESOLVED_TASK — resolved VARIANT=$RESOLVED_VARIANT\"");
+        expect(script).toContain("  $GRADLE_CMD :app:$RESOLVED_TASK");
+    });
+    test("keeps a nested module's path", () => {
+        const script = variantTaskCommands({
+            module: "feature:store",
+            variant: "debug",
+            prefix: "lint",
+            suffix: "",
+            escape,
+        }).join("\n");
+        expect(script).toContain("$GRADLE_CMD :feature:store:lintDebug");
+    });
+});
+describe("gradleTaskPath and gradleProjectDir", () => {
+    test("compose a task path", () => {
+        expect(gradleTaskPath(":", "lintDebug")).toBe(":lintDebug");
+        expect(gradleTaskPath("", "lintDebug")).toBe(":lintDebug");
+        expect(gradleTaskPath("app", "lintDebug")).toBe(":app:lintDebug");
+    });
+    test("compose the project directory, empty for the root", () => {
+        expect(gradleProjectDir(":")).toBe("");
+        expect(gradleProjectDir("")).toBe("");
+        expect(gradleProjectDir("app")).toBe("app/");
+    });
+});
+describe("the steps, for the root project", () => {
+    test("lint runs :lintDebug and reports under build/", async () => {
+        const result = await new AndroidLintStepExecutor().execute({ module: ":" }, {}, testConfig);
+        const script = result.script;
+        expect(script).toContain("$GRADLE_CMD :lintDebug");
+        expect(script).not.toContain("::");
+        expect(script).toContain('echo "Lint report location: build/reports/lint-results-debug.html"');
+        expect(script).toContain('echo "Running lint on the root project"');
+    });
+    test("unit tests run :testDebugUnitTest and report under build/", async () => {
+        const result = await new AndroidUnitTestStepExecutor().execute({ module: ":" }, {}, testConfig);
+        const script = result.script;
+        expect(script).toContain("$GRADLE_CMD :testDebugUnitTest");
+        expect(script).not.toContain("::");
+        expect(script).toContain('echo "Test report location: build/reports/tests/testDebugUnitTest/index.html"');
+    });
+    test("an absent module still means app", async () => {
+        const lint = await new AndroidLintStepExecutor().execute({}, {}, testConfig);
+        const unit = await new AndroidUnitTestStepExecutor().execute({}, {}, testConfig);
+        expect(lint.script).toContain("$GRADLE_CMD :app:lintDebug");
+        expect(lint.script).toContain('echo "Running lint on module: app"');
+        expect(unit.script).toContain('echo "Running unit tests on module: app"');
+        expect(lint.script).toContain('echo "Lint report location: app/build/reports/lint-results-debug.html"');
+        expect(unit.script).toContain("$GRADLE_CMD :app:testDebugUnitTest");
+    });
+    test("UI testing assembles on the root and finds its APKs", async () => {
+        const result = await new AndroidBuildForUITestingStepExecutor().execute({ module: ":" }, {}, testConfig);
+        const script = result.script;
+        expect(script).toContain("$GRADLE_CMD :assembleDebug :assembleDebugAndroidTest");
+        expect(script).not.toContain("::");
+        const find = script.split("\n").find((l) => l.startsWith("APP_APK="));
+        const pattern = find.match(/-path "([^"]+)"/)[1];
+        const dir = mkdtempSync(join(tmpdir(), "cibuild-root-apk-"));
+        try {
+            mkdirSync(join(dir, "build/outputs/apk/debug"), { recursive: true });
+            writeFileSync(join(dir, "build/outputs/apk/debug/x.apk"), "");
+            const found = execFileSync("find", [".", "-path", pattern], { cwd: dir, encoding: "utf-8" }).trim();
+            expect(found).toBe("./build/outputs/apk/debug/x.apk");
+        }
+        finally {
+            rmSync(dir, { recursive: true, force: true });
+        }
+    });
+});
+/**
+ * gradle-build takes a bare `assembleDebug`, which a root project satisfies.
+ * Its artifact collection must still see `./build/outputs/...`.
+ */
+describe("gradle-build's artifact collection", () => {
+    test("finds an APK the root project wrote", async () => {
+        const result = await new GradleBuildStepExecutor().execute({}, {}, testConfig);
+        const script = result.script;
+        const pattern = script.match(/-path "(\*\/build\/outputs\/apk\/[^"]+)"/)[1];
+        const dir = mkdtempSync(join(tmpdir(), "cibuild-root-apk-"));
+        try {
+            mkdirSync(join(dir, "build/outputs/apk/debug"), { recursive: true });
+            writeFileSync(join(dir, "build/outputs/apk/debug/x.apk"), "");
+            const found = execFileSync("find", [".", "-path", pattern], { cwd: dir, encoding: "utf-8" }).trim();
+            expect(found).toBe("./build/outputs/apk/debug/x.apk");
+        }
+        finally {
+            rmSync(dir, { recursive: true, force: true });
+        }
+    });
+});
 //# sourceMappingURL=android-variant-resolution.test.js.map

package/dist/src/yaml/steps/android.d.ts

@@ -85,6 +85,14 @@ export declare class GradleBuildStepExecutor extends BaseStepExecutor {
  * Written as a shell function so what ships is what the test runs.
  */
 export declare const PICK_VARIANT_TASK_FUNCTION = "cibuild_pick_variant_task() {\n  # $1 task prefix (lint, test), $2 task suffix (empty, UnitTest),\n  # $3 the build type to prefer (Debug). Gradle's output arrives on stdin.\n  awk '/Candidates are:/ { sub(/.*Candidates are:/, \"\"); print; exit }' |\n    tr ',' '\\n' |\n    tr -d \" '\\\".\" |\n    grep -E \"^${1}[A-Za-z0-9]*${3}${2}$\" |\n    head -1\n}";
+/**
+ * The Gradle path of a task in a module: `:app:lintDebug`, or `:lintDebug`
+ * for an app whose root project is the application. The leading `:` keeps it
+ * the root project's own task rather than every project's task of that name.
+ */
+export declare function gradleTaskPath(module: string, task: string): string;
+/** A module's directory with a trailing slash, or `''` for the root project. */
+export declare function gradleProjectDir(module: string): string;
 /**
  * The commands that run a variant-specific Gradle task and, if the variant
  * turns out to be a guess Gradle cannot resolve, resolve it and retry once.

package/dist/src/yaml/steps/android.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"android.d.ts","sourceRoot":"","sources":["../../../../src/yaml/steps/android.ts"],"names":[],"mappings":"AAAA;;GAEG;AAUH,OAAO,EAAE,gBAAgB,EAAuB,MAAM,WAAW,CAAC;AAClE,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,KAAK,EAAE,qBAAqB,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AAEhF;;GAEG;AACH,MAAM,WAAW,oBAAoB;IACnC,YAAY,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;CAChC;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,MAAM,CAAC;IAExB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED;;;GAGG;AACH,qBAAa,0BAA2B,SAAQ,gBAAgB;IAC9D,yBAAyB,CACvB,MAAM,EAAE,oBAAoB,EAC5B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC5B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAyB1B;;;OAGG;YACW,4BAA4B;IAwF1C,UAAU,IAAI,UAAU,EAAE;IAUpB,OAAO,CAAC,MAAM,EAAE,oBAAoB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CA6H7G;AAED;;;GAGG;AACH,qBAAa,uBAAwB,SAAQ,gBAAgB;IAC3D,UAAU,IAAI,UAAU,EAAE;IAe1B,yBAAyB,CACvB,OAAO,EAAE,iBAAiB,EAC1B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAiBpB,OAAO,CAAC,MAAM,EAAE,iBAAiB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CA8P1G;AAED;;;GAGG;AACH;;;;;;;;;;;;;;;;;;GAkBG;AACH,eAAO,MAAM,0BAA0B,0WAQrC,CAAC;AAEH;;;;;;;GAOG;AACH,wBAAgB,mBAAmB,CAAC,OAAO,EAAE;IAC3C,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,wDAAwD;IACxD,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,MAAM,CAAC;CACnC,GAAG,MAAM,EAAE,CAuCX;AAED,qBAAa,uBAAwB,SAAQ,gBAAgB;IAC3D,yBAAyB,CACvB,OAAO,EAAE,iBAAiB,EAC1B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAiBpB,OAAO,CAAC,MAAM,EAAE,iBAAiB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CA6L1G;AAED;;;GAGG;AACH,qBAAa,2BAA4B,SAAQ,gBAAgB;IAC/D,yBAAyB,CACvB,OAAO,EAAE,qBAAqB,EAC9B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAiBpB,OAAO,CAAC,MAAM,EAAE,qBAAqB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CAwM9G;AAMD,MAAM,WAAW,8BAA8B;IAC7C,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;GAGG;AACH,qBAAa,oCAAqC,SAAQ,gBAAgB;IACxE,yBAAyB,CACvB,OAAO,EAAE,8BAA8B,EACvC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAM1B,UAAU,IAAI,UAAU,EAAE;IAOpB,OAAO,CAAC,MAAM,EAAE,8BAA8B,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,OAAO,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CAsEzH"}
+{"version":3,"file":"android.d.ts","sourceRoot":"","sources":["../../../../src/yaml/steps/android.ts"],"names":[],"mappings":"AAAA;;GAEG;AAUH,OAAO,EAAE,gBAAgB,EAAuB,MAAM,WAAW,CAAC;AAClE,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,KAAK,EAAE,qBAAqB,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AAEhF;;GAEG;AACH,MAAM,WAAW,oBAAoB;IACnC,YAAY,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;CAChC;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,MAAM,CAAC;IAExB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED;;;GAGG;AACH,qBAAa,0BAA2B,SAAQ,gBAAgB;IAC9D,yBAAyB,CACvB,MAAM,EAAE,oBAAoB,EAC5B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC5B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAyB1B;;;OAGG;YACW,4BAA4B;IAwF1C,UAAU,IAAI,UAAU,EAAE;IAUpB,OAAO,CAAC,MAAM,EAAE,oBAAoB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CA6H7G;AAED;;;GAGG;AACH,qBAAa,uBAAwB,SAAQ,gBAAgB;IAC3D,UAAU,IAAI,UAAU,EAAE;IAe1B,yBAAyB,CACvB,OAAO,EAAE,iBAAiB,EAC1B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAiBpB,OAAO,CAAC,MAAM,EAAE,iBAAiB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CA8P1G;AAED;;;GAGG;AACH;;;;;;;;;;;;;;;;;;GAkBG;AACH,eAAO,MAAM,0BAA0B,0WAQrC,CAAC;AAYH;;;;GAIG;AACH,wBAAgB,cAAc,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAEnE;AAED,gFAAgF;AAChF,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAEvD;AAOD;;;;;;;GAOG;AACH,wBAAgB,mBAAmB,CAAC,OAAO,EAAE;IAC3C,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,wDAAwD;IACxD,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,MAAM,CAAC;CACnC,GAAG,MAAM,EAAE,CAuCX;AAED,qBAAa,uBAAwB,SAAQ,gBAAgB;IAC3D,yBAAyB,CACvB,OAAO,EAAE,iBAAiB,EAC1B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAiBpB,OAAO,CAAC,MAAM,EAAE,iBAAiB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CA6L1G;AAED;;;GAGG;AACH,qBAAa,2BAA4B,SAAQ,gBAAgB;IAC/D,yBAAyB,CACvB,OAAO,EAAE,qBAAqB,EAC9B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAiBpB,OAAO,CAAC,MAAM,EAAE,qBAAqB,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CAwM9G;AAMD,MAAM,WAAW,8BAA8B;IAC7C,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;GAGG;AACH,qBAAa,oCAAqC,SAAQ,gBAAgB;IACxE,yBAAyB,CACvB,OAAO,EAAE,8BAA8B,EACvC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC3B,OAAO,EAAE,QAAQ,GAChB,qBAAqB,EAAE;IAM1B,UAAU,IAAI,UAAU,EAAE;IAOpB,OAAO,CAAC,MAAM,EAAE,8BAA8B,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,OAAO,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;CAsEzH"}

package/dist/src/yaml/steps/android.js

@@ -544,6 +544,31 @@ export const PICK_VARIANT_TASK_FUNCTION = `cibuild_pick_variant_task() {
     grep -E "^\${1}[A-Za-z0-9]*\${3}\${2}$" |
     head -1
 }`;
+/**
+ * Whether a `module` input names the root project. Gradle's own path for the
+ * root is `:`; an explicit empty string is taken to mean the same, since it
+ * would otherwise compose `::<task>`. An absent input never reaches here —
+ * the steps default it to `app`.
+ */
+function isRootProject(module) {
+    return module === ':' || module === '';
+}
+/**
+ * The Gradle path of a task in a module: `:app:lintDebug`, or `:lintDebug`
+ * for an app whose root project is the application. The leading `:` keeps it
+ * the root project's own task rather than every project's task of that name.
+ */
+export function gradleTaskPath(module, task) {
+    return isRootProject(module) ? `:${task}` : `:${module}:${task}`;
+}
+/** A module's directory with a trailing slash, or `''` for the root project. */
+export function gradleProjectDir(module) {
+    return isRootProject(module) ? '' : `${module}/`;
+}
+/** How a log line names the project a step runs on. */
+function gradleProjectLabel(module) {
+    return isRootProject(module) ? 'the root project' : `module: ${module}`;
+}
 /**
  * The commands that run a variant-specific Gradle task and, if the variant
  * turns out to be a guess Gradle cannot resolve, resolve it and retry once.
@@ -555,7 +580,7 @@ export const PICK_VARIANT_TASK_FUNCTION = `cibuild_pick_variant_task() {
 export function variantTaskCommands(options) {
     const { module, variant, prefix, suffix, gradleArgs = '', escape } = options;
     const capitalized = variant.charAt(0).toUpperCase() + variant.slice(1);
-    const task = `:${escape(module)}:${prefix}${capitalized}${suffix}`;
+    const task = gradleTaskPath(escape(module), `${prefix}${capitalized}${suffix}`);
     const args = gradleArgs === '' ? '' : ` ${gradleArgs}`;
     return [
         '',
@@ -580,10 +605,10 @@ export function variantTaskCommands(options) {
         `  RESOLVED_VARIANT=\${RESOLVED_TASK#${prefix}}`,
         ...(suffix === '' ? [] : [`  RESOLVED_VARIANT=\${RESOLVED_VARIANT%${suffix}}`]),
         `  RESOLVED_VARIANT="$(echo "\${RESOLVED_VARIANT:0:1}" | tr '[:upper:]' '[:lower:]')\${RESOLVED_VARIANT:1}"`,
-        `  echo "Using :${escape(module)}:$RESOLVED_TASK — resolved VARIANT=$RESOLVED_VARIANT"`,
+        `  echo "Using ${gradleTaskPath(escape(module), '$RESOLVED_TASK')} — resolved VARIANT=$RESOLVED_VARIANT"`,
         // Best-effort: the discovery is worth having even where envman is not.
         '  envman add --key VARIANT --value "$RESOLVED_VARIANT" || true',
-        `  $GRADLE_CMD :${escape(module)}:$RESOLVED_TASK${args}`,
+        `  $GRADLE_CMD ${gradleTaskPath(escape(module), '$RESOLVED_TASK')}${args}`,
         'elif [ "$VARIANT_STATUS" -ne 0 ]; then',
         '  rm -f "$VARIANT_LOG"',
         '  exit "$VARIANT_STATUS"',
@@ -736,7 +761,7 @@ export class AndroidLintStepExecutor extends BaseStepExecutor {
             commands.push('PROJECT_DIR="${CIBUILD_SOURCE_DIR:-.}"');
         }
         commands.push('echo "Project directory: $PROJECT_DIR"');
-        commands.push(`echo "Running lint on module: ${this.escapeBash(module)}"`);
+        commands.push(`echo "Running lint on ${gradleProjectLabel(this.escapeBash(module))}"`);
         commands.push(`echo "Variant: ${this.escapeBash(variant)}"`);
         // Change to project directory
         commands.push('');
@@ -769,7 +794,7 @@ export class AndroidLintStepExecutor extends BaseStepExecutor {
         commands.push('');
         commands.push('# Lint completed');
         commands.push('echo "Lint analysis completed successfully"');
-        commands.push(`echo "Lint report location: ${this.escapeBash(module)}/build/reports/lint-results-${this.escapeBash(variant)}.html"`);
+        commands.push(`echo "Lint report location: ${gradleProjectDir(this.escapeBash(module))}build/reports/lint-results-${this.escapeBash(variant)}.html"`);
         const script = this.createBashScriptFromCommands(commands, stepName);
         return this.createScriptStep(script, stepName);
     }
@@ -925,7 +950,7 @@ export class AndroidUnitTestStepExecutor extends BaseStepExecutor {
             commands.push('PROJECT_DIR="${CIBUILD_SOURCE_DIR:-.}"');
         }
         commands.push('echo "Project directory: $PROJECT_DIR"');
-        commands.push(`echo "Running unit tests on module: ${this.escapeBash(module)}"`);
+        commands.push(`echo "Running unit tests on ${gradleProjectLabel(this.escapeBash(module))}"`);
         commands.push(`echo "Variant: ${this.escapeBash(variant)}"`);
         // Change to project directory
         commands.push('');
@@ -964,7 +989,7 @@ export class AndroidUnitTestStepExecutor extends BaseStepExecutor {
         commands.push('');
         commands.push('# Tests completed');
         commands.push('echo "Unit tests completed successfully"');
-        commands.push(`echo "Test report location: ${this.escapeBash(module)}/build/reports/tests/test${variantCapitalized}UnitTest/index.html"`);
+        commands.push(`echo "Test report location: ${gradleProjectDir(this.escapeBash(module))}build/reports/tests/test${variantCapitalized}UnitTest/index.html"`);
         const script = this.createBashScriptFromCommands(commands, stepName);
         return this.createScriptStep(script, stepName);
     }
@@ -1013,8 +1038,8 @@ export class AndroidBuildForUITestingStepExecutor extends BaseStepExecutor {
         commands.push('fi');
         commands.push('');
         // Build both app and test APKs
-        const appTask = `:${this.escapeBash(module)}:assemble${variantCap}`;
-        const testTask = `:${this.escapeBash(module)}:assemble${variantCap}AndroidTest`;
+        const appTask = gradleTaskPath(this.escapeBash(module), `assemble${variantCap}`);
+        const testTask = gradleTaskPath(this.escapeBash(module), `assemble${variantCap}AndroidTest`);
         let cmd = `$GRADLE_CMD ${appTask} ${testTask}`;
         if (extraArgs) {
             cmd += ` ${extraArgs}`;
@@ -1024,8 +1049,8 @@ export class AndroidBuildForUITestingStepExecutor extends BaseStepExecutor {
         commands.push('');
         // Locate APKs
         commands.push('# Locate generated APKs');
-        commands.push(`APP_APK=$(find . -path "*/${this.escapeBash(module)}/build/outputs/apk/*/*.apk" ! -name "*androidTest*" | head -1)`);
-        commands.push(`TEST_APK=$(find . -path "*/${this.escapeBash(module)}/build/outputs/apk/*/*.apk" -name "*androidTest*" | head -1)`);
+        commands.push(`APP_APK=$(find . -path "*/${gradleProjectDir(this.escapeBash(module))}build/outputs/apk/*/*.apk" ! -name "*androidTest*" | head -1)`);
+        commands.push(`TEST_APK=$(find . -path "*/${gradleProjectDir(this.escapeBash(module))}build/outputs/apk/*/*.apk" -name "*androidTest*" | head -1)`);
         commands.push('');
         commands.push('if [ -z "$APP_APK" ]; then');
         commands.push('  echo "⚠️  App APK not found"');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invarn/cibuild",
-  "version": "2.8.2",
+  "version": "2.8.4",
   "description": "CI Build CLI — local pipeline orchestration and validation",
   "type": "module",
   "main": "dist/cli.cjs",