Skip to content

Conversation

codefromthecrypt
Copy link
Contributor

@codefromthecrypt codefromthecrypt commented Oct 1, 2025

Update imports to use beeai_framework.agents.requirement instead of
beeai_framework.agents.experimental, reflecting the namespace change
in beeai-framework 0.1.51+. Remove type workarounds that were handling
the experimental module's potential absence.

  • Update minimum beeai-framework version to >= 0.1.51
  • Change imports from .experimental to .requirement namespace
  • Remove TYPE_CHECKING and runtime ImportError handling

Note

Updates BeeAI instrumentation to use the stable agents.requirement APIs and raises the minimum beeai-framework version to 0.1.51.

  • Instrumentation (BeeAI):
    • Switch imports from beeai_framework.agents.experimental.* to beeai_framework.agents.requirement.* in processors/agents/requirement_agent.py, processors/requirement.py, and processors/locator.py.
    • Update locator registrations to reference RequirementAgent and Requirement from the new agents.requirement paths.
  • Versioning:
    • Bump beeai-framework minimum version to >=0.1.51 in pyproject.toml (core, instruments, and test extras).
    • Update _instruments requirement in __init__.py to beeai-framework >= 0.1.51.

Written by Cursor Bugbot for commit 17a1e78. This will update automatically on new commits. Configure here.

@codefromthecrypt codefromthecrypt requested a review from a team as a code owner October 1, 2025 06:27
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 1, 2025
cursor[bot]

This comment was marked as outdated.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Oct 1, 2025
cursor[bot]

This comment was marked as outdated.

@mikeldking
Copy link
Collaborator

Thanks @codefromthecrypt for the fix. Intro'ing you to @Tomas2D who helps maintain this. Ping me and I'll get it deployed!

@codefromthecrypt
Copy link
Contributor Author

hi @Tomas2D I noticed existing code walking carefully around experimental symbols going away and assume that's the way forward vs bumping the minimum version. However you like to proceed game.. just this is currently a broken build.

@Tomas2D
Copy link
Contributor

Tomas2D commented Oct 2, 2025

@codefromthecrypt we moved the RequiremetAgent out of the experimental module. See how it can be fixed i-am-bee/beeai-framework-py-starter@6514b20. I would just update the imports and bump the minimal version to >= 0.1.51

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Oct 3, 2025
@codefromthecrypt codefromthecrypt changed the title fix(beeai): handle removal of experimental features fix(beeai): update for RequirementAgent move from experimental to stable Oct 3, 2025
@codefromthecrypt
Copy link
Contributor Author

done @mikeldking fire when ready!

@Tomas2D
Copy link
Contributor

Tomas2D commented Oct 3, 2025

Seems good to me. Can we release? 🙏🏻

Copy link
Collaborator

@mikeldking mikeldking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛼

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 3, 2025
@mikeldking mikeldking merged commit f511485 into Arize-ai:main Oct 3, 2025
15 checks passed
@github-actions github-actions bot mentioned this pull request Oct 3, 2025
@codefromthecrypt codefromthecrypt deleted the fix-beeai-experimental branch October 5, 2025 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants