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] TYPE-016: Conformance checker regex false-positives on enforceCommentLimits signature (comment_limit_helpers.cjs)

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#70,450 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
76/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ệ
bash, github-actions
Lĩnh vực
ci-cd, testing

Hướng nghiên cứu

Đọc kiểm tra TYPE-016 quanh dòng 2503 trong scripts/check-safe-outputs-conformance.sh và so sánh mẫu của nó với chữ ký enforceCommentLimits được mô tả trong issue. Điều chỉnh kiểm tra để vẫn xác minh cả hai tham số bắt buộc khi có thêm tham số theo sau. Sau đó chạy bash scripts/check-safe-outputs-conformance.sh và xác nhận TYPE-016 đạt yêu cầu, đồng thời vẫn phát hiện khi một trong hai tham số bị xóa.

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

Mô tả

automated conformance high-priority safe-outputs
Conformance Check Failure

Check ID: TYPE-016
Severity: HIGH
Category: Specification/Implementation (add_comment Configurable Link Limit, Section 7.1 / 8.3 MCE5, v1.29.9)

Problem Description

scripts/check-safe-outputs-conformance.sh reports:

[HIGH] TYPE-016: enforceCommentLimits does not accept a configurable maxLinks and agent-only linkBody (generated links must be excluded)

Investigation result: this is a checker regex false positive, not a product defect. The actual implementation in actions/setup/js/comment_limit_helpers.cjs (function enforceCommentLimits, around line 64) is:

function enforceCommentLimits(body, { maxLinks = MAX_LINKS, linkBody = body, maxLength = MAX_COMMENT_LENGTH } = {})

This already supports a configurable maxLinks, a separate linkBody (so generated/footer content can be excluded from the link count), and the E008 error message correctly interpolates the configured limit: "E008: Comment contains \${links} links, maximum is \${maxLinks}..." (line 79).

The checker regex at scripts/check-safe-outputs-conformance.sh line 2503 is:

grep -qE "function enforceCommentLimits\(body, \{ maxLinks = MAX_LINKS, linkBody = body \}" "$helpers"

This requires the destructured parameter list to end immediately after "linkBody = body" (literal " }" right after). Since a third parameter, maxLength = MAX_COMMENT_LENGTH, was added between linkBody = body and the closing brace (to support the body-length limit, E006), the regex no longer matches even though the required maxLinks/linkBody behavior is fully intact. The sibling checks in the same TYPE-016 function (handler enforcement, MCP gateway enforcement, schema max-links definition) all still pass.

Affected Components
  • Files: scripts/check-safe-outputs-conformance.sh (line 2503, TYPE-016 check)
  • Verified correct (no change needed): actions/setup/js/comment_limit_helpers.cjs
Current vs Expected Behavior
Current Behavior

The checker regex anchors on an exact, dated function-signature substring that breaks whenever a new parameter is inserted before the closing brace, even when maxLinks/linkBody configurability is preserved.

Expected Behavior

The checker should verify presence of the required destructured parameters (maxLinks = MAX_LINKS and linkBody = body) without requiring them to be the last parameters in the signature, so legitimate signature extensions (like the maxLength parameter) do not trip a false HIGH finding.

Remediation Steps

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

  1. In scripts/check-safe-outputs-conformance.sh, update the TYPE-016 regex (around line 2503) to drop the trailing closing-brace anchor, or split it into two independent greps: one for "maxLinks = MAX_LINKS" and one for "linkBody = body", both scoped to the enforceCommentLimits function definition.
  2. Re-run bash scripts/check-safe-outputs-conformance.sh and confirm TYPE-016 reports PASS.
  3. Keep the check meaningful: ensure it still fails if a future refactor removes maxLinks or linkBody configurability from enforceCommentLimits.
  4. No change needed in actions/setup/js/comment_limit_helpers.cjs itself - the implementation already satisfies the TYPE-016 requirement.
Verification

After remediation, verify the fix by running: bash scripts/check-safe-outputs-conformance.sh

The check TYPE-016 should pass without errors.

References
  • Safe Outputs Specification: docs/src/content/docs/specs/safe-outputs-specification.md
  • Conformance Checker: scripts/check-safe-outputs-conformance.sh
  • Run ID: 39376524099
  • Date: 2026-10-20

Generated by ✅ Daily Safe Outputs Conformance Checker · claude · agent · 98 AIC · ⌖ 0.625 AIC · ⊞ 5.4K · ◷

  • expires on Oct 20, 2026, 9:53 PM UTC-08:00
Ngôn ngữ chính
Go
Star
5.4k
Fork
596
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.