Conversation
sahilmob
commented
Apr 6, 2026
- Debarrel sharding
- Move the skill installation step to a separate node
There was a problem hiding this comment.
Pull request overview
Adds sharding support to the debarrel codemod workflow and separates skill installation into its own workflow node to enable running the transform in shard-based PRs.
Changes:
- Introduces shard evaluation (by CODEOWNERS or directory) and a matrix-based PR strategy in the codemod workflow.
- Adds
getSelectorto support selecting candidate files for sharded execution. - Adds
codemodctldependency and adjusts TypeScript config for the codemod package.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-lock.yaml | Locks new dependencies (notably codemodctl) and updated dependency graph. |
| codemods/debarrel/workflow.yaml | Adds shard evaluation node, matrix shard execution node that opens draft PRs, and separates skill installation into its own node. |
| codemods/debarrel/tsconfig.json | Removes a couple of compiler options and enables skipLibCheck. |
| codemods/debarrel/scripts/codemod.ts | Adds getSelector and updates transform signature to accept options. |
| codemods/debarrel/package.json | Adds codemodctl as a runtime dependency. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "noFallthroughCasesInSwitch": true, | ||
| "noUncheckedIndexedAccess": true | ||
| "noUncheckedIndexedAccess": true, | ||
| "skipLibCheck": true |
There was a problem hiding this comment.
Setting skipLibCheck: true weakens the check-types gate by skipping type-checking of all declaration files, which can mask real type issues introduced by dependency updates. If this was added to work around a specific library typing problem, it would be better to address that directly (or at least document why this relaxation is necessary).
| "skipLibCheck": true | |
| "skipLibCheck": false |