don't imply running builds all the time in AGENTS.md#7865
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3eef38b17e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d40ff59ca4
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| # 7. [if server] just generate-openapi | ||
| ``` | ||
|
|
||
| ### Run these only if the user has asked you to build/test your changes: |
There was a problem hiding this comment.
Keep OpenAPI generation mandatory for server edits
This new heading makes just generate-openapi conditional on users explicitly asking for build/test steps, which conflicts with the existing Server: Changes need just generate-openapi rule and can leave server PRs with stale API artifacts when users request only code changes. In that common workflow, agents can now follow the document and skip regeneration, causing backend/UI contract drift that is not guaranteed to be caught before review.
Useful? React with 👍 / 👎.
…ct-ui * origin/main: Improve the formatting of tool calls, show thinking, treat Reasoning and Thinking as the same thing (sorry Kant) (#7626) don't imply running builds all the time in AGENTS.md (#7865) fix: unregister goosed child process's listener (#7956) # Conflicts: # ui/desktop/src/api/index.ts
* main: (32 commits) Revert message flush & test (block#7966) docs: add Remote Access section with Telegram Gateway documentation (block#7955) fix: update webmcp blog post metadata image URL (block#7967) fix: clean up OAuth token cache on provider deletion (block#7908) fix: hard-coded tool call id in code mode callback (block#7939) Fix SSE parsers to accept optional space after data: prefix (block#7929) docs: add GOOSE_INPUT_LIMIT to config-files.md (block#7961) Add WebMCP for Beginners blog post (block#7957) Fix download manager (block#7933) Improve the formatting of tool calls, show thinking, treat Reasoning and Thinking as the same thing (sorry Kant) (block#7626) don't imply running builds all the time in AGENTS.md (block#7865) fix: unregister goosed child process's listener (block#7956) feat: adversarial agent for preventing leaking of info and more (block#7948) Update contributing.md (block#7927) docs: add credit balance monitoring section (block#7952) docs: add Cerebras provider to supported providers list (block#7953) docs: add TUI client documentation to ACP clients guide (block#7950) fix: removed double dash in pnpm command (block#7951) docs: polish ACP docs (block#7946) claude adaptive thinking (block#7944) ...
Signed-off-by: esnyder <[email protected]>
Signed-off-by: esnyder <[email protected]>
The current language makes interactive work with goose quite slow. I think what you want is for goose to do this when it's running more autonomously. I'm not sure of the best way to signal this.