Repository navigation
feat(OMN-20758): the Overview titles Actual spend, Agent calls and Tokens processed as the mockup does - #378
Merged
Merged
Conversation
…kens processed as the mockup does The OMN-20223 re-walk failed AC4 on three titles: the Overview said Spend, Measured runs and Tokens in and out where the demo mockup says Actual spend, Agent calls and Tokens processed. Only the titles change; each card keeps its component id, binding and served figure. Savings keeps its modelled label (OMN-20008). Bumps to 1.1.8 so onex dashboard can pin the renamed page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jake-b-omni
enabled auto-merge (squash)
October 9, 2026 03:13
Contributor
|
OCC autobind did not mint a companion for this PR: no changed-file candidate could be proven RED against the merge base, and emitting a PR-existence probe instead would be non-falsifiable evidence (OMN-15247). Hand-authored evidence is required. Considered 9 changed file(s):
This verdict is occ-autobind's, derived from this head and its merge base alone. A second producer, the occ-companion-effect backstop, consumes its own command on the lab bus and may still mint a companion under the generic binding. Whether it does depends on that consumer running, not on this diff (OMN-18876). |
jake-b-omni
pushed a commit
that referenced
this pull request
Oct 9, 2026
#378 (OMN-20758) took 1.1.8 first, so the table fit releases as 1.1.9. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jake-b-omni
added a commit
that referenced
this pull request
Oct 9, 2026
…440x900 (#379) * fix(OMN-20759): the Runs and Recent runs tables fit their panels at 1440x900 The OMN-20223 re-walk found both run tables wider than their panels at 1440: Runs (17 columns) scrolled sideways with Host clipped, and the Overview's Recent runs clipped the Run id. Runs had no fit treatment at all, and Recent runs gave the run id and timestamp a 9ch floor while the host, a 33-character token with no break opportunity, could not wrap. Both tables now use the fit wrapper. Run id, time, model, backend and host may break inside themselves; every other cell still wraps at spaces, and no column is dropped. A Playwright check at 1440x900 asserts each table's scrollWidth fits its container, no cell is clipped and every header is present; it fails on dev with 1230 px and 82 px of overflow. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * chore(OMN-20759): bump to 1.1.9 so onex dashboard can pin the table fit #378 (OMN-20758) took 1.1.8 first, so the table fit releases as 1.1.9. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(OMN-20759): the local dashboard contract spec matches the fitted Runs table and the merged pages The scroll test asserted the Runs table scrolls sideways, the behaviour OMN-20759 removes; it now asserts the table needs no sideways scroll. Two stale expectations from merged work also failed this spec on dev: per-run cost prints $0.0004 since #376, and Savings and Baseline spend (#375) both carry the baseline caption and modelled note. The e2e suite is not in CI, so neither break was caught when it merged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Jake Brower <jake@bodegalogic.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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
1 participant
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.
The OMN-20223 re-walk on 2026-10-09 (bundle v1.1.7) reconciled every in-scope figure but failed AC4 on three titles. The Overview said Spend, Measured runs and Tokens in and out, where the demo mockup (walkthrough step 5) says Actual spend, Agent calls and Tokens processed.
Change. In
overview.page.yaml(widget title and label) andoverview.contracts.yaml(component title),overview-spend,overview-measuredandoverview-tokensare retitled. Component ids, bindings, required fields, formats and captions are unchanged, so each card shows the same served figure as before. Savings keeps its modelled label (OMN-20008), as the ticket specifies.Release. 1.1.7 → 1.1.8 (project-level lockfile fields only) with a CHANGELOG entry, so
onex dashboardcan pin the renamed page.Tests.
src/pages/LocalDashboardPage.omn20758.test.tsx(5) renders the Overview from a served metering-summary all-time row. All 5 fail without the rename.$0.0123(spend_usd), Agent calls shows23(runs_measured), and Tokens processed shows5,756 in · 378 out.omn19980,live) and the e2e speclocal-dashboard-contracts.spec.tsuse the new titles.Evidence-Source: OCC#13380
Evidence-Ticket: OMN-20758
🤖 Generated with Claude Code