feat(skills): offer to update stale skills after commands

[gjtorikian]

Summary

• Skills ship inside the binary and are copied into each agent's skills/ dir, so every release that bumps @workos/skills leaves installed copies behind. Until now the only signal was workos doctor, and the setup offer never re-fires once completed — a user who set up once would never learn their skills were stale.
• After each successful command in a human TTY, the CLI now offers to refresh agents whose .workos-skill-version marker trails the bundled version. ("latest" is whatever this binary bundles, so there is no remote fetch, cache, or cooldown.)
• Reuses what already exists — checkSkills() (doctor's staleness check) for detection and refreshWorkOSSkills() (atomic per-skill backup/rename) for the refresh. The new code is the prompt and its gating.
• Consent: defaults to No (AUTH-6734: "an absent-minded Enter never writes to an agent dir"), honors a prior setup decline, and is silent in JSON / agent / CI / non-TTY. Asked at most once per bundled version — a "no" is remembered in preferences.json (setup.skillsUpdateOfferedVersion) until a newer CLI ships newer skills. Ctrl-C is not remembered, matching setup.
• Exempt commands: root (--help / --version), skills, setup, doctor.

[greptile-apps]

Confidence Score: 5/5

The changes since the previous review appear safe to merge, with both prior findings resolved and no new actionable defect identified.

Summary

• Runs only after successful, non-exempt commands in a prompt-capable human TTY.
• Defaults the update confirmation to No and remembers declines per bundled skills version.
• Detects partial multi-agent refreshes, reports the remaining stale agents, and allows one additional offer.
• The changes since the previous review correctly address partial-refresh retry eligibility without violating mounted repository rules.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Successful CLI command] --> B{Prompt-capable human TTY?}
  B -- No --> Z[Remain silent]
  B -- Yes --> C{Command exempt or setup declined?}
  C -- Yes --> Z
  C -- No --> D[Check installed skill versions]
  D --> E{Stale agents and version not already offered?}
  E -- No --> Z
  E -- Yes --> F[Offer update, default No]
  F -->|Cancel| Z
  F -->|Decline| G[Record offered version]
  F -->|Accept| H[Record offered version and refresh stale agents]
  H --> I{Refresh result}
  I -->|All updated| J[Report success]
  I -->|None updated| K[Report explicit retry command]
  I -->|Partial, first attempt| L[Clear offered version and record retry marker]
  I -->|Partial, retry spent| M[Keep offered version and report remaining agents]
  L --> N[Allow one more offer]

Loading

_{Reviews (3) · Last reviewed commit: "fix(skills): give an incomplete skill re..."}