docs: add LEARNINGS.md and PR template for AI agent guidance#98
Conversation
Agent-Logs-Url: https://github.com/nitrocode/token-deathclock/sessions/664ec08c-cf62-4170-9017-7aac277650a9 Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 8 minutes and 42 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |


No structured way existed for AI agents to learn from past PRs or produce consistent PR descriptions. Adds a living learnings log and a GitHub PR template to close both gaps.
Summary
docs/LEARNINGS.mdgives agents a categorised reference of 28 lessons extracted from PRs #54–#94 (mobile/CSS quirks, build system rules, test reliability, architecture boundaries, security, feature design, release flow) plus a per-PR log with Problem/Approach/Learning/Key-files entries..github/pull_request_template.mdenforces a uniform Summary → Changes → Agent Checklist structure on every PR going forward.AGENTS.mdis updated to surface both documents immediately and adds a "must add learning entry" rule to the session-end requirements.Changes
docs/LEARNINGS.md— PR entry template, PR description template, 28-rule categorised learnings tables, pre-populated PR log (feat: add JSDoc + checkJs TypeScript type checking (Option B) #54–docs: add Phase 3 PRDs — 8 satirical engagement features #94).github/pull_request_template.md— Summary, Changes checklist, 10-item Agent Checklist (including learning-entry requirement)AGENTS.md— companion-document table at top, new "Documenting PR learnings" subsection, new "What NOT to Do" ruleAgent Checklist
npm run test:cipasses (all unit tests green, coverage not decreased)npm run build && npm run test:e2epasses (all E2E tests green)script.js,styles.css,*-data.js)innerHTMLvalues pass throughescHtml()death-clock-core.jsuses:pins use a full commit SHA + inline semver commentfeat:,fix:,docs:,chore:, etc.)project-stats.yamlupdated if this session merges one or more PRsdocs/LEARNINGS.md(new pattern or lesson learned)