Skip to content

fix: guard against empty API response on project fetch

Rajasekhar Ponakala requested to merge feat/speckit-compliance into main

What does this MR do and why?

fix: guard against empty API response on project fetch

  • compliance.py: check project is a valid dict before accessing project["id"] — was crashing with TypeError when _get returned []
  • client.py: never cache empty [] or {} responses, only cache valid non-empty results so a failed lookup doesn't poison the cache for 5 minutes

Co-Authored-By: Claude Sonnet 4.6 [email protected]

References

Screenshots or screen recordings

Before After

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports