Skip to content

fix(version): untagged-package warning matches resolved changelog range - #749

Merged
goosewobbler merged 5 commits into
mainfrom
fix/727-untagged-changelog-warning
Oct 24, 2026
Merged

goosewobbler merged 5 commits into
mainfrom
fix/727-untagged-changelog-warning

Conversation

@goosewobbler

@goosewobbler goosewobbler commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner

Description

The warning for a package with no tag of its own always said its changelog "will include the full git history". That's only true when no tag is reachable at all. Otherwise BaselineResolver starts the range at the nearest reachable tag, so in a tagged monorepo the warning overstated the problem.

The warning now runs after the range is resolved, and describes it:

  • No reachable tag (range HEAD): "its changelog will include the full git history."
  • A reachable tag (range <tag>..HEAD): "its changelog will start from <tag>."

Both still suggest the git tag command that sets a baseline. Runs with a baseRef, such as PR previews, still skip the warning.

Related Issue(s)

Closes #727.

Type of Change

  • feat — New feature
  • fix — Bug fix
  • docs — Documentation change
  • refactor — Code refactoring
  • test — Adding or updating tests
  • chore — Maintenance, dependencies, tooling
  • Breaking change — Requires major version bump

Scope

  • @releasekit/version
  • @releasekit/notes
  • @releasekit/publish
  • @releasekit/release
  • @releasekit/config
  • @releasekit/core
  • CI / workflows
  • Root / monorepo

Checklist

  • Lint passes (pnpm lint)
  • Tests pass (pnpm test)
  • Typecheck passes (pnpm typecheck)
  • Documentation updated (if applicable): none needed; the warning text is the only change
  • Breaking changes noted in description: none

Testing Notes

  • processPackages tests cover both messages: full history when no tag is reachable, and start from pkg-b@v1.0.0 when one is. The second fails if the message ignores the resolved range.
  • @releasekit/version typecheck, unit tests (747) and integration tests pass, and pnpm lint passes.

Made with Cursor

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge.

Summary

Updates the untagged-package warning to describe the resolved changelog range.

  • Untagged-package warnings now name the changelog's actual starting point.

No actionable issues found. Tests were inspected, not run.

Reviews (1) · Last reviewed commit: "fix(version): untagged-package warning m..." · Reviewed by Greptile

Fixes #727

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Standing release PR: #730 · v0.42.1 queued (4 packages) · open 6h 22m · ✅ ready to merge

Release Preview — v0.42.0 → v0.42.1

Note: Labels on this PR are advisory in standing-pr mode. Bumps come from conventional commits in the standing PR; override by editing labels on the standing PR itself. Add release:immediate to bypass the standing PR and release this PR directly.

These changes will be added to the release PR (#730) when merged:

Changelog

monorepo 0.42.0 → 0.42.1

Changed

  • vary only the reachable tag between the two warning tests (version)
  • drop the baseRef comment on the untagged-package warning (version)
  • say why the untagged-package warning skips baseRef runs (version)
  • cover jsonOutput when disabled; fix inverted guard test (version) (PR #754 · closes #725)
  • drop versionCalculator tests that mocked a missing helper (version) (PR #755 · closes #726)
  • link standing-pr.yml comments to published ci-setup URLs (templates) (PR #756 · closes #728)
  • build the untagged-package warning in one call (version)

Fixed

  • commit the standing PR's release branch with the PR title (release) (PR #729)
  • keep root changelog history when several packages release (notes) (PR #775 · closes #736)
  • untagged-package warning matches resolved changelog range (version) (#727)

All packages → 0.42.1 (sync versioning)

  • @releasekit/version
  • @releasekit/notes
  • @releasekit/publish
  • @releasekit/release

After merge — predicted release

No version escalation — this PR's changes will be included in the queued release without affecting the projected versions.

Package Standing PR This PR After merge
monorepo 0.42.1 0.42.1 0.42.1

Updated automatically by ReleaseKit

github-actions Bot and others added 4 commits October 23, 2026 22:17
The resolved range is always `HEAD` or `<ref>..HEAD`, so the two branches
differ only in the phrase naming the range. Restores the comment on why
the warning exists, which moved with it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… runs

The old comment cited a 422 from an oversized standing-PR body, but the
body has been truncated below GitHub's limit since the warning was added.
The full-history test now sets the no-reachable-tag case explicitly.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…arning

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
With a manifest version, the processor never calls getLatestTag, so the
global tag in the nearest-tag test had no effect.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

version: untagged-package warning claims full git history when the changelog is bounded by the nearest tag

1 participant