Update OpenRouter model handling and enhance documentation#1249
Update OpenRouter model handling and enhance documentation#1249nblog wants to merge 1 commit intogithub:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the nano-banana-pro-openrouter skill to make the OpenRouter model configurable at runtime and refreshes the skill documentation to clarify default behavior.
Changes:
- Add a
DEFAULT_MODELconstant and a--modelCLI option to override the OpenRouter model used for image generation. - Update SKILL.md to clarify the default model and document model selection.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| skills/nano-banana-pro-openrouter/SKILL.md | Updates overview text and adds a “Model selection” section; adjusts frontmatter description wording. |
| skills/nano-banana-pro-openrouter/scripts/generate_image.py | Introduces DEFAULT_MODEL, adds --model argument, and wires it into the OpenRouter request. |
aaronpowell
left a comment
There was a problem hiding this comment.
It looks like you've incorrectly branched from the main branch not staged, and as a result all the materialised plugins are included in this PR.
You can attempt to fix this with a rebase:
git fetch origin staged
git rebase --onto origin/staged origin/main <branch name>
git push --force-with-lease
If that does not resolve it, you can run npm run plugin:clean which will delete the materialised plugins and you can commit that change.
d6cb762 to
0c3c5bb
Compare
|
@aaronpowell Thanks for the review and guidance. The diff should now contain only these two files:
Could you please take another look when you have a moment? |
Enhance the handling of the OpenRouter model and improve related documentation for better clarity and usability.