Skip to content

Update OpenRouter model handling and enhance documentation#1249

Open
nblog wants to merge 1 commit intogithub:mainfrom
nblog:feat/model-select
Open

Update OpenRouter model handling and enhance documentation#1249
nblog wants to merge 1 commit intogithub:mainfrom
nblog:feat/model-select

Conversation

@nblog
Copy link
Copy Markdown
Contributor

@nblog nblog commented Apr 1, 2026

Enhance the handling of the OpenRouter model and improve related documentation for better clarity and usability.

@nblog nblog requested a review from aaronpowell as a code owner April 1, 2026 08:31
Copilot AI review requested due to automatic review settings April 1, 2026 08:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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_MODEL constant and a --model CLI 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.

Copy link
Copy Markdown
Contributor

@aaronpowell aaronpowell left a comment

Choose a reason for hiding this comment

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

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.

@github-actions github-actions bot force-pushed the main branch 5 times, most recently from d6cb762 to 0c3c5bb Compare April 1, 2026 23:04
@nblog nblog force-pushed the feat/model-select branch from 8302bef to d1ee647 Compare April 2, 2026 01:45
@nblog
Copy link
Copy Markdown
Contributor Author

nblog commented Apr 2, 2026

@aaronpowell Thanks for the review and guidance.
I have rebuilt this PR branch from upstream/staged, restored only the intended changes under skills/nano-banana-pro-openrouter, and force-pushed the cleaned branch.

The diff should now contain only these two files:

  • skills/nano-banana-pro-openrouter/SKILL.md
  • skills/nano-banana-pro-openrouter/scripts/generate_image.py

Could you please take another look when you have a moment?

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.

3 participants