docs(changelog): credit Control UI fixes#100740
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ea4b6ab6d7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - **Control UI agent model labels:** show each selected agent's effective model in the Default picker option instead of the global model. (#100719, #77690, #77440) Thanks @hyspacex. | ||
| - **Control UI inbound image previews:** render canonical inbound media references through the authenticated ticket route after chat-history reloads. (#100725, #90172, #89591) Thanks @sweetcornna. |
There was a problem hiding this comment.
Remove changelog entries for unmerged fixes
When publishing a release from this tree, these two new entries advertise #100719 and #100725 as shipped Control UI fixes even though GitHub currently marks both PRs open and their referenced head commits are not present in the reviewed history. That would produce release notes claiming unavailable behavior and contributor credit before the runtime fixes actually land; please add these entries only with, or after, the corresponding fixes are merged.
Useful? React with 👍 / 👎.
f43e080 to
403b941
Compare
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: docs(changelog): credit Control UI fixes This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
|
Land-ready maintainer closeout for exact head
Known proof gaps: none for this changelog-only follow-up. |
|
Merged via squash.
|
What Problem This Solves
The clean landing branches for #100719 and #100725 intentionally exclude
CHANGELOG.mdso their already-proven runtime patches do not repeatedly conflict with OpenClaw's high-traffic release-notes hunk. Their user-visible fixes and contributor credit still need release-note coverage.Why This Change Was Made
Keep the two code PRs as one focused runtime commit each, then own their changelog entries in this maintainer-only closeout. This preserves release notes and contributor attribution without coupling exact-head UI CI to unrelated concurrent changelog edits.
User Impact
No runtime change. The next release notes describe the corrected per-agent default-model label and canonical inbound-image preview behavior.
Evidence
git diff --checkpassed.