mirror of
https://github.com/vitorpamplona/amethyst.git
synced 2026-10-30 07:37:38 +00:00
Claude
4eae92964e
Cut the doc from 411 to ~75 lines. The performance, coroutine, caching, logging and KMP rules already live in CLAUDE.md and the skills that agents load, so they are dropped here. The doc now leads with one rule: no PR is opened until a human has reviewed the diff and tested the app by hand on a device, and an agent never opens the PR itself. CONTRIBUTING.md, the PR template and CLAUDE.md now point to that rule. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DbNeTtN2RpXHeGwNzPfvbQ
69 lines
2.8 KiB
Markdown
69 lines
2.8 KiB
Markdown
<!--
|
||
Thanks for contributing to Amethyst! Before opening this PR, please skim
|
||
CONTRIBUTING.md — especially the "Proof of testing" and "Interoperability
|
||
tests" sections if you are not a regular contributor to this repo.
|
||
|
||
Delete any section below that doesn't apply.
|
||
-->
|
||
|
||
## Summary
|
||
|
||
<!-- 1–3 sentences. What changed and why. Not "what files changed" — the
|
||
diff already shows that. -->
|
||
|
||
## Test plan
|
||
|
||
<!-- Required. What did you actually run, on what platform, with what result?
|
||
"CI is green" is necessary but not sufficient — show the new path firing.
|
||
|
||
For UI changes, attach screenshots (light + dark) or a short recording.
|
||
For Android: device model + Android version. For Desktop: OS + window size.
|
||
For build/packaging changes: paste the `./gradlew` command + tail of output.
|
||
|
||
If you are NOT a regular contributor to this repo, this section is required
|
||
regardless of how small the change is — see CONTRIBUTING.md § Proof of
|
||
testing. -->
|
||
|
||
- [ ] Ran `./gradlew spotlessApply` — repo is formatted
|
||
- [ ] Ran `./gradlew test` (or the relevant module's tests)
|
||
- [ ] Manually exercised the change (see notes below)
|
||
|
||
Notes / screenshots:
|
||
|
||
<!-- paste here -->
|
||
|
||
## Interop suites
|
||
|
||
<!-- The interop suites listed in CONTRIBUTING.md § Interoperability tests
|
||
are NOT run in CI. If your change touches the relevant code paths, run them
|
||
locally and tick the box. If your change can't possibly affect them
|
||
(docs-only, UI-only on unrelated screens, etc.), tick "N/A". -->
|
||
|
||
- [ ] N/A — change can't affect wire bytes / decoded audio / MLS state / DM envelopes
|
||
- [ ] Marmot / MLS — `cli/tests/marmot/marmot-interop-headless.sh` (Marmot / MDK `wn`)
|
||
- [ ] NIP-17 DM — `cli/tests/dm/dm-interop-headless.sh`
|
||
- [ ] Audio rooms manual — `cli/tests/nests/nests-interop.sh` (Amethyst ↔ nostrnests.com)
|
||
- [ ] MoQ-lite hang-tier — `:nestsClient:jvmTest -DnestsHangInterop=true`
|
||
- [ ] MoQ-lite browser-tier — `:nestsClient:jvmTest -DnestsBrowserInterop=true`
|
||
- [ ] QUIC interop-runner — `quic/interop/run-matrix.sh -s {aioquic,picoquic,quic-go,quinn}`
|
||
|
||
## AI assistance
|
||
|
||
<!-- Optional disclosure. We accept AI-assisted PRs (Claude Code, Copilot,
|
||
Cursor, Codex, etc.) under the same rules as human PRs — see
|
||
CONTRIBUTING.md § Human and AI contributions. A one-line note here is
|
||
appreciated when an assistant did the bulk of the diff. -->
|
||
|
||
- [ ] Drafted with AI assistance, manually reviewed and tested
|
||
- [ ] Written by hand
|
||
|
||
If "Drafted with AI assistance" is ticked, a human must have reviewed the
|
||
whole diff and tested the app by hand on a device before this PR was
|
||
opened. See [`CONTRIBUTING-WITH-AI.md`](CONTRIBUTING-WITH-AI.md).
|
||
|
||
## License
|
||
|
||
- [ ] By submitting this PR, I agree to license my contribution under the
|
||
MIT license. Any code I did not author personally carries its
|
||
original license header.
|