Skip to content

Bump strnum to 2.4.2 - #64

Merged
ww-mw merged 1 commit into
mainfrom
bump-strnum
Oct 3, 2026
Merged

ww-mw merged 1 commit into
mainfrom
bump-strnum

Conversation

@ww-mw

@ww-mw ww-mw commented Oct 3, 2026

Copy link
Copy Markdown
Member

Lockfile-only patch bump of a direct runtime dependency. 2.4.2 is in range of the existing "^2.4.1", so there is no package.json edit and no overrides entry — the same shape as the earlier brace-expansion work.

resolved rewritten back to registry.npmjs.org after the install (the configured registry here writes an internal host); integrity is content-based and unaffected. npm run check:leak and npm run typecheck clean.

Note for the record: npm test on this Mac fails 9 tests in test/mcosParser.test.ts and test/publicTypeSurface.test.ts under full-suite load — identically on clean main with this change stashed, and both files pass in isolation with the change applied. Pre-existing local flake, not this bump.

A direct runtime dependency: the number coercion in DictionaryXmlFast is
strnum's, declared beside fast-xml-parser because the fast reader calls it
with the same options fxp would. 2.4.2 is in range of the existing
"^2.4.1", so this is the lockfile alone — no package.json edit and no
overrides entry, the same shape as the brace-expansion work.

The resolved URL was rewritten back to registry.npmjs.org after the
install, which the configured registry here does not do on its own; the
integrity hash is content-based and unaffected. npm run check:leak clean.
@ww-mw
ww-mw merged commit 16d8edc into main Oct 3, 2026
4 checks passed
@ww-mw
ww-mw deleted the bump-strnum branch October 3, 2026 07:13
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