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

[safeoutputs] Fix incorrect safe output usage in Smoke Built-in Ledgers prompt

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#71,952 3 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

@pelikhan đang làm issue này rồi.

Từ ngày 26/10/2026.

  • #72095 của @copilot-swe-agent — đã merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
75/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ệ
github-actions, go
Lĩnh vực
tooling

Hướng nghiên cứu

Đọc các dòng 98–114 trong .github/workflows/smoke-builtin-ledgers.md và pkg/workflow/js/safe_outputs_tools.json. Cập nhật prompt để dùng ledger_map_put và ledger_map_delete cho các thao tác trên map, đồng thời làm rõ các giá trị ledger được phép trong phần mô tả ledger_append. Hoàn tất khi prompt và phần mô tả công cụ phân biệt các thao tác trên map với thao tác trên counter, log, set và table.

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

Mô tả

automated-analysis bug cookie safe-outputs tool-improvement workflow-issue

Prompt in .github/workflows/smoke-builtin-ledgers.md (lines 98-114) tells the agent to send map-ledger put/delete via ledger_append, e.g. ledger=map, operation=put. But ledger_append only accepts ledger in [counter,log,set,table]; map requires the separate ledger_map_put/ledger_map_delete tools, which the prompt never mentions even though the compiled lock file exposes them. Evidence: run 40146986920 (Smoke Built-in Ledgers) got McpError -32602 ledger must be one of: counter, log, set, table, received map, on ledger_append, then self-recovered by calling ledger_map_put successfully. Fix: rewrite the prompt to call ledger_map_put/ledger_map_delete explicitly for map, not ledger_append. Secondary: ledger_append description in pkg/workflow/js/safe_outputs_tools.json only says the ledger field is a Configured ledger name, without stating the counter/log/set/table restriction or pointing to ledger_map_put/ledger_map_delete for map - worth clarifying too. This was the only safe-output tool error found across the full last-24h window (727 run directories scanned).

Generated by ⚡ Daily Safe Output Tool Optimizer · claude · agent · ~251 AIC · ⌖ ~1 AIC · ⊞ 4.7K · ◷

  • expires on Oct 27, 2026, 1:20 PM UTC-08:00
Ngôn ngữ chính
Go
Star
5.5k
Fork
606
Merge trung bình
4 giờ 13 phút
Pull request đã merge (30 ngày)
823

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

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

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 github/gh-aw

Tất cả issue của github/gh-aw

Issue tương tự

Thêm issue về Go

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.