Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

fix(performance): go-faster counts paged reads and re-reads of growing task output as repeats

Đang mở
  • Có gì không ổn?
#6,337 1 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
72/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
performance, tooling

Hướng nghiên cứu

Start in plugins/performance/scripts/findings.py around the Read and Bash transcript-count keys (near lines 897-898). Key reads on (file_path, offset, limit) and skip or downgrade Bash reads of session background-task output (tasks/*.output) and commands whose target changed. Add a regression test covering paged reads of one file and re-reads of a growing task output that expects no measured now-repeat findings.

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

Mô tả

needs-triage priority: medium

Problem

/performance:go-faster's session-work area offers "Adopt now" batching findings for reads and commands that were not repeats.

findings.py transcript-counts keys a Read on file_path alone (plugins/performance/scripts/findings.py:897-898) and a Bash call on its command text. So:

  • Paged reads of one large file (same path, different offset/limit) count as the file "read 3 times".
  • Re-reading a background task's output file while the task is still writing to it counts as "the same command run 2 times", although its input changed between the two runs.

Both were offered as now items (session-work-2, session-work-3) with the advice "reuse the earlier output while its inputs are unchanged". Following it would have been wrong in both cases: the reused output would have been a different section of the file, or stale task output.

Evidence

Sweep run 20261004T190412Z, session 4c1577ca:

  • repeated_reads: a 109 KB handoff file read 3 times, at three different offsets, once each.
  • repeated_commands: cat <background task output> run twice, minutes apart, while the task appended to the file.

Fix

  • Key a Read on (file_path, offset, limit), so only an identical read counts as a repeat.
  • Do not count a Bash read of a file the session's background tasks write to (tasks/*.output), or a command whose target changed between runs, as a repeat. Where change cannot be detected, label the finding a candidate, not a measured now item.
  • Add a regression test with both shapes that expects no repeat finding.
Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
5 giờ 11 phút
Pull request đã merge (30 ngày)
838

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 melodic-software/claude-code-plugins

Tất cả issue của melodic-software/claude-code-plugins

Issue tương tự

Thêm issue về Shell/Bash

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.