Skip to content

Remove Super Context from UI and core - #5678

Merged
senamakel merged 6 commits into
tinyhumansai:mainfrom
senamakel:remove-super-context
Aug 22, 2026
Merged

Remove Super Context from UI and core#5678
senamakel merged 6 commits into
tinyhumansai:mainfrom
senamakel:remove-super-context

Conversation

@senamakel

@senamakel senamakel commented Aug 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the Super Context toggle, write coordination, translations, and E2E setup from the desktop UI.
  • Remove the Super Context configuration, RPC surface, context-manager state, first-turn injection, and TinyAgents middleware from the Rust core.
  • Preserve the on-demand agent_prepare_context path while simplifying prepared-source state to immutable per-turn data.
  • Remove Super Context documentation and update adjacent architecture and routing references.

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

  • Tests added or updated (happy path + at least one failure / edge case) per Testing Strategy — removed feature-specific tests and updated surviving prepared-context coverage.
  • Diff coverage >= 80% — focused affected suites pass; CI will enforce the merged diff gate.
  • Coverage matrix updated — N/A: the matrix contains no Super Context feature row.
  • All affected feature IDs from the matrix are listed in the PR description under ## Related — N/A: no matrix feature ID covers Super Context.
  • No new external network dependencies introduced (mock backend used per Testing Strategy)
  • Manual smoke checklist updated if this touches release-cut surfaces (docs/RELEASE-MANUAL-SMOKE.md) — N/A: the checklist contains no Super Context step.
  • Linked issue closed via Closes #NNN in the ## Related section — 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

  • Closes: N/A
  • Follow-up PR(s)/TODOs: None

AI Authored PR Metadata (required for Codex/Linear PRs)

Linear Issue

  • Key: N/A
  • URL: N/A

Commit & Branch

  • Branch: remove-super-context
  • Commit SHA: dfc9a4d

Validation Run

  • pnpm --filter openhuman-app format:check
  • pnpm typecheck
  • Focused tests: Vitest related suite (163 tests); Rust catalog, context config, and prepared-context tests
  • Rust fmt/check (if changed): cargo check --manifest-path Cargo.toml, pre-push Clippy
  • Tauri fmt/check (if changed): pre-push desktop Clippy

Validation Blocked

  • command: N/A
  • error: N/A
  • impact: None

Behavior Changes

  • Intended behavior change: Super Context is removed completely from the UI and core.
  • User-visible effect: the chat toggle disappears and automatic first-turn Super Context retrieval no longer runs.

Parity Contract

  • Legacy behavior preserved: explicit agent_prepare_context remains available.
  • Guard/fallback/dispatch parity checks: prepared-source short-circuit test passes; RPC/config registrations were removed together.

Duplicate / Superseded PR Handling

  • Duplicate PR(s): None found
  • Canonical PR: This PR
  • Resolution (closed/superseded/updated): N/A

Summary by CodeRabbit

  • Removed Features

    • Removed the Super Context chat toggle and automatic context-gathering behavior.
    • Removed related configuration controls, environment settings, and API endpoints.
    • Removed Super Context references from the chat interface and product descriptions.
  • Documentation

    • Updated documentation and translations to reflect the feature’s removal.
  • Tests

    • Updated end-to-end and navigation checks to stop expecting Super Context functionality.

senamakel and others added 2 commits August 22, 2026 12:17
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
@senamakel
senamakel requested a review from a team August 22, 2026 09:29
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ab427a94-a87a-4cb0-a0f1-fb3be5723d95

📥 Commits

Reviewing files that changed from the base of the PR and between eaa0250 and 20c4aeb.

📒 Files selected for processing (1)
  • tests/config_auth_app_state_connectivity_e2e.rs
💤 Files with no reviewable changes (1)
  • tests/config_auth_app_state_connectivity_e2e.rs

Included review availability: Your plan provides up to 10 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The pull request removes SuperContext from the Rust runtime, configuration and RPC surfaces, frontend references, tests, translations, and documentation. It retains explicit context preparation through AgentPrepareContextTool and updates burst-tier descriptions.

Changes

SuperContext removal

Layer / File(s) Summary
Remove SuperContext runtime path
src/openhuman/agent/context/..., src/openhuman/agent/harness/..., src/openhuman/agent/tinyagents/...
Removes first-turn SuperContext execution, middleware configuration, context-bundle injection, and mutable prepared-source registration.
Remove configuration and RPC contracts
src/openhuman/config/..., app/src/services/rpcMethods.ts, app/src/utils/tauriCommands/config.ts
Removes the configuration field, environment overrides, UI operations, controller schemas, handlers, and RPC mappings.
Remove frontend surface
app/src/components/..., app/src/lib/i18n/*, app/test/playwright/...
Removes SuperContext references, translations, and chat-page expectations.
Update test and harness setup
app/scripts/..., app/test/e2e/..., tests/...
Removes SuperContext setup calls and simplifies test configuration and bootstrap helpers.
Align documentation and references
README.md, docs/..., gitbooks/..., scripts/debug/...
Removes SuperContext descriptions and updates context-scout and burst-tier references.

Estimated code review effort: 4 (Complex) | ~45 minutes

Merge Risk: ⚪ Minimal · up to 20c4a

This change removes Super Context while preserving explicit context preparation, and no actionable merge-blocking risk remains after normal checks and review.

Poem

I’m a rabbit with a tidy trail,
SuperContext leaves the grail.
Toggles fade and scouts retire,
Clean configs rise a little higher.
Hop through tests, docs, and code—
A lighter path now bears the load.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 39.13% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 23 functions across 17 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the primary change: removing Super Context from the UI and Rust core.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

senamakel and others added 2 commits August 22, 2026 12:31
# Conflicts:
#	src/openhuman/config/ops/mod.rs
Co-authored-by: Medulla <medulla@tinyhumans.ai>
coderabbitai[bot]
coderabbitai Bot previously approved these changes Aug 22, 2026
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

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>
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

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.

@tinysweeper tinysweeper Bot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Aug 22, 2026

@tinysweeper tinysweeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out · 803 embedded · openrouter/openai/text-embedding-3-small

coderabbitai[bot]
coderabbitai Bot previously approved these changes Aug 22, 2026
Co-authored-by: Medulla <medulla@tinyhumans.ai>
@senamakel
senamakel merged commit 9c347ae into tinyhumansai:main Aug 22, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant