Skip to content

Sadkoeni/fix update check marker#217

Open
sadkoeni wants to merge 2 commits intogarrytan:mainfrom
sadkoeni:sadkoeni/fix-update-check-marker
Open

Sadkoeni/fix update check marker#217
sadkoeni wants to merge 2 commits intogarrytan:mainfrom
sadkoeni:sadkoeni/fix-update-check-marker

Conversation

@sadkoeni
Copy link

No description provided.

sadkoeni and others added 2 commits March 19, 2026 14:13
Two bugs in bin/gstack-update-check:

1. --force did not clear the marker file, so forced checks still
   short-circuited on a stale just upgraded marker.

2. The marker handler wrote UP_TO_DATE to cache and exited, never
   checking if an even newer version was available on remote.

Now: marker handler prints JUST_UPGRADED but falls through to the
remote check. --force clears both cache and marker. The marker
handler also clears the cache to prevent stale cache entries from
suppressing the remote check.

Adds 4 regression tests covering --force+marker, marker+newer
remote, marker+same remote, and marker+fresh cache scenarios.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.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.

1 participant