From 58a30d913fff1cd824391b0142068a0ba8186e1a Mon Sep 17 00:00:00 2001
From: Anne He <anne@comfy.org>
Date: Tue, 15 Sep 2026 12:40:27 -0700
Subject: [PATCH 1/2] fix(knowledge): check an oss pick's weights before
 recommending it (BE-8974)

available_locally only covers templates and nodes, so on a local install
the skill could steer the agent to a rank 1 oss pick whose weights are
not on disk. Rule 2 now tells it to run `comfy --json templates check`
on the pick's template when routing is local. On missing-models it
still names the pick, lists models.missing[] with each directory, and
names the first oss pick that comes back runnable.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 comfy_cli/skills/comfy/SKILL.md | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/comfy_cli/skills/comfy/SKILL.md b/comfy_cli/skills/comfy/SKILL.md
index 4517faa3..e1ef0840 100644
--- a/comfy_cli/skills/comfy/SKILL.md
+++ b/comfy_cli/skills/comfy/SKILL.md
@@ -80,6 +80,13 @@ rather than an answer to anything. Five rules:
    it resolves to. A row flagged this way also pulls in `picks[]` for the
    capabilities that rank it, so the highest-ranked entry *without* the flag is
    the runnable alternative. Say what is missing, then name that alternative.
+   The flag never looks at model files. When routing is local, run
+   `comfy --json templates check <template>` on an `oss` pick's `template`
+   before recommending it. On `missing-models` the pick is still the answer:
+   name it, say it is not installed, and list `models.missing[]` with each
+   file's `directory`. Then check the next `oss` pick down and name the first
+   `runnable` one as what works today. Ask before downloading anything. A pick
+   with no `template` has nothing to check.
 3. **Verify before denying.** A missing `knowledge` key or a `nudge` means
    nothing is curated for that query, not that it is unsupported. A `nudge` on
    a block that still carries rows means your search term matched nothing

From 768d83de80d5ce9108055d50a1dc6a64b45e1649 Mon Sep 17 00:00:00 2001
From: Anne He <anne@comfy.org>
Date: Tue, 15 Sep 2026 13:56:52 -0700
Subject: [PATCH 2/2] fix(knowledge): walk past any non-runnable pick, not just
 missing-models

`templates check` exits with `template_not_found` before it computes a
verdict when a pick's template is not in the gallery, and `unknown` is a
verdict too. The rule only said what to do after `missing-models`, so the
walk down to the first runnable oss pick was undefined for those. Any
result that is not `runnable` now moves on, and the agent says so when no
oss pick is runnable.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 comfy_cli/skills/comfy/SKILL.md | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/comfy_cli/skills/comfy/SKILL.md b/comfy_cli/skills/comfy/SKILL.md
index e1ef0840..a1d5b951 100644
--- a/comfy_cli/skills/comfy/SKILL.md
+++ b/comfy_cli/skills/comfy/SKILL.md
@@ -84,9 +84,11 @@ rather than an answer to anything. Five rules:
    `comfy --json templates check <template>` on an `oss` pick's `template`
    before recommending it. On `missing-models` the pick is still the answer:
    name it, say it is not installed, and list `models.missing[]` with each
-   file's `directory`. Then check the next `oss` pick down and name the first
-   `runnable` one as what works today. Ask before downloading anything. A pick
-   with no `template` has nothing to check.
+   file's `directory`. Any other result that is not `runnable`, including a
+   `template_not_found` error, means that pick is not confirmed to run here.
+   Either way, check the next `oss` pick down and name the first `runnable` one
+   as what works today, or say that none is. Ask before downloading anything. A
+   pick with no `template` has nothing to check.
 3. **Verify before denying.** A missing `knowledge` key or a `nudge` means
    nothing is curated for that query, not that it is unsupported. A `nudge` on
    a block that still carries rows means your search term matched nothing