chore: update dash — agno 2.5.14→2.7.0a6, replica/hygiene sweep#28
Draft
ashpreetbedi wants to merge 1 commit into
Draft
chore: update dash — agno 2.5.14→2.7.0a6, replica/hygiene sweep#28ashpreetbedi wants to merge 1 commit into
ashpreetbedi wants to merge 1 commit into
Conversation
- pin agno[os]==2.7.0a6 in pyproject.toml (was unpinned; lock held 2.5.14) - regenerate requirements.txt via ./scripts/generate_requirements.sh (agno 2.7.0a6 + agnoctl 0.1.0a5; fastapi[standard] extras dropped upstream) - evals/run.py: guard avg_score None — AccuracyEval.avg_score is float | None in agno 2.7 (mypy-clean again) - hygiene sweep verified clean at public HEAD: railway.json numReplicas already 1, no .claude/settings.local.json, no *.egg-info, no tmp/, no AGENTS.md byte-copy Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Author
|
🤖 Automated re-review No blocking issues found. Re-checked the changed files for correctness, simplification, and efficiency regressions; none of the previously flagged concerns reproduced. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brings dash up to the
agentos-railwayStarter reference standard (agno pin + hygiene), perprompts/apps-update.md. Overnight automated pass — draft on purpose; do not merge before running the reviewer checklist below.What changed
pyproject.tomlhadagno[os]unpinned — the lock was what held 2.5.14. Now pinned explicitly:agno[os]==2.7.0a6.requirements.txtregenerated with the repo's own./scripts/generate_requirements.sh. Notable lock movement:agno==2.7.0a6, new transitiveagnoctl==0.1.0a5.fastapi[standard]extras, souvloop,watchfiles,httptools,jinja2,email-validator,fastapi-cli,gitpython,sentry-sdketc. fell out of the lock — this matches the reference's lock shape.openinference-instrumentation-agnostayed at0.1.29(compile keeps compatible pins; the reference lock carries0.1.38). It is transitive, not declared — bump with./scripts/generate_requirements.sh upgradeif you want lock parity.evals/run.pycompat fix (1 line):AccuracyEvalresults'avg_scoreisfloat | Nonein agno 2.7 — added aNoneguard so mypy is clean again. No other code changes.Known drift bugs — verified against public HEAD
The stale local-clone findings did not reproduce at public HEAD; nothing needed fixing:
railway.jsonnumReplicasis already1(in-process scheduler must never run 2 replicas) — verified, unchanged..claude/settings.local.json(the/Users/ab/code/codareference existed only in a local checkout) — nothing to scrub.*.egg-info(incl. the reporteddata_agent.egg-info), notmp/scratch — nothing to delete.CLAUDE.mdis standalone (noAGENTS.mdexists), so the symlink rule doesn't apply.Migration hazards to keep in mind (from
templates/docs/decisions.md)This is a multi-version jump (2.5.14 → 2.7.0a6); these are the known landmines:
421 Misdirected Requestfor/mcpon any non-localhost host; fixed as of 2.7.0a5. Dash does not enable the MCP server, so no fastmcp is in the lock — but if you enableenable_mcp_serverlater, use theagno[mcp]extra and probe/mcpover the public domain, never just localhost./,/health,/info,/docs,/redoc,/openapi.json. Dash runsauthorization=TruewhenRUNTIME_ENV=prd— verify after the bump that only those routes are open and everything else still 401s (including the customPOST /knowledge/reloadendpoint).knowledge-refreshcron viaScheduleManagerin the lifespan. Single replica is mandatory (double-fires on 2 —numReplicasstays 1), andAGENTOS_URLis load-bearing in prod: left at the localhost default, scheduled jobs silently never fire.agno_service_accounts.user_id). First boot against an existing 2.5.14 database should be watched — check startup logs for schema-migration errors, and back up prod Postgres before deploying this.Reviewer checklist — full verify gate before merge
Needs a real
.env(OPENAI_API_KEYat minimum) and free ports 8000/5432 (gates are serial across templates):docker compose up -d --build— API + Postgres come up healthy, startup logs clean (watch for agno schema-migration errors against pre-existing data)curl localhost:8000/docs→ 200enable_mcp_serveris off in dash; if enabled, run the MCP check over the public domain too (421 guard)./scripts/venv_setup.sh && ./scripts/validate.sh→ ruff + mypy clean (passed in this run, re-confirm)python -m evals --tag smoke(or dash'sevals/run.pyflow) → all passknowledge-refreshschedule registered once,POST /knowledge/reloadreturns{"status": "ok"}Not verified autonomously
.envin this fresh clone →docker compose up, live/docs, evals, and the Slack path were not run. Only validated: ruff clean, mypy clean,docker compose configparses, andimport app.mainunder agno 2.7.0a6 loads all module wiring (it fails only at the Postgres connection, as expected with no DB running).avg_scoreguard is type-level verified only.🤖 Generated with Claude Code