Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bcd90e7360
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1,317 +0,0 @@ | |||
| # How to Use AI with goose | |||
There was a problem hiding this comment.
Keep legacy HOWTOAI links from breaking
Deleting HOWTOAI.md without updating inbound links leaves public documentation with dead URLs: README.md:30 and documentation/blog/2025-09-26-hacktoberfest-2025/index.md:37 both still point to .../HOWTOAI.md, which now returns 404 after this commit. This regresses contributor onboarding because readers from those entry points can no longer reach the AI guidance content.
Useful? React with 👍 / 👎.
|
|
|
||
| ## Quick Responsible AI Tips | ||
|
|
||
| There's no need to tell us you used AI in your work. You are contributing to an agent, it would be odd if |
There was a problem hiding this comment.
we should also update the PR template, which asks this
There was a problem hiding this comment.
or leave just that piece for AI attribution metrics (see my comment below)
There was a problem hiding this comment.
I have updated the PR template.
| ### Discussions | ||
|
|
||
| --- | ||
| If you have an idea but are not yet sure how it should work, open a discussion instead. Discussions are a good |
There was a problem hiding this comment.
should we maybe push them to Discord? is the team actively monitoring Discussions? we could also set up automation to post in Discord when a Discussion is created. cc @blackgirlbytes
There was a problem hiding this comment.
yeah we can do that. good idea.
|
|
||
| ## Quick Responsible AI Tips | ||
|
|
||
| There's no need to tell us you used AI in your work. You are contributing to an agent, it would be odd if |
There was a problem hiding this comment.
this was for internal AI attribution detection. Do we have a better way to know which PRs had AI-assistance? or maybe we just dont care anymore?
Signed-off-by: Angie Jones <[email protected]>
Signed-off-by: Angie Jones <[email protected]>
Co-authored-by: Angie Jones <[email protected]> Signed-off-by: Angie Jones <[email protected]>
Signed-off-by: Angie Jones <[email protected]>
bdbad15 to
008131e
Compare
* origin/main: feat: adversarial agent for preventing leaking of info and more (#7948) Update contributing.md (#7927) docs: add credit balance monitoring section (#7952) docs: add Cerebras provider to supported providers list (#7953) docs: add TUI client documentation to ACP clients guide (#7950) fix: removed double dash in pnpm command (#7951) docs: polish ACP docs (#7946) claude adaptive thinking (#7944) feat: new onboarding flow (#7266) Add DCO git commit command to AGENTS.md (#7945) fix(claude-code): remove incorrect agent_visible filter on user message (#7931) No Check do Check (#7942) Log 500 errors and also show error for direct download (#7936) fix: retry on authentication failure with credential refresh (#7812) Remove java/.ai-usage-marker directory (#7925) test(acp): add terminal delegation fixtures and fix shell singleton (#7923) fix: bump pctx_code_mode to 0.3.0 for iterator type checking fix (#7892) feat: persist GooseMode per-session via session DB (#7854)
* 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: Angie Jones <[email protected]> Co-authored-by: Douwe Osinga <[email protected]> Co-authored-by: Angie Jones <[email protected]> Signed-off-by: esnyder <[email protected]>
Signed-off-by: Angie Jones <[email protected]> Co-authored-by: Douwe Osinga <[email protected]> Co-authored-by: Angie Jones <[email protected]> Signed-off-by: esnyder <[email protected]>
Summary
Update the text, remove the outdated HOWTOAI.md