Repository navigation
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Understand this PR’s impact Explore downstream dependencies and potential security impact with Blast Radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (4)
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. WalkthroughThe pull request adds a staged design for a disabled-by-default Zero Hour Bot/Agent API. It defines engine seams, fog-safe observations, validated commands, bridge transport, match setup, headless execution, SDK integrations, referee controls, and verification. ChangesBot Agent API
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
|
…its, stealth Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 2
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: af90624e-333d-49f1-891c-6e38fc914e1d
📒 Files selected for processing (12)
docs/plans/2026/09/21/101-bot-agent-api/00-prior-art.mddocs/plans/2026/09/21/101-bot-agent-api/01-engine-seams.mddocs/plans/2026/09/21/101-bot-agent-api/02-observation.mddocs/plans/2026/09/21/101-bot-agent-api/03-commands-and-hardening.mddocs/plans/2026/09/21/101-bot-agent-api/04-bridge-protocol.mddocs/plans/2026/09/21/101-bot-agent-api/05-match-setup-and-lobby.mddocs/plans/2026/09/21/101-bot-agent-api/06-headless-and-step.mddocs/plans/2026/09/21/101-bot-agent-api/07-sdk-gym-mcp.mddocs/plans/2026/09/21/101-bot-agent-api/08-referee-and-tournament.mddocs/plans/2026/09/21/101-bot-agent-api/09-verification.mddocs/plans/2026/09/21/101-bot-agent-api/overview.mddocs/plans/2026/09/21/101-bot-agent-api/status.yml
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
We don't add these types of plans to the code. If you want to discuss these type of plans it is best to open a discussion. Definitive plans can be added by editing the main issue post (e.g. with a checklist). Be aware that at the moment we do not implement large features. We are still mainly working on bugs, performance, code optimization and modernization and it is unlikely that the feature will be worked on any time in the near future. Additionally, as some mentioned in the related issue, there are a lot of pre-conditions that need to be checked and worked on before such an API can be provide. Last, I would like to say that AI is notoriously bad in writing good feature and user stories unless explicitly guided and prompted. The docs added in this PR are difficult to read for a human. They are summarizations of an explorative AI. Closing this PR as this type of information does not belong in the code repository. |
|
Yes lets not add these plans to the repository. Seems unnecessary. |
TL;DR
overview.mdhas a table mapping each point to where it is answered.Core/; the bot runs out of process (any language) over loopback TCP (Windows, Linux native/Wine, macOS via Wine); orders go only through the existingGameMessagepath after UI-parity checks; observation is fog-respecting by default; SDKs / Gym env / MCP server / referee live in a separate repo.overview.md. A plan is much cheaper to change than code.Full description
All files are under
docs/plans/2026/09/21/101-bot-agent-api/. Everyfile:linereference was checked againstmain@241f01926.overview.md00-prior-art.md01-engine-seams.mdObject::getShroudedStatus,getDelayBetweenShots,createGroup, …); platform status02-observation.mdGameLogic::update(so it also works in headless replay);player/camera/fullvisibility modes; records, feature planes and entity arrays for ML03-commands-and-hardening.mdGameMessages, injected afterpropagateMessages(); client-side UI-parity checks (e.g. no attacking shrouded targets); action masks; optional logic-side hardening behindRETAIL_COMPATIBLE_CRC04-bridge-protocol.mdobserve/act, realtime vs step timing — including what happens when a bot stalls05-match-setup-and-lobby.md[BOT]tag in the player name06-headless-and-step.md07-sdk-gym-mcp.md08-referee-and-tournament.md09-verification.mdKey decisions (all open to change)
GameMessages for the local player, so replays, CRC and network sync are unchanged.fullvisibility is refused in network games.About the plan layout
The directory follows the planning layout we use at developerz-ai for AI-assisted work (dated dir,
overview.md+ one slice per area,status.ymlas the only tracker,file:linereferences instead of pasted code): https://github.com/developerz-ai/gold-standards-in-ai. If the project prefers a different place or shape — the wiki, a Discussion, or only the external repo — I'll move it.🤖 Generated with Claude Code