Conversation
…ew features - Add agent_as_workflow.ipynb: Demonstrates WorkflowAgent pattern with reflection and retry - Update azure_responses_client.ipynb: Azure OpenAI Responses Client with function calling - Update openai_responses_example.ipynb: OpenAI Responses Client direct usage examples - Update mem0_basic.ipynb: Mem0 provider integration for long-term memory - Update magentic.ipynb: Complete Magentic orchestration with multi-agent coordination - Update yaml_workflow.ipynb: YAML-based workflow configuration and execution - Clear all output cells to keep notebooks clean Resolves #220 and #221
Add frontend-install, dev, haxui-server, frontend-dev and test-e2e targets to the Makefile and include them in .PHONY. Update the help output to document the new commands. Implement frontend-install (runs npm install), dev (starts backend + frontend concurrently), dedicated backend/frontend dev targets, and a test-e2e target to run the Playwright workflow. Tweak install target output messaging for clarity.
…ckfile - Update pyproject.toml version to 0.5.3 - Regenerate uv.lock: update agent-framework-core version, bump openai to 2.5.0 - Add/record runtime deps for ipykernel, azure-ai-evaluation, tiktoken and their transitive dependencies - Add many new lock entries and wheels as part of dependency resolution
Signed-off-by: Zachary BENSALEM <zachary@qredence.ai>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files
|
There was a problem hiding this comment.
Pull Request Overview
Adds notebook and evaluation related Python dependencies and introduces frontend development and E2E test targets to the Makefile, preparing for notebook examples and a full-stack dev workflow.
- Bumps project version to 0.5.3 while adding new runtime dependencies (ipykernel, openai, azure-ai-evaluation, tiktoken).
- Expands Makefile with frontend install/run targets and an end-to-end test runner.
- Removes direct ad-hoc installation of openai in favor of pyproject-managed dependency.
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| pyproject.toml | Version bump and addition of four new dependencies to support notebooks/evaluation. |
| Makefile | Adds frontend/dev orchestration targets and E2E test command; adjusts install messaging. |
| [project] | ||
| name = "agentic-fleet" | ||
| version = "0.5.2" | ||
| version = "0.5.3" |
There was a problem hiding this comment.
[nitpick] Adding new user-facing dependencies alters the install surface and is more than a patch-level change; consider bumping the minor version (e.g., 0.6.0) per semantic versioning rather than a patch increment.
| version = "0.5.3" | |
| version = "0.6.0" |
| "ipykernel>=7.0.1", | ||
| "openai==2.5.0", | ||
| "azure-ai-evaluation>=1.12.0", | ||
| "tiktoken>=0.12.0", |
There was a problem hiding this comment.
[nitpick] ipykernel is only needed for notebook/examples usage; placing it in core dependencies forces all installations to pull Jupyter stack. Recommend moving ipykernel (and potentially other example-only libs) to an extras group (e.g., [project.optional-dependencies].notebooks) to keep the core footprint minimal.
There was a problem hiding this comment.
@copilot Fix and resolve conversation
| "rich>=13.7.0", | ||
| "prompt-toolkit>=3.0.47", | ||
| "ipykernel>=7.0.1", | ||
| "openai==2.5.0", |
There was a problem hiding this comment.
[nitpick] Exact pinning of openai while other dependencies use ranges can cause avoidable resolution conflicts and delay security/bugfix updates; prefer a compatible range like openai>=2.5.0,<3.0 unless a strict freeze is required.
| "openai==2.5.0", | |
| "openai>=2.5.0,<3.0", |
![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.
ℹ️ 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".
| "mem0ai==1.0.0", | ||
| "rich>=13.7.0", | ||
| "prompt-toolkit>=3.0.47", | ||
| "ipykernel>=7.0.1", | ||
| "openai==2.5.0", | ||
| "azure-ai-evaluation>=1.12.0", | ||
| "tiktoken>=0.12.0", |
There was a problem hiding this comment.
The dependency list now contains both "tiktoken>=0.7.0,<0.8" and "tiktoken>=0.12.0". Package installers treat duplicate requirements as an intersection, so there is no version that satisfies both ranges and uv sync/pip install . will fail with an unsatisfiable requirement before the project can even be set up. Drop the old constraint or move the upgraded version to an optional extra to keep installs workable.
Useful? React with 👍 / 👎.
| frontend-install: | ||
| @echo "Installing frontend dependencies..." | ||
| cd src/frontend && npm install | ||
| @echo "✓ Frontend dependencies installed" | ||
|
|
||
| # Run application | ||
| run: | ||
| uv run python -m agenticfleet | ||
|
|
||
| # Full stack development (backend + frontend) | ||
| dev: | ||
| @echo "Starting AgenticFleet Full Stack Development..." | ||
| @echo "" | ||
| @echo "Backend: http://localhost:8000" | ||
| @echo "Frontend: http://localhost:5173" | ||
| @echo "" | ||
| @echo "Press Ctrl+C to stop both services" | ||
| @echo "" | ||
| @trap 'kill 0' INT; \ | ||
| uv run uvicorn agenticfleet.haxui.api:app --reload --port 8000 & \ | ||
| cd src/frontend && npm run dev | ||
|
|
||
| # Examples | ||
| demo-hitl: | ||
| uv run python examples/demo_hitl.py | ||
|
|
||
| # DevUI backend server only | ||
| haxui-server: | ||
| @echo "Starting HaxUI backend on http://localhost:8000" | ||
| uv run uvicorn agenticfleet.haxui.api:app --reload --port 8000 | ||
|
|
||
| # Frontend dev server only | ||
| frontend-dev: | ||
| @echo "Starting frontend on http://localhost:5173" | ||
| cd src/frontend && npm run dev |
There was a problem hiding this comment.
frontend-install, dev, and frontend-dev all cd src/frontend, but no such directory exists in the repository (find . -name frontend returns nothing). Each advertised command will fail immediately with cd: src/frontend: No such file or directory, so developers cannot run the new full‑stack targets at all. Either add the frontend sources or guard these targets so they only run when the directory is present.
Useful? React with 👍 / 👎.
Description
Type of Change
Related Issues
Fixes #
Changes Made
Testing
uv run pytest)uv run python tests/test_config.py)uv run ruff check .)uv run black .)uv run mypy .)Test Commands Run
Documentation
Screenshots (if applicable)
Checklist
Additional Notes