Skip to content

chore: drop changesets that only bump ignored packages - #99

Merged
elkaix merged 1 commit into
mainfrom
chore/drop-ignored-only-changesets
Aug 17, 2026
Merged

chore: drop changesets that only bump ignored packages#99
elkaix merged 1 commit into
mainfrom
chore/drop-ignored-only-changesets

Conversation

@elkaix

@elkaix elkaix commented Aug 17, 2026

Copy link
Copy Markdown
Member

Related Issue

No issue. The problem is described below.

Problem

The Release workflow run for ci: release packages (#98) failed:

HttpError: Validation Failed: {"resource":"PullRequest","code":"custom",
"message":"No commits between main and changeset-release/main"}

Two changeset entries stayed on main after #98 merged:

  • .changeset/general-session-mode.md — bumps @pymodel/agent-core, @pymodel/protocol
  • .changeset/model-catalog-thinking-metadata.md — bumps @pymodel/protocol, @pymodel/agent-core, @pymodel/pythinker-web

Every package they name is in the ignore list of .changeset/config.json, so changeset version writes no files. The action pushes changeset-release/main at the same commit as main and the PR creation fails.

The larger effect: because pending changesets exist, the action stays on the version-PR path and never reaches the publish path. main is at 0.20.0 while the npm latest dist-tag is 0.19.1, so 0.20.0 is unpublished and all publish-gated jobs (CDN redeploy, Homebrew tap, native assets, docs) are skipped.

What changed

Delete the two changeset files. With no pending changesets, the next push to main takes the publish path and releases 0.20.0.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works. — not applicable, this only removes release metadata.
  • Ran gen-changesets skill, or this PR needs no changeset. — this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update. — no user-facing behavior change.

Summary by CodeRabbit

  • Chores
    • Removed pending release notes for general session mode.
    • Removed pending release notes for model thinking-effort levels and adaptive-thinking metadata.

Both entries bumped only packages in the .changeset ignore list, so 'changeset version' produced an empty diff. The changesets action stayed on the version-PR path, failed with 'No commits between main and changeset-release/main', and never reached the publish path, leaving 0.20.0 unpublished.
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: ccd612d2-8d0f-47c2-ada6-1ba5e6d90df9

📥 Commits

Reviewing files that changed from the base of the PR and between 0aae708 and f10edaa.

📒 Files selected for processing (2)
  • .changeset/general-session-mode.md
  • .changeset/model-catalog-thinking-metadata.md
💤 Files with no reviewable changes (2)
  • .changeset/model-catalog-thinking-metadata.md
  • .changeset/general-session-mode.md

Included review availability: Your plan includes up to 3 reviews per rolling hour; 2 remain after this review.


📝 Walkthrough

Walkthrough

Removed two Changeset files that declared minor releases for package changes related to session mode and model thinking metadata.

Changes

Cohort / File(s) Summary
Changeset cleanup
.changeset/general-session-mode.md, .changeset/model-catalog-thinking-metadata.md
Removed two release declarations. No public entities changed.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f10ed

This change removes two ignored release entries so the release workflow can proceed normally; it is a localized metadata-only update, and no actionable merge-blocking risk remains.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the valid chore prefix, imperative mood, stays within 72 characters, and accurately describes the changeset removals.
Description check ✅ Passed The description includes the required issue, problem, changes, and checklist sections with sufficient technical context.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Aug 17, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@pymodel/pythinker-code@f10edaa
npx https://pkg.pr.new/@pymodel/pythinker-code@f10edaa

commit: f10edaa

@elkaix
elkaix merged commit 2d5691c into main Aug 17, 2026
13 checks passed
@elkaix
elkaix deleted the chore/drop-ignored-only-changesets branch August 17, 2026 02:44
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