chore(changeset): trim the route-groups changeset to one line - #4143
Conversation
The changeset body is copied verbatim into the changelog, where the rationale paragraphs from #4070 are noise. Keep the line that says what changed and where it matters; the reasoning is already in the commit, the PR and issue #3991. This was reviewed on #4070 and fixed on #4140, a CI mirror of that PR. #4070 is the one that merged and #4140 was closed, so the fix never landed. The changeset is still unconsumed on `main`, so correcting the file is enough — the open release PR regenerates from it and no CHANGELOG.md has been written yet. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Pranay Prakash <1797812+pranaygp@users.noreply.github.com> Co-Authored-By: Pranay Prakash <1797812+pranaygp@users.noreply.github.com>
|
| Name | Type |
|---|---|
| @workflow/core | Patch |
| @workflow/builders | Patch |
| @workflow/cli | Patch |
| @workflow/next | Patch |
| @workflow/nitro | Patch |
| @workflow/vitest | Patch |
| @workflow/web-shared | Patch |
| @workflow/web | Patch |
| workflow | Patch |
| @workflow/world-testing | Patch |
| @workflow/astro | Patch |
| @workflow/nest | Patch |
| @workflow/rollup | Patch |
| @workflow/sveltekit | Patch |
| @workflow/vite | Patch |
| @workflow/nuxt | Patch |
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Sim WorldSimulated world deterministic testing for races. Traces 🟠 world-sim scenario book — 1 fail of 41 total
Full trace: |
There was a problem hiding this comment.
🟢 Approval recommended
The reviewed change only shortens the unconsumed changeset entry and has no unresolved review comments.
Pull request overview
This PR trims the pending @workflow/core changeset to a concise one-line changelog entry without code changes.
Changes:
- Replaces the verbose explanation with a single sentence.
- Preserves package and patch metadata.
File summaries
| File | Description |
|---|---|
.changeset/workflow-name-nextjs-route-segments.md |
Concise description of the workflow-name fix |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
No backport to This commit touches only To override, re-run the Backport to stable workflow manually via |
Follow-up to #4070 (merged as d427c47) — no code change, one changeset file.
What happened
You left this review comment on #4070's changeset: "too verbose. let's just have the 1 line here since this will go into changelog and we don't need all the paragraphs below."
I fixed it on #4140, the CI mirror of #4070 — but #4070 is the one that merged and #4140 was closed, so the trim went with it. Nothing went wrong in the merge; the fix was just sitting on the wrong PR. (Good outcome otherwise: merging #4070 means @torsello kept authorship.)
Why the changeset file is still the right place to fix it
The changeset has not been consumed yet — it's still at
.changeset/workflow-name-nextjs-route-segments.mdonmain, andpackages/core/CHANGELOG.mdcontains none of this text. The open release PR #4142 has it staged (renames to.changeset/pre/…and writes the long version into the changelog), and it regenerates on every push tomain, so landing this will refresh #4142 with the one-liner. No changelog surgery needed, and nothing has been published with the verbose text.Before / after
Also lost with #4140
The
world-verceltest from #4140 didn't land either — the one pinning that a workflow name still reaches VQS as a valid topic ([A-Za-z0-9_-]) afterprepareSendfolds it. That guarantee is what makes the parens/brackets fix safe on the Vercel world, and it's still untested. Deliberately not included here to keep this mergeable on sight; happy to send it as its own PR.PR Checklist - Required to merge
choreon an unconsumed changeset produces no release of its own)@vercel/workflow🤖 Generated with Claude Code