chore: Greptile deep-label repo config (plan 028)#10
Conversation
Plan 028: org Reviews UI cannot set commentTypes/ignorePatterns. Repo config enforces review:deep-only, App authors, no auto on updates, status checks, and lockfile/dist ignores. Flex $0 already set org-wide.
|
Required label not found on this PR. |
📝 WalkthroughWalkthroughAdds ChangesGreptile Review Policy
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In @.greptile/config.json:
- Line 25: Update the ignorePatterns configuration in .greptile/config.json so
dist, .next, and node_modules patterns use recursive directory matching with
**/, ensuring these directories are ignored at every nesting depth while
preserving the existing lockfile exclusions.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository: ZenSystemAI/coderabbit/.coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: f90a4456-a0fa-438e-8407-26d016d980d1
📒 Files selected for processing (1)
.greptile/config.json
| "triggerOnUpdates": false, | ||
| "statusCheck": true, | ||
| "fileChangeLimit": 100, | ||
| "ignorePatterns": "**/package-lock.json\n**/pnpm-lock.yaml\ndist/**\n.next/**\nnode_modules/**", |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Match ignored build and dependency directories at every depth.
Because these patterns contain /, dist/**, .next/**, and node_modules/** only target root-level directories under .greptile/’s governed root; nested paths such as packages/web/dist/** can still be reviewed. Greptile applies .gitignore-style matching, where **/ is needed for all directory levels. (greptile.com)
Proposed fix
- "ignorePatterns": "**/package-lock.json\n**/pnpm-lock.yaml\ndist/**\n.next/**\nnode_modules/**",
+ "ignorePatterns": "**/package-lock.json\n**/pnpm-lock.yaml\n**/dist/**\n**/.next/**\n**/node_modules/**",📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "ignorePatterns": "**/package-lock.json\n**/pnpm-lock.yaml\ndist/**\n.next/**\nnode_modules/**", | |
| "ignorePatterns": "**/package-lock.json\n**/pnpm-lock.yaml\n**/dist/**\n**/.next/**\n**/node_modules/**", |
🤖 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 @.greptile/config.json at line 25, Update the ignorePatterns configuration in
.greptile/config.json so dist, .next, and node_modules patterns use recursive
directory matching with **/, ensuring these directories are ignored at every
nesting depth while preserving the existing lockfile exclusions.
Adds
.greptile/config.jsonso Greptile only runs onreview:deepApp-authored PRs.Org Flex Usage Limit is $0. CodeRabbit remains the standard path (labels review:standard + review:deep via central coderabbit config).
Summary by CodeRabbit