Repository navigation
Conversation
The top-bar LIVE badge and catalog listing metrics used "live" in a way a first user can read as a tradable feed. Rename them to SSE connected, Observed catalog, and Catalog listings without changing what they measure or the analysis-only authority boundary. Co-authored-by: RainMona <RainMona@users.noreply.github.com>
RainMona
marked this pull request as ready for review
August 18, 2026 08:33
This was referenced Aug 18, 2026
Author
|
Superseded by topic PR #42 (Studio is research, not a live trading desk). This small PR is folded into that branch from origin/main. Do not merge this one. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #3
Problem
Studio showed Live data when projection sync was
LIVE(SSE connected) and Live markets for the anonymous catalog listing count. Those labels sit next to Research mode · Analysis only · no execution and the footer that opportunities are research evidence, not orders. A first user can still read “live” as a live book / tradable feed. This is a safety misread, not a missing feature.What changed
Copy only. The badges still measure the same things, and authority is unchanged.
LIVE)Plan notes that taught the old top-bar words now say SSE connected.
Kept: Research mode · Analysis only · no execution. No orders, signing, funds, OAuth, campaigns, or model spend.
liveExecutionEnabledis untouched.Independent of #1 / #2
This branch starts from current
main(c1f63ce). It does not include Discover first-step or Findings CTA work fromcursor/first-operator-next-step-044b/ PR #5.Out of scope: issues #1, #2, and #4.
How to check in Studio
Open the Vite-printed URL (usually
http://127.0.0.1:5173/):?view=overviewor the default page) should show Observed catalog on the snapshot card and Catalog listings for the listing-count metric, not Live markets.?view=discover) should use Catalog listings for the same kind of count.pnpm --filter @pmh/studio testExisting Studio tests do not assert the old strings. This PR does not add product behavior, so no new test file was required.