skilldrop-cli 0.16.43 → 0.16.45

package/packs/solution-architect/skills/event-contract-draft/scripts/check_asyncapi.py

@@ -113,9 +113,11 @@ class Report:
 
 def load(path):
     try:
-        text = sys.stdin.read() if path == "-" else Path(path).read_text(encoding="utf-8")
+        text = sys.stdin.read() if path == "-" else Path(path).read_text(encoding="utf-8-sig")
     except OSError as exc:
         raise SystemExit(f"error: cannot read {path}: {exc.strerror}")
+    except UnicodeDecodeError:
+        raise SystemExit(f"error: {path} is not UTF-8 text; save it as UTF-8")
     stripped = text.lstrip()
     if not stripped:
         raise SystemExit(f"error: {path} is empty")
@@ -376,7 +378,8 @@ def collect_messages(doc):
         if id(m) not in seen:
             seen[id(m)] = key
             out.append((m.get("name") or key, f"message {key}", m))
-    for cname, channel in (doc.get("channels") or {}).items():
+    channels = doc.get("channels") or {}
+    for cname, channel in (channels.items() if isinstance(channels, dict) else ()):
         ch = deref(doc, channel)
         for key, msg in (ch.get("messages") or {}).items():
             m = deref(doc, msg)
@@ -429,6 +432,9 @@ def check(doc):
     if not version.startswith("3."):
         rep.fail("asyncapi", f"version is '{version or 'missing'}'; this checks AsyncAPI 3.x")
     info = doc.get("info") or {}
+    if not isinstance(info, dict):
+        rep.fail("info", "must be an object with title and version")
+        info = {}
     for field in ("title", "version"):
         if not info.get(field):
             rep.fail("info", f"missing info.{field}")
@@ -437,7 +443,10 @@ def check(doc):
     check_refs(doc, doc, rep)
 
     channels = doc.get("channels") or {}
-    if not channels:
+    if not isinstance(channels, dict):
+        rep.fail("channels", "must be an object keyed by channel name, not a list")
+        channels = {}
+    elif not channels:
         rep.fail("channels", "no channels")
     for name, channel in channels.items():
         check_channel(doc, name, deref(doc, channel), rep)
@@ -477,7 +486,7 @@ def check(doc):
     return rep, counts
 
 
-def main(argv=None):
+def _main(argv=None):
     parser = argparse.ArgumentParser(description=__doc__.split("\n\n")[0])
     parser.add_argument("path", help="AsyncAPI 3.0 document as JSON, or - for stdin")
     parser.add_argument("-o", "--output", help="write the report to this path as well as stdout")
@@ -498,5 +507,15 @@ def main(argv=None):
     return 1 if rep.fails else 0
 
 
+
+def main(argv=None):
+    """Run, and turn an input that parses but has the wrong shape into one clear error."""
+    try:
+        return _main(argv)
+    except (KeyError, TypeError, AttributeError, IndexError, ValueError) as e:
+        print("error: couldn't read the input as an AsyncAPI document (%s: %s). If it is one, this is a bug: "
+              "please report it with the file." % (type(e).__name__, e), file=sys.stderr)
+        return 2
+
 if __name__ == "__main__":
     sys.exit(main())

package/packs/solution-architect/skills/figma-diagrams/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "figma-diagrams",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Work with Figma and FigJam files via the Figma REST API — read existing file structure, list frames/pages, post comments, and produce FigJam-importable diagram specs. Use when the user mentions a Figma URL, asks to inspect/audit a Figma file, wants to comment on a design programmatically, or wants their architecture diagram in Figma.",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/solution-architect/skills/figma-diagrams/scripts/frame_to_mermaid.py

@@ -82,7 +82,7 @@ def main(argv: list[str]) -> int:
     node_id = argv[1]
 
     data = get(f"/v1/files/{file_key}/nodes", params={"ids": node_id})
-    node = data.get("nodes", {}).get(node_id, {}).get("document")
+    node = (data.get("nodes", {}).get(node_id) or {}).get("document")
     if not node:
         sys.stderr.write(f"error: node {node_id} not found in {file_key}\n")
         return 1

package/packs/solution-architect/skills/figma-diagrams/scripts/post_comment.py

@@ -28,7 +28,7 @@ def main(argv: list[str]) -> int:
 
     if args.node_id:
         nodes = get(f"/v1/files/{file_key}/nodes", params={"ids": args.node_id})
