[PR #70] [MERGED] feat(groom): default the agent model to claude-opus-5 #79

New issue

Closed

opened 2026-10-26 22:36:00 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 22:36:00 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/github-workflows/pull/70
Author: @mattmillerai
Created: 7/24/2026
Status: ✅ Merged
Merged: 7/26/2026
Merged by: @mattmillerai

Base: main ← Head: matt/groom-model-opus-5

📝 Commits (1)

70242b4 feat(groom): default the agent model to claude-opus-5

📊 Changes

1 file changed (+6 additions, -2 deletions)

View changed files

📝 .github/workflows/groom.yml (+6 -2)

📄 Description

Bumps the groom model input default claude-opus-4-8 → claude-opus-5, and documents the trade-off in the input description.

The finder, verifier, and builder agents are groom's output quality — surfacing real refactors, adjudicating CONFIRM/DOWNGRADE/REJECT, and authoring the patch — so the default should track the current flagship. Callers that want to hold a run steady can still pin an older id via the model: input; neither current caller (comfy-cloud-mcp-server, cloud) overrides it, so both pick this up on their next SHA bump.

ELI5

Groom uses an AI model to read the repo, decide which cleanups are real, and write the fix. This points it at the newest, strongest model instead of the previous one, because the quality of what it finds and writes depends almost entirely on that choice. Anyone who wants the old model can still ask for it by name.

Cost note: a finder pass is ~$7.50 at the previous model; expect this to shift with Opus 5 pricing. Runtime is unchanged (~12 min, well inside the 40-min timeout).

actionlint clean.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/github-workflows/pull/70 **Author:** [@mattmillerai](https://github.com/mattmillerai) **Created:** 7/24/2026 **Status:** ✅ Merged **Merged:** 7/26/2026 **Merged by:** [@mattmillerai](https://github.com/mattmillerai) **Base:** `main` ← **Head:** `matt/groom-model-opus-5` --- ### 📝 Commits (1) - [`70242b4`](https://github.com/Comfy-Org/github-workflows/commit/70242b4d983f43f68a760aec40dcd86a313ab1b5) feat(groom): default the agent model to claude-opus-5 ### 📊 Changes **1 file changed** (+6 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/groom.yml` (+6 -2) </details> ### 📄 Description Bumps the groom `model` input default `claude-opus-4-8` → `claude-opus-5`, and documents the trade-off in the input description. The finder, verifier, and builder agents *are* groom's output quality — surfacing real refactors, adjudicating CONFIRM/DOWNGRADE/REJECT, and authoring the patch — so the default should track the current flagship. Callers that want to hold a run steady can still pin an older id via the `model:` input; neither current caller (comfy-cloud-mcp-server, cloud) overrides it, so both pick this up on their next SHA bump. ## ELI5 Groom uses an AI model to read the repo, decide which cleanups are real, and write the fix. This points it at the newest, strongest model instead of the previous one, because the quality of what it finds and writes depends almost entirely on that choice. Anyone who wants the old model can still ask for it by name. Cost note: a finder pass is ~$7.50 at the previous model; expect this to shift with Opus 5 pricing. Runtime is unchanged (~12 min, well inside the 40-min timeout). `actionlint` clean. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-26 22:36:00 +00:00