Skip to content

sync: upstream v0.51.231 (needs review)#49

Merged
Du7chManiac merged 2 commits into
masterfrom
sync/upstream-v0.51.231
Jun 5, 2026
Merged

sync: upstream v0.51.231 (needs review)#49
Du7chManiac merged 2 commits into
masterfrom
sync/upstream-v0.51.231

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions Bot commented Jun 4, 2026

Auto-generated by .github/workflows/sync-upstream.yml.

Merging upstream tag v0.51.231 from nesquena/hermes-webui into master.

Needs human review.

  • Conflicts in: CHANGELOG.md

See CLAUDE.mdSyncing with upstream for the conflict playbook.

Validation

  • Merge: conflict
  • Tests: skipped (0 = pass; skipped = not run because merge had conflicts)

Next step

  • Squash-merge is forbidden — keep the merge commit so future syncs see the correct merge-base.
  • After merge, the next workflow run will pick up the tag after v0.51.231.

## Release v0.51.231 — Release GY (stage-q1)

Four independent, low-risk fixes batched into one release. All gates green.

### Fixes
| PR | Author | Fix |
|----|--------|-----|
| nesquena#3368 | @nesquena-hermes | `/model` resolves models from the truncated `extra_models` tail of a large provider catalog (completes the half-done nesquena#3368 fix; @garyd9's `mimo-v2.5` case) |
| nesquena#3457 | @pix0127 | Auto-hide the Plugins settings tab when no plugins are installed; deep-link fallback to Conversation |
| nesquena#3474 | @Mubashirrrr | Guard malformed/negative `depth` on `/api/sessions/search` (no more 500 on `?depth=deep`, no silent newest-message drop on negative) |
| nesquena#3433 | @sjh9714 | Resolve symlinked home before the trust comparison so `~/` workspace suggestions work when the process home is a symlink |

### Note on nesquena#3457
The original contributor patch reassigned a `const section` (a **runtime `TypeError` brick** on the settings panel) and placed the hidden-tab fallback *after* the panes were already toggled. Refactored to `let` + hoist the fallback above all consumers of `section`. ESLint runtime gate now clean.

### Gate results
- **Full pytest suite**: 7441 passed, 7 skipped, 3 xpassed, **0 failed**
- **ESLint runtime gate**: CLEAN (caught + fixed the nesquena#3457 const-reassign)
- **ruff forward gate**: CLEAN (no new violations on changed lines)
- **browser-smoke gate**: CLEAN (`/`, `/#settings`, `/#sessions` — zero console errors)
- **Codex (regression)**: SAFE TO SHIP
- **Opus (correctness)**: SAFE TO SHIP

Closes nesquena#3368. Closes nesquena#3457. Closes nesquena#3433.

Co-authored-by: pix0127 <pix0127@users.noreply.github.com>
Co-authored-by: Mubashirrrr <Mubashirrrr@users.noreply.github.com>
Co-authored-by: sjh9714 <sjh9714@users.noreply.github.com>
@github-actions github-actions Bot added the sync-upstream PRs from .github/workflows/sync-upstream.yml label Jun 4, 2026
@Du7chManiac Du7chManiac force-pushed the sync/upstream-v0.51.231 branch from d0ef796 to 6ef0fd0 Compare June 5, 2026 06:08
@Du7chManiac Du7chManiac marked this pull request as ready for review June 5, 2026 06:09
Copy link
Copy Markdown

Babysitter resolved — ready to merge.

What happened

The workflow had already merged v0.51.252 (PR #48) before v0.51.231 was discovered as un-merged. Merge-base between master and v0.51.231 is the v0.51.230 commit, so git's "their" diff (v0.51.230→v0.51.231) is a strict subset of what master already carries.

Conflict resolved

  • CHANGELOG.md — only conflict. Union/additive: kept our fork's ## [Unreleased] section with the sync-upstream.yml workflow-fix note, kept all the v0.51.252–v0.51.232 entries from HEAD, then let v0.51.231's own entry (already outside the conflict block) follow naturally.

Validation

  • Full suite: 7593 passed, 44 failed, 102 skipped — all 44 failures are pre-existing in master (git commit-signing server unavailable in CI environment + permission-test sandbox quirks), confirmed by running the same failing tests against master directly.
  • No new failures introduced by this merge.
  • Files changed by v0.51.231 (api/routes.py, api/workspace.py, static/commands.js, static/panels.js, tests) merged cleanly with no conflicts.

Force-pushed the clean merge commit to replace the conflict-markers commit.


Generated by Claude Code

@Du7chManiac Du7chManiac merged commit 04f4e80 into master Jun 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sync-upstream PRs from .github/workflows/sync-upstream.yml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants