docs: update documentation to reflect optional RulesPath#48
docs: update documentation to reflect optional RulesPath#48oshorefueled merged 2 commits intomainfrom
Conversation
- README.md: Simplify Quick Start to 3 steps using bundled preset - README.md: Remove manual rule creation step (now uses VectorLint preset) - README.md: Update terminology from 'Global API keys' to 'LLM provider API keys' - CONFIGURATION.md: Mark RulesPath as optional in settings table - CONFIGURATION.md: Update example to show RulesPath commented out - init-command.ts: Reorder next steps to prioritize API key setup
📝 WalkthroughWalkthroughDocumentation and CLI messaging were updated: RulesPath is no longer required, examples and quick-start flow were simplified to emphasize bundled presets and running a check, and post-init instructions now point users to configure API keys, run linting, or optionally edit local rules. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Poem
Pre-merge checks✅ Passed checks (3 passed)
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📚 Learning: 2025-12-28T19:43:51.189ZApplied to files:
📚 Learning: 2025-12-28T19:43:51.189ZApplied to files:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/cli/init-command.ts (1)
67-69: Consider alignment with README.md example.The CLI guidance uses a placeholder
<file.md>while README.md uses the concrete exampledoc.md. For consistency and clarity, consider either:
- Using the same concrete example
doc.mdhere, or- Ensuring the README also uses angle-bracket placeholders
The current approach is functional but could momentarily confuse users cross-referencing documentation.
🔎 Suggested alignment with README.md
- console.log(` 2. Run 'vectorlint <file.md>' to start linting your content`); + console.log(` 2. Run 'vectorlint doc.md' to start linting your content`);
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
CONFIGURATION.mdREADME.mdsrc/cli/init-command.ts
🧰 Additional context used
📓 Path-based instructions (1)
src/**/*.ts
📄 CodeRabbit inference engine (AGENTS.md)
src/**/*.ts: Use TypeScript ESM with explicit imports and narrow types
Use 2-space indentation; avoid trailing whitespace
Maintain strict TypeScript with noany; useunknown+ schema validation for external data
Use custom error types with proper inheritance; catch blocks useunknowntype
Files:
src/cli/init-command.ts
🧠 Learnings (2)
📚 Learning: 2025-12-28T19:43:51.189Z
Learnt from: CR
Repo: TRocket-Labs/vectorlint PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-12-28T19:43:51.189Z
Learning: Organize rules into subdirectories (packs) within RulesPath; use company names (e.g., Acme, TechCorp, Stripe) to indicate which style guide the rules implement
Applied to files:
CONFIGURATION.md
📚 Learning: 2025-12-28T19:43:51.189Z
Learnt from: CR
Repo: TRocket-Labs/vectorlint PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-12-28T19:43:51.189Z
Learning: Applies to .vectorlint.ini : Use `[glob/pattern]` sections in `.vectorlint.ini` to specify which packs run on which files
Applied to files:
CONFIGURATION.mdREADME.md
🧬 Code graph analysis (1)
src/cli/init-command.ts (2)
src/config/global-config.ts (1)
getGlobalConfigPath(9-11)src/config/constants.ts (1)
DEFAULT_CONFIG_FILENAME(6-6)
🔇 Additional comments (3)
CONFIGURATION.md (1)
15-17: Clear guidance on optional RulesPath.The updated comments effectively explain when to use or omit RulesPath, making the optional nature immediately clear to users.
README.md (2)
63-63: Improved terminology clarity.The updated description "LLM provider API keys and settings" is more specific and descriptive than "Global API keys and provider settings," making the config file's purpose immediately clear.
79-87: Excellent simplification of the Quick Start flow.The restructured step 3 significantly improves the getting-started experience by:
- Prioritizing immediate usage over configuration
- Leveraging the bundled preset to remove setup friction
- Maintaining clear pathways to customization
This aligns well with the PR's objective to make RulesPath optional and simplify onboarding.
* docs: update documentation to reflect optional RulesPath - README.md: Simplify Quick Start to 3 steps using bundled preset - README.md: Remove manual rule creation step (now uses VectorLint preset) - README.md: Update terminology from 'Global API keys' to 'LLM provider API keys' - CONFIGURATION.md: Mark RulesPath as optional in settings table - CONFIGURATION.md: Update example to show RulesPath commented out - init-command.ts: Reorder next steps to prioritize API key setup * docs: use '(none)' instead of 'No' for RulesPath default value
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.