Skip to content

Commit 7ad03fe

Browse files
committed
docs(forge): correct build_exec_args docstring on failure scope
The docstring claimed every dispatched command/prompt step exits 2, but prompt dispatch passes output_json=False, so the inherited implementation only fails when a model is configured or output_json=True actually appends an unsupported flag.
1 parent 010a961 commit 7ad03fe

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

‎src/specify_cli/integrations/forge/__init__.py‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,10 @@ def build_exec_args(
106106
107107
``MarkdownIntegration``'s default appends ``--model`` and
108108
``--output-format``, neither of which exists in the Forge CLI (see
109-
issue #4666) — every dispatched command/prompt step exits 2 at
110-
argument parsing. Forge only accepts ``-p/--prompt``; ``model`` is
111-
deliberately dropped rather than remapped, since Forge selects it
109+
issue #4666) — a dispatched step exits 2 at argument parsing
110+
whenever either flag ends up appended (a configured ``model``, or
111+
``output_json=True``). Forge only accepts ``-p/--prompt``; ``model``
112+
is deliberately dropped rather than remapped, since Forge selects it
112113
out of band via ``forge config set model`` and its ``--agent`` flag
113114
selects an agent ID, not a model.
114115
"""

0 commit comments

Comments
 (0)