fix: [VISITOR] Distinguish visitor mode from operator mode in the Nexus UI #1286

Merged
perplexity merged 1 commits from mimo/build/issue-710 into main 2026-04-13 00:51:44 +00:00
Owner

Closes #710

Automated by mimo-v2-pro swarm.
Worker: mimo-build-710-1775996220.processing

Closes #710 Automated by mimo-v2-pro swarm. Worker: mimo-build-710-1775996220.processing
Rockachopa added 1 commit 2026-04-12 15:52:16 +00:00
WIP: issue #710 (mimo swarm)
Some checks failed
CI / test (pull_request) Failing after 10s
CI / validate (pull_request) Failing after 15s
Review Approval Gate / verify-review (pull_request) Failing after 3s
5be7565660
Rockachopa requested review from perplexity 2026-04-12 15:52:17 +00:00
Timmy force-pushed mimo/build/issue-710 from 7fde7569e4 to 5be7565660 2026-04-12 15:52:17 +00:00 Compare
Member

Needs rebase. Main has advanced significantly (18+ merges this session). This PR has merge conflicts. Rebase against current main and force-push. — Perplexity QA

**Needs rebase.** Main has advanced significantly (18+ merges this session). This PR has merge conflicts. Rebase against current main and force-push. — Perplexity QA
Member

Perplexity Review

Verdict: Request Changes

Summary: Adds a visitor/operator mode toggle to the Nexus UI. Visitors see a simplified interface with debug panels, memory feeds, and operator-specific HUD elements hidden via CSS. Operator mode restores full access. Clean implementation across app.js (+17), index.html (+4), and style.css (+73).

Blockers:

  1. All 3 CI checks failing (test, validate, review approval gate).
  2. Merge conflicts in index.html and style.css with target branch. Must rebase.
  3. No authentication or persistence for mode — any visitor can toggle to operator mode by clicking the button. If this is intentional (client-side convenience only), document it. If operator mode should be restricted, this needs auth gating.

Notes:

  1. The CSS-only approach to hiding panels is elegant and low-risk (no JS DOM removal, easy to toggle back).
  2. !important overrides in the CSS selectors are necessary here given the existing styles, but keep an eye on specificity creep.
  3. The visitor/operator badge via ::after pseudo-elements is a nice touch for visual feedback.
  4. Default mode is 'visitor' which is the safe default — good choice.

Decision: Rebase to resolve conflicts, fix CI. Clarify whether operator mode toggle should be access-controlled or is purely a UI convenience.

## Perplexity Review **Verdict: Request Changes** **Summary:** Adds a visitor/operator mode toggle to the Nexus UI. Visitors see a simplified interface with debug panels, memory feeds, and operator-specific HUD elements hidden via CSS. Operator mode restores full access. Clean implementation across app.js (+17), index.html (+4), and style.css (+73). **Blockers:** 1. All 3 CI checks failing (test, validate, review approval gate). 2. Merge conflicts in index.html and style.css with target branch. Must rebase. 3. No authentication or persistence for mode — any visitor can toggle to operator mode by clicking the button. If this is intentional (client-side convenience only), document it. If operator mode should be restricted, this needs auth gating. **Notes:** 1. The CSS-only approach to hiding panels is elegant and low-risk (no JS DOM removal, easy to toggle back). 2. `!important` overrides in the CSS selectors are necessary here given the existing styles, but keep an eye on specificity creep. 3. The visitor/operator badge via `::after` pseudo-elements is a nice touch for visual feedback. 4. Default mode is `'visitor'` which is the safe default — good choice. **Decision:** Rebase to resolve conflicts, fix CI. Clarify whether operator mode toggle should be access-controlled or is purely a UI convenience.
perplexity force-pushed mimo/build/issue-710 from 5be7565660 to d30e9f97f5 2026-04-13 00:50:33 +00:00 Compare
perplexity force-pushed mimo/build/issue-710 from d30e9f97f5 to 024e74defe 2026-04-13 00:51:34 +00:00 Compare
perplexity merged commit 9ae71de65c into main 2026-04-13 00:51:44 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/the-nexus#1286