Add analyze plugins to CLI dependencies for npx resolution#45
Merged
YusukeHirao merged 3 commits intomainfrom Mar 5, 2026
Merged
Add analyze plugins to CLI dependencies for npx resolution#45YusukeHirao merged 3 commits intomainfrom
YusukeHirao merged 3 commits intomainfrom
Conversation
When running via `npx @nitpicker/cli analyze`, the dynamic `import()` calls in `@nitpicker/core` fail with ERR_MODULE_NOT_FOUND because the analyze plugin packages are not in the CLI's dependency tree. Add all six standard analyze plugins as direct dependencies of `@nitpicker/cli`. Closes #34 https://claude.ai/code/session_013sn9ycemnanzocpDRtT689
…encies The CLI now directly depends on analyze-* plugins for npx module resolution. Update dependency graphs in ARCHITECTURE.md and CLAUDE.md to reflect this, and add a note about maintaining CLI dependencies when adding new plugins. https://claude.ai/code/session_013sn9ycemnanzocpDRtT689
… plugin note - Add @d-zero/dealer to the dependency graph (was missing) - Add note about CLI depending on analyze-* plugins for npx module resolution - Add explanation to @nitpicker/cli section about bundled plugin dependencies https://claude.ai/code/session_013sn9ycemnanzocpDRtT689
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updated the CLI package to explicitly include all analyze plugins as direct dependencies, enabling proper module resolution when running commands via
npx.closes: #34
Key Changes
Added all 6 analyze plugins (
@nitpicker/analyze-*) to@nitpicker/clidependencies:@nitpicker/analyze-axe@nitpicker/analyze-lighthouse@nitpicker/analyze-main-contents@nitpicker/analyze-markuplint@nitpicker/analyze-search@nitpicker/analyze-textlintUpdated architecture documentation (ARCHITECTURE.md and CLAUDE.md) to clarify:
dependenciesfornpxexecution@nitpicker/coredynamically imports plugins at runtimeImplementation Details
The CLI now acts as a dependency aggregator for all analyze plugins. This ensures that when users run
npx @nitpicker/cli analyze, the dynamicimport()calls in@nitpicker/corecan properly resolve plugin modules without requiring users to install them separately. The documentation includes a note for future maintainers to add new analyze plugins to this list when they are created.https://claude.ai/code/session_013sn9ycemnanzocpDRtT689