chore(preflight): add ktlintCheck + detekt to the fast gate #55

Merged
JMR-dev merged 1 commits from chore-preflight-static-analysis into main 2026-07-01 20:38:18 +00:00
JMR-dev commented 2026-07-01 20:28:39 +00:00 (Migrated from github.com)

Why

PR #54''s CI failed on Static analysis (ktlint) even though the local /preflight gate was green. The gate ran assembleDebug + testDebugUnitTest + lintDebug, but CI''s Static analysis job runs :app:ktlintCheck :app:detekt — and lintDebug does not cover the test/androidTest source sets, so a style violation there slipped through locally and only failed after push.

Change

Docs/tooling only — no code:

  • .claude/skills/preflight/SKILL.md — add :app:ktlintCheck :app:detekt to the steps, note it mirrors CI''s Static analysis job and covers the source sets lintDebug skips, and point at the ktlint/detekt reports on failure. Updated the skill description accordingly.
  • CLAUDE.md — add the same command to the build/test/lint block and update the "fast CI gate" sentence to match.

🤖 Generated with Claude Code

## Why PR #54''s CI failed on **Static analysis** (ktlint) even though the local `/preflight` gate was green. The gate ran `assembleDebug` + `testDebugUnitTest` + `lintDebug`, but CI''s Static analysis job runs `:app:ktlintCheck :app:detekt` — and `lintDebug` does not cover the `test`/`androidTest` source sets, so a style violation there slipped through locally and only failed after push. ## Change Docs/tooling only — no code: - `.claude/skills/preflight/SKILL.md` — add `:app:ktlintCheck :app:detekt` to the steps, note it mirrors CI''s Static analysis job and covers the source sets `lintDebug` skips, and point at the ktlint/detekt reports on failure. Updated the skill description accordingly. - `CLAUDE.md` — add the same command to the build/test/lint block and update the "fast CI gate" sentence to match. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.