Skip to content

Bump Open WebUI subpath build to v0.12.0 + fix port-owner classification - #51

Merged
SushantGautam merged 2 commits into
mainfrom
agents/https-github-com-sushantgautam-open-webui-has-ne
Oct 14, 2026
Merged

SushantGautam merged 2 commits into
mainfrom
agents/https-github-com-sushantgautam-open-webui-has-ne

Conversation

@SushantGautam

Copy link
Copy Markdown
Collaborator

Summary

Bumps SimpleAudit Studio's Open WebUI subpath integration to v0.12.0 and fixes a latent port-owner classification bug the full test run surfaced.

1. Open WebUI → v0.12.0 (the ask)

The port + GHCR publish were already completed in the SushantGautam/open-webui fork (tag v0.12.0-subpath, functional head 92f0108 == main). This wires it into Studio:

  • docker-compose.yml: OPEN_WEBUI_IMAGE default → ghcr.io/sushantgautam/open-webui:v0.12.0-subpath
  • build.sh: pinned SHA → 15818d88 (pristine v0.12.0)
  • subpath.patch: regenerated as 15818d88..92f0108 excluding .github/+FORK.md (198 files); old patch kept as subpath-v0.11.4.patch
  • README.md / UPGRADE.md: current base + outcome log updated

2. Port-owner classification fix (surfaced by full test run)

_classify_command substring-matched the whole command line against ('open-webui', 'open_webui'). A checkout whose path contains open-webui (this worktree) made every listener process classify as Studio's own Open WebUI — a force-kill hazard, and the cause of 3 test_ports failures. Now matches the underscore spelling only (uvicorn open_webui.main:app is what a running instance carries), so real detection is unchanged while the path-based false positive is removed. Mirrors the existing documented rationale for the Studio marker.

Verification

  • Full suite uv run pytest -m "not embedded_hatchet": 1120 passed, 2 skipped (expected env-specific), 0 failed
  • uv run ruff check .: clean
  • Chat module: 104/104
  • GHCR v0.12.0-subpath multi-arch image (amd64 + arm64) confirmed present via docker manifest inspect
  • All three patches pass git apply --check on pristine v0.12.0

Not done (non-blocking): live 7-point browser E2E of /chat/ in a running container — the subpath code is unchanged from what was already published/E2E-verified for v0.12.0.

SushantGautam and others added 2 commits October 14, 2026 16:09
The port + GHCR publish were already done in the SushantGautam/open-webui
fork (tag v0.12.0-subpath, functional head 92f0108 == main). This wires it
into Studio:

- docker-compose.yml: OPEN_WEBUI_IMAGE default -> :v0.12.0-subpath
- build.sh: pinned SHA -> 15818d88 (pristine v0.12.0)
- subpath.patch: regenerated as 15818d88..92f0108 EXCLUDING .github/FORK.md
  (198 files); git apply --check clean on pristine alongside the node-heap
  and otel patches. Old patch kept as subpath-v0.11.4.patch.
- README.md / UPGRADE.md: current base + outcome log updated

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
_classify_command substring-matched the whole command line against
the open-webui and open_webui spellings. A checkout whose path contains
open-webui (e.g. this worktree) made every listener process classify as
Studio's own Open WebUI - a force-kill hazard and the cause of 3
test_ports failures.

Match the underscore spelling only: that is what a running instance
carries (uvicorn open_webui.main:app), so real detection is unchanged
while the path-based false positive is removed. Mirrors the existing note
for the Studio marker. Test now asserts the underscore form a real
process uses.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@SushantGautam
SushantGautam merged commit 8e35ed1 into main Oct 14, 2026
2 checks passed
@SushantGautam
SushantGautam deleted the agents/https-github-com-sushantgautam-open-webui-has-ne branch October 14, 2026 14:15
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