[repo-assist:issue-triage] githubnext/gh-cao issue 20097 triage guidance

未关闭
#20,103 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
半天
新手友好度
15/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
javascript

调研方向

Start with the aicc branch of formatNumber in dashboard/site/src/view-formatters.js, which calls formatAicc after rounding to 2 significant figures. Then read DLS-UNIT-004 in docs/dashboard-language-specification.md, since the spec mandates that rounding and a maintainer must approve any change before code is touched. Done means 39 AIC renders $0.39, the formatter tests and significant: 2 fixtures are updated, and the dashboard test suite passes.

由索引模型根据 Issue 内容生成。

描述

repo-assist repo-assist:issue-triage

Target githubnext/gh-cao issue 20097 (Cost page AIC rounding): the behavior is real but spec-mandated, so a fix needs a maintainer decision on DLS-UNIT-004 first. Confidence: high.

Action: A dashboard maintainer should decide whether to amend DLS-UNIT-004 (cents-grain rounding); if approved, assign the issue to Copilot with the prompt below. Acceptance: 39 AIC renders $0.39, 117 AIC renders $1.17, and dashboard unit tests and npm run check pass.

Evidence
  • docs/dashboard-language-specification.md DLS-UNIT-004 requires an aicc presenter to round the AIC value to significant: 2 before dividing by 100. 39 AIC becomes 40 and renders $0.40, matching the report.
  • dashboard/site/src/view-formatters.js formatNumber calls formatAicc(rounded) for aicc units, which implements that rule. It is not an implementation bug.
  • Rounding to 2 AIC gives $0.02 granularity, so sums of displayed parts cannot reconcile with totals. Rounding after the divide by 100 (cents) fixes this.
  • Rejected: tooltip-only unrounded AIC. It leaves the visible figures inconsistent.
  • Breaking-spec risk: changing a MUST rule requires updating the spec, docs/, formatter tests and contract fixtures together.
Agent prompt

In githubnext/gh-cao, change aicc formatting so the AIC value is divided by 100 and then rounded to cents, instead of being rounded to 2 AIC first. Update formatNumber in dashboard/site/src/view-formatters.js, amend DLS-UNIT-004 in docs/dashboard-language-specification.md, and update any significant: 2 fixtures or tests. Add a unit test showing 39 AIC renders $0.39 and three of them sum to $1.17. Run npm test, npm run lint and npm run typecheck in dashboard/site/, then npm run docs:build.

Control Plane

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

Generated by 🏷️ Repo Assist / Issue Triage · copilot · auto · ~20 AIC · ⌖ ~1 AIC · ⊞ 14K · ◷

  • expires on Nov 16, 2026, 12:32 PM UTC
主要语言
JavaScript
星标
14
派生
5
平均合并
3 小时 24 分钟
30 天内合并 PR
796

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

githubnext/gh-cao 的其他 Issue

查看 githubnext/gh-cao 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。