Skip to content

fix(cli): prevent Typer from collapsing 'create' command#6

Merged
trisdoan merged 1 commit intotrobz:mainfrom
dzungtran89:fix-missing-command
Feb 2, 2026
Merged

fix(cli): prevent Typer from collapsing 'create' command#6
trisdoan merged 1 commit intotrobz:mainfrom
dzungtran89:fix-missing-command

Conversation

@dzungtran89
Copy link
Copy Markdown
Contributor

When a Typer app has only one command, it collapses it into the main app entry point.
This causes odoo-venv create 18.0 to fail, as 'create' is interpreted as the
version argument and '18.0' as an unexpected extra argument.

This commit adds a callback to the Typer app to force the subcommand structure,
ensuring odoo-venv create ... works as expected.

When a Typer app has only one command, it collapses it into the main app entry point.
This causes `odoo-venv create 18.0` to fail, as 'create' is interpreted as the
version argument and '18.0' as an unexpected extra argument.

This commit adds a callback to the Typer app to force the subcommand structure,
ensuring `odoo-venv create ...` works as expected.
@nilshamerlinck
Copy link
Copy Markdown
Contributor

  • nice catch!
  • looking at my notes, I realize indeed that I took the habit not to add the create command when running odoo-venv
  • given that more commands will come and that we teached agents to use create already in our skill, we should honor create
  • ok for you @trisdoan ?

@trisdoan
Copy link
Copy Markdown
Collaborator

trisdoan commented Feb 2, 2026

Thanks anh Dung

@trisdoan trisdoan merged commit c6dd41f into trobz:main Feb 2, 2026
1 check passed
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