Combining command options does not work

[thiagokoster]

Bug Description

Hey,
I'm trying to create a reversible sequential migration file. By reading the --help menu I thought that executing the the following command: sqlx migrate add -rs post would create two files, 0001_post.up.sql and 0001_post.down.sql.
But it only creates one file, 0001_post.up.sql.

Using just the -r flag works but it does not create a sequential file.
I don't know if this is the expected behavior or a bug.
Thanks in advance!

Minimal Reproduction

Execute:
sqlx migrate add -rs post

Info

• SQLx version: sqlx-cli 0.7.4
• SQLx features enabled: sqlx = { version = "0.7", features = [ "postgres", "uuid", "runtime-tokio", "tls-native-tls" ] }
• Database server and version: PostgreSQL 16.3
• Operating system: Linux
• rustc --version: rustc 1.76.0

[abonander]

This would appear to be a bug, yes.

[CommanderStorm]

I tried writing a testcase for this but failed to reproduce the results.
Could you re-check your sqlx-cli version?

[thiagokoster]

Sure, here is a screenshot of my current version and problem

[CommanderStorm]

What does your directory look like? Maybe there is a combination of additions/removals that I am missing.
Could you try to also reproduce this in a fresh directory? (Thus ruling out that there is a platform/.. issue)

[thiagokoster]

Hey, sorry for the delay.
It works fine in a fresh directory. And I think I found the issue.
I have a non-reversible migration in my migrations directory. After removing it, the -rs flag works.

I thought having a combination of reversible and non-reversible migration files was possible.

[CommanderStorm]

@abonander so that I can provide the correct fix as a PR. Should the behavior be:

• create a down migration for all in the current directory
• create down only for the added migration (mixing up+down)