Implement Arena command console surface #425
No reviewers
Labels
No labels
agent:fix_bugfix
agent:fix_conflicts
agent:fix_security
agent:gap_analysis
agent:implement
agent:open_issues
agents
agent:security_audit
agent:verify
area
timecurve
arena
arena-v2
audit
block:feature_review
block:security
bug
contracts
design
devops
docs
documentation
enhancement
epic
feature
frontend
indexer
low priority
megaeth
operations
ops
performance
QA
qa checklist
referrals
removal
security
smart-contracts
tech-debt
testing
time-arena
timecurve
visual-qa
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
code/yieldomega!425
Loading…
Reference in a new issue
No description provided.
Delete branch "cursor/arena-command-console-c09e"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary of changes
Implements GitLab #291 by replacing the
/arenaconcept-mock-plus-live-stack mount with one productionarena-command-consolesurface. The liveArenaSimplePageread/write paths remain the source of truth, while the layout now makes Last Buy and inline CHARM buying primary, moves CRED/timers/WarBow into a secondary operations rail, corrects visible DOUB/CHARM/CRED decision labels, and documents the new invariant/QA path.Issue #291 should remain open for maintainer/product acceptance.
Design review notes
/arenarendering, and visible CL8Y-facing labels where the Arena v2 user decision is DOUB.Checklist
glab issue view 291 -R PlasticDigits/yieldomega --comments/arenarenders a single unified command-console surfacearenaCommandConsoleStatic.test.ts;npm run test:e2e -- e2e/arena.spec.ts --workers=5; Rabby Chromium smoke (commandConsoleCount=1,conceptCount=0)bash scripts/e2e-anvil.shwallet-write/CRED specscd frontend && npm run typecheck && npm run lint && npm testcd frontend && npm run test:e2e -- e2e/arena.spec.ts --workers=5export PATH="$HOME/.foundry/bin:$PATH" && bash scripts/e2e-anvil.shcd frontend && npm run buildbash scripts/check-arena-naming.sh && bash scripts/check-art-readme-consumers.sh && bash scripts/check-doc-anchors.shVisual/design verification checklist for third parties
/arenahas onearena-command-consolesurface and no stacked.arena-final-conceptmock./arena/protocolremains the AUDIT route and/timecurve*redirects still work.Accessibility/responsive notes
ChainMismatchWriteBarrier, wallet connect, AddressInline/blockie, and profile-modal paths.arena-command-console/primary test IDs and labelled primary/secondary regions.Connect wallet).Blockers
None.
marked this merge request as ready
changed the description
mentioned in commit
3e9819bfddSecurity review
Commit reviewed:
c2fd2c09868b96f4d3e70b40c78cbe2403ba39eb(MR !425 head)Scope: Arena command-console UI (#291) —
TimeArenaPage,ArenaSimplePage,index.css, E2E/tests, docs,rabby_playwright.mjs.Outcome: FINDINGS: 0 medium+
Summary
This MR is a frontend layout and presentation refactor. It removes the stacked concept mock, reorganizes Arena components into a single
arena-command-consolesurface, and corrects visible spend-asset labels (CL8Y → DOUB on Arena v2).Paths reviewed
ChainMismatchWriteBarrier;submitBuy/useArenaSaleSessionnot in diffdangerouslySetInnerHTML; labels derived fromsession.isArenaV2, not user inputpayWith: "cl8y"still routes via existing Arena v2 DOUB plumbing (payTokenOptionsForSimpleBuy,swapOutToken: "doub")Inline threads
None — no medium+ findings to anchor on specific diff lines.
Automated security review (Cursor Cloud Agent).
Security review
Commit reviewed:
c2fd2c09868b96f4d3e70b40c78cbe2403ba39eb(MR !425 head)Scope: Arena command-console UI (#291) —
TimeArenaPage,ArenaSimplePage,index.css, E2E/tests, docs,rabby_playwright.mjs.Outcome: FINDINGS: 0 medium+
Summary
This MR is a frontend layout and presentation refactor. It removes the stacked concept mock, reorganizes Arena components into a single
arena-command-consolesurface, and corrects visible spend-asset labels (CL8Y → DOUB on Arena v2).Paths reviewed
ChainMismatchWriteBarrier;submitBuy/useArenaSaleSessionnot in diffdangerouslySetInnerHTML; labels derived fromsession.isArenaV2, not user inputpayWith: "cl8y"still routes via existing Arena v2 DOUB plumbing (payTokenOptionsForSimpleBuy,swapOutToken: "doub")Inline threads
None — no medium+ findings to anchor on specific diff lines.
Automated security review (Cursor Cloud Agent).
mentioned in commit
cc02efcf4a