@andespindola/crow-skills 0.19.1 → 0.20.1

package/crow_skills/__init__.py

@@ -1,3 +1,3 @@
 """Portable, local-first installer for the crow-skills catalogue."""
 
-__version__ = "0.19.1"
+__version__ = "0.20.1"

package/crow_skills/autoupdate.py

@@ -319,6 +319,71 @@ def broken_projections(agent, home, store, rows):
     return [{"name": n, "status": "uniflow_link_modified", "fix": broken_fix(n)} for n in sorted(names)]
 
 
+
+def catalog_gap(agent, home, source, rows, projected, broken):
+    """What this agent is missing from the catalog, and whether upgrade may install it.
+
+    WHY THIS EXISTS. `upgrade` only ever touched skills already RECORDED for the agent, so a skill
+    that is new in the catalog was published and never arrived: 0.17.0 shipped crow-be-human, every
+    CLI reported "atualizadas" and `doctor` stayed healthy while the skill existed nowhere on disk.
+    A release that the command the user runs cannot deliver is not delivered.
+
+    Returns {"missing": [names], "install": [names]}. The decision is per skill and read from the
+    GROUPS, because "has everything else" is vacuous — the set of missing names is by definition
+    everything it does not have. A missing skill is installed when some group of it is otherwise
+    COMPLETE here: the user took that group wholesale, and a new member belongs to it. A skill whose
+    every group is partial was curated by hand, so it is only reported, with the command.
+    """
+    if source is None:
+        return {"missing": [], "install": []}
+    present = {r.get("name") for r in rows if r.get("name")}
+    present |= set(projected or [])
+    present |= {k.get("name") for k in (broken or []) if isinstance(k, dict) and k.get("name")}
+    missing = sorted(set(source.catalog) - present)
+    if not missing:
+        return {"missing": [], "install": []}
+    try:
+        from . import groups as groups_mod
+        available = [g for g in groups_mod.load(source) if g["name"] != groups_mod.ALL]
+    except (CrowError, OSError, ValueError, KeyError):
+        available = []
+    install = []
+    for name in missing:
+        for group in available:
+            members = [s for s in group["skills"] if s != name]
+            if name in group["skills"] and members and all(m in present for m in members):
+                install.append(name)
+                break
+    return {"missing": missing, "install": install}
+
+
+def install_new(home, entry, names, agent, dry_run, trust):
+    """Install catalog skills the agent is missing — only when it already has every other one.
+
+    The decision is read from the machine, not assumed: an agent holding the whole catalog minus the
+    new names has said "all of them", so a new skill belongs there. An agent with a SUBSET chose that
+    subset, and adding to it would be deciding for the user, so those names are only reported.
+    """
+    if not names:
+        return
+    argv = ["install", *names, "--agents", agent, "--global", "--json", "--home", str(home),
+            "--dry-run" if dry_run else "--yes"]
+    if not trust:
+        argv.append("--no-trust")
+    from .cli import parser, run as cli_run
+    try:
+        data = cli_run(parser().parse_args(argv))
+    except CrowError as exc:
+        entry["newInCatalogError"] = {"code": exc.code, "message": str(exc)}
+        return
+    done = [c["name"] for c in data.get("changes") or [] if c.get("action") != "unchanged"]
+    # NÃO entra em "updated": verificado com o binário publicado, a linha do CLI dizia as duas coisas
+    # da mesma skill — "a atualizar: crow-be-human · instalaria novas: crow-be-human". Instalar agora
+    # e atualizar o que já existia são fatos diferentes e cada um tem a sua coluna.
+    entry["installedNew"] = sorted(done)
+    if data.get("transaction"):
+        entry["newTransaction"] = data["transaction"]
+
 def reinstall_behaviour(home, entry, dry_run, trust):
     """Codex without a working crow-behaviour (its projection is broken, no native copy): install it natively,
     as setup does, so the guide and hooks keep a coordinator to point to."""
