From e2157998d7c04c765d32623cb04596a871e2dd5a Mon Sep 17 00:00:00 2001
From: Kevin Veen-Birkenbach <kevin@veen.world>
Date: Tue, 3 Nov 2026 17:32:48 +0100
Subject: [PATCH] fix(update): pass the source before --agent

Every scheduled Update run since 2026-10-31 failed with 'Missing required argument: source': --agent takes several values and swallowed the source that followed it. update.sh now passes the source first. test_update.py had pinned the source as the last argument and so encoded the bug; it now requires the source right after add and claude-code as the only agent (negative control against the old order fails).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 scripts/update.sh    | 2 +-
 tests/test_update.py | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/scripts/update.sh b/scripts/update.sh
index bf6fe0b..921ed98 100644
--- a/scripts/update.sh
+++ b/scripts/update.sh
@@ -30,6 +30,6 @@ fi
 log "skills: updating from ${lockfile}..."
 jq -r '.skills[].source' "${lockfile}" | sort -u | while read -r source; do
 	log "skills: updating ${source}..."
-	npx --yes skills add --yes --agent claude-code "${source}" </dev/null
+	npx --yes skills add "${source}" --yes --agent claude-code </dev/null
 done
 log "skills: update complete."
diff --git a/tests/test_update.py b/tests/test_update.py
index 6fe44cc..89dee03 100644
--- a/tests/test_update.py
+++ b/tests/test_update.py
@@ -36,10 +36,10 @@ class TestUpdate(unittest.TestCase):
             env={"PATH": f"{self.bin}:/usr/bin:/bin"},
             stdin=subprocess.DEVNULL,
         )
-        calls = self.log.read_text(encoding="utf-8").splitlines()
-        self.assertEqual([c.split()[-1] for c in calls], sources)
+        calls = [c.split() for c in self.log.read_text(encoding="utf-8").splitlines()]
+        self.assertEqual([c[c.index("add") + 1] for c in calls], sources)
         for call in calls:
-            self.assertIn("--agent claude-code", call)
+            self.assertEqual(call[call.index("--agent") + 1 :], ["claude-code"])
 
 
 if __name__ == "__main__":