[repo-assist:issue-fix] githubnext/gh-cao issue 20097 fix review

Open
  • Is something wrong?
#20,104 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
Half a day
Newbie friendliness
25/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
javascript
Domain
documentation, frontend

Research direction

Start with the formatNumber function in dashboard/site/src/view-formatters.js, which now formats the unrounded AIC value, and with the DLS-UNIT-004 entry in docs/dashboard-language-specification.md, whose spec change needs maintainer sign-off. The matching test is in dashboard/site/test/unit/view-formatters.test.js. Done means npm test and npm run lint pass in dashboard/site and the Cost page per-repository slices sum to the total.

Written by the indexing model from the issue text.

Description

repo-assist repo-assist:issue-fix

The Cost page rounded aicc values to 2 AIC before dividing by 100, so per-repository slices ($0.40 each) disagreed with the total ($1.18). The attached patch formats the unrounded AIC value (rounded to cents by Intl), giving $0.39 each and $1.17, and updates DLS-UNIT-004 and its unit test. Validation: direct formatNumber check passed (341→$3.41, 39→$0.39, 117→$1.17); vitest/eslint could not run because dashboard/site dependencies failed to install.

Action: A maintainer should review the attached review bundle and confirm the spec change to DLS-UNIT-004 is acceptable; acceptance: npm test and npm run lint pass in dashboard/site/ and the Cost page totals reconcile.

Agent prompt

Apply the patch from the repo-assist-issue-fix-issue-20097 review bundle to githubnext/gh-cao (base f8b2fc99), then run npm ci && npm test && npm run lint && npm run typecheck in dashboard/site and npm run docs:build; fix any failures without weakening tests, then open a PR that Closes #20097.

Changed files
  • dashboard/site/src/view-formatters.js
  • dashboard/site/test/unit/view-formatters.test.js
  • docs/dashboard-language-specification.md
Control Plane

cao: githubnext/gh-cao, correlation: 41444087799-128

Generated by :tools: Repo Assist / Issue Fix · copilot · auto · ~28 AIC · ⌖ ~1 AIC · ⊞ 14.2K · ◷

  • expires on Nov 16, 2026, 12:34 PM UTC
Dominant language
JavaScript
Stars
14
Forks
5
Avg merge
3h 24m
Merged PRs (30d)
796

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from githubnext/gh-cao

All issues in githubnext/gh-cao

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.