Skip to content

fix: reject non-object workflow caches (import of github/spec-kit#3860) - #138

Open
one-kash wants to merge 2 commits into
mainfrom
import/pr-3860
Open

fix: reject non-object workflow caches (import of github/spec-kit#3860)#138
one-kash wants to merge 2 commits into
mainfrom
import/pr-3860

Conversation

@one-kash

Copy link
Copy Markdown

Imported from upstream PR github#3860
Original author: @marcelsafin
Original head branch: fix/workflow-cache-shapes


Summary

  • treat non-object workflow and step cache metadata as invalid
  • refetch when a cached workflow catalog is not a JSON object
  • reject non-object stale cache fallbacks instead of returning unusable data

Testing

  • uvx ruff@0.15.0 check src tests
  • .venv/bin/python -m pytest -q tests/test_workflows.py (821 passed)
  • .venv/bin/python -m pytest -q (6011 passed, 173 skipped)

AI disclosure

GitHub Copilot helped identify edge cases and review the implementation. I reproduced the failures and validated the final change with targeted and full test suites.

marcelsafin and others added 2 commits July 29, 2026 19:57
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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