Skip to content

Fix request handler and routing state isolation - #76

Merged
dkropachev merged 6 commits into
mainfrom
fix/repository-review-findings
Sep 9, 2026
Merged

dkropachev merged 6 commits into
mainfrom
fix/repository-review-findings

Conversation

@dkropachev

Copy link
Copy Markdown
Collaborator

Summary

  • disable AWS middleware handler caching so mutable retry/query-plan state remains request-scoped
  • distinguish handler instances from handler option objects and merge Node/Fetch options with Alternator connection settings
  • recognize FetchHttpHandler instances across duplicate package installations to avoid decoding compressed responses twice
  • keep foreground discovery deadline timers referenced so awaited refreshes always settle
  • correct the README TLS example to use the documented HTTPS scheme and port

Review validation

Four reported runtime defects were reproduced before applying these fixes and now have regression coverage. The reported claim that TLS options should automatically select HTTPS was not treated as a runtime defect: the public API and documentation define scheme as the transport selector and tls as HTTPS-agent configuration. The contradictory options example was corrected instead.

Verification

  • npm run typecheck
  • npm run lint
  • npm test (82 tests)
  • npm run build

@dkropachev
dkropachev force-pushed the fix/repository-review-findings branch from 7d18e9f to 65a629d Compare September 9, 2026 14:55
@dkropachev
dkropachev merged commit 5304ec7 into main Sep 9, 2026
4 checks passed
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