fix: retarget orphan changeset from manifest-model-router to manifest#1546
Merged
brunobuddy merged 1 commit intomainfrom Apr 13, 2026
Merged
fix: retarget orphan changeset from manifest-model-router to manifest#1546brunobuddy merged 1 commit intomainfrom
brunobuddy merged 1 commit intomainfrom
Conversation
The release workflow was failing again with:
🦋 error Found changeset remove-local-mode for package
manifest-model-router which is not in the workspace
.changeset/remove-local-mode.md was added in #1539 but targets
manifest-model-router, which was deleted in #1528. Changesets
fast-fails on any dangling package reference.
The changeset content itself is genuinely useful — it documents the
local-mode removal, the Docker hardening, the new AUTO_MIGRATE env
var, and the unified EMAIL_* scheme. Rather than deleting it (which
is what #1534 did for a similar situation), retarget it to 'manifest'
so the content lands in packages/manifest/CHANGELOG.md on the next
release.
Also drop the now-stale final sentence that described
manifest-model-router as 'the recommended way to route OpenClaw
requests through Manifest' — that package no longer exists. Replaced
with a correct note pointing users at configuring Manifest as a
generic OpenAI-compatible provider (which is what the dashboard
setup modal shows them).
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1546 +/- ##
=======================================
Coverage 97.93% 97.93%
=======================================
Files 118 118
Lines 8581 8581
Branches 3230 3230
=======================================
Hits 8404 8404
Misses 175 175
Partials 2 2
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Release workflow failing on the merge of #1545:
```
🦋 error Found changeset remove-local-mode for package manifest-model-router which is not in the workspace
```
`.changeset/remove-local-mode.md` was added in #1539 (`chore/remove-local-mode`) but targets `"manifest-model-router": patch`. That package was deleted in #1528, so changesets fast-fails on the dangling reference. Same class of problem as #1534.
Fix
Unlike #1534 where I deleted the orphans outright, this one has genuinely useful changelog content — it documents the local-mode removal, the Docker hardening, the new `AUTO_MIGRATE` env var, and the unified `EMAIL_*` scheme. Those are real user-facing changes and they belong in the `manifest` CHANGELOG.
So: retarget the changeset from `manifest-model-router` to `manifest` (the canonical version from #1541). Keep the `patch` bump type — if you want `minor` or `major` instead, that's a one-line follow-up, but I don't want to make that call for you.
Also dropped the final sentence of the body, which described `manifest-model-router` as "the recommended way to route OpenClaw requests through Manifest" — that package no longer exists. Replaced with a correct note pointing users at the dashboard's setup modal (generic OpenAI-compatible provider config).
Verified locally
```
$ npx changeset status --verbose
🦋 info Packages to be bumped at patch
🦋 - manifest 5.46.1
🦋 - .changeset/remove-local-mode.md
```
Clean resolve. Release workflow should succeed on the merge.
Test plan
Follow-up thought
After this lands and the version-packages PR is merged, it might be worth a quick audit to make sure no other PRs in flight have orphan changesets. The pattern is always the same: a PR branched off main before #1528 landed adds a changeset targeting a now-deleted package. A one-liner `grep -rl '"manifest-model-router"' .changeset/` on any release-blocking branch would catch this early.
Summary by cubic
Retargeted
.changeset/remove-local-mode.mdfrommanifest-model-routertomanifestto unblock the release workflow and put the changelog in the right package. Updated the final note to remove the deleted plugin and direct OpenClaw users to configure Manifest as a generic OpenAI-compatible provider.Written for commit 5344510. Summary will update on new commits.