-        node = nodes.get("nodes", {}).get(args.node_id, {}).get("document", {})
+        node = (nodes.get("nodes", {}).get(args.node_id) or {}).get("document") or {}
         box = node.get("absoluteBoundingBox")
         if not box:
             sys.stderr.write(

package/packs/solution-architect/skills/interactive-diagram/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "interactive-diagram",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "Turn a codebase, a system description, a process or a database schema into one self-contained interactive HTML diagram (architecture, workflow, data flow, lifecycle, ERD, sequence, class, tree, timeline or waterfall) where every box and arrow cites the file and line or the words it came from, anything inferred is drawn dashed as an assumption, and each cited line is checked against the repository before the page is written. Laid out automatically, opens offline with no CDN, and exports SVG, PNG and Mermaid. Use when the user asks to \"visualize\", \"map\" or \"make an interactive diagram of\" a repo, a request path, a pipeline, an approval process, a set of tables, an incident's events or a slow trace, or wants a diagram they can click through and share as one file.",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/solution-architect/skills/interactive-diagram/scripts/build_diagram.py

@@ -1287,7 +1287,7 @@ def timeline_findings(spec, evs, kind):
     out = []
     order = sorted(evs, key=lambda e: e["_t"][0])
     if len(order) >= 2:
-        first = order[0]["when"]
+        first = order[0]["when"].split(" to ")[0]
         if kind.startswith("abs") and ":" in first and not re.match(r"^~?\d{2} [A-Z][a-z]{2} ", first):
             first = fmt_when(order[0]["_t"][0], kind, 86400) + " " + first  # one day: give its date once
         out.append("From %s to %s: %s." % (first, order[-1]["when"].split(" to ")[0], fmt_span(order[-1]["_t"][0] - order[0]["_t"][0], kind)))
@@ -2495,7 +2495,10 @@ def build_html(spec, svg, mermaid, issues, theme):
 
 def load(path):
     with open(path, encoding="utf-8") as fh:
-        return json.load(fh)
+        spec = json.load(fh)
+    if not isinstance(spec, dict):
+        raise ValueError("the spec must be a JSON object at the top level, not a %s" % type(spec).__name__)
+    return spec
 
 
 

package/packs/sre-oncall/skills/cloud-cost-review/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "cloud-cost-review",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Review actual cloud spend from a cost export (AWS CUR or Data Exports, GCP billing export, Azure cost details) and produce a ranked savings report — idle and unattached resources, oversized instances backed by utilisation evidence, storage tiering, data transfer hot spots, commitment coverage and untagged spend — each finding with a monthly saving computed from the bill, a confidence level and an ease rating. Use when the user shares a cost CSV and says \"where are we wasting money\", \"review our AWS bill\", \"why did the cloud bill jump this month\", \"find savings in our GCP/Azure spend\", or \"should we buy savings plans\".",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/sre-oncall/skills/cloud-cost-review/scripts/cost_summary.py

@@ -110,7 +110,7 @@ FORMATS = {
     },
 }
 
