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

[deep-report] Make shared-workflow forbidden-field validation errors deterministic in schema_validation.go

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#66,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
Dưới một 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ệ
go
Lĩnh vực
backend

Hướng nghiên cứu

Bắt đầu với pkg/parser/schema_validation.go:52-86, đặc biệt là validateSharedWorkflowFields, và kiểm tra cách hàm này xây dựng forbiddenFound cũng như định dạng lỗi. Sắp xếp các trường bị cấm trước khi định dạng; cân nhắc xem sharedWorkflowAllowedOnFieldList có nên định hướng phần hướng dẫn hay không. Hoàn thành khi lỗi có tính xác định với nhiều trường bị cấm và mọi hướng dẫn được thêm vào đều được các bài kiểm thử phù hợp kiểm tra.

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

Mô tả

automation code-quality cookie deep-report improvement quick-win task-mining
Description

validateSharedWorkflowFields (pkg/parser/schema_validation.go:52-86) iterates frontmatter via for key := range frontmatter — a Go map, whose iteration order is randomized per-run. When a shared workflow has multiple forbidden fields, the resulting error message (built from forbiddenFound at lines 83/85) lists them in non-deterministic order across compiler runs. The generic forbidden-field error also omits any "here's what IS allowed" guidance, even though the file already has that list (sharedWorkflowAllowedOnFieldList, used for the on-field case at line 155).

Expected Impact

Deterministic, reproducible error messages (useful for golden tests and bug reports that currently can't rely on message stability), plus more actionable guidance for authors fixing a shared-workflow frontmatter.

Suggested Fix

Sort forbiddenFound before formatting the multi-field error (line 85). Consider enriching the single/multi-field forbidden-field messages with a short pointer to what IS allowed in shared workflows, mirroring the existing on-field error's use of sharedWorkflowAllowedOnFieldList.

Code Region: pkg/parser/schema_validation.go:52-86

Suggested Agent

Developer (Go)

Estimated Effort

Quick (< 1 hour)

Data Source

DeepReport Intelligence Briefing, 2026-10-08 (cycle 3). Source discussion #66921. Live-verified today: validateSharedWorkflowFields does iterate a map with for key := range frontmatter, and the forbidden-field error at lines 83/85 does not reference the allowed-field list.

Generated by 🔬 Deep Report · claude · agent · 343.4 AIC · ⌖ 9.25 AIC · ⊞ 7.1K · ◷

  • expires on Oct 10, 2026, 10:47 AM UTC-08:00
Ngôn ngữ chính
Go
Star
5.4k
Fork
576
Merge trung bình
8 giờ 29 phút
Pull request đã merge (30 ngày)
783

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.