Menu

#259 fix(github): guard against null nodes in ProjectV2 GraphQL response

closed
nobody
None
2026-06-11
2026-06-11
Anonymous
No

Originally created by: adityaharishch

Summary

  • ProjectV2ItemConnection.nodes and UserConnection.nodes are typed [T] in GitHub's GraphQL schema — both the list and individual elements can be null
  • The Rust structs had them as Vec<T> (non-optional elements), so a null element would cause a serde deserialization failure and silently drop the entire project's tasks
  • Changed both to Vec<Option<T>> and added .flatten() at the two iteration sites

Test plan

  • [ ] Verify cargo clippy passes (no warnings)
  • [ ] Trigger a GitHub sync and confirm tasks from all configured projects are fetched correctly
  • [ ] Confirm behaviour is unchanged when no null elements are present (normal case)

🤖 Generated with Claude Code

Discussion

  • Anonymous

    Anonymous - 2026-06-11

    Originally posted by: Akarsh-Hegde

    Superseded by [#257]

     

    Related

    Tickets: #257

  • Anonymous

    Anonymous - 2026-06-11

    Ticket changed by: Akarsh-Hegde

    • status: open --> closed
     

Log in to post a comment.