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

Đang mở
  • Có gì không ổn?
#20,103 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
Nửa ngày
Mức phù hợp với người mới
15/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript
Lĩnh vực
documentation, frontend

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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
Ngôn ngữ chính
JavaScript
Star
14
Fork
5
Merge trung bình
3 giờ 24 phút
Pull request đã merge (30 ngày)
796

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của githubnext/gh-cao

Tất cả issue của githubnext/gh-cao

Issue tương tự

Thêm issue về JavaScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.