Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (31)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR bumps core/framework/plugin versions, updates multiple changelogs, adds a runtime-gated PostgreSQL migration step to initialize Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsTimed out fetching pipeline failures after 30000ms Comment |
| <Warning> | ||
| **This release contains 4 breaking changes** related to model aliasing. See the [v1.5.0 Migration Guide](/migration-guides/v1.5.0#breaking-change-9-provider-deployments-removed-migrate-to-aliases) for full before/after examples and migration instructions. | ||
| </Warning> | ||
| - **Realtime Support** — Add WebSocket, WebRTC, and client secret handlers with session state management and transport context helpers |
There was a problem hiding this comment.
Model alias and routing rule chaining
Confidence Score: 5/5Safe to merge — all findings are P2; no active tests are broken and no runtime behavior is affected. The bulk of the PR is version bumps and changelogs. The two substantive findings are both P2: skipped tests with incorrect assertions (no CI impact until t.Skip() is removed) and a minor SQLite migration coverage gap. UI and migration script changes are additive and low risk. framework/modelcatalog/overrides_test.go — skipped tests need pointer-dereference fixes before they can be unskipped.
|
| Filename | Overview |
|---|---|
| framework/modelcatalog/overrides_test.go | New pricing override tests; several skipped tests contain incorrect assertions that compare float64 values against *float64 pointer fields without dereferencing — will fail when t.Skip() is removed. |
| framework/modelcatalog/pricing_test.go | New test helpers (chatPricing, makeChatResponse, etc.) — straightforward and correct. |
| .github/workflows/scripts/run-migration-tests.sh | Adds whitelisted_routes_json PostgreSQL dynamic-column coverage for migration tests; SQLite counterpart is absent, but mcp_disable_auto_tool_inject (same v1.5.0 group) is present in both sections, suggesting this is a minor gap. |
| ui/app/_fallbacks/enterprise/components/user-groups/businessUnitsView.tsx | New enterprise fallback view for Business Units — simple ContactUsView wrapper, no issues. |
| ui/components/sidebar.tsx | Adds Business Units entry to Governance navigation group and new compareVersions helper for semantic version comparison — logic and access control look correct. |
Reviews (3): Last reviewed commit: "prerelease 2 cut" | Re-trigger Greptile
|
|
a8311d8 to
094be9a
Compare
094be9a to
befc659
Compare
Merge activity
|

Release v1.5.1 and v1.3.1
This release bumps versions across all components and includes significant new features, bug fixes, and improvements to the platform.
Changes
Type of change
Affected areas
How to test
Breaking changes
Security considerations
Checklist
docs/contributing/README.mdand followed the guidelines