|
| 1 | +# AGENTS.md |
| 2 | + |
| 3 | +Agent skills for Red Hat Developer Hub (RHDH) plugin development, overlay management, and local testing. Orchestrator skill (`rhdh`) routes to specialized sub-skills (`overlay`, `rhdh-local`, `create-*`). Skills follow the [Agent Skills open standard](https://agentskills.io/specification). See `CONTEXT.md` for domain language and `docs/adr/` for architectural decisions. |
| 4 | + |
| 5 | +## 1. Think Before Coding |
| 6 | + |
| 7 | +**Don't assume. Don't hide confusion. Surface tradeoffs.** |
| 8 | + |
| 9 | +Before implementing: |
| 10 | +- State your assumptions explicitly. If uncertain, ask. |
| 11 | +- If multiple interpretations exist, present them — don't pick silently. |
| 12 | +- If a simpler approach exists, say so. Push back when warranted. |
| 13 | +- If something is unclear, stop. Name what's confusing. Ask. |
| 14 | + |
| 15 | +## 2. Simplicity First |
| 16 | + |
| 17 | +**Minimum code that solves the problem. Nothing speculative.** |
| 18 | + |
| 19 | +- No features beyond what was asked. |
| 20 | +- No abstractions for single-use code. |
| 21 | +- No "flexibility" or "configurability" that wasn't requested. |
| 22 | +- If you write 200 lines and it could be 50, rewrite it. |
| 23 | + |
| 24 | +CLIs in this project use stdlib-only Python (see `docs/adr/0002-stdlib-only-python-clis.md`). Don't introduce dependencies. |
| 25 | + |
| 26 | +## 3. Surgical Changes |
| 27 | + |
| 28 | +**Touch only what you must. Clean up only your own mess.** |
| 29 | + |
| 30 | +When editing existing code: |
| 31 | +- Don't "improve" adjacent code, comments, or formatting. |
| 32 | +- Don't refactor things that aren't broken. |
| 33 | +- Match existing style, even if you'd do it differently. |
| 34 | +- If you notice unrelated dead code, mention it — don't delete it. |
| 35 | + |
| 36 | +When your changes create orphans: |
| 37 | +- Remove imports/variables/functions that YOUR changes made unused. |
| 38 | +- Don't remove pre-existing dead code unless asked. |
| 39 | + |
| 40 | +The test: every changed line should trace directly to what was asked. |
| 41 | + |
| 42 | +## 4. Goal-Driven Execution |
| 43 | + |
| 44 | +**Define success criteria. Loop until verified.** |
| 45 | + |
| 46 | +Transform tasks into verifiable goals: |
| 47 | +- "Add validation" → "Write tests for invalid inputs, then make them pass" |
| 48 | +- "Fix the bug" → "Write a test that reproduces it, then make it pass" |
| 49 | +- "Refactor X" → "Ensure tests pass before and after" |
| 50 | + |
| 51 | +Run `uv run pytest` before reporting any task complete. Do not report completion based on code existing — verify it works. |
| 52 | + |
| 53 | +## 5. No Irreversible Commands Without Confirmation |
| 54 | + |
| 55 | +Never force push, reset HEAD, merge branches, or run destructive commands without asking. If unsure whether a command is destructive, ask. |
| 56 | + |
| 57 | +## 6. Learn From Corrections |
| 58 | + |
| 59 | +If told an implementation was wrong, apply the correction and then record what went wrong so the same mistake is not repeated. Patterns and gotchas specific to this project belong in the relevant `references/` file under each skill. |
| 60 | + |
| 61 | +--- |
| 62 | + |
| 63 | +## Versioning |
| 64 | + |
| 65 | +Single version (`0.2.0`) kept in sync across three files: |
| 66 | + |
| 67 | +- `pyproject.toml` — Python package version |
| 68 | +- `.claude-plugin/plugin.json` — plugin manifest |
| 69 | +- `.claude-plugin/marketplace.json` — marketplace listing (2 occurrences) |
| 70 | + |
| 71 | +Bump all three when releasing. |
| 72 | + |
| 73 | +## Agent skills |
| 74 | + |
| 75 | +### Issue tracker |
| 76 | + |
| 77 | +GitHub Issues via `gh` CLI. See `docs/agents/issue-tracker.md`. |
| 78 | + |
| 79 | +### Triage labels |
| 80 | + |
| 81 | +Default labels (`needs-triage`, `needs-info`, `ready-for-agent`, `ready-for-human`, `wontfix`). See `docs/agents/triage-labels.md`. |
| 82 | + |
| 83 | +### Domain docs |
| 84 | + |
| 85 | +Single-context (`CONTEXT.md` + `docs/adr/` at root). See `docs/agents/domain.md`. |
0 commit comments