Skip to content

feat: add npm update notifier - #46

Merged
jlitola merged 1 commit into
mainfrom
coal-yogurt
Apr 28, 2026
Merged

feat: add npm update notifier#46
jlitola merged 1 commit into
mainfrom
coal-yogurt

Conversation

@jlitola

@jlitola jlitola commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a background npm dist-tags update check for sticky CLI installs
  • cache non-secret update-check state under XDG config and print stale notices to stderr only
  • skip CI, non-TTY stderr, help/version, ephemeral package runners, and MCP stdio invocations
  • keep package/plugin version metadata at the currently released 0.2.3 value

Notes

  • stale-version notices intentionally print on every eligible invocation once a newer npm latest is known
  • update-check cache now uses ~/.config/githits; auth/client storage still uses ~/.githits and should be migrated in a follow-up before public release
  • lint currently exits 0 but reports existing ToolDefinition<any, any> warnings in src/commands/mcp.ts; follow-up will clean those up

Verification

  • bun test
  • bun run typecheck
  • bun run build
  • bun run lint

@jlitola
jlitola marked this pull request as ready for review April 28, 2026 16:34
Checks npm dist-tags in the background for sticky CLI installs and prints stale-version notices to stderr only. Stores non-secret cache state under the XDG config path and skips CI, non-TTY, ephemeral runner, help/version, and MCP stdio invocations.
@jlitola
jlitola merged commit 6df78a4 into main Apr 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant