-
Notifications
You must be signed in to change notification settings - Fork 0
refactor(npm): Deduplicate and categorize NPM packages with feature tags #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
- Removed 736 duplicate entries (1609 → 861 unique packages) - Organized into 20 functional categories aligned with README - Added feature tags for each category (e.g., [ai-agent], [mcp-server], [rag]) - Separated 12 invalid/external entries (URLs, user profiles) - Preserved original RAW_LIST for reference - Created proper module exports (GROUPED_PACKAGES, ALL_PACKAGES, metadata) Categories created: - ClaudeEcosystem (19 packages): claude-integration, anthropic - MCPServers (80 packages): mcp-server, model-context-protocol - AgentFrameworks (50 packages): ai-agent, orchestration, multi-agent - RAGCodebaseIndexing (26 packages): rag, vector-db, code-indexing - StaticCodeAnalysis (24 packages): static-analysis, linting, code-quality - TestingQA (26 packages): testing, test-automation, qa - DataCrawlingScraping (65 packages): web-scraping, data-extraction - BrowserAutomation (6 packages): browser-automation - CodeGeneration (22 packages): code-generation, ide-integration - Research (6 packages): research, knowledge-management - Memory (2 packages): memory, caching, persistence - Sandboxing (4 packages): sandboxing, code-execution - APIs (23 packages): api, sdk, integration - DatabaseORM (26 packages): database, orm - UIFrameworks (14 packages): ui, frontend, components - CLI (15 packages): cli, command-line - DevTools (27 packages): dev-tools, productivity - Proxy (2 packages): proxy, networking - Security (3 packages): security, encryption - ObservabilityMonitoring (7 packages): observability, telemetry - Uncategorized (414 packages): require manual review Co-authored-by: Zeeeepa <[email protected]>
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 issues found across 1 file
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="NPMs.js">
<violation number="1" location="NPMs.js:2148">
P2: Invalid NPM package name: `claude-flow@alpha` contains a distribution tag suffix (`@alpha`) which is not part of the actual package name. This should be `claude-flow` instead, as `@alpha` is installation syntax (e.g., `npm install claude-flow@alpha`).</violation>
<violation number="2" location="NPMs.js:2498">
P2: Invalid entry: `yuusoft-org/kanbatte` is a GitHub repository reference (owner/repo format), not a valid NPM package name. According to the PR's own categorization logic, this should be moved to `INVALID_OR_EXTERNAL` and removed from `ALL_PACKAGES` and `Uncategorized`.</violation>
</file>
Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR
| "yuniq-cookie", | ||
| "yuniqkv", | ||
| "yutani", | ||
| "yuusoft-org/kanbatte", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Invalid entry: yuusoft-org/kanbatte is a GitHub repository reference (owner/repo format), not a valid NPM package name. According to the PR's own categorization logic, this should be moved to INVALID_OR_EXTERNAL and removed from ALL_PACKAGES and Uncategorized.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At NPMs.js, line 2498:
<comment>Invalid entry: `yuusoft-org/kanbatte` is a GitHub repository reference (owner/repo format), not a valid NPM package name. According to the PR's own categorization logic, this should be moved to `INVALID_OR_EXTERNAL` and removed from `ALL_PACKAGES` and `Uncategorized`.</comment>
<file context>
@@ -1608,5 +1631,2285 @@
+ "yuniq-cookie",
+ "yuniqkv",
+ "yutani",
+ "yuusoft-org/kanbatte",
+ "ziri",
+ "zterm"
</file context>
| "claude-code-sugar", | ||
| "claude-flow", | ||
| "claude-flow-novice", | ||
| "claude-flow@alpha", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Invalid NPM package name: claude-flow@alpha contains a distribution tag suffix (@alpha) which is not part of the actual package name. This should be claude-flow instead, as @alpha is installation syntax (e.g., npm install claude-flow@alpha).
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At NPMs.js, line 2148:
<comment>Invalid NPM package name: `claude-flow@alpha` contains a distribution tag suffix (`@alpha`) which is not part of the actual package name. This should be `claude-flow` instead, as `@alpha` is installation syntax (e.g., `npm install claude-flow@alpha`).</comment>
<file context>
@@ -1608,5 +1631,2285 @@
+ "claude-code-sugar",
+ "claude-flow",
+ "claude-flow-novice",
+ "claude-flow@alpha",
+ "claude-hydra",
+ "claude-skill",
</file context>
📊 Summary
Completely refactored
NPMs.jsto organize 1,600+ package entries into a maintainable, well-structured module with feature tags.📈 Statistics
✨ Key Changes
1. Deduplication
2. Categorization with Feature Tags
Organized packages into 20 functional categories aligned with README sections:
claude-integration,anthropicmcp-server,model-context-protocolai-agent,orchestration,multi-agentrag,vector-db,code-indexing,semantic-searchstatic-analysis,linting,code-quality,asttesting,test-automation,qa,e2e-testingweb-scraping,data-crawling,data-extractionbrowser-automation,web-automationcode-generation,code-transformation,ide-integrationresearch,knowledge-management,documentationmemory,caching,persistencesandboxing,code-execution,runtimeapi,sdk,client,integrationdatabase,orm,data-persistenceui,frontend,components,visualizationcli,command-line,terminaldev-tools,productivity,ideproxy,networking,routingsecurity,encryption,vulnerability-scanningobservability,monitoring,telemetry,logging3. Invalid Entry Handling
Separated 12 invalid/external entries:
www.npmjs.com/~username)https)tinkaton?activeTab=code)yuusoft-org/kanbatte)@contrast/,@askman-dev/)memory,management,system)4. Module Structure
Converted from raw array to proper module exports:
🎯 Benefits
📝 Usage Examples
🔍 Next Steps
📂 Files Changed
NPMs.js- Complete refactor (2,307 insertions, 4 deletions)🔗 Related
💻 View my work • 👤 Initiated by @Zeeeepa • About Codegen
⛔ Remove Codegen from PR • 🚫 Ban action checks
Summary by cubic
Refactored NPMs.js to deduplicate 1,609 entries, group packages into 20 categories with feature tags, and export a structured module. This makes the list maintainable, searchable, and ready for programmatic use.
Refactors
Migration
Written for commit a9f06f3. Summary will update automatically on new commits.