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

[workshop-sync] workshop/side-quest-17-06-output-injection.md: stale add_comment.cjs line references

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#4,679 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
78/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github, javascript
Lĩnh vực
documentation

Hướng nghiên cứu

Bắt đầu với workshop/side-quest-17-06-output-injection.md, đặc biệt là khối chi tiết quanh dòng 70 và bài tập “Inspect the Validation Source” quanh các dòng 92–94. Đối chiếu hành vi được mô tả với actions/setup/js/add_comment.cjs, comment_limit_helpers.cjs và pkg/workflow/safe_outputs_parser.go; cập nhật các liên kết hoặc hướng dẫn tìm kiếm, đồng thời làm rõ rằng tất cả nhãn bắt buộc được liệt kê đều phải khớp. Hoàn thành khi các tham chiếu trong workshop dẫn người học đến những bước kiểm tra hiện tại.

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

Mô tả

documentation

Workshop file reviewed

workshop/side-quest-17-06-output-injection.md (reviewed together with side-quest-17-05-supply-chain-mcp.md and side-quest-17-07-repo-poisoning.md; the other two had no accuracy issues). Checked against gh-aw release v0.92.0.

Problem

The file links to hard-coded line ranges in actions/setup/js/add_comment.cjs on main, and they no longer point at the described code:

add_comment.cjs#L582-L650, #L582-L583 ("the required-labels target check") and #L646-L650 ("comment sanitization and limits")

In the current main file (1184 lines), lines 582-583 and 646-650 are inside handleAddComment setup and comment-ID/target resolution code. They do not contain the label check or sanitization.

Current behaviour

  • The required-labels filter (does not match required-labels filter) is at about lines 724-762.
  • sanitizeContent(processedBody, ...) is at about line 817.
  • Mention and link limits come from comment_limit_helpers.cjs (enforceCommentLimits).
  • required-labels is still valid and is documented as "ALL must match" in pkg/workflow/safe_outputs_parser.go.

Suggested fix

  • Update the line anchors in the details block (line 70) and in the "Inspect the Validation Source" exercise (lines 92-94) to the current locations.
  • Better, pin the links to a release tag or commit SHA instead of main so they stay stable. Alternatively, link to the file without line anchors and tell learners to search for required-labels and sanitizeContent.
  • Line 93 says the check is "required-labels target check". The text could also note that all listed labels must be present.

Generated by 🔍 Workshop Sync Check · copilot · auto · 23.4 AIC · ⌖ 0.552 AIC · ⊞ 9.2K · ◷

  • expires on Oct 18, 2026, 8:39 AM UTC
Ngôn ngữ chính
JavaScript
Star
56
Fork
40
Merge trung bình
2 ngày 7 giờ
Pull request đã merge (30 ngày)
16

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.

  • Không có Dockerfile hay tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

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-aw-workshop

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

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.