-COMPUTE_RE = re.compile(r"BoxUsage|DedicatedUsage|HostUsage|Fargate-(vCPU|GB)-Hours|Lambda-GB-Second|InstanceUsage|NodeUsage|"
+COMPUTE_RE = re.compile(r"BoxUsage|SpotUsage|DedicatedUsage|HostUsage|Fargate-(vCPU|GB)-Hours|Lambda-GB-Second|InstanceUsage|NodeUsage|"
                         r"Instance Core|Instance Ram|vCPU", re.I)
 GENERIC_COMPUTE_SERVICE_RE = re.compile(r"compute|virtual machine|\bvm\b|ec2|fargate|lambda|functions", re.I)
 HOTSPOTS = [
@@ -603,7 +603,12 @@ def main(argv=None):
             raise InputError("--month must be YYYY-MM, got %r" % args.month)
         if args.top < 1:
             raise InputError("--top must be at least 1")
-        fmt, used, rows = load(Path(args.csv), args.format)
+        try:
+            fmt, used, rows = load(Path(args.csv), args.format)
+        except UnicodeDecodeError:
+            raise InputError("%s is not UTF-8 text; save it as UTF-8 (CSV UTF-8) and retry" % args.csv)
+        except csv.Error as e:  # e.g. a NUL byte, which Python before 3.11 rejects
+            raise InputError("%s is not a readable CSV: %s" % (args.csv, e))
         a = analyse(rows, args.tag, args.month, args.top, fmt)
     except InputError as e:
         print("error: %s" % e, file=sys.stderr)

package/packs/sre-oncall/skills/delivery-metrics-report/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "delivery-metrics-report",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Produce a delivery metrics report for one service from deployment, incident and pull-request exports — the DORA metrics (deployment frequency, change lead time, change fail rate, failed deployment recovery time) plus flow metrics (WIP, cycle time p50/p85, throughput), computed by a script with per-week trends and explicit data gaps, then interpreted for what changed and why. Use when the user wants DORA metrics, says \"how are we doing on the four keys\", \"what's our change failure rate\", \"compute lead time and deploy frequency from this CSV\", or \"build our delivery metrics report\".",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/sre-oncall/skills/delivery-metrics-report/scripts/delivery_metrics.py

@@ -50,6 +50,8 @@ def parse_offset(text: str) -> timezone:
         return timezone.utc
     if not m:
         raise InputError("--assume-tz must look like +00:00, -05:00 or Z, got %r" % text)
+    if int(m.group(2)) > 23 or int(m.group(3)) > 59:
+        raise InputError("--assume-tz offset out of range (hours 00-23, minutes 00-59), got %r" % text)
     sign = 1 if m.group(1) == "+" else -1
     return timezone(sign * timedelta(hours=int(m.group(2)), minutes=int(m.group(3))))
 
@@ -179,6 +181,8 @@ def compute(args) -> dict:
     dep_rows = read_csv(Path(args.deployments), ["deploy_id", "service", "deployed_at", "status"], "deployments")
     inc_rows = read_csv(Path(args.incidents), ["incident_id", "service", "started_at", "resolved_at"], "incidents") if args.incidents else None
     pr_rows = read_csv(Path(args.prs), ["pr_id", "opened_at", "merged_at"], "prs") if args.prs else None
+    if pr_rows is not None and not pr_rows:
+        raise InputError("prs file %s has a header but no rows; add PR rows or drop --prs" % args.prs)
 
     services_seen = sorted({r["service"] for r in dep_rows if r["service"]})
     svc = args.service

package/packs/trackers/skills/backlog-triage/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "backlog-triage",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Triage a whole backlog export from Jira, Linear or GitHub Projects (CSV, JSON or an MCP tool) — duplicate and near-duplicate pairs with the reason, items missing acceptance criteria or an owner, stale items, oversized items to split, priority conflicts, and a proposed ordering with a reason per item — ending in a change list the user can apply. A stdlib script computes the mechanical checks. Use when the user wants to clean up, groom or refine a backlog, find duplicate tickets, or says \"triage our backlog\", \"what should we pull next?\".",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/trackers/skills/backlog-triage/scripts/triage_backlog.py

@@ -102,6 +102,8 @@ def load_rows(path: Path) -> list:
         text = path.read_text(encoding="utf-8-sig")
     except OSError as e:
         die("cannot read {}: {}".format(path, e))
+    except UnicodeDecodeError:
+        die("{} is not UTF-8 text; save it as UTF-8 and retry".format(path))
     if suffix in (".csv", ".tsv"):
         delim = "\t" if suffix == ".tsv" else ","
         reader = csv.reader(io.StringIO(text, newline=""), delimiter=delim)
@@ -175,6 +177,8 @@ def parse_date(s: str):
     if not s:
         return None
     s2 = s[:-1] + "+00:00" if s.endswith("Z") else s
+    # Python < 3.11 needs a colon in the offset (Jira REST writes +0000).
+    s2 = re.sub(r"(\d{2}:\d{2}(?::\d{2}(?:\.\d+)?)?[+-]\d{2})(\d{2})$", r"\1:\2", s2)
     try:
         return datetime.fromisoformat(s2).date()
     except ValueError:
@@ -250,7 +254,10 @@ def main(argv=None) -> int:
 
     if not args.export.exists():
         die("{} does not exist".format(args.export))
-    rows = load_rows(args.export)
+    try:
+        rows = load_rows(args.export)
+    except csv.Error as e:  # e.g. a NUL byte, which Python before 3.11 rejects
+        die("{} is not a readable export: {}".format(args.export, e))
     if not rows:
         die("{} has no issues in it".format(args.export))
     headers = []
@@ -283,8 +290,9 @@ def main(argv=None) -> int:
         })
     open_items = [it for it in items if not it["done"]]
 
