docs: update Claude docs from PR review analysis

[claude]

Summary

Weekly analysis of PR comments (2026-04-20 -> 2026-04-27).
Analyzed 7 PRs with 56 resolved/effectively-resolved review threads. Found 3 actionable insights.

Changes

agent_docs/rules.md

• Never assign a void return to a variable in JSDoc examples — await service.method() not const result = await service.method() for Promise<void> methods. Assigning void implies the return is usable, misleading users.
  Source: PR feat(jobs): add Jobs.stop() method [PLT-100574] #337 — reviewer noted that const result = await jobs.stop(...) in JSDoc examples was misleading since stop() returns Promise<void>.

agent_docs/conventions.md

• Parallelize independent API calls with Promise.all — when a method makes multiple independent API calls (e.g., chunk-based key resolution), use Promise.all instead of sequential calls to avoid compounding latency.
  Source: PR feat(jobs): add Jobs.stop() method [PLT-100574] #337 — reviewer noted that chunk fetches were independent and should be parallelized.

Agents.md

• Version bumps go in a separate PR — never include a version bump in a feature or fix PR. Other in-flight changes may need to ship in the same version.
  Source: PR feat(conversations): expose disconnect method for clean process exit #340 — reviewer said "Let's not bump up the version with this change. We might have a few other changes we want to ship with the next version. We'll raise a separate PR for the version change."

No changes

• CLAUDE.md — no relevant insights found
• agent_docs/architecture.md — no relevant insights found

PRs Analyzed

PR	Title	Comments
#371	feat(jobs): add Jobs.resume method	19 threads
#380	docs: update Claude docs from PR review analysis	0 threads
#389	fix: skip download attachment tests due to API bug	0 threads
#340	feat(conversations): expose disconnect method for clean process exit	4 threads
#385	fix: pick user for task assignment	7 threads
#337	feat(jobs): add Jobs.stop() method	29 threads
#347	chore: remove CLI and codedapp-tool packages	1 thread

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud