Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[workshop-sync] side-quest-21-01-sub-agent-syntax.md: incorrect claim that unsupported sub-agent fields are stripped with a warning

クローズ 初心者向け
  • 何か問題がありますか？
#4,132 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
88/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
go
領域
documentation

調査の方向性

まず workshop/side-quest-21-01-sub-agent-syntax.md から始め、その主張を pkg/parser/sub_agent_extractor.go および名前が示されている sub-agent の検証テストと比較します。誤った説明とチェックポイントを parser の動作に一致するよう更新し、発生しない警告を避けるよう学習者に教える内容が、演習にもう含まれていないことを確認します。

索引モデルが issue の本文から書いたものです。

説明

documentation

Workshop file reviewed

workshop/side-quest-21-01-sub-agent-syntax.md

Problem

The file states, regarding sub-agent frontmatter:

Any fields other than description and model are stripped from sub-agent frontmatter at runtime with a warning.

And later, as a checkpoint item:

  • I did not see warnings about stripped sub-agent fields in that run

This is incorrect. The exercise itself asks learners to remove engine: openai from the sub-agent block as if it would otherwise cause a stripped-field warning, but the actual compiler does not strip or warn about extra fields at all.

Current correct syntax

Per pkg/parser/sub_agent_extractor.go in github/gh-aw:

Sub-agent frontmatter keys and their order are preserved without filtering; boundary whitespace is trimmed.

And ValidateInlineSubAgentsFrontmatter is explicitly "best-effort" and only warns on frontmatter parse errors (e.g. unterminated YAML strings) or duplicate agent names — never on unsupported/arbitrary fields. Unit tests (sub_agent_validation_test.go) confirm this: TestValidateInlineSubAgentsFrontmatter_ArbitraryFields and _PreservedFields explicitly include engine: copilot and other arbitrary fields in a sub-agent block and assert assert.Empty(t, warnings, ...).

The official Inline Sub-Agents reference only lists model and description as documented frontmatter fields, but does not state that other fields are stripped or produce a warning — they are simply preserved/ignored by the compiler without validation.

Suggested fix

Remove or rephrase the "stripped... with a warning" claim and the related checkpoint item. Instead, state that only description and model are documented/supported sub-agent fields, and that other fields (like engine) are not validated or used by the runtime, but will not produce a compiler warning. Consider reframing the exercise's "fix" step around correctness/clarity rather than avoiding a warning that does not actually occur.

Generated by 🔍 Workshop Sync Check · copilot · auto · 87.5 AIC · ⌖ 5.89 AIC · ⊞ 9.2K · ◷

  • expires on Oct 4, 2026, 4:53 AM UTC
主要言語
JavaScript
スター
55
フォーク
36
平均マージ
2日 8時間
マージ済み PR（30日）
17

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートなし
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

githubnext/gh-aw-workshop のほかの issue

githubnext/gh-aw-workshop の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。