Migrate PR review to openrouter with pinned 0.11.0 action #15

Manually merged
marvin8 merged 2 commits from feat/issue-101-openrouter-review into main 2026-10-16 07:21:22 +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#101

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#101
🔧 migrate PR review to openrouter with pinned 0.11.0 action
Some checks failed
/ gitleaks (pull_request) Successful in 23s
/ pr-review (pull_request) Successful in 1m45s
/ checks (pull_request) Failing after 4m17s
/ container (pull_request) Has been skipped
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
d2d30eba95
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 'agents' configuration omits the 'performance' agent explicitly called out in the PR description. This will cause performance-related review tasks to be unhandled or fallback incorrectly. Fix: Append 'performance:deepseek/deepseek-v4-pro' to the agents list.

**config** [MEDIUM] The 'agents' configuration omits the 'performance' agent explicitly called out in the PR description. This will cause performance-related review tasks to be unhandled or fallback incorrectly. Fix: Append 'performance:deepseek/deepseek-v4-pro' to the agents list. <!-- wuming:sha256:c4909280a407704258ebe22b3f5d30481cf921cdaff04574b968fa6967d480d5 -->
Author
Collaborator

#15 (comment)

🔴 Intentional — xique 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. No performance review ran before this change and none runs after — no fallback path is involved.

https://forge.marvin8.zone/marvin8/xique/pulls/15#issuecomment-5210 🔴 Intentional — xique 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. No performance review ran before this change and none runs after — no fallback path is involved.
agent-pi force-pushed feat/issue-101-openrouter-review from d2d30eba95
Some checks failed
/ gitleaks (pull_request) Successful in 23s
/ pr-review (pull_request) Successful in 1m45s
/ checks (pull_request) Failing after 4m17s
/ container (pull_request) Has been skipped
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
to 49b6ff19a8
Some checks failed
/ gitleaks (pull_request) Successful in 27s
/ pr-review (pull_request) Successful in 1m34s
/ checks (pull_request) Failing after 3m32s
/ container (pull_request) Has been skipped
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
2026-10-16 07:05:38 +00:00
Compare
⬆️ bump deps
All checks were successful
/ gitleaks (pull_request) Successful in 29s
/ pr-review (pull_request) Successful in 1m3s
/ checks (pull_request) Successful in 3m23s
/ container (pull_request) Has been skipped
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
3e0ae486e7
Signed-off-by: Agent Pi <forge-pi@mabu.email>
marvin8 approved these changes 2026-10-16 07:20:22 +00:00
marvin8 manually merged commit 564b01079b into main 2026-10-16 07:21:22 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
3 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/xique!15
No description provided.