fix: use reference price for market baseline

[ibruno]

Summary by cubic

Use the reference opening price as the market baseline for live charts. Removes the WebSocket-derived baseline to keep the chart stable and consistent.

• Bug Fixes
  • Always resolve the baseline from referenceOpeningPrice (no fallback to first live tick).
  • Removed baseline state and setter from useLiveSeriesPriceSnapshot and useLiveSeriesWebSocket, and updated the chart accordingly.
  • Updated tests to reflect the new baseline logic.

^{Written for commit e33e29e. Summary will update on new commits.}

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[cubic-dev-ai]

1 issue found across 4 files

Confidence score: 4/5

• In src/app/[locale]/(platform)/event/[slug]/_components/EventLiveSeriesChart.tsx, if the reference snapshot is missing opening_price, the chart can still stream live prices but lose the Price To Beat, delta, and target line, which weakens key trading context for users viewing live events. Restore a fallback baseline (or guard with an explicit default behavior) before merging to avoid this regression.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/app/[locale]/(platform)/event/[slug]/_components/EventLiveSeriesChart.tsx">

<violation number="1" location="src/app/[locale]/(platform)/event/[slug]/_components/EventLiveSeriesChart.tsx:463">
P2: When the reference snapshot is unavailable or has no `opening_price`, the chart still receives live prices but now renders no Price To Beat, delta, or target line because the previous WebSocket baseline fallback was removed. Preserving the live baseline only while the canonical reference is unavailable would avoid blank baseline UI without overriding a valid reference price.</violation>
</file>

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[cubic-dev-ai]

P2: When the reference snapshot is unavailable or has no opening_price, the chart still receives live prices but now renders no Price To Beat, delta, or target line because the previous WebSocket baseline fallback was removed. Preserving the live baseline only while the canonical reference is unavailable would avoid blank baseline UI without overriding a valid reference price.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At src/app/[locale]/(platform)/event/[slug]/_components/EventLiveSeriesChart.tsx, line 463:

<comment>When the reference snapshot is unavailable or has no `opening_price`, the chart still receives live prices but now renders no Price To Beat, delta, or target line because the previous WebSocket baseline fallback was removed. Preserving the live baseline only while the canonical reference is unavailable would avoid blank baseline UI without overriding a valid reference price.</comment>

<file context>
@@ -463,9 +460,7 @@ function EventLiveSeriesChartContent({
-  const resolvedBaselinePrice = isEventClosed
-    ? referenceOpeningPrice
-    : baselinePrice ?? referenceOpeningPrice
+  const resolvedBaselinePrice = referenceOpeningPrice
   const precisionReferencePrice = currentPrice
     ?? resolvedBaselinePrice
</file context>