Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughSummary by CodeRabbit
WalkthroughAdds an Alembic migration wired to a new data-migration that canonicalizes built-in application workflow schemas, a SQL reference for the migration, UI/type changes to allow nullable commit messages, package version/dependency bumps, and CI workflow cache/install adjustments. ChangesWorkflow Schema Canonicalization
Commit Message Nullability
Repo Version, Dependency, and CI Updates
Sequence Diagram(s)sequenceDiagram
participant Alembic as Alembic migration
participant DataMig as Data-migration module
participant Catalog as Catalog (catalog storage)
participant DB as Postgres (workflow_revisions)
Alembic->>DataMig: upgrade(session=connection)
DataMig->>Catalog: fetch canonical entry for URI
Catalog-->>DataMig: canonical schema JSON
DataMig->>DB: UPDATE workflow_revisions SET schemas = canonical WHERE uri IN (...)
DB-->>DataMig: rows updated count
DataMig-->>Alembic: return / print summary
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
[fix] Replace non-canonical schemas in chat and completion workflows
[fix] Remove default empty message for commits
Railway Preview Environment
Updated at 2026-05-06T15:09:14.980Z |
…60506 [fix] Resolve all open vulnerabilities
There was a problem hiding this comment.
🧹 Nitpick comments (3)
api/oss/tests/legacy/requirements.test.txt (1)
1-9: ⚡ Quick winLegacy requirements file appears disconnected from the active test install path.
Line 1 and Line 8 adjust pins here, but CI installs API test deps via
uv sync --lockedfromapi/pyproject.toml/api/uv.lock(.github/workflows/12-check-unit-tests.yml, Lines 142–161). This creates drift risk where edits in this file don’t affect real test environments. Consider removing this file or explicitly wiring a job/script that consumes it.web/packages/agenta-ui/package.json (1)
56-56: ⚡ Quick winConsider aligning peerDependency version with the actual dependency.
The
peerDependenciesspecifiesuuid: ">=9.0.0"(line 56) while thedependenciessection now usesuuid: "^11.1.1"(line 94). If uuid 11.x introduced API changes that your code depends on, consumers using uuid 9.x or 10.x might encounter runtime issues. Consider updating the peer dependency to>=11.0.0to accurately reflect the minimum supported version.♻️ Suggested fix
"peerDependencies": { ... - "uuid": ">=9.0.0" + "uuid": ">=11.0.0" },Also applies to: 94-94
sdk/oss/tests/legacy/new_tests/requirements.test.txt (1)
1-9: 💤 Low valueRemove this orphaned requirements file.
This file is no longer used by the test infrastructure. The CI workflows now use
uv sync --lockedwhich reads frompyproject.tomlanduv.lock, not from this requirements file. No test configuration or scripts in the codebase reference it. Removing it will eliminate confusion about which dependency specifications are authoritative.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 646a949b-954f-497c-b608-f662219c1476
⛔ Files ignored due to path filters (9)
api/uv.lockis excluded by!**/*.lockdocs/package-lock.jsonis excluded by!**/package-lock.jsondocs/pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlexamples/node/observability-opentelemetry/pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlexamples/node/observability-vercel-ai/package-lock.jsonis excluded by!**/package-lock.jsonexamples/python/RAG_QA_chatbot/frontend/pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlsdk/uv.lockis excluded by!**/*.lockservices/uv.lockis excluded by!**/*.lockweb/pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (19)
.github/workflows/12-check-unit-tests.yml.github/workflows/44-railway-tests.ymlapi/oss/tests/legacy/requirements.test.txtapi/oss/tests/requirements.txtapi/pyproject.tomldocs/package.jsonsdk/oss/tests/legacy/new_tests/requirements.test.txtsdk/oss/tests/requirements.txtsdk/pyproject.tomlservices/pyproject.tomlweb/ee/package.jsonweb/oss/package.jsonweb/package.jsonweb/packages/agenta-entities/package.jsonweb/packages/agenta-entity-ui/package.jsonweb/packages/agenta-playground-ui/package.jsonweb/packages/agenta-playground/package.jsonweb/packages/agenta-shared/package.jsonweb/packages/agenta-ui/package.json
💤 Files with no reviewable changes (2)
- api/oss/tests/requirements.txt
- sdk/oss/tests/requirements.txt
🚧 Files skipped from review as they are similar to previous changes (1)
- web/ee/package.json
New version v0.99.2 in