4.4 KiB
4.4 KiB
Repository Guidelines
Project Structure & Module Organization
- Source code:
src/(CLI wiring insrc/cli, commands insrc/commands, web provider insrc/provider-web.ts, infra insrc/infra, media pipeline insrc/media). - Tests: colocated
*.test.ts. - Docs:
docs/(images, queue, Pi config). Built output lives indist/.
Build, Test, and Development Commands
- Install deps:
pnpm install - Run CLI in dev:
pnpm clawdis ...(tsx entry) orpnpm devforsrc/index.ts. - Type-check/build:
pnpm build(tsc) - Lint/format:
pnpm lint(biome check),pnpm format(biome format) - Tests:
pnpm test(vitest); coverage:pnpm test:coverage
Coding Style & Naming Conventions
- Language: TypeScript (ESM). Prefer strict typing; avoid
any. - Formatting/linting via Biome; run
pnpm lintbefore commits. - Keep files concise; extract helpers instead of “V2” copies. Use existing patterns for CLI options and dependency injection via
createDefaultDeps. - Keep every file ≤ 500 LOC; refactor or split before exceeding and check frequently.
Testing Guidelines
- Framework: Vitest with V8 coverage thresholds (70% lines/branches/functions/statements).
- Naming: match source names with
*.test.ts; e2e in*.e2e.test.ts. - Run
pnpm test(orpnpm test:coverage) before pushing when you touch logic. - Pure test additions/fixes generally do not need a changelog entry unless they alter user-facing behavior or the user asks for one.
Commit & Pull Request Guidelines
- Follow concise, action-oriented commit messages (e.g.,
CLI: add verbose flag to send). - Group related changes; avoid bundling unrelated refactors.
- PRs should summarize scope, note testing performed, and mention any user-facing changes or new flags.
Security & Configuration Tips
- Web provider stores creds at
~/.clawdis/credentials/; rerunclawdis loginif logged out. - Pi/Tau sessions live under
~/.clawdis/sessions/by default; the base directory is not configurable. - Never commit or publish real phone numbers, videos, or live configuration values. Use obviously fake placeholders in docs, tests, and examples.
Agent-Specific Notes
- Relay is managed by launchctl (label
com.steipete.clawdis). After code changes restart withlaunchctl kickstart -k gui/$UID/com.steipete.clawdisand verify vialaunchctl list | grep clawdis. When debugging on macOS, start/stop the relay via the Clawdis Mac app (or its launchd helper), not ad-hoc tmux sessions. Avoid tmux relays unless explicitly requested, and kill any temporary tunnels before handoff. - macOS logs: use
./scripts/clawlog.sh(akavtlog) to query unified logs for subsystemcom.steipete.clawdis; it supports follow/tail/category filters and expects passwordless sudo for/usr/bin/log. - Also read the shared guardrails at
~/Projects/oracle/AGENTS.mdand~/Projects/agent-scripts/AGENTS.MDbefore making changes; align with any cross-repo rules noted there. - When asked to open a “session” file, open the Pi/Tau session logs under
~/.tau/agent/sessions/clawdis/*.jsonl(newest unless a specific ID is given), not the defaultsessions.json. - Menubar dimming + restart flow mirrors Trimmy: use
scripts/restart-mac.sh(kills all Clawdis variants, runsswift build, packages, relaunches). Icon dimming depends on MenuBarExtraAccess wiring in AppMain; keepappearsDisabledupdates intact when touching the status item. - Voice wake forwarding tips:
- Command template should stay
clawdis-mac agent --message "${text}" --thinking low;VoiceWakeForwarderalready shell-escapes${text}. Don’t add extra quotes. - launchd PATH is minimal; ensure the app’s launch agent sets PATH to include
/opt/homebrew/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Users/steipete/Library/pnpmsopnpm/clawdisbinaries resolve when invoked viaclawdis-mac. - For manual
clawdis sendmessages that include!, use the heredoc pattern noted below to avoid the Bash tool’s escaping.
- Command template should stay
Exclamation Mark Escaping Workaround
The Claude Code Bash tool escapes ! to \\! in command arguments. When using clawdis send with messages containing exclamation marks, use heredoc syntax:
# WRONG - will send "Hello\\!" with backslash
clawdis send --to "+1234" --message 'Hello!'
# CORRECT - use heredoc to avoid escaping
clawdis send --to "+1234" --message "$(cat <<'EOF'
Hello!
EOF
)"
This is a Claude Code quirk, not a clawdis bug.