Remove Super Context from UI and core - #5678
Conversation
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe pull request removes SuperContext from the Rust runtime, configuration and RPC surfaces, frontend references, tests, translations, and documentation. It retains explicit context preparation through ChangesSuperContext removal
Estimated code review effort: 4 (Complex) | ~45 minutes Merge Risk: ⚪ Minimal · up to This change removes Super Context while preserving explicit context preparation, and no actionable merge-blocking risk remains after normal checks and review. Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
# Conflicts: # src/openhuman/config/ops/mod.rs
Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Summary
agent_prepare_contextpath while simplifying prepared-source state to immutable per-turn data.Problem
Super Context was a cross-layer feature with UI controls, persisted core configuration, RPC methods, and agent middleware. Removing only the UI would leave dormant core behavior and an unsupported configuration surface.
Solution
The feature is deleted end to end. The remaining explicit context-preparation tool retains its existing behavior, but no automatic first-turn Super Context retrieval or feature flag remains.
Submission Checklist
## Related— N/A: no matrix feature ID covers Super Context.docs/RELEASE-MANUAL-SMOKE.md) — N/A: the checklist contains no Super Context step.Closes #NNNin the## Relatedsection — N/A: no issue was provided.Impact
Desktop UI and core/CLI configuration no longer expose or execute Super Context. Existing configuration files containing the removed field remain forward-compatible because unknown TOML fields are ignored. Explicit on-demand agent context preparation remains available.
Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
remove-super-contextdfc9a4dValidation Run
pnpm --filter openhuman-app format:checkpnpm typecheckcargo check --manifest-path Cargo.toml, pre-push ClippyValidation Blocked
command:N/Aerror:N/Aimpact:NoneBehavior Changes
Parity Contract
agent_prepare_contextremains available.Duplicate / Superseded PR Handling
Summary by CodeRabbit
Removed Features
Documentation
Tests