Migrate PR review to openrouter with pinned 0.11.0 action #84

Manually merged
marvin8 merged 1 commit from feat/issue-99-openrouter-review into main 2026-10-15 23:15:43 +00:00
Collaborator

Migrates the wuming PR review to the openrouter backend with the fleet-standard two-tier model mapping (code/tests/performance on deepseek-v4-pro, security on kimi-k2-thinking, config/docs/shell on qwen3.7-flash) and pins the action to 0.11.0.

Tracking: marvin8/wuming#99

Migrates the wuming PR review to the openrouter backend with the fleet-standard two-tier model mapping (code/tests/performance on deepseek-v4-pro, security on kimi-k2-thinking, config/docs/shell on qwen3.7-flash) and pins the action to `0.11.0`. Tracking: marvin8/wuming#99
🔧 migrate PR review to openrouter with pinned 0.11.0 action
All checks were successful
/ gitleaks (pull_request) Successful in 19s
/ pr-review (pull_request) Successful in 1m9s
/ checks (pull_request) Successful in 14m32s
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
/ container (pull_request) Successful in 1m40s
1e0ad3250f
forgejo-actions left a comment

WuMing

Found 1 issue(s). See inline comments below.

## WuMing Found **1** issue(s). See inline comments below.
@ -24,1 +23,3 @@
agents: code:deepseek-flash,config:deepseek-flash,docs:deepseek-flash,shell:deepseek-flash,security:deepseek-flash,tests:deepseek-flash
openrouter_key: ${{ secrets.wuming_openrouter_key }}
backend: openrouter
agents: code:deepseek/deepseek-v4-pro,config:qwen/qwen3.7-flash,docs:qwen/qwen3.7-flash,shell:qwen/qwen3.7-flash,security:moonshotai/kimi-k2-thinking,tests:deepseek/deepseek-v4-pro

config [MEDIUM]

The PR description explicitly states that performance reviews should map to deepseek-v4-pro, but the agents configuration on line 25 omits the performance key. Add performance:deepseek/deepseek-v4-pro to the agents string to align with the documented intent.

**config** [MEDIUM] The PR description explicitly states that `performance` reviews should map to `deepseek-v4-pro`, but the `agents` configuration on line 25 omits the `performance` key. Add `performance:deepseek/deepseek-v4-pro` to the agents string to align with the documented intent. <!-- wuming:sha256:c4909280a407704258ebe22b3f5d30481cf921cdaff04574b968fa6967d480d5 -->
Author
Collaborator

#84 (comment)

🔴 Intentional — yunjin did not run a performance agent before this migration, and each repo's existing agent set is preserved. The fleet mapping assigns a model to whichever agents a repo runs; it does not add agents. (As a side effect of this migration, yunjin's stale deepseek-flash model assignments were also modernised to the verified current slugs.)

https://forge.marvin8.zone/marvin8/yunjin/pulls/84#issuecomment-5205 🔴 Intentional — yunjin did not run a `performance` agent before this migration, and each repo's existing agent set is preserved. The fleet mapping assigns a model to whichever agents a repo runs; it does not add agents. (As a side effect of this migration, yunjin's stale `deepseek-flash` model assignments were also modernised to the verified current slugs.)
agent-pi force-pushed feat/issue-99-openrouter-review from 1e0ad3250f
All checks were successful
/ gitleaks (pull_request) Successful in 19s
/ pr-review (pull_request) Successful in 1m9s
/ checks (pull_request) Successful in 14m32s
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
/ container (pull_request) Successful in 1m40s
to 71b166618a
All checks were successful
/ gitleaks (pull_request) Successful in 32s
/ pr-review (pull_request) Successful in 42s
/ checks (pull_request) Successful in 21m12s
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
/ container (pull_request) Successful in 3m0s
2026-10-15 23:11:06 +00:00
Compare
marvin8 manually merged commit 77e3aa757f into main 2026-10-15 23:15:43 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
marvin8/yunjin!84
No description provided.