Skip to content

chore: release the removed web flag as a minor bump - #218

Merged
elkaix merged 1 commit into
mainfrom
chore/release-1-4-0
Aug 27, 2026
Merged

chore: release the removed web flag as a minor bump#218
elkaix merged 1 commit into
mainfrom
chore/release-1-4-0

Conversation

@elkaix

@elkaix elkaix commented Aug 27, 2026

Copy link
Copy Markdown
Member

Related Issue

None — release version policy.

Problem

The pending release PR (#193) resolves to @pymodel/pythinker-code@2.0.0 because drop-allow-remote-terminals.md carries a major bump. We do not want to spend the 2.0.0 version on this change.

What changed

Changed that changeset from major to minor. The release PR regenerates from .changeset/ on main, so the next release becomes 1.4.0 instead of 2.0.0. The changelog sentence stays the same, so users still read that the flag is gone.

Note the trade-off: removing a CLI flag under a minor bump gives users no major-version signal. Anyone passing --allow-remote-terminals to pythinker web still gets a clear error, and PTY terminal routes remain available on loopback binds.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue (external PRs: the issue must have a maintainer's /approve).
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset. [skip changeset]
  • Ran gen-docs skill, or this PR needs no doc update.

Summary by CodeRabbit

  • Breaking Changes

    • Removed the --allow-remote-terminals option.
    • PTY terminal routes are now available only when bound to loopback addresses.
  • Release

    • Reclassified the package update from a major to a minor release.

@coderabbitai

coderabbitai Bot commented Aug 27, 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: a281fa48-fee9-4a6a-a639-00a1d38b14fd

📥 Commits

Reviewing files that changed from the base of the PR and between 30a4647 and c07a3fa.

📒 Files selected for processing (1)
  • .changeset/drop-allow-remote-terminals.md

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The changeset changes @pymodel/pythinker-code from a major to a minor release and documents removal of --allow-remote-terminals. PTY terminal routes remain available only on loopback binds.

Changes

Release metadata

Layer / File(s) Summary
Update package release metadata
.changeset/drop-allow-remote-terminals.md
The changeset changes the package release classification from major to minor and documents removal of remote terminal access.

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

Merge Risk: ⚪ Minimal · up to c07a3

This localized release-metadata change preserves the existing changelog message while changing the next version from 2.0.0 to 1.4.0; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the required conventional-commit prefix, stays within 72 characters, uses imperative mood, and accurately describes the release bump change.
Description check ✅ Passed The description includes all required sections, explains why the changeset changes from major to minor, describes the user impact, and records checklist status. The missing issue link is explained bec…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Description check

Explanation

The description includes all required sections, explains why the changeset changes from major to minor, describes the user impact, and records checklist status. The missing issue link is explained because this is a release-policy change.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)


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

@pkg-pr-new

pkg-pr-new Bot commented Aug 27, 2026

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

commit: c07a3fa

@elkaix
elkaix enabled auto-merge (squash) August 27, 2026 03:43
@elkaix
elkaix merged commit ff2b614 into main Aug 27, 2026
26 checks passed
@elkaix
elkaix deleted the chore/release-1-4-0 branch August 27, 2026 03: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