Ignoring convention that `--` separate non-flag arguments in bash (apart from last `--` for fire flags)

Offen

#580 erstellt am 22.1.2025 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 42/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: cli

Rechercherichtung

Start by reproducing the minimal test.py example and then inspect fire/core.py around line 376, where argument parsing is identified. Trace how multiple -- arguments are handled. Done means the earlier -- causes remaining arguments, including --flag 3, to be treated positionally while the final -- remains handled as shown in the expected output.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Beyond the fact that fire uses the last -- to separate flag argument from command arguments, it seems that previous -- are not parsed correctly. At least according to the common convention that a double dash ( -- ) is used in most Bash built-in commands and many other commands to signify the end of command options, after which only positional ("non-option") arguments are accepted (source).

I would like to use this to pass some other commands to a fire command, without the flags targeted to the extra command being parsed by fire. Here is a minimal example test.py:

import fire


def foo(*args: str, flag: int = 2) -> None:
    print(f"{args=} {flag=}")


fire.Fire(foo)

Current behavior:

$ python test.py -- other args 1 2 --flag 3 --
args=('args', 1, 2) flag=3
ERROR: Could not consume arg: --
Usage: test.py -- other args 1 2 -

For detailed information on this command, run:
  test.py -- other args 1 2 - --help

Expected behavior:

python test.py -- other args 1 2 --flag 3 --
args=('args', 1, 2, "--flag", 3) flag=2

Would you be fine with updating the parsing rules in https://github.com/google/python-fire/blob/6cf45c663075c96b20dd0dfa733c2374545a4ad6/fire/core.py#L376 to consider all remaining args as positional arguments if it encounter a -- ? I could work on a PR if there is no major blocker.

Vorherrschende Sprache: Python
Sterne: 28.2k
Forks: 1.5k
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus google/python-fire

Test suite fails on Windows: unescaped path regex in testArgPassing, NamedTemporaryFile reopen in testFileNameFire Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100

google/python-fire#693 · Erstellt am 16.8.2026
Release 0.7.2? Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100

google/python-fire#698 · Erstellt am 30.8.2026
Uncaught exception when a component has a property whose getter raises (intended?) Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 58/100

google/python-fire#672 · 5 Kommentare · Erstellt am 06.6.2026
python-fire: (1) PAGER+shell=True RCE in console_io.More (2) bash $(...) runs when sourcing --completion from maliciously named .py file. PoC verified. Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100

google/python-fire#665 · 2 Kommentare · Erstellt am 05.5.2026
**kwargs unexpectedly parses flags for later chained method Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100

google/python-fire#659 · 1 Kommentar · Erstellt am 03.4.2026

Alle Issues in google/python-fire