-    no_owner = [it for it in open_items if not it["owner"] or it["owner"].lower() in ("unassigned", "none")]
-    no_ac = [it for it in open_items if not it["has_ac"]]
+    no_owner = [it for it in open_items if not it["owner"] or it["owner"].lower() in ("unassigned", "none")] \
+        if "owner" in mp else []
+    no_ac = [it for it in open_items if not it["has_ac"]] if ("description" in mp or "acceptance" in mp) else []
     no_date = [it for it in open_items if it["updated"] is None]
     stale = []
     for it in open_items:

package/packs/trackers/skills/team-status-report/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "team-status-report",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Write a weekly team status report from tracker data (Jira, Linear or GitHub Projects, via CSV, JSON or an MCP tool) — shipped, in progress, blocked with the blocker and owner, slipped against plan, risks and asks — with every number computed from the data by a stdlib script, never estimated, and a RAG status that states the rule that set it. Use when the user wants a weekly status update, sprint report or stakeholder update from the tracker, or says \"write this week's status report\", \"are we on track?\".",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/trackers/skills/team-status-report/scripts/status_counts.py

@@ -96,6 +96,8 @@ def load_rows(path: Path) -> list:
         text = path.read_text(encoding="utf-8-sig")
     except OSError as e:
         die("cannot read {}: {}".format(path, e))
+    except UnicodeDecodeError:
+        die("{} is not UTF-8 text; save it as UTF-8 and retry".format(path))
     if suffix in (".csv", ".tsv"):
         delim = "\t" if suffix == ".tsv" else ","
         reader = csv.reader(io.StringIO(text, newline=""), delimiter=delim)
@@ -232,7 +234,10 @@ def main(argv=None) -> int:
 
     if not args.export.exists():
         die("{} does not exist".format(args.export))
-    rows = load_rows(args.export)
+    try:
+        rows = load_rows(args.export)
+    except csv.Error as e:  # e.g. a NUL byte, which Python before 3.11 rejects
+        die("{} is not a readable export: {}".format(args.export, e))
     if not rows:
         die("{} has no issues in it".format(args.export))
     headers = []
@@ -309,7 +314,9 @@ def main(argv=None) -> int:
         planned = [by_key[k] for k in keys if k in by_key]
         plan_source = "plan file {} ({} keys)".format(args.plan_file.name, len(keys))
     elif args.sprint:
-        planned = [it for it in items if args.sprint.lower() in it["sprint"].lower()]
+        # Whole-name match; repeated Sprint columns are joined with ", " (carried-over items).
+        want = args.sprint.strip().lower()
+        planned = [it for it in items if want in [x.strip().lower() for x in it["sprint"].split(",")]]
         plan_source = "items in sprint '{}'".format(args.sprint)
     elif "due" in mp:
         planned = [it for it in items if it["due"] and start <= it["due"] <= end]

package/packs/trackers/skills/tracker-brief-sync/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "tracker-brief-sync",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Turn a brief or PRD (such as prd-draft output) into tracker-ready epics and issues — title, description, acceptance criteria, labels and a link back to the doc section — as a CSV for Jira, a mapped CSV for other importers, or gh commands for GitHub; and in reverse, report drift between the doc and the tracker (issues with no section, sections with no issue, issues for Won't-have requirements). Use when the user wants to load a PRD into Jira, Linear or GitHub, says \"create tickets from this brief\", or asks \"does the tracker still match the PRD?\".",
   "entrypoint": "SKILL.md",
   "deps": {

package/packs/trackers/skills/tracker-brief-sync/scripts/brief_sync.py

@@ -194,7 +194,7 @@ def parse_doc(path: Path, id_pattern: str, headings: bool) -> dict:
             continue
         if s.startswith("|"):
             cells = [c.strip().strip("*").strip() for c in s.strip("|").split("|")]
-            if cells and broad.match(cells[0]):
+            if cells and broad.match(cells[0]) and not headings:
                 moscow = ""
                 for c in cells[2:]:
                     if c.lower() in MOSCOW: