Skip to content

feat: add more completions for package managers#34

Merged
AmirSa12 merged 1 commit intomainfrom
feat/pm-completions-options
Aug 11, 2025
Merged

feat: add more completions for package managers#34
AmirSa12 merged 1 commit intomainfrom
feat/pm-completions-options

Conversation

@AmirSa12
Copy link
Copy Markdown
Member

No description provided.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Aug 11, 2025

⚠️ No Changeset found

Latest commit: b9103aa

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Aug 11, 2025

Open in StackBlitz

npm i https://pkg.pr.new/bombshell-dev/tab/@bombsh/tab@34

commit: b9103aa

@AmirSa12 AmirSa12 merged commit d3d8c5d into main Aug 11, 2025
9 checks passed
@AmirSa12 AmirSa12 deleted the feat/pm-completions-options branch August 11, 2025 06:20
completion.command('audit', 'Run security audit');
completion.command('list', 'List installed packages');
// Package management
const addCmd = completion.command('add', 'Install packages');
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

as i mentioned, these options and commands them themselves have values which we need autocompletions for. only adding names won't show the library as a more feasible option compared to the traditional autocompletion scripts that these package managers provide.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

again, descriptions should be retrieved from --help.

because these descriptions are volatile and we should always show what the actual package manager shows. otherwise, it'd be an inconsistency between actual descriptions and what we provide.

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.

2 participants