Skip to content

greptile: Always emit sequence diagram in PR reviews#244

Merged
suphawitwanl merged 1 commit intomasterfrom
greptile-first-roll-out-4
Apr 30, 2026
Merged

greptile: Always emit sequence diagram in PR reviews#244
suphawitwanl merged 1 commit intomasterfrom
greptile-first-roll-out-4

Conversation

@yutanaofficial
Copy link
Copy Markdown

Adds an explicit sequenceDiagramSection block to .greptile/config.json
so the diagram is rendered on every PR review.

No rule changes — the slim three-rule config from roll-out-3 (#240) is
unchanged

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 30, 2026

Greptile Summary

Adds a sequenceDiagramSection block to .greptile/config.json so Greptile emits a sequence diagram on every PR review. The new block is consistent with the existing section definitions (summarySection, issuesTableSection, confidenceScoreSection) in structure and field naming. No rule changes; the three C-language enforcement rules from the prior roll-out are untouched.

Confidence Score: 5/5

Safe to merge — pure config addition with no logic, security, or behavioral risk.

Single config-only change that adds a well-structured section block consistent with existing patterns. No C code touched, no rules modified, no functional behavior changed in FRR itself.

No files require special attention.

Important Files Changed

Filename Overview
.greptile/config.json Adds sequenceDiagramSection config block; structure and values are consistent with all other section definitions in the file.

Sequence Diagram

sequenceDiagram
    participant GH as GitHub PR
    participant Greptile as Greptile Review Bot
    participant Config as .greptile/config.json

    GH->>Greptile: PR opened / updated
    Greptile->>Config: Load configuration
    Config-->>Greptile: sequenceDiagramSection.included = true
    Greptile->>Greptile: Analyze changed files
    Greptile->>GH: Post summary section
    Greptile->>GH: Post issues table section
    Greptile->>GH: Post confidence score section
    Greptile->>GH: Post sequence diagram section (new)
Loading

Reviews (1): Last reviewed commit: "greptile: enforce greptile to show seque..." | Re-trigger Greptile

@suphawitwanl suphawitwanl merged commit ce3c305 into master Apr 30, 2026
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants