Skip to content

Trim CLAUDE.md to caveman-terse, add add-mobile-module skill - #36

Merged
Miyamura80 merged 1 commit into
mainfrom
claude/mobile-mcp-hardware-gateway-ujnkuw
Aug 28, 2026
Merged

Trim CLAUDE.md to caveman-terse, add add-mobile-module skill#36
Miyamura80 merged 1 commit into
mainfrom
claude/mobile-mcp-hardware-gateway-ujnkuw

Conversation

@Miyamura80

@Miyamura80 Miyamura80 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

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.md both buried them and had drifted stale (it still listed Gradle 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:

  • No Android SDK in the cloud sandbox (dl.google.com blocked) → ./gradlew assembleDebug|lintDebug|testDebugUnitTest fail locally; CI is the sole build+lint+test gate.
  • lint MissingPermission is per-call-site → catch SecurityException in the same function.
  • Trifecta/policy classification for these tools lives in the edison-watch repo, not here.
  • Points at gradle/libs.versions.toml as 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 (*Source interface + Android*.kt impl + 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.md from ~800 words to ~180 and moves the procedural module-building detail into a new .claude/skills/add-mobile-module skill 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.md now 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 lint MissingPermission, and policy classification living in edison-watch.
  • Drops the hard-coded stack versions that had rotted and points to gradle/libs.versions.toml as the source of truth.
  • The skill carries the module contract (*Source interface + Android*.kt impl + JVM test), in-band error and hex-payload rules, the same-function SecurityException gotcha, and a JVM scratch-project recipe for verifying logic without the SDK.

Written for commit a558fc2. Summary will update on new commits.

Review in cubic

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-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@Miyamura80
Miyamura80 merged commit ee8878d into main Aug 28, 2026
3 checks passed
@github-actions
github-actions Bot deleted the claude/mobile-mcp-hardware-gateway-ujnkuw branch August 28, 2026 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants