Skip to content

[pull] master from daquinoaldo:master - #13

Merged
pull[bot] merged 6 commits into
MelnikovAG:masterfrom
daquinoaldo:master
Sep 6, 2026
Merged

pull[bot] merged 6 commits into
MelnikovAG:masterfrom
daquinoaldo:master

Conversation

@pull

@pull pull Bot commented Sep 6, 2026 •

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

daquinoaldo and others added 6 commits September 6, 2026 15:40
# Pull Request Details

## Related Issue
Addresses #77.

Importing `https-localhost` currently installs a process-wide
`uncaughtException` handler even when the package is only used as a
module. That changes application-level error handling and can hide the
original stack-trace behavior.

This change scopes the existing handler to the CLI path (`require.main
=== module`) so the CLI keeps its friendly `EACCES` / `EADDRINUSE`
messages while library consumers no longer get a global process listener
as a side effect of `require("https-localhost")`.

A regression test re-imports the module after clearing the require cache
and verifies that the `uncaughtException` listener count does not
change.

## Types of changes
- [ ] Docs change / refactoring / dependency upgrade
- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)

## Checklist
- [x] My code follows the code style of this project.
- [x] I have updated the documentation or my changes dont require it.
- [x] I have read the CONTRIBUTING document.
- [x] I have added tests to cover my changes.
- [ ] All new and existing tests passed.
Configure the release workflow for npm Trusted Publishing via GitHub
OIDC. Supports publishing the already-tagged v4.7.2 without creating
another version bump.
Bump release runner to Node 24 for npm Trusted Publishing OIDC support,
and consolidate the release action into a single `upgrade` input with
options: `patch`, `minor`, `major`, and `existing` (publish without
re-bumping).
@pull pull Bot locked and limited conversation to collaborators Sep 6, 2026
@pull pull Bot added the ⤵️ pull label Sep 6, 2026
@pull
pull Bot merged commit dd59376 into MelnikovAG:master Sep 6, 2026
1 check was pending
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants