Skip to content

Reject overlapping same-owner deployment claims - #506

Open
Kewe63 wants to merge 2 commits into
github:mainfrom
Kewe63:fix/504-reentrant-nonsticky-lock
Open

Kewe63 wants to merge 2 commits into
github:mainfrom
Kewe63:fix/504-reentrant-nonsticky-lock

Conversation

@Kewe63

@Kewe63 Kewe63 commented Sep 19, 2026

Copy link
Copy Markdown

Summary

  • reject a distinct non-sticky deployment claim while an earlier claim from the same actor is active
  • keep exact-claim reruns idempotent and preserve legacy lock behavior when no claim ID exists
  • regenerate the committed action bundle

Testing

  • regression test failed against the previous behavior and passes with the fix
  • npm run all
  • npm run acceptance (114 scenarios)

Compatibility

Sticky locks and legacy lock records retain their existing owner behavior. The behavior change is limited to distinct overlapping non-sticky claims that would otherwise share cleanup ownership.

Fixes #504

@Kewe63

Kewe63 commented Oct 4, 2026

Copy link
Copy Markdown
Author

@GrantBirki I merged current main into this branch without rewriting history, regenerated the committed bundle from the combined source, and reran the repository validation on Node 24.18.0: npm run all passed (927 tests, 100% line/branch/function coverage) and npm run acceptance passed all 114 scenarios. The PR is conflict-free again at 0798df5. Could you please take another look?

@Kewe63
Kewe63 requested a review from GrantBirki October 4, 2026 21:28

This branch has not been deployed

No deployments
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.

Reentrant non-sticky deployments can release their shared lock before all runs finish

2 participants