Skip to content

Guard nullable GraphQL responses#831

Merged
kitsuyui merged 1 commit into
mainfrom
guard-null-graphql-responses
May 23, 2026
Merged

Guard nullable GraphQL responses#831
kitsuyui merged 1 commit into
mainfrom
guard-null-graphql-responses

Conversation

@kitsuyui
Copy link
Copy Markdown
Owner

Summary

  • guard nullable viewer and history GraphQL response fields before reading nested values
  • return clear errors when the current user login or commit count cannot be resolved
  • add regression coverage and regenerate the checked-in action bundle

Verification

  • bun install --frozen-lockfile
  • bun run format
  • bun run lint
  • bun run test
  • bun run build
  • git diff --check

@github-actions
Copy link
Copy Markdown

🎉 Happy commit!

@github-actions
Copy link
Copy Markdown

gh-counter

PR gate

Removed Added +/-
TODO/FIXME 0 0 0

Repo dashboard

main (d8fc92c) #831 (1b896a4) +/-
TODO/FIXME 0 0 0

Reported by gh-counter

@github-actions
Copy link
Copy Markdown

Code Metrics Report

main (d8fc92c) #831 (1b896a4) +/-
Coverage 99.0% 99.0% +0.0%
Code to Test Ratio 1:0.5 1:0.5 +0.0
Test Execution Time 1s 1s 0s
Details
  |                     | main (d8fc92c) | #831 (1b896a4) |  +/-  |
  |---------------------|----------------|----------------|-------|
+ | Coverage            |          99.0% |          99.0% | +0.0% |
  |   Files             |              5 |              5 |     0 |
  |   Lines             |            107 |            109 |    +2 |
+ |   Covered           |            106 |            108 |    +2 |
+ | Code to Test Ratio  |          1:0.5 |          1:0.5 |  +0.0 |
  |   Code              |           1438 |           1464 |   +26 |
+ |   Test              |            777 |            799 |   +22 |
  | Test Execution Time |             1s |             1s |    0s |

Code coverage of files in pull request scope (100.0% → 100.0%)

Files Coverage +/- Status
src/github.ts 100.0% 0.0% modified

Reported by octocov

@github-actions
Copy link
Copy Markdown

gh-build-size

Target main #831 +/-
Total action artifacts 23,291,895 B 23,295,317 B +3,422 B
Runtime bundle 1,152,247 B 1,153,079 B +832 B
Source maps 17,124,104 B 17,126,694 B +2,590 B

Reported by gh-build-size

@kitsuyui kitsuyui merged commit 8b69839 into main May 23, 2026
5 checks passed
@kitsuyui kitsuyui deleted the guard-null-graphql-responses branch May 23, 2026 11:44
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.

1 participant