Skip to content

Comments

docs: update AGENTS.md with accurate bot architecture and config#1612

Merged
simonkberg merged 1 commit intomainfrom
docs/update-agents-md
Feb 21, 2026
Merged

docs: update AGENTS.md with accurate bot architecture and config#1612
simonkberg merged 1 commit intomainfrom
docs/update-agents-md

Conversation

@simonkberg
Copy link
Owner

Summary

  • Fix misleading simon-bot description — it runs via instrumentation.ts WebSocket subscription at server boot, not via after()
  • Add undocumented next.config.ts settings (output, cacheComponents, reactCompiler, globalNotFound)
  • Add config.ts and instrumentation.ts to directory structure
  • Document rate limit gate and Redis-based message deduplication in Discord/bot sections

Test plan

  • Verify AGENTS.md renders correctly on GitHub
  • Confirm all documented config flags match next.config.ts

🤖 Generated with Claude Code

Fix misleading simon-bot description (runs via instrumentation.ts
WebSocket subscription, not after()), add undocumented next.config.ts
settings, and document rate limit gate and message deduplication.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@railway-app railway-app bot temporarily deployed to simon.dev / simon.dev-pr-1612 February 21, 2026 21:15 Destroyed
@railway-app
Copy link

railway-app bot commented Feb 21, 2026

🚅 Deployed to the simon.dev-pr-1612 environment in simon.dev

Service Status Web Updated (UTC)
simon.dev ✅ Success (View Logs) Web Feb 21, 2026 at 9:16 pm

@claude
Copy link
Contributor

claude bot commented Feb 21, 2026

Claude finished @simonkberg's task in 1m 28s —— View job


Code Review

  • Gather context and read changed files
  • Verify documented settings match actual source files
  • Review documentation accuracy and completeness
  • Post final review

All documented changes verified against source. The PR is accurate and clean — here are my findings:

Verified correct:

  • Bot startup via instrumentation.tsstartBotSubscription() is confirmed (instrumentation.ts:3-6, bot.ts:83-87)
  • All five next.config.ts settings (output, cacheComponents, reactCompiler, typedRoutes, experimental.globalNotFound) match the actual config (next.config.ts:6-12)
  • Rate limit gate documented correctly — backed by the rateLimitUntil map in api.ts:28
  • Redis deduplication: SEEN_TTL = 60 and markSeen() in bot.ts:25-33 confirm the 60s TTL claim
  • Claude Haiku 4.5 and 5-second timeout both confirmed in anthropic.ts:20,229
  • Regex pattern /\bsimon[- ]?bot\b/i matches BOT_MENTION_PATTERN in bot.ts:15
  • config.ts and instrumentation.ts both exist at their documented paths

Minor omissions (not blocking, intentional omissions are fine):

Two next.config.ts settings are undocumented in the "Key settings" list:

  • logging: { fetches: { fullUrl: true } } — dev-only debug logging, omitting is reasonable
  • turbopack: { root: import.meta.dirname } — since pnpm dev uses Turbopack and this sets the root resolution path, it could be worth a bullet point to help agents understand why the Turbopack config is there. Not critical though.

Overall: Documentation is accurate and a clear improvement over what it replaced. The bot architecture fix in particular removes genuinely misleading information. Ready to merge.

@codecov
Copy link

codecov bot commented Feb 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.81%. Comparing base (9a2f348) to head (4d71c5d).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1612   +/-   ##
=======================================
  Coverage   97.81%   97.81%           
=======================================
  Files          49       49           
  Lines         916      916           
  Branches      161      161           
=======================================
  Hits          896      896           
  Misses          8        8           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@simonkberg simonkberg merged commit 1182b31 into main Feb 21, 2026
8 checks passed
@simonkberg simonkberg deleted the docs/update-agents-md branch February 21, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant