• docs: improve tool descriptions with review comment summary instructions #172
• feat: check-env CLI command for config validation #171
• fix: per-PR error handling in review_pr_descriptions #170
• feat: support dismissing PR-level reviews in resolve_comment #169
• fix: apply _check_graphql_errors to query paths #168 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Greptile Summary

Added missing GraphQL error checks to _fetch_raw_threads and _fetch_thread_detail functions using _check_graphql_errors. This prevents silently swallowing GraphQL-level errors like auth failures and rate limits. The change ensures both query paths now properly raise GhError with descriptive messages when GraphQL returns errors.

• Added _check_graphql_errors call in _fetch_raw_threads after the paginated GraphQL query (line 432)
• Added _check_graphql_errors call in _fetch_thread_detail after the thread detail query (line 605)
• Comprehensive test coverage added with 3 tests covering both error and success scenarios
• Tests properly validate error messages and successful data flow

Confidence Score: 5/5

• This PR is safe to merge with no concerns
• The changes are minimal, well-tested, and follow the existing error handling pattern used elsewhere in the codebase. The fix addresses a real bug where GraphQL errors were being silently ignored, and the implementation correctly applies the existing _check_graphql_errors helper that's already used in other GraphQL calls in the same file
• No files require special attention

Important Files Changed

Flowchart

flowchart TD
    A[GraphQL Query] --> B{Response Received}
    B --> C[_check_graphql_errors]
    C --> D{Errors Field Present?}
    D -->|Yes| E[Raise GhError with context]
    D -->|No| F[Extract data from response]
    F --> G[Process & return data]
    
    style E fill:#ff6b6b
    style G fill:#51cf66
    style C fill:#ffd43b

_{Last reviewed commit: a877d2f}

@greptileai review

Merge activity

• Feb 17, 11:17 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 17, 11:17 AM UTC: @ichoosetoaccept merged this pull request with Graphite.