Trim CLAUDE.md to caveman-terse, add add-mobile-module skill - #36
Merged
Conversation
The old CLAUDE.md had drifted stale (it still listed Gradle 8.9 / AGP 8.7 / compileSdk 35, all superseded by #32) and buried the facts a fresh agent actually can't discover by reading code. Cut it to a short, high-signal file and move the procedural detail into a skill. CLAUDE.md now leads with the environment facts that are invisible from the source: no Android SDK in the cloud sandbox (gradle build/lint/test fail locally; CI is the sole gate), the per-call-site lint MissingPermission rule, and that trifecta/policy classification lives in the edison-watch repo. Points at libs.versions.toml as the version source of truth instead of copying numbers that rot. New .claude/skills/add-mobile-module skill carries the module contract (*Source interface + Android*.kt impl + JVM test), the in-band-error and hex-payload rules, the SecurityException-in-the-same-function lint gotcha, and the JVM scratch-project recipe for verifying module logic without the SDK. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NgN7YdfLT41kWnWqe1MPZa
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
github-actions
Bot
deleted the
claude/mobile-mcp-hardware-gateway-ujnkuw
branch
August 28, 2026 17:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Onboarding a fresh coding agent to this repo: most of what it needs is discoverable by reading the (clean, modular) code. A handful of things are not — they're facts about the environment, not the source — and the old
CLAUDE.mdboth buried them and had drifted stale (it still listedGradle 8.9 / AGP 8.7 / compileSdk 35, all superseded by #32). A stale CLAUDE.md is worse than a short one.What
CLAUDE.md→ caveman-terse (~180 words of prose, down from ~800). Leads with the facts an agent can't read its way to:dl.google.comblocked) →./gradlew assembleDebug|lintDebug|testDebugUnitTestfail locally; CI is the sole build+lint+test gate.MissingPermissionis per-call-site → catchSecurityExceptionin the same function.edison-watchrepo, not here.gradle/libs.versions.tomlas the version source of truth instead of copying numbers that rot.New
.claude/skills/add-mobile-module/SKILL.md— the procedural detail, loaded only when relevant. Carries the module contract (*Sourceinterface +Android*.ktimpl + JVM test), the in-band-error and hex-payload rules, the same-function lint gotcha, and the JVM scratch-project recipe for verifying module logic without the SDK.No code or behavior changes — docs + agent config only.
🤖 Generated with Claude Code
https://claude.ai/code/session_01NgN7YdfLT41kWnWqe1MPZa
Generated by Claude Code
Summary by cubic
Trims
CLAUDE.mdfrom ~800 words to ~180 and moves the procedural module-building detail into a new.claude/skills/add-mobile-moduleskill file. The old doc had drifted stale and buried the environment facts a fresh agent can't discover by reading code. Docs and agent config only — no code or behavior changes.Content split
CLAUDE.mdnow opens with the environment facts not visible in source: no Android SDK in the cloud sandbox (gradle build/lint/test fail locally; CI is the sole gate), per-call-site lintMissingPermission, and policy classification living inedison-watch.gradle/libs.versions.tomlas the source of truth.*Sourceinterface +Android*.ktimpl + JVM test), in-band error and hex-payload rules, the same-functionSecurityExceptiongotcha, and a JVM scratch-project recipe for verifying logic without the SDK.Written for commit a558fc2. Summary will update on new commits.