@andespindola/crow-skills 0.19.0 → 0.19.1

package/crow_skills/__init__.py

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

package/crow_skills/broadcast.py

@@ -352,6 +352,14 @@ def edit_paths(host, tool, tool_input, cwd):
 GIT_WORD = re.compile(r"^(?:\S*/)?git$")
 DIR_WORD = re.compile(r"(?:^|[\s;&|()])(?:cd|[^\s;&|()]*/?git)(?:\s|$)")
 TREE_SUBCOMMANDS = ("checkout", "switch", "rebase", "stash", "merge", "pull", "cherry-pick", "bisect", "restore", "clean")
+# A git call is not by itself a sign of working in a clone: `git log`, `git status` and `git diff`
+# are reading. Measured right after shipping occupancy(): a peer session that only INSPECTED two
+# clones was still published as occupying them, because any git verb marked the clone as driven.
+# Everything outside this list keeps marking occupancy, so an unknown verb errs toward warning.
+READ_SUBCOMMANDS = ("log", "status", "diff", "show", "rev-parse", "rev-list", "ls-tree", "ls-files",
+                    "ls-remote", "cat-file", "describe", "blame", "shortlog", "for-each-ref",
+                    "name-rev", "merge-base", "symbolic-ref", "whatchanged", "grep", "count-objects",
+                    "check-ignore", "verify-commit", "verify-tag", "var", "help", "version")
 
 
 def tree_command(host, tool, tool_input, cwd=None):
@@ -423,7 +431,7 @@ def _note(visited, directory, source="cd"):
     if visited is None or not directory or not directory.startswith("/"):
         return
     if directory in visited:
-        if source == "git":
+        if source == "git":  # working outranks reading and the incidental cd
             visited[directory] = "git"
         return
     if len(visited) < MAX_PATTERNS:
@@ -488,7 +496,8 @@ def _segment(words, here, targets, depth=0, visited=None):
     if i >= len(words):
         return here
     sub, rest = words[i], words[i + 1:]
-    _note(visited, directory, "git")  # any git call names the clone the session is working in
+    # "git" means the session WORKS in that clone; a read-only verb means it only looked.
+    _note(visited, directory, "read" if sub in READ_SUBCOMMANDS else "git")
     if sub in TREE_SUBCOMMANDS or (sub == "reset" and "--hard" in rest):
         if directory and directory not in targets:
             targets.append(directory)
@@ -703,14 +712,21 @@ def fold(snap, event, touched=(), now=None, ppid=None, cli_pid=None, tty_runner=
         repos = [r for r in repos if r["root"] != found["root"]]
         entry = {"id": found["id"], "root": found["root"], "ts": now_iso(now),
                  "hits": int((previous or {}).get("hits") or 0) + 1}
-        if (previous or {}).get("driven") or (isinstance(dirs, dict) and dirs.get(directory) == "git"):
+        source = dirs.get(directory) if isinstance(dirs, dict) else None
+        # TWO different questions about the same clone. `driven`: a git call named it, which is a
+        # stronger signal than an incidental `cd` and wins the eviction contest — a `git log` counts.
+        # `working`: the verb CHANGES something there, which is what occupancy() may warn about.
+        if (previous or {}).get("driven") or source in ("git", "read"):
             entry["driven"] = True
+        if (previous or {}).get("working") or source == "git":
+            entry["working"] = True
         repos.append(entry)
     if repos or presence.get("repos") is not None:
         # Eviction is not FIFO: a clone a git command actually drove, and one visited many times,
         # outrank the incidental `cd` that would otherwise push it out of the eight slots.
-        keep = sorted(repos, key=lambda r: (bool(r.get("driven")), int(r.get("hits") or 0),
-                                            parse_time(r.get("ts")) or 0), reverse=True)[:MAX_REPOS]
+        keep = sorted(repos, key=lambda r: (bool(r.get("working")), bool(r.get("driven")),
+                                            int(r.get("hits") or 0), parse_time(r.get("ts")) or 0),
+                      reverse=True)[:MAX_REPOS]
         presence["repos"] = [r for r in repos if r in keep]
     snap["presence"] = presence
     return snap
@@ -929,8 +945,8 @@ def occupancy(snap, now=None, window=None):
             ids.add(repo["id"])
         if repo.get("root"):
             roots.add(repo["root"])
-    for entry in presence.get("repos") or []:  # only the clones a git command drove
-        if not isinstance(entry, dict) or not entry.get("driven"):
+    for entry in presence.get("repos") or []:  # only the clones a WRITING git command drove
+        if not isinstance(entry, dict) or not entry.get("working"):
             continue
         if now - (parse_time(entry.get("ts")) or 0) > window:
             continue

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andespindola/crow-skills",
-  "version": "0.19.0",
+  "version": "0.19.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": "7720abacfffca6315ecfee7b1dfced20c3c6e7387f00113ce2c437ed63f8e0bc"
+      "sha256": "0fac3e997aef5911a1a83e61ec8abc3b92abc0e02cdff187a430b3277e40b80f"
     },
     "crow_skills/__main__.py": {
       "executable": false,
@@ -38,7 +38,7 @@
     "crow_skills/broadcast.py": {
       "executable": false,
       "kind": "file",
-      "sha256": "998cdd9bc67d52e63dfcde5528c5caaa04c509ec6e1e7d6afb298c8bcf4a442f"
+      "sha256": "135fb80925dea8e6c0790ee830c6b7e440cc5c0f116a4c9dcc4cf36bab1c22e3"
     },
     "crow_skills/cli.py": {
       "executable": false,
@@ -128,7 +128,7 @@
     "package.json": {
       "executable": false,
       "kind": "file",
-      "sha256": "b70074096ec8ed3994a6cff3eeb2b72885ec366d01269f9006d0e492fccf649a"
+      "sha256": "25d9d44e380c13b64264f33b18d0b722a1b465c693fd92d819581b78df7f6fb8"
     },
     "skills/crow-accessibility/SKILL.md": {
       "executable": false,
@@ -721,7 +721,7 @@
       "sha256": "e37132ef8b060548191fbf14a5a18a9dd3acd84ea9a35e802174455b9e374d3d"
     }
   },
-  "revision": "9c7d5fe418c5c993eef7df75790263b3c77d400d",
+  "revision": "2a8fb8a28c5b1073fd5faea2da0e2b4fe631fab7",
   "schema_version": 1,
-  "version": "0.19.0"
+  "version": "0.19.1"
 }