Repository navigation
docs: record final deployed follow-up and OCI acceptance blocker - #114
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Bounded review of the two changed evidence excerpts completed: Standards0 findings, Spec0 findings. No runtime code changed. Observed production frontend322c115 and mainCI38023941212 evidence is recorded separately from blocked OCI rollout; oldfinancialvalues remain explicitly unvalidated. Required PR CI38024679049 is in progress. No unrelateduser changes were staged. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a2e954b59a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| [PR #113](https://github.com/Kohnnn/vnibb/pull/113) merged as `322c115a85581888bb783b75c8d3e52461e8beb4` after resolving the squash-ancestry conflicts in an isolated worktree. Its changes are limited to release evidence, the plan and removal of an incidental object-identity assertion; runtime code is unchanged. All 38 financial-service tests passed. Standards and Spec reviews found zero findings. The [synchronized PR CI](https://github.com/Kohnnn/vnibb/actions/runs/38021759796) and [merged main CI](https://github.com/Kohnnn/vnibb/actions/runs/38023941212) passed all four jobs; main completed nine root steps in 371.6 seconds with 157 frontend suites/1,044 tests and 1,621 backend passes/four skips. | ||
|
|
||
| Production frontend deployment `dpl_2MjCKpdHruKZfwnvwvxjZBhuNzXq` is READY at that exact Git revision and serves `vnibb-web.vercel.app`. Certificate-validating `/api/live` and `/api/ready` returned HTTP 200. A fresh production browser replay confirmed ten MSR Quant widgets left loading on warm reload; momentum direction, score and returns, signal, gap statistics, backtest and sweep remained withheld on uncertified history. Sampled legacy extreme-return, BUY and Strong Downtrend tokens were absent. An invalid tab rewrote to the real Quant tab. MSR Technical ATR/MACD/Bollinger/RSI remained unavailable. VNM shared TTM selected the period and disclosed unsupported ratios. This bounded replay does not repeat or expand the earlier 40/40 navigation coverage. |
There was a problem hiding this comment.
This says the ten Quant widgets were left loading after the warm reload, contradicting both the immediately preceding replay result (“ten unavailable widgets and zero loading” on line 67) and the release-plan acceptance statement in qa-ticket-plan.md that zero widgets remained loading. If the intended claim is that all ten widgets exited the loading state and became unavailable, state that explicitly; as written, this records the deferred-mount failure as still present and makes the final acceptance evidence internally inconsistent.
Useful? React with 👍 / 👎.
|
Final evidence PR114 merged as bbad7c0. Main CI https://github.com/Kohnnn/vnibb/actions/runs/38025299364 passed all four jobs. Final canonical production deployment is READY dpl_CJttv1fvzKkTcGemchzXtHXxMGC5, URL https://vnibb-5a5m03y1h-vphk2001-gmailcoms-projects.vercel.app, Gitrevision bbad7c. Canonical vnibb-web.vercel.app /api/live and /api/ready HTTP200; actual browser MSR Quant warm reload again reached ten terminal widget states with withheld momentum and no sampled legacy extreme-return/BUY/StrongDowntrend tokens. Managed browser tab closed. No runtime changes beyond previously merged PR112; published image0e5dfdd/b1ec3141 remains backend rollout target. Every OCI additional-check request expired without authentication; no active approval URL, no backend service replacement. Fresh human-approved Tailscale SSH and new-backend financial acceptance remain the only unfinished release gates. User glossary/Matrix work remains untouched. |
This branch was successfully deployed
1 participant
Refs #111. Records exact frontend322c115 production deployment, green main CI and the bounded actual browser replay. Runtime code is unchanged. API/MCP/scheduler rollout and new-backend financial acceptance remain blocked on human-approved Tailscale SSH. Evidence-only commit a2e954b; unrelated glossary and Matrix changes remain untouched. Verification: main CI38023941212 all four jobs passed; actual production warm reload, invalid-tab resolution, Technical unavailable states and shared TTM ratio disclosure observed. No QA acceptance issue is closed.