refactor(deps): isolate documentation dependencies into an npm workspace
Maintainers usually reply within 1 day
A pull request for this has already been merged.
- #549 by @Kuldeep2822k — merged
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 48/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- node.js, typescript
- Domain
- build-system, cli, documentation
Research direction
Start by reading the root package.json and .github/workflows/deploy-docs.yml, then trace the existing documentation scripts and install commands. Move the listed documentation dependencies and overrides into a docs workspace, keep the named CLI tools at the root, and update the scripts and workflow to use that workspace. Done when documentation builds and deploys through the workspace while the root retains only the listed CLI development tools.
Written by the indexing model from the issue text.
Description
Problem Statement
The root package.json currently includes both core CLI development dependencies and the documentation site toolchain:
vitepress(^1.6.4)vitepress-plugin-mermaid(^2.0.17)mermaid(^11.16.1)markdown-it-mathjax3(^4.3.2)
These doc-generation tools account for ~85% of total packages in node_modules (~400 of 462 packages) and form deep transitive dependency chains (up to 12 levels deep via Vue/Babel/PostCSS, plus nested CVE surfaces such as DOMPurify and KaTeX).
Core CLI tasks (compiling TypeScript, linting, executing unit/invariant tests via tsx, and computing coverage with c8) do not need any of these packages.
Proposed Solution
Isolate the documentation toolchain into its own npm workspace (e.g., docs/package.json or via workspaces: ["docs"]).
- Move Docs Dependencies: Transfer
vitepress,mermaid,vitepress-plugin-mermaid,markdown-it-mathjax3, and their related overrides (katex,dompurify) todocs/package.json. - Keep Root Lean: Retain only core CLI development tools (
typescript,eslint,tsx,c8,diff-cover) in rootdevDependencies. - Update Workflows: Update doc build/dev scripts and
.github/workflows/deploy-docs.ymlto operate against thedocsworkspace.
Benefits
- Zero Supply-Chain Bloat in CLI Toolchain: Eliminates deep transitive dependency trees and reduces noise from third-party audit advisories affecting only doc rendering.
- Faster CI/Dev Installs: CI jobs running tests/linting don't need to resolve or unpack hundreds of docs-only packages.
- Decoupled Tooling: Documentation site dependencies can be updated independently without affecting CLI release gates.
- Dominant language
- TypeScript
- Stars
- 1
- Forks
- 0
- Avg merge
- 17h 14m
- Merged PRs (30d)
- 170
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Kuldeep2822k/cli
-
Task: Full-repository improvement audit at 24947bc — verified P1/P2/P3 remediation checklistPossibly taken @Kuldeep2822k claimed this 5 days ago. Openchore ci cli core dependencies P1 security storage tests
Kuldeep2822k/cli#565 · 2 comments · 1 reaction · 1 assignee ·
Maintainers usually reply within 1 day
-
chore priority: low refactor
Difficulty 5/5 Over a week Newbie friendliness 25/100
Kuldeep2822k/cli#548 ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
Kuldeep2822k/cli#522 ·
Maintainers usually reply within 1 day
-
bug tests
Difficulty 5/5 Over a week Newbie friendliness 25/100
Kuldeep2822k/cli#504 · 1 comment ·
Maintainers usually reply within 1 day
-
Dependency DashboardOpenci dependencies security
Difficulty 4/5 3-5 days Newbie friendliness 8/100
Kuldeep2822k/cli#463 · 1 comment ·
Maintainers usually reply within 1 day
All issues in Kuldeep2822k/cli
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
hugomrdias/playwright-test#753 ·
Maintainers usually reply within 1 day
-
Review view: Daily/Weekly Review header buttons use the app-wide toolbar control stylePossibly taken @colherdesopa claimed this today. Open
Difficulty 1/5 1-3 hours Newbie friendliness 90/100
dongdongbh/Mindwtr#1515 ·
Maintainers usually reply within 1 day
-
pending-triage
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
kirodotdev/Kiro#12301 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 83/100