chore: remove outdated documentation and screenshot scripts - #651
chore: remove outdated documentation and screenshot scripts#651priyanshu6238 wants to merge 2 commits into
Conversation
📝 WalkthroughWalkthroughThe pull request removes the documentation skill and its supporting guides, screenshot recipes, and Playwright screenshot runner. It also removes screenshot and video npm scripts and the 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
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.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
package.json (1)
32-32: 🩺 Stability & Availability | 🟠 Major | ⚡ Quick winKeep
dotenvas a direct dependency.
docusaurus.config.jsexecutesrequire("dotenv").config()during Docusaurus startup. Removingdotenvcauses a clean installation to fail withCannot find module "dotenv". Restore the previously declareddotenvdevelopment dependency.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@package.json` at line 32, Restore dotenv as a direct development dependency in package.json so docusaurus.config.js can load require("dotenv").config() during startup after a clean installation.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@package.json`:
- Line 32: Restore dotenv as a direct development dependency in package.json so
docusaurus.config.js can load require("dotenv").config() during startup after a
clean installation.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: d7baca1e-f5a0-436c-81d4-53ea3ce0b3c1
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (11)
.claude/skills/documentation/SKILL.md.claude/skills/documentation/screenshots.md.claude/skills/documentation/sections.md.claude/skills/documentation/style-images.md.claude/skills/documentation/style-page.md.claude/skills/documentation/style-writing.md.claude/skills/documentation/workflow.mdpackage.jsonscripts/recipes/chats.yamlscripts/recipes/flows.yamlscripts/screenshot.js
💤 Files with no reviewable changes (10)
- .claude/skills/documentation/style-writing.md
- .claude/skills/documentation/sections.md
- .claude/skills/documentation/screenshots.md
- .claude/skills/documentation/style-images.md
- scripts/screenshot.js
- .claude/skills/documentation/SKILL.md
- scripts/recipes/flows.yaml
- .claude/skills/documentation/style-page.md
- .claude/skills/documentation/workflow.md
- scripts/recipes/chats.yaml
Summary by CodeRabbit
Documentation
Chores