fix(kiro-cli): replace literal $ARGUMENTS with prose fallback

[eldar702]

Summary

Replace the literal "$ARGUMENTS" value in KiroCliIntegration.registrar_config["args"] with a prose fallback string so that rendered Kiro CLI prompt files no longer ship a broken shell-style placeholder.

Fixes #1926

Problem

When SpecKit prompts are invoked via Kiro CLI (kiro-cli chat), the literal string $ARGUMENTS was passed to the AI model unchanged. Every parameterized SpecKit slash-command under Kiro CLI was effectively unusable — the agent either errored out or asked the user to repeat their input.

The integration code feeds registrar_config["args"] straight into process_template() (base.py:717-719), which then substitutes both {ARGS} and $ARGUMENTS tokens in command bodies with that value. Setting it to "$ARGUMENTS" made the substitution a no-op, so the literal token remained in every rendered prompt.

Solution

Kiro CLI file-based prompts do not natively support any argument-substitution syntax — $ARGUMENTS, {{userMessage}}, and $INPUT are all delivered to the model verbatim (see kirodotdev/Kiro#4141 and kiro.dev/docs/cli/chat/manage-prompts). Switching to a different shell-style token would just ship a different broken literal.

Instead, set args to a prose fallback that gets substituted into the rendered prompt and tells the model to take its argument from the user's next conversational message:

_KIRO_ARG_FALLBACK = "(the user will provide the argument in this conversation)"

This is the approach used by other integrations whose target CLI lacks native argument injection.

Test plan

• New regression test test_rendered_prompts_do_not_contain_raw_arguments — calls setup(), walks every rendered .kiro/prompts/*.md, asserts $ARGUMENTS is not present. Fails on main, passes after the fix.
• New regression test test_rendered_prompts_contain_kiro_arg_placeholder — asserts the new prose value is substituted in (proves substitution fired, not just that the placeholder was removed).
• Inherited test_registrar_config from MarkdownIntegrationTests overridden in TestKiroCliIntegration because the base assertion args == "$ARGUMENTS" no longer holds for kiro-cli.
• pytest full suite: 2788 passed, 34 skipped — zero regressions.

Notes

This change was AI-assisted. The fix was selected after a 3-agent solution-design debate in which two alternative approaches ({{userMessage}} and $INPUT placeholder swaps) were rejected after web research confirmed Kiro file-based prompts do not substitute any token; the prose-fallback was the only approach that actually solves the bug. The deliberation also produced the regression-test pair that proves both directions — $ARGUMENTS must be absent and the new value must be present — so we can't accidentally regress by changing the fallback to an empty string.

[Copilot]

Pull request overview

This PR fixes Kiro CLI prompt rendering by replacing the literal "$ARGUMENTS" registrar args value with a prose fallback, preventing Kiro’s file-based prompts from shipping an un-substituted placeholder to the model (fixes #1926).

Changes:

• Update KiroCliIntegration.registrar_config["args"] to a prose fallback string suitable for Kiro CLI.
• Add/adjust integration tests to ensure rendered .kiro/prompts/*.md files no longer contain raw $ARGUMENTS and that substitution occurs.
• Override the shared Markdown integration registrar-config assertion for kiro-cli.

Show a summary per file

File	Description
src/specify_cli/integrations/kiro_cli/__init__.py	Switches Kiro CLI args placeholder from "$ARGUMENTS" to a prose fallback constant.
tests/integrations/test_integration_kiro_cli.py	Updates/extends tests to validate Kiro prompt rendering and overrides the base registrar-config expectation for kiro-cli.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comments suppressed due to low confidence (1)

tests/integrations/test_integration_kiro_cli.py:64

• test_rendered_prompts_contain_kiro_arg_placeholder derives expected from integration.registrar_config["args"], so it will pass even if args is changed later to a different (still-broken) placeholder token as long as that token is substituted into the rendered files. Consider asserting the specific prose fallback value you intend to ship (or a predicate that enforces “prose” rather than “placeholder token”) so the test fails if someone swaps back to another literal token that Kiro won’t substitute.

        expected = integration.registrar_config["args"]
        prompts_dir = tmp_path / self.REGISTRAR_DIR
        contents = "\n".join(
            p.read_text(encoding="utf-8") for p in prompts_dir.glob("*.md")
        )
        assert expected in contents, (
            f"none of the rendered prompts contain the configured args fallback "
            f"({expected!r})"
        )

• Files reviewed: 2/2 changed files
• Comments generated: 1

[mnriem]

This PR should also update docs/reference/integrations.md to document the argument limitation in the Kiro CLI notes column. Currently it only says Alias: --integration kiro. It should note that Kiro CLI file-based prompts do not support argument substitution and that $ARGUMENTS placeholders are replaced with a prose fallback at install time.

This is consistent with how other integrations document their quirks in that table (e.g., Goose noting YAML recipe format, Pi noting MCP limitations, skills-based integrations noting their layout).