docs: add agent-feedback system - #889
Conversation
Record actionable observations that are out of scope for the task that surfaced them, instead of dropping them or widening an unrelated diff. Ported from the marko repo, with an a11y category and a rule requiring entries to name which framework packages a defect reaches. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
There was a problem hiding this comment.
Pull request overview
Adds an agent-feedback/ system to the repo so out-of-scope but actionable findings are captured in a structured place instead of being silently dropped or fixed in unrelated diffs, and links it from CLAUDE.md.
Changes:
- Documented the “agent feedback” workflow and rules in
agent-feedback/README.md. - Added empty category stubs (
bugs,a11y,cleanup,perf,dx,unclear) for consistent capture. - Added a pointer to the system from
CLAUDE.md(and reinforced the “don’t widen diffs” constraint).
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| CLAUDE.md | Adds the out-of-scope fix constraint and a new “Agent Feedback” section linking to the system. |
| agent-feedback/README.md | Defines when/how to add entries, required metadata, and the entry template. |
| agent-feedback/bugs.md | Stub category file for suspected bugs. |
| agent-feedback/a11y.md | Stub category file for accessibility gaps. |
| agent-feedback/cleanup.md | Stub category file for cleanup/refactors. |
| agent-feedback/perf.md | Stub category file for performance/bundle/CSS size opportunities. |
| agent-feedback/dx.md | Stub category file for developer-experience friction. |
| agent-feedback/unclear.md | Stub category file for confusing code/docs notes. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
PR Preview Deployed (removed)Website • commit de0867c |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
Removing in favor of a more comprehensive agent system that @ArtBlue is working on |
Adds an
agent-feedback/directory so actionable observations that are out of scope for the task that surfaced them get recorded instead of dropped — or, worse, fixed inside an unrelated diff. Ported from the marko repo, which has been running this system for a while.Each category file (
bugs,a11y,cleanup,perf,dx,unclear) starts empty;README.mdcovers when to add an entry, the entry format, and the rule that entries are deleted in the same PR as the fix. Two things differ from marko's version: ana11ycategory, since WCAG 2.2 AA is a non-negotiable here and would otherwise be buried inbugs, and a rule that an entry must name which of the five framework packages a defect reaches and which were actually checked.CLAUDE.mdgets a pointer to it, matching how marko surfaces the system.🤖 Generated with Claude Code