chore: sync dev to main for v1.0.39 - #518
Merged
Merged
Conversation
chore: sync main to dev after v1.0.37 notes fix
The 200ms bound around a 40ms connectTimeout flaked at 288ms on a loaded linux runner (raw setTimeout, no TestClock possible). 2000ms still separates fail-fast from hang — a hang trips the test timeout instead — and the Stalled error-class assertion remains the semantic check.
Closes the dependabot high advisory (alert 110): mysql2 < 3.22.0 leaks plaintext credentials on an auth-plugin downgrade to mysql_clear_password.
The RouteProbe seam test mirrors the production subscription shape, so removing an onCleanup wrapper from the real Session/Prompt routes left it green. Mount the actual route components inside the production provider stack and assert the SDK event bus returns to its baseline listener count after unmount; verified red when either wrapper is removed. The two bridge lint suppressions carry reasons (restoration point; overloaded on widened through bind).
win32 taskkill /T /F vs negative-pid kill existed as diverging copies in util/process.stop and the hook execShell killGroup. Extract killGroupPid (POSIX group signal; win32 awaits the taskkill exit code and throws, leaving fallback and logging to callers) and reuse it in both. The SDK sync copy stays as documented in #503. No behavior change: same signals, ordering, and grace constants; the #500 process-group regression suite passes.
…ening # Conflicts: # .specgit.yaml
test: make the memory stall timing assertion load-tolerant
LeXwDeX
marked this pull request as ready for review
September 2, 2026 14:10
This was referenced Sep 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #517
Why
Summarize the problem or need this delivery addresses.
What changed
Evidence
Checklist
specgit finishexits 0.