Skip to content

feat(demo): simulate trading order pushes - #746

Merged
luokerenx4 merged 1 commit into
devfrom
codex/demo-trading-push
Jul 29, 2026
Merged

feat(demo): simulate trading order pushes#746
luokerenx4 merged 1 commit into
devfrom
codex/demo-trading-push

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

Summary

  • return deterministic, non-persistent demo push results for place, close, and cancel order endpoints
  • mark demo results explicitly as simulated so the order dialog never claims a broker accepted them
  • add demo handler contract coverage for all three order mutations

Why

The demo exposes the complete manual order form and says mutations are simulated, but every trading submission previously ended in a hard-coded read-only validation error. That prevented visitors from seeing the stage → commit → push result surface. This keeps the demo safe and non-persistent while allowing the workflow to complete honestly.

Verification

  • git diff --check
  • npx tsc --noEmit
  • pnpm test (358 passed, 1 skipped; 3391 tests passed, 9 skipped)
  • cd ui && npx tsc -b
  • pnpm -F open-alice-ui build:demo
  • walked pnpm -F open-alice-ui dev:demo --host 127.0.0.1 in the real Portfolio → Alpaca Paper route
    • Place Order shows 1 operation simulated
    • AAPL Close Position shows 1 operation simulated
    • both state that no order was sent to a broker and portfolio data was not changed

Safety

Demo MSW handlers only. No UTA state-machine, broker adapter, configured account, or live-paper path was exercised or changed.

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview, Comment Jul 28, 2026 7:37pm

Request Review

@luokerenx4 luokerenx4 added area:demo Demo runtime, fixtures, handlers, or banner theme:demo Demo fidelity, fixtures, or simulated interactions workflow:parallel Autonomous parallel contribution; leave open for later acceptance labels Jul 29, 2026 — with ChatGPT Codex Connector
@luokerenx4
luokerenx4 merged commit 4b56a57 into dev Jul 29, 2026
7 checks passed
@luokerenx4
luokerenx4 deleted the codex/demo-trading-push branch July 29, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:demo Demo runtime, fixtures, handlers, or banner theme:demo Demo fidelity, fixtures, or simulated interactions workflow:parallel Autonomous parallel contribution; leave open for later acceptance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant