[deep-report] Make shared-workflow forbidden-field validation errors deterministic in schema_validation.go
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず pkg/parser/schema_validation.go:52-86、特に validateSharedWorkflowFields を確認し、forbiddenFound の構築方法とエラーのフォーマット方法を調べてください。フォーマットする前に禁止フィールドをソートしてください。また、sharedWorkflowAllowedOnFieldList をガイダンスに反映すべきか検討してください。複数の禁止フィールドがある場合にエラーが決定的になり、追加したガイダンスが関連するテストでカバーされていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- Go
- スター
- 5.4k
- フォーク
- 576
- 平均マージ
- 8時間 29分
- マージ済み PR（30日）
- 783
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/gh-aw のほかの issue
-
[duplicate-code] Duplicate Code: pull_request event detection duplicated across safe_update filesオープンautomated-analysis code-quality cookie refactoring
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
[deep-report] Migrate manual os.Setenv/Unsetenv restore patterns to t.Setenv in 2 pkg/cli test filesオープンautomation code-quality cookie deep-report improvement quick-win task-mining
難易度 2/5 1時間未満 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
automation code-quality cookie deep-report improvement quick-win task-mining
難易度 2/5 1時間未満 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
ai-generated cookie high-priority security
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/gh-aw#66933 · コメント 12 件 ·
メンテナーはふだん 1 日以内に返信
-
automation cli cookie documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
-
[Chore] Remove dead AutogenV2 feature flag対応中かも @geeknishantkyeus が今日担当しました。 オープンbug triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
kyverno/kyverno#17936 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100