chore: reduce Greptile review noise from defensive coding suggestions

[andreatgretel]

Summary

Reduce noise from Greptile's automated code reviews by raising the severity threshold, filtering to logic-only comments, and adding explicit instructions to skip defensive coding suggestions.

Changes

Changed

• greptile.json - Added three noise-reduction settings:
  • strictness: 3 - only surface critical issues (was default 2)
  • commentTypes: ["logic"] - skip style and syntax comments
  • instructions - tell Greptile to avoid suggesting error handling, null checks, try/catch, input validation, logging, comments, or docstrings unless there's a concrete bug

---

Generated with AI

[greptile-apps]

Greptile Summary

This PR tightens the Greptile reviewer configuration to reduce low-signal comments by raising strictness to 3, restricting emitted comment types to logic only, and adding an explicit instructions string that mirrors custom instruction instruction-1 already active in this org.

Key changes:

• strictness: 3 — raises the bar above the default (2) so only high-confidence issues are surfaced.
• commentTypes: ["logic"] — suppresses style and syntax category comments entirely. Worth keeping in mind that real compilation/type errors classified as syntax by Greptile would no longer be reported inline; however, the accompanying instructions text directs the model to focus on concrete bugs, which should compensate.
• instructions — aligns the per-repo instruction with the org-level instruction-1 custom rule, making the behaviour explicit and stable even if org-level rules change.
• sequenceDiagramSection.defaultOpen — quietly changed from true to false (not mentioned in the PR description). This is a harmless UX tweak that collapses the sequence diagram section by default.

Confidence Score: 5/5

• This PR is safe to merge — it only modifies a Greptile configuration file with no functional code changes.
• Single-file, config-only change. The three new fields (strictness, commentTypes, instructions) are additive and well-understood. The defaultOpen tweak is cosmetic. No logic, tests, or production code are affected.
• No files require special attention.

Important Files Changed

Filename	Overview
greptile.json	Adds three noise-reduction settings (strictness: 3, commentTypes: ["logic"], instructions) and collapses the sequence diagram section by default — straightforward config-only change with no logic issues.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR opened / updated] --> B[Greptile triggered]
    B --> C{strictness >= 3?}
    C -- "Yes (new)" --> D[Filter low-confidence findings]
    C -- "No (old default 2)" --> E[Surface all findings]
    D --> F{commentTypes check}
    F -- "logic only (new)" --> G[Emit logic comments]
    F -- "logic + style + syntax (old)" --> H[Emit all comment types]
    G --> I{instructions filter}
    I -- "No defensive-coding suggestions" --> J[Post final review comments]
    H --> J

Loading

_{Last reviewed commit: 17079b6}

[nabinchha]

unrelated: may be we can collapse the sequence diagram by default?

[andreatgretel]

I think so, hasn't been super helpful! Added.