Release 13.2.4, so latest stops being the lowest published version - #7
Merged
Conversation
The published version was 13.2.1-fix.1, which sorts below every other version this fork has on the registry: 13.2.1-fix.1 < 13.2.1 < 13.2.2-dev.0 < 13.2.3-dev.0 So `latest` pointed at the oldest of the four, and a consumer with ^13.2.1 resolved to upstream's plain 13.2.1 -- a build with none of this fork's changes -- rather than to anything published here. That came from applying "upstream's version plus -fix.N" without noticing develop had already shipped 13.2.3-dev.0. 13.2.4 sorts above everything published so far and carries no prerelease suffix, which settles both problems and gets consumers off pinning a -dev version exactly. The library code does not change: 13.2.1-fix.1 and 13.2.3-dev.0 are byte-identical apart from README.md and package.json, so this is a renumbering, not a release of new work. The root package.json stays at 13.2.1. It is the private workspace (ng-recaptcha-base) and is never published, so its version has no consumer-visible meaning. Also records in the README a reason this fork exists that was not listed: provideRecaptchaV3Services and provideEnvironmentRecaptchaV3Services do not exist upstream, which exports only RecaptchaV3Module -- verified against the published ng-recaptcha@13.2.1 tarball. Without that line the list implies the fork is only an npm scope and an Angular bump, and so disposable. CHANGELOG.md is reformatted only inside the new entry; README.md was already failing prettier before this change and is left alone apart from the added line. Generated-by: Claude Opus 5 (Anthropic)
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
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.
The published version is
13.2.1-fix.1, which sorts below every other version this fork has on the registry:So
latestpoints at the oldest of the four, and a consumer with^13.2.1resolves to upstream's plain13.2.1— a build with none of this fork's changes — rather than to anything published here.ngx-apponly escapes that by pinning13.2.3-dev.0exactly.The cause was applying the "upstream's version plus
-fix.N" convention without noticingdevelophad already shipped13.2.3-dev.0. Worth a caveat in that convention: the suffix has to sort above everything already published, not just above upstream.What this changes
projects/ng-recaptcha/package.json→13.2.4. It sorts above everything published so far, carries no prerelease suffix, and lets consumers stop pinning a-devversion exactly.No library code changes.
13.2.1-fix.1and13.2.3-dev.0are byte-identical apart fromREADME.mdandpackage.json— I unpacked both tarballs and compared all 33 files — so this is a renumbering, not a release of new work. The two-devversions are left on the registry; nothing should be depending on them.The root
package.jsonstays at13.2.1: it is the private workspace (ng-recaptcha-base), never published, so its version carries no consumer-visible meaning.One thing the README was missing
The "Changes from upstream" list said the fork exists for the npm scope, Angular 18 and the publish workflow — which reads as disposable. It omitted the actual API difference:
provideRecaptchaV3ServicesandprovideEnvironmentRecaptchaV3Servicesdo not exist upstream, which exports onlyRecaptchaV3Module. Verified against the publishedng-recaptcha@13.2.1tarball.ngx-app's login page uses the first of those, so dropping the fork would break it.Added as a line rather than a rewrite, per the instruction to keep that list current.
After this merges
The publish job will put
13.2.4onlatest. Two follow-ups that need registry access and so cannot be done from here:nextstill points at13.2.2-dev.0and will stay there — a develop push now carries the same version as master, so the guard correctly skips it. Moving it needsnpm dist-tag add @herdwatch/ng-recaptcha@13.2.4 next.13.2.3-dev.0to13.2.4.prettier --checkpasses on the changelog and manifest.README.mdwas already failing prettier before this change and is left alone apart from the added line, so the one-line addition stays readable in the diff.