@@ -402,6 +467,9 @@ def skills_update(home, trust=True, dry_run=False):
                 from . import machine
                 entry = {"agent": agent, "target": target["path"], "updated": [], "unchanged": 0, "kept": list(broken), "error": None,
                          "via_uniflow": projected}
+                gap = catalog_gap(agent, home, current, rows, projected, broken)
+                if gap["missing"]:
+                    entry["newInCatalog"] = gap["missing"]
                 if dry_run:
                     entry["dry_run"] = True
                 try:
@@ -413,6 +481,8 @@ def skills_update(home, trust=True, dry_run=False):
                     entry["error"] = {"code": exc.code, "message": str(exc)}
                 except (OSError, ValueError, KeyError, TypeError):
                     entry["error"] = {"code": "io_error", "message": "Falha de leitura/gravação; nada concluído para este agente."}
+                if gap["install"] and not entry.get("error"):
+                    install_new(home, entry, gap["install"], agent, dry_run, trust)
                 if agent == "codex" and not entry.get("error"):
                     # Ramo mais provável de quem vinha da 0.14.0: no diretório do Codex só há links do
                     # Uniflow, e TODAS as cópias do codex estão na pasta compartilhada. Sem migrar aqui,
@@ -431,6 +501,9 @@ def skills_update(home, trust=True, dry_run=False):
                 names.append(r["name"])
         entry = {"agent": agent, "target": target["path"], "updated": [], "unchanged": 0, "kept": kept + list(broken),
                  "error": None, "via_uniflow": projected}
+        gap = catalog_gap(agent, home, current, rows, projected, broken)
+        if gap["missing"]:
+            entry["newInCatalog"] = gap["missing"]
         if dry_run:
             entry["dry_run"] = True
         if agent == "codex":
@@ -462,6 +535,8 @@ def skills_update(home, trust=True, dry_run=False):
                 entry["error"] = {"code": exc.code, "message": str(exc)}
             except (OSError, ValueError, KeyError, TypeError):
                 entry["error"] = {"code": "io_error", "message": "Falha de leitura/gravação; nada concluído para este agente."}
+        if gap["install"] and not entry.get("error"):
+            install_new(home, entry, gap["install"], agent, dry_run, trust)
         if agent == "codex" and not entry.get("error"):
             # Mesma migração do setup, e pelo mesmo motivo: sem isto, uma máquina que só roda
             # autoupdate/upgrade mantém a pasta compartilhada antiga para sempre. Depois da

package/crow_skills/machine.py

@@ -778,6 +778,17 @@ def upgrade_summary(skills, dry_run):
                                           k["name"] + " --agents " + agent + " --global --yes")
             else:
                 item["needs_user"].append(k["name"] + ": " + k["status"])
+        # Skill nova no catálogo: instalada quando o agente já tinha todas as outras, e NOMEADA com o
+        # comando quando ele tem um subconjunto escolhido. Antes disto o upgrade dizia "atualizadas" e
+        # a skill não existia em disco em lugar nenhum.
+        item["installed_new"] = s.get("installedNew") or []
+        pending = [n for n in (s.get("newInCatalog") or []) if n not in item["installed_new"]]
+        if pending:
+            item["needs_user"].append(str(len(pending)) + " skill(s) nova(s) no catálogo não instalada(s) aqui (" +
+                                      ", ".join(pending[:3]) + "); este agente tem um subconjunto escolhido. Para instalar: "
+                                      "crow-skills install " + " ".join(pending[:3]) + " --agents " + agent + " --global --yes")
+        if s.get("newInCatalogError"):
+            item["needs_user"].insert(0, "skill nova do catálogo não instalada: " + s["newInCatalogError"]["message"])
         if s.get("error"):
             item["needs_user"].insert(0, s["error"]["message"])
         item["state"] = "fail" if s.get("error") else ("warn" if item["needs_user"] else ("plan" if dry_run and (item["changed"] or item["removed"]) else "ok"))
@@ -904,9 +915,14 @@ def render_upgrade(data, stream=None):
         changed = ", ".join(item["changed"]) if len(item["changed"]) <= 3 else str(len(item["changed"])) + " skills"
         removed = ((" · " if changed else "") + str(len(item["removed"])) + (" a remover" if dry else " removidas") +
                    " (duplicavam o Uniflow)") if item.get("removed") else ""
+        fresh = item.get("installed_new") or []
+        # Dito na própria linha do CLI: uma skill nova do catálogo não é "atualizada", é instalada
+        # agora — e foi exatamente isso que faltou quando a 0.17.0 publicou a crow-be-human.
+        new_text = ((" · " if changed or removed else "") + (("instalaria " if dry else "instaladas ") +
+                    ("novas: " + ", ".join(fresh[:3]) if len(fresh) <= 3 else str(len(fresh)) + " novas"))) if fresh else ""
         rows.append([ui.mark(STATE_MARK[item["state"]], stream), item["label"],
-                     ((("a atualizar: " if dry else "atualizadas: ") + changed) if changed else ("" if removed else "sem mudanças"))
-                     + removed,
+                     ((("a atualizar: " if dry else "atualizadas: ") + changed) if changed else ("" if removed or new_text else "sem mudanças"))
+                     + removed + new_text,
                      str(item["unchanged"]), str(item.get("via_uniflow", 0)), str(len(item["kept"]))])
     if rows:
         lines.append("")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andespindola/crow-skills",
-  "version": "0.19.1",
+  "version": "0.20.1",
   "description": "Versioned, complementary agent skills with a protected multi-agent installer",
   "license": "UNLICENSED",
   "repository": {

package/release.json

@@ -18,7 +18,7 @@
     "crow_skills/__init__.py": {
       "executable": false,
       "kind": "file",
-      "sha256": "0fac3e997aef5911a1a83e61ec8abc3b92abc0e02cdff187a430b3277e40b80f"
+      "sha256": "fe7f42f4b197f8dbf5e96979a3d2d2ef6ff79c7d2b099d45415d8b2a9e96364c"
     },
     "crow_skills/__main__.py": {
       "executable": false,
@@ -33,7 +33,7 @@
     "crow_skills/autoupdate.py": {
       "executable": false,
       "kind": "file",
-      "sha256": "8de4a6f3f7ce773dafb4c4f9b217d399becbd98b0b0638a80e27e13450e3a0f6"
+      "sha256": "c192495c895a95b957cad04c2d7fc058aa86cd67e76077200fa202d7c96fb039"
     },
     "crow_skills/broadcast.py": {
       "executable": false,
@@ -73,7 +73,7 @@
     "crow_skills/machine.py": {
       "executable": false,
       "kind": "file",
-      "sha256": "8b7187afae9ad85369fdc8a918e19b32bab9d95776246cda6670a963dff19df3"
+      "sha256": "83d3f6fa0e296b4de87d90ff90ff50784241cf346bc1947ab78dd415409ebc0d"
     },
     "crow_skills/npm_entry.py": {
       "executable": false,
@@ -128,7 +128,7 @@
     "package.json": {
       "executable": false,
       "kind": "file",
-      "sha256": "25d9d44e380c13b64264f33b18d0b722a1b465c693fd92d819581b78df7f6fb8"
+      "sha256": "1d95441781de5db5184c2962d48feaecb9a8c31b15f1b9afab21c7bbb08e34ca"
     },
     "skills/crow-accessibility/SKILL.md": {
       "executable": false,
@@ -721,7 +721,7 @@
       "sha256": "e37132ef8b060548191fbf14a5a18a9dd3acd84ea9a35e802174455b9e374d3d"
     }
   },
-  "revision": "2a8fb8a28c5b1073fd5faea2da0e2b4fe631fab7",
+  "revision": "d8b071036d6d9bda4f894d0764dfca45556ce9ff",
   "schema_version": 1,
-  "version": "0.19.1"
+  "version": "0.20.1"
 }