Skip to content

Commit 2f9ffd5

Browse files
tidy kiro-cli multi-install-safe declaration + add docs row and test
kiro-cli is declared multi_install_safe (it uses a fully isolated .kiro/ root, .kiro/prompts command dir, a stable '.' separator, and a dedicated manifest β€” #3471). main ended up with the flag assigned twice in the class body; this drops the bare duplicate and keeps a single declaration with the comment explaining why it's safe. also adds the missing kiro-cli row to the multi-install-safe table in docs/reference/integrations.md, and a test asserting the flag is set (the registry contract tests already enforce the actual path isolation against every other safe integration).
1 parent 7bdf6c5 commit 2f9ffd5

3 files changed

Lines changed: 11 additions & 1 deletion

File tree

β€Ždocs/reference/integrations.mdβ€Ž

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -271,6 +271,7 @@ The currently declared multi-install safe integrations are:
271271
| `grok` | `.grok/skills` |
272272
| `junie` | `.junie/commands`, `.junie/AGENTS.md` |
273273
| `kilocode` | `.kilocode/workflows`, `.kilocode/rules/specify-rules.md` |
274+
| `kiro-cli` | `.kiro/prompts` |
274275
| `qodercli` | `.qoder/commands`, `QODER.md` |
275276
| `qwen` | `.qwen/commands`, `QWEN.md` |
276277
| `shai` | `.shai/commands`, `SHAI.md` |

β€Žsrc/specify_cli/integrations/kiro_cli/__init__.pyβ€Ž

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313

1414
class KiroCliIntegration(MarkdownIntegration):
1515
key = "kiro-cli"
16-
multi_install_safe = True
1716
config = {
1817
"name": "Kiro CLI",
1918
"folder": ".kiro/",

β€Žtests/integrations/test_integration_kiro_cli.pyβ€Ž

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,16 @@ def test_registrar_config(self):
5656
assert i.registrar_config["format"] == "markdown"
5757
assert i.registrar_config["extension"] == ".md"
5858

59+
def test_declared_multi_install_safe(self):
60+
"""kiro-cli uses a fully isolated .kiro/ root, a stable "." separator,
61+
and a dedicated manifest, so it must be declared multi-install safe β€”
62+
otherwise co-installing it (e.g. alongside claude) leaves
63+
`integration status` permanently in ERROR with no way to resolve it
64+
(#3471). The registry contract tests enforce the actual path isolation
65+
against every other safe integration."""
66+
i = get_integration(self.KEY)
67+
assert i.multi_install_safe is True
68+
5969
def test_registrar_config_args_is_exact_prose_fallback(self):
6070
"""Layer 1 β€” pin the exact fallback so wording drift requires a
6171
deliberate paired commit (production constant + test update)."""

0 commit comments

Comments
Β (0)