cleanup: drop deprecated v0/v1 APIs and align docs/tests#512
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 14b68da00a
ℹ️ 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".
nicko-ai
force-pushed
the
chore/remove-deprecated-cleanup
branch
from
14b68da to
7cad16e
Compare
nicko-ai
force-pushed
the
chore/remove-deprecated-cleanup
branch
from
bb3b518 to
5e2e8a7
Compare
nicko-ai
changed the title
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 673656ebcc
ℹ️ 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".
ArtemShatokhin
left a comment
ArtemShatokhin
left a comment
There was a problem hiding this comment.
- Agent docsting in src/agency_swarm/agent/core.py mentions Agency Chart
- docs\core-framework\agents\advanced-configuration.mdx and docs\additional-features\few-shot-examples.mdx mention examples param
- src\agency_swarm\utils\create_agent_template.py creates an agent with deprecated params
- docs\migration\guide.mdx incorrectly references old params on line 167
- docs\additional-features\guardrails.mdx and examples\guardrails_input.py mention reasoning_effort
- temperature param is in incorrect spot at line 166 in docs\migration\guide.mdx
- Remove deprecated Agency APIs (agency_chart, completions proxy, message_files) - Enforce strict Agent kwargs and remove deprecated agent params - Remove BaseTool _shared_state and legacy reminder_override behavior - Update tests/docs/examples for v1.x-only surface (keep SendMessageHandoff deprecated alias) - Skip live OpenAI flake test on CI
- reorder agent core imports after formatter
- drop deprecated completions module/tests - rename platform API docs and OpenAPI to /get_response
- Add `import warnings` to fix F821 lint error in CI - Add mandatory reproduce-before-fix rule to AGENTS.md
- fail fast when message_files is passed
nicko-ai
force-pushed
the
chore/remove-deprecated-cleanup
branch
from
673656e to
4b613e0
Compare
- Reduce migration confusion by using agent-level model selection, max_tokens, and current reasoning config in examples and templates.
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
- avoid shared time.sleep patch interference - patch per-instance sleep in the test
2 participants
Uh oh!
There was an error while loading. Please reload this page.