@pushary/agent-hooks 0.72.0 → 0.73.0

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # Changelog
 
+## 0.73.0
+
+### Codex could finish setup with no skill installed
+
+If your machine already uses skills.sh, setup installs the Pushary skill
+through that CLI so the install registers rather than being invisible. For
+Codex, `skills add --agent codex` prints "copy to Codex", says "Installation
+complete" and exits 0, then writes only `~/.agents/skills/pushary`. Nothing
+lands in `~/.codex/skills`, which is where Codex reads. Setup took the exit
+code as proof, skipped the copy it ships, and Codex ended up with no skill.
+
+Doctor caught it, and then gave advice that could not work: clean and set up
+again reinstalled exactly the same way and failed the same check.
+
+The exit code no longer decides this. What decides it is whether the run left
+a SKILL.md where that agent reads, which is the same thing doctor looks at,
+and setup writes its own copy when it did not. Anything skills.sh really did
+install is left exactly as it installed it, symlinks included.
+
+Everything else about the wiring was fine, so this cost you the skill and
+nothing more. Approvals, hooks and the instructions in `~/.codex/AGENTS.md`
+were all in place.
+
+### A skill that will not install no longer stops setup
+
+The skill is the one piece of the wiring nothing else depends on. It used to
+be able to abort setup partway, leaving an agent half configured. Now it warns
+and setup finishes the rest, and `pushary doctor` tells you the skill is
+missing.
+
 ## 0.72.0
 
 ### Setup tells you when Codex was already quarantined

package/dist/bin/pushary-setup.js

@@ -294,6 +294,17 @@ var installSkillViaSkillsCli = (agent) => {
     return false;
   }
 };
+var installAgentSkill = async (request, effects) => {
+  if (request.skillsCliInUse) {
+    const startedAt = effects.now();
+    if (effects.runSkillsCli(request.agent)) {
+      const modifiedAt = effects.skillModifiedAt(request.agentSkillDir);
+      if (modifiedAt !== null && modifiedAt >= startedAt) return "skills-cli";
+    }
+  }
+  await effects.writePackagedSkill(request.agentSkillDir);
+  return "packaged-copy";
+};
 
 // src/setup/telemetry.ts
 var currentStep = "start";
@@ -607,14 +618,22 @@ var fetchSkillContent = async () => {
   _cachedSkillContent = readFileSync(source, "utf-8");
   return _cachedSkillContent;
 };
+var writePackagedSkill = async (dir) => {
+  const content = await fetchSkillContent();
+  if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
+  writeFileSync(join2(dir, "SKILL.md"), content, "utf-8");
+};
 var skillsCliInUse = () => isInstalled("skills") || existsSync(join2(homedir(), ".agents", "skills"));
-var installSkill = async (agent, fallbackDir) => {
+var skillInstallEffects = {
+  runSkillsCli: installSkillViaSkillsCli,
+  skillModifiedAt: (dir) => statSync(join2(dir, "SKILL.md"), { throwIfNoEntry: false })?.mtimeMs ?? null,
+  writePackagedSkill,
+  now: () => Date.now()
+};
+var installSkill = async (agent, agentSkillDir) => {
   await spinner("Installing Pushary skill", async () => {
-    if (skillsCliInUse() && installSkillViaSkillsCli(agent)) return;
-    const content = await fetchSkillContent();
-    if (!existsSync(fallbackDir)) mkdirSync(fallbackDir, { recursive: true });
-    writeFileSync(join2(fallbackDir, "SKILL.md"), content, "utf-8");
-  });
+    await installAgentSkill({ agent, agentSkillDir, skillsCliInUse: skillsCliInUse() }, skillInstallEffects);
+  }, { optional: true });
 };
 var setupClaudeCode = async (apiKey) => {
   console.log(`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushary/agent-hooks",
-  "version": "0.72.0",
+  "version": "0.73.0",
   "description": "Permission hooks for AI coding agents: route tool approvals through Pushary push notifications",
   "keywords": [
     "pushary",