[deep-report] Make shared-workflow forbidden-field validation errors deterministic in schema_validation.go
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 76/100
Línea de trabajo
Empieza por pkg/parser/schema_validation.go:52-86, especialmente por validateSharedWorkflowFields, e inspecciona cómo construye forbiddenFound y da formato al error. Ordena los campos prohibidos antes de darles formato; considera si sharedWorkflowAllowedOnFieldList debería influir en las indicaciones. La tarea estará terminada cuando el error sea determinista para varios campos prohibidos y cualquier indicación añadida esté cubierta por pruebas pertinentes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Go
- Estrellas
- 5.4k
- Forks
- 576
- Merge medio
- 8 h 29 min
- PR fusionados (30 d)
- 783
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de github/gh-aw
-
[duplicate-code] Duplicate Code: pull_request event detection duplicated across safe_update filesAbiertoautomated-analysis code-quality cookie refactoring
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
[deep-report] Migrate manual os.Setenv/Unsetenv restore patterns to t.Setenv in 2 pkg/cli test filesAbiertoautomation code-quality cookie deep-report improvement quick-win task-mining
Dificultad 2/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
[deep-report] Migrate os.Chdir+t.Parallel() test patterns to t.Chdir in 5 pkg/cli test filesAbiertoautomation code-quality cookie deep-report improvement quick-win task-mining
Dificultad 2/5 Menos de una hora Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
ai-generated cookie high-priority security
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
github/gh-aw#66933 · 12 comentarios ·
Los mantenedores suelen responder en 1 día
-
automation cli cookie documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
Todos los issues de github/gh-aw
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Broken Claude manifestAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
[Chore] Remove dead AutogenV2 feature flagPosiblemente ocupada @geeknishantkyeus la tomó hoy. Abiertobug triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
kyverno/kyverno#17936 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100