Skip to content

fix: nest model under engine: in PR reviewer YAML example#1251

Merged
pelikhan merged 4 commits into
mainfrom
copilot/workshop-sync-fix-frontmatter
Jul 16, 2026
Merged

fix: nest model under engine: in PR reviewer YAML example#1251
pelikhan merged 4 commits into
mainfrom
copilot/workshop-sync-fix-frontmatter

Conversation

Copilot AI commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

model: was used as an invalid top-level frontmatter field in the annotated YAML example in workshop/11c-build-pr-reviewer.md. Per the agentic workflow syntax reference, model must be a sub-field of engine:.

Changes

  • workshop/11c-build-pr-reviewer.md: Replace top-level model: gpt-4o with a proper engine: block; add a clarifying note that model is a sub-field of engine:, not a top-level key.
# Before
model: gpt-4o # LLM used for the review

# After
engine:
  id: copilot
  model: gpt-4o   # LLM used for the review

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix invalid top-level frontmatter field in workshop file fix: nest model under engine: in PR reviewer YAML example Jul 15, 2026
Copilot AI requested a review from pelikhan July 15, 2026 22:06
Comment thread workshop/11c-build-pr-reviewer.md Outdated
model: gpt-4o # LLM used for the review
engine:
id: copilot
model: gpt-4o # LLM used for the review

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot suggest other model, newer

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to gpt-4.1 in commit 449e818's follow-up — it's the current successor to gpt-4o for the Copilot engine.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan July 16, 2026 04:16
Comment thread workshop/11c-build-pr-reviewer.md Outdated
model: gpt-4o # LLM used for the review
engine:
id: copilot
model: gpt-4.1 # LLM used for the review

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot use a gpt-5.4-mini

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to gpt-5.4-mini in the latest commit.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan July 16, 2026 04:19
@pelikhan
pelikhan marked this pull request as ready for review July 16, 2026 04:25
@pelikhan
pelikhan merged commit ec5716f into main Jul 16, 2026
@pelikhan
pelikhan deleted the copilot/workshop-sync-fix-frontmatter branch July 16, 2026 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[workshop-sync] workshop/11c-build-pr-reviewer.md: model: used as invalid top-level frontmatter field

2 participants