Skip to content

Ship prebuilt Docker deployment - #14

Draft
ClePol wants to merge 29 commits into
mainfrom
monolith_refactor
Draft

Ship prebuilt Docker deployment#14
ClePol wants to merge 29 commits into
mainfrom
monolith_refactor

Conversation

@ClePol

@ClePol ClePol commented Jul 23, 2026

Copy link
Copy Markdown
Member

Summary

  • ship the monolithic application as a prebuilt GHCR Docker image
  • simplify Linux installation and startup to Docker-only shell scripts
  • support Apple Silicon through explicit linux/amd64 emulation
  • move frontend deployment settings to runtime configuration
  • remove default FreeSurfer license handling and temporarily hide manual labeling
  • update vulnerable Python and npm dependencies
  • make npm and Python vulnerability scans blocking
  • configure Dependabot for the root uv project

Verification

  • npm run lint
  • npm run test:node (63 passed)
  • npm run build
  • .venv/bin/pyright
  • .venv/bin/pytest tests/ -v (278 passed, 26 skipped)
  • npm audit --audit-level=low (0 vulnerabilities)
  • uvx pip-audit --path .venv/lib/python3.12/site-packages (0 known vulnerabilities)
  • production Docker image build
  • container health check at http://localhost:8000

Notes

  • NIIVUE_REPORT.md is intentionally excluded.
  • Dependabot alerts on the default branch will close after this PR is merged.

Copilot AI review requested due to automatic review settings July 23, 2026 22:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Comment thread api-service/api_service/main.py Fixed
Comment thread api-service/api_service/main.py Fixed
Copilot AI review requested due to automatic review settings July 23, 2026 23:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings July 23, 2026 23:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings July 23, 2026 23:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Comment thread api-service/api_service/runtime_tools/handlers.py Fixed
Copilot AI review requested due to automatic review settings July 23, 2026 23:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings July 24, 2026 12:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings July 28, 2026 09:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

try {
return await creation;
} finally {
if (state.creation === creation) state.creation = undefined;
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