Skip to content

Guarantee the push service worker is included in release images - #36

Merged
Muukuro merged 1 commit into
mainfrom
codex/investigate-missing-service-worker-for-notifications
Jul 24, 2026
Merged

Guarantee the push service worker is included in release images#36
Muukuro merged 1 commit into
mainfrom
codex/investigate-missing-service-worker-for-notifications

Conversation

@Muukuro

@Muukuro Muukuro commented Jul 24, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Recent image-size optimizations made including public/ assets incidental to the builder stage and risked releasing images without public/push-sw.js, breaking browser push notifications.

Description

  • Copy public/ from the builder into ./.next/standalone/public in the final Docker stage and add a build-time assertion test -f ./.next/standalone/public/push-sw.js so the Docker build fails if the push service worker is missing, while preserving the reduced standalone image layout.

Testing

  • Ran git diff --check and npm run lint, both succeeded; npm run build could not fully complete here because the environment could not fetch Google Fonts; and a full Docker image build could not be run because the Docker CLI is not available in this environment.

Codex Task

@Muukuro
Muukuro merged commit fe4ce93 into main Jul 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant