chore(preflight): compile androidTest source set in the fast gate #209

Merged
JMR-dev merged 2 commits from chore-preflight-compile-androidtest into main 2026-07-03 14:54:16 +00:00
JMR-dev commented 2026-07-03 14:27:36 +00:00 (Migrated from github.com)

Closes #211

Problem

The /preflight fast gate (assembleDebug + testDebugUnitTest + lintDebug +
ktlintCheck + detekt) never compiles the androidTest source set. ktlintCheck/
detekt lint that source set, but linting isn't compiling — a change that breaks
androidTest compilation (E2E/instrumented test compile errors, e.g. from a UI signature
change) passes preflight locally yet only fails once CI runs connectedDebugAndroidTest
or the managed-device matrix. This has already bitten a couple of PRs.

Change

  • .claude/skills/preflight/SKILL.md: add ./gradlew :app:compileDebugAndroidTestKotlin
    to the ## Steps block (right after testDebugUnitTest), with prose explaining what it
    catches that the rest of the gate skips. Also updated the skill's frontmatter
    description to list the new step.
  • CLAUDE.md: updated the "Before treating a change as done" fast-gate summary so it
    matches the skill.

No other files touched.

🤖 Generated with Claude Code

Closes #211 ## Problem The `/preflight` fast gate (`assembleDebug` + `testDebugUnitTest` + `lintDebug` + `ktlintCheck` + `detekt`) never compiles the `androidTest` source set. `ktlintCheck`/ `detekt` lint that source set, but linting isn't compiling — a change that breaks `androidTest` compilation (E2E/instrumented test compile errors, e.g. from a UI signature change) passes preflight locally yet only fails once CI runs `connectedDebugAndroidTest` or the managed-device matrix. This has already bitten a couple of PRs. ## Change - `.claude/skills/preflight/SKILL.md`: add `./gradlew :app:compileDebugAndroidTestKotlin` to the `## Steps` block (right after `testDebugUnitTest`), with prose explaining what it catches that the rest of the gate skips. Also updated the skill's frontmatter `description` to list the new step. - `CLAUDE.md`: updated the "Before treating a change as done" fast-gate summary so it matches the skill. No other files touched. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.