Skip to content

refactor: move webview/ into src/anthias_webview#2896

Merged
vpetersson merged 1 commit into
masterfrom
refactor/move-webview-into-src
May 14, 2026
Merged

refactor: move webview/ into src/anthias_webview#2896
vpetersson merged 1 commit into
masterfrom
refactor/move-webview-into-src

Conversation

@vpetersson
Copy link
Copy Markdown
Contributor

Issues Fixed

N/A — repo layout cleanup.

Description

Move the Qt webview source from webview/ into src/anthias_webview/ so it sits alongside the other src/anthias_* packages. All references updated:

  • docker/Dockerfile.viewer.j2 and docker/Dockerfile.qt5-webview-builder.j2 COPY/WORKDIR paths
  • bin/rebuild_qt5_toolchain.sh WEBVIEW_DIR (in-container /webview mount target left alone — arbitrary mount name)
  • ruff.toml and pyproject.toml sysroot-relativelinks.py exclusion paths
  • CLAUDE.md, README.md, CONTRIBUTING.md, website/content/docs/developer-documentation.md, the moved README.md, Dockerfile, build_qt5.sh headers
  • Comment pointers in src/anthias_server/app/models.py, src/anthias_viewer/__init__.py, tools/image_builder/utils.py

Verified: uv run python -m tools.image_builder --dockerfiles-only --service viewer renders cleanly for both a Qt6 board (default x86) and a Qt5 board (pi3); uv run ruff check . passes.

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have done an end-to-end test for Raspberry Pi devices.
  • I have tested my changes for x86 devices.
  • I added a documentation for the changes I have made (when necessary).

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vpetersson vpetersson requested a review from a team as a code owner May 14, 2026 09:48
@vpetersson vpetersson self-assigned this May 14, 2026
@vpetersson vpetersson requested a review from Copilot May 14, 2026 09:48
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR relocates the Qt-based WebView source from the top-level webview/ directory to src/anthias_webview/, aligning it with the other src/anthias_* packages. It updates all path references in Dockerfiles, build scripts, config files (ruff/mypy excludes), comment pointers, and documentation.

Changes:

  • Moves webview/src/anthias_webview/ (no source code changes).
  • Updates Dockerfile COPY/WORKDIR paths and shell WEBVIEW_DIR to point at the new location.
  • Updates references in ruff.toml, pyproject.toml, comments in Python/C++ files, and Markdown docs (CLAUDE.md, README.md, CONTRIBUTING.md, developer docs).

Reviewed changes

Copilot reviewed 14 out of 30 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/anthias_webview/** Relocated webview package (Dockerfile, build_qt5.sh, README, .pro, sysroot script, src/, res/) with internal comment path fixes.
docker/Dockerfile.viewer.j2 Updated COPY/WORKDIR for the Qt6 in-tree webview source.
docker/Dockerfile.qt5-webview-builder.j2 Updated COPY paths and comments for the Qt5 cross-build pipeline.
bin/rebuild_qt5_toolchain.sh WEBVIEW_DIR repointed at src/anthias_webview.
ruff.toml / pyproject.toml Updated exclude paths for the vendored sysroot script.
src/anthias_server/app/models.py Updated comment pointer to view.cpp.
src/anthias_viewer/init.py Updated comment pointer to main.cpp.
tools/image_builder/utils.py Updated comment pointer to build_qt5.sh.
CLAUDE.md / README.md / CONTRIBUTING.md / website/.../developer-documentation.md Updated doc references and links.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vpetersson vpetersson merged commit fb2d790 into master May 14, 2026
14 checks passed
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.

2 participants