refactor: replace simple-git with tinyexec#950
Conversation
Replace the simple-git dependency with inline git commands via tinyexec to reduce dependencies and align with the project's existing tinyexec usage after the execa migration.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe pull request removes the Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
Replace the
simple-gitdependency with inline git commands viatinyexec. This removes an unnecessary dependency and aligns with the project's existing tinyexec usage across devtools and devtools-kit.The three git operations (
branch,rev-parse,status) are now inlined ingenerateAnalyzeBuildName()using tinyexec's subprocess API, with proper error handling and stdout parsing.Changes:
packages/devtools/src/server-rpc/analyze-build.tssimple-gitfrom dependencies and pnpm catalog🤖 Generated with Claude Code