Skip to content

fix: include instrument.server.mjs in docker image#525

Merged
SvenVw merged 2 commits into
mainfrom
hotfix/FDM523-2
Mar 20, 2026
Merged

fix: include instrument.server.mjs in docker image#525
SvenVw merged 2 commits into
mainfrom
hotfix/FDM523-2

Conversation

@SvenVw
Copy link
Copy Markdown
Collaborator

@SvenVw SvenVw commented Mar 20, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Fixed Docker image build to ensure all required runtime files are properly included.

@github-actions
Copy link
Copy Markdown
Contributor

👋 Hotfix Branch PR Detected!

Before merging this Pull Request into main, please ensure you have finalized the hotfix by manually running the 'Release' workflow on this hotfix/FDM523-2 branch.

This will:

  1. Bump package versions.
  2. Generate changelogs.
  3. Create Git tags.

You can trigger the workflow from the 'Actions' tab, selecting the 'Release' workflow, and choosing this hotfix/FDM523-2 branch.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 20, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ceeb9b9d-3090-408f-bd7f-ace399062df5

📥 Commits

Reviewing files that changed from the base of the PR and between f63a4de and b490528.

📒 Files selected for processing (3)
  • fdm-app/CHANGELOG.md
  • fdm-app/dockerfile
  • fdm-app/package.json

📝 Walkthrough

Walkthrough

This patch release (v0.28.6) ensures instrument.server.mjs is included in the Docker image by adding a copy step in the Dockerfile. The version is bumped from 0.28.5 to 0.28.6 and the CHANGELOG is updated to document the fix.

Changes

Cohort / File(s) Summary
Patch Release v0.28.6
fdm-app/CHANGELOG.md, fdm-app/dockerfile, fdm-app/package.json
Version bump to 0.28.6 with Dockerfile modification to copy instrument.server.mjs from builder to runner stage and corresponding changelog entry documenting the fix.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

fdm-app, bug, branch:main

Suggested reviewers

  • gerardhros

Poem

🐰 A tiny file was lost from sight,
In Docker's build, not packed quite right,
Now bundled snug, included with care,
Version bumped—the fix is there! 📦

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/FDM523-2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@SvenVw SvenVw requested a review from BoraIneviNMI March 20, 2026 15:46
@SvenVw SvenVw self-assigned this Mar 20, 2026
@SvenVw SvenVw merged commit 580d1a8 into main Mar 20, 2026
3 of 4 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.

1 participant