Skip to content

chore: rewrite increment-version in typescript with an auto-commit op… - #1533

Merged
joshunrau merged 4 commits into
DouglasNeuroInformatics:mainfrom
joshunrau:devel
Sep 10, 2026
Merged

chore: rewrite increment-version in typescript with an auto-commit op…#1533
joshunrau merged 4 commits into
DouglasNeuroInformatics:mainfrom
joshunrau:devel

Conversation

@joshunrau

Copy link
Copy Markdown
Collaborator

adjust increment-version script and release new version

joshunrau and others added 4 commits September 10, 2026 12:03
…tion

Replaces scripts/increment-version.sh with scripts/increment-version.ts, run
directly by node with no dependencies beyond the standard library.

- `--commit` makes the `chore: release v<version>` commit itself, by explicit
  pathspec so unrelated staged work stays staged. It commits only; the tag
  remains the release workflow's.
- `--bump` and `--yes` replace piping answers into the bash `select` prompt,
  and are required when stdin is not a terminal.
- The version fields are re-read after the rewrite and asserted equal, so the
  root-only bump that publishes nothing while CI stays green fails here.
- Output is styled through node:util styleText, which degrades to plain text
  under NO_COLOR or when piped.

The version field is still rewritten textually rather than through a JSON
round-trip, and list-publishable.sh remains the only source of truth for what
is publishable. Every in-repo reference to the old path is updated.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Cz7sSmTSL4tgT6wd3ZpUzi
The commit was reachable only through --commit. It is now a y/N prompt in the
interactive run, with --commit as its non-interactive answer, the way --yes
answers the confirmation; --yes on its own leaves it at its default of no.

The prompt sequence moves into planRelease, which returns the bump and the
commit choice together, so main no longer carries a half-assigned newVersion
across a try/finally.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Cz7sSmTSL4tgT6wd3ZpUzi
@joshunrau
joshunrau merged commit f97706e into DouglasNeuroInformatics:main Sep 10, 2026
2 checks passed
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