wehub-resource-sync
b4fbd6fe9f
Deploy Site / deploy-vercel (push) Has been skipped
Deploy Site / deploy-docs (push) Has been skipped
Build Skills Index / build-index (push) Has been skipped
CI / Deny unrelated histories (push) Has been skipped
CI / Detect affected areas (push) Successful in 27m35s
CI / OSV scan (push) Failing after 4s
CI / Build&Test Docker image (push) Successful in 9s
CI / Supply-chain scan (push) Has been skipped
CI / Lint Docker scripts (push) Failing after 5m13s
CI / Check contributors (push) Failing after 12m8s
CI / Docs Site (push) Failing after 12m8s
CI / TypeScript (push) Failing after 12m8s
CI / Python lints (push) Failing after 12m9s
CI / Python tests (push) Failing after 12m9s
CI / Check uv.lock (push) Failing after 23m22s
CI / CI timing report (push) Has been cancelled
Build Skills Index / trigger-deploy (push) Has been cancelled
CI / All required checks pass (push) Has been cancelled
23 lines
872 B
Python
23 lines
872 B
Python
"""Contract test: Docker TUI must not require writable node_modules.
|
|
|
|
Older images made /opt/hermes/ui-tui and /opt/hermes/node_modules writable so a
|
|
runtime npm install could repair stale dependencies. The hosted install tree is
|
|
now immutable, so the Docker image must take the prebuilt TUI bundle path
|
|
instead of writing to node_modules at runtime.
|
|
"""
|
|
from __future__ import annotations
|
|
|
|
from pathlib import Path
|
|
|
|
REPO_ROOT = Path(__file__).resolve().parents[2]
|
|
DOCKERFILE = REPO_ROOT / "Dockerfile"
|
|
|
|
|
|
def test_dockerfile_uses_prebuilt_tui_instead_of_writable_node_modules() -> None:
|
|
text = DOCKERFILE.read_text()
|
|
|
|
assert "ENV HERMES_TUI_DIR=/opt/hermes/ui-tui" in text
|
|
assert "cd ../ui-tui && npm run build" in text
|
|
assert "chown -R hermes:hermes /opt/hermes/ui-tui" not in text
|
|
assert "chown -R hermes:hermes /opt/hermes/node_modules" not in text
|