Skip to content

fix(docker): add missing service-worker.js into built container image#39596

Open
hainenber wants to merge 3 commits intomasterfrom
fix/add-missing-service-worker-file-to-built-container-image
Open

fix(docker): add missing service-worker.js into built container image#39596
hainenber wants to merge 3 commits intomasterfrom
fix/add-missing-service-worker-file-to-built-container-image

Conversation

@hainenber
Copy link
Copy Markdown
Contributor

fix(docker): add missing service-worker.js into built container image

SUMMARY

Fixes #39431

Add a COPY clause to copy missing service-worker.js into built container image, namely lean.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Signed-off-by: hainenber <dotronghai96@gmail.com>
@bito-code-review
Copy link
Copy Markdown
Contributor

bito-code-review Bot commented Apr 23, 2026

Code Review Agent Run #f2624e

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 3d16776..3d16776
    • Dockerfile
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

@dosubot dosubot Bot added the install:docker Installation - docker container label Apr 23, 2026
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
@bito-code-review
Copy link
Copy Markdown
Contributor

The flagged issue is valid — the COPY command at line 205 in Dockerfile will fail when DEV_MODE=true because the superset-node stage skips npm run build, leaving /app/superset/static/service-worker.js uncreated. To resolve, ensure the file is always generated in the node stage. Fix: Modify the superset-node stage to always execute npm run ${BUILD_CMD}, even in dev mode. There are no other comments on this PR.

Dockerfile

RUN npm run ${BUILD_CMD}

@bito-code-review
Copy link
Copy Markdown
Contributor

bito-code-review Bot commented Apr 25, 2026

Code Review Agent Run #651bb5

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 3d16776..fce5e35
    • Dockerfile
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

install:docker Installation - docker container size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(Dockerfile): service-worker.js missing from lean/production image

1 participant