Skip to content

feat: complete pnpm scripts directly - #165

Merged
versenilvis merged 2 commits into
versenilvis:mainfrom
bingoYB:codex/pnpm-direct-script-completion
Sep 29, 2026
Merged

versenilvis merged 2 commits into
versenilvis:mainfrom
bingoYB:codex/pnpm-direct-script-completion

Conversation

@bingoYB

@bingoYB bingoYB commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Suggest package.json scripts directly after pnpm and while typing a script name.
  • Keep built-in pnpm commands and avoid duplicate entries when a script shares a built-in name.
  • Reuse the existing script reader for pnpm run and keep its fallback behavior.

Closes #164.

Verification

  • go test ./commands/js ./spec passed.
  • go vet ./commands/js ./spec passed.
  • git diff --check passed.
  • Full go test ./... was attempted. It could not complete because downloading modernc.org/libc@v1.73.4 ended with unexpected EOF; the environment-dependent TestZoxideGenerator also failed because no zoxide history results were available.

@versenilvis versenilvis left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just fix lint

@versenilvis
versenilvis merged commit 40256d5 into versenilvis:main Sep 29, 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.

Complete package.json scripts directly after pnpm

2 participants