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

[Safe Outputs Conformance] MCE-001: add_comment tool description wording does not match conformance checker 50-link disclosure pattern

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#68,972 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
68/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ệ
bash, go, json
Lĩnh vực
devtools

Hướng nghiên cứu

Mô tả của add_comment và mô tả của trường body nằm trong pkg/workflow/js/safe_outputs_tools.json, còn regex bị lỗi nằm trong check_mce_constraint_disclosure ở scripts/check-safe-outputs-conformance.sh. Bắt đầu bằng cách so sánh cách diễn đạt max-links hiện tại với mẫu 50 link mà bộ kiểm tra tìm kiếm, rồi quyết định nên viết lại JSON hay mở rộng regex; việc này cần maintainer xác nhận. Hoàn thành khi bộ kiểm tra báo MCE-001 là đạt khi chạy.

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

Mô tả

automated conformance safe-outputs
Conformance Check Failure

Check ID: MCE-001
Severity: MEDIUM
Category: Implementation (MCP Constraint Enforcement, Section 8.3 MCE2)

Problem Description

scripts/check-safe-outputs-conformance.sh (check check_mce_constraint_disclosure) requires the add_comment tool description in pkg/workflow/js/safe_outputs_tools.json to contain the literal substring 50 link (case-insensitive) as proof that the 50-URL constraint is disclosed to the agent per spec Section 8.3 MCE2.

The description does disclose the limit, but with different wording:

"...the configured max-links URL occurrences in agent content (default: 50; generated footer and metadata links excluded)..."

Because the digit 50 and the word link/links never appear adjacent, the checker's grep -iE "50 link" does not match, and the check fails with a MEDIUM-severity false-positive-style result even though the underlying requirement (informing the agent of the link limit before it calls the tool) is satisfied.

This will keep re-triggering on every scheduled run of this conformance workflow until either the description wording or the checker pattern is adjusted, generating recurring noise.

Affected Components
  • Files:
    • pkg/workflow/js/safe_outputs_tools.json (canonical source, embedded via go:embed in pkg/workflow/safe_outputs_tools.go) — add_comment tool description (~line 525) and body field description (~line 533)
    • scripts/check-safe-outputs-conformance.sh — check_mce_constraint_disclosure (around line 684)
🔍 Current vs Expected Behavior
Current Behavior

The add_comment description states the link limit as "...(default: 50; generated footer and metadata links excluded)" — correct in substance, but it does not contain a string matching 50 link so the automated checker flags it as missing disclosure.

Expected Behavior

Either:

  1. The tool description text explicitly includes a phrase like "maximum 50 links" / "50 link limit" so the disclosure is unambiguous to both the agent and the automated conformance check, OR
  2. The checker's regex is broadened to also accept the existing "max-links ... default: 50" phrasing (and similar equivalent wording) so it does not flag substantively-conformant descriptions.
Remediation Steps

This task can be assigned to a Copilot coding agent with the following steps:

  1. Open pkg/workflow/js/safe_outputs_tools.json and locate the add_comment tool description and its body field description (two occurrences of the max-links wording).
  2. Reword the link-limit clause to literally include "50 links" (or "50 link limit"), e.g. change "...the configured max-links URL occurrences in agent content (default: 50; generated footer and metadata links excluded)..." to "...a maximum of 50 links (configurable via max-links; generated footer and metadata links excluded)...", keeping the meaning unchanged.
  3. Regenerate/sync actions/setup/js/safe_outputs_tools.json from the canonical source if it is a generated copy (check pkg/workflow/safe_outputs_tools_generation.go / generate_safe_outputs_tools.cjs for the regeneration step), and run make fmt.
  4. Re-run the conformance checker to confirm MCE-001 passes.
  5. If, instead, the maintainers prefer to keep the current wording, update the checker regex in scripts/check-safe-outputs-conformance.sh (check_mce_constraint_disclosure) to accept the existing "max-links...default: 50" phrasing instead of requiring the literal "50 link" substring.
Verification

After remediation, verify the fix by running:

bash scripts/check-safe-outputs-conformance.sh

The check MCE-001 should pass without errors.

References
  • Safe Outputs Specification: docs/src/content/docs/specs/safe-outputs-specification.md (Section 8.3 MCE2)
  • Conformance Checker: scripts/check-safe-outputs-conformance.sh
  • Run ID: §38688391715
  • Date: 2026-10-15

Generated by ✅ Daily Safe Outputs Conformance Checker · claude · agent · 83.3 AIC · ⊞ 5.3K · ◷

  • expires on Oct 15, 2026, 9:49 PM UTC-08:00
Ngôn ngữ chính
Go
Star
5.4k
Fork
584
Merge trung bình
3 giờ 26 phút
Pull request đã merge (30 ngày)
822

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.