Skip to content

feat: add docker-compose for reproducible builds#889

Open
khushigoel44-afk wants to merge 2 commits into
shouri123:mainfrom
khushigoel44-afk:main
Open

feat: add docker-compose for reproducible builds#889
khushigoel44-afk wants to merge 2 commits into
shouri123:mainfrom
khushigoel44-afk:main

Conversation

@khushigoel44-afk

@khushigoel44-afk khushigoel44-afk commented Jul 22, 2026

Copy link
Copy Markdown

Description

Added a docker-compose.yml file to support reproducible local builds. This configuration automates the build process inside a lightweight Alpine Node container and seamlessly extracts the compiled, production-ready extension bundle into the host machine's ./dist folder.

Fixes #851

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • [✓] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • [✓] My code follows the style guidelines of this project
  • [✓] I have performed a self-review of my own code
  • [✓] I have commented my code, particularly in hard-to-understand areas
  • [✓] My changes generate no new warnings

Summary by CodeRabbit

  • Chores
    • Added a Docker Compose configuration to build the application and export generated files to the host.
    • Added build completion feedback and an output file listing to make it easier to verify results.

@github-actions github-actions Bot added gssoc Official GSSoC contribution issue gssoc:approved GSSoC: PR approved and scored bug Something isn't working size/XS type:design Type: Design / UI / UX (Optional bonus) type:docs Type: Documentation type:feature Type: New Feature labels Jul 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Thank You for Contributing to Late-Meet

Please ensure that:

  • the issue was assigned to you before opening this PR
  • the PR references the related issue
  • your changes follow repository contribution guidelines
  • the project builds successfully before submission

Unassigned, duplicate, or low-quality PRs may be closed.

Thank you for contributing 💙

@github-actions github-actions Bot added type:config Type: Configuration change and removed bug Something isn't working type:feature Type: New Feature type:docs Type: Documentation type:design Type: Design / UI / UX (Optional bonus) size/XS labels Jul 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

👋 Thank you @khushigoel44-afk for your contribution to Late-Meet!

✅ Verified: You are assigned to the linked issue #851.

Please review any automated suggestions or code review comments that may appear below! We will review your PR as soon as possible!


Please consider starring the repository ⭐ to show your support!

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5c6adf55-4d4d-4101-a21f-8bb14c108617

📥 Commits

Reviewing files that changed from the base of the PR and between efc6570 and 859f1f8.

📒 Files selected for processing (1)
  • docker-compose.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • docker-compose.yml

📝 Walkthrough

Walkthrough

Adds a Docker Compose service that builds the project, copies /app/dist to the host ./dist directory, and lists the exported files.

Changes

Docker Compose build workflow

Layer / File(s) Summary
Build and host output export
docker-compose.yml
Defines the build_meet service, builds from the repository Dockerfile, mounts ./dist at /host_dist, copies generated output, and reports the resulting files.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: shouri123

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR adds docker-compose.yml, but it does not implement the requested npm run dev hot-reload workflow or docs update for #851. Add a compose service for npm run dev hot reloading, use a Node 18 alpine/slim image, and update docs/GETTING_STARTED.md with docker compose up --build.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: adding a Docker Compose setup for builds.
Out of Scope Changes check ✅ Passed The changes are limited to the Compose setup and do not show unrelated additions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Checkov (3.3.8)
docker-compose.yml

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'


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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docker-compose.yml`:
- Line 8: Update the docker-compose build command to resolve /host_dist write
permissions, clear its existing contents, and then copy the current /app/dist
contents into it. Preserve the existing success message and directory listing
while ensuring stale exported assets are removed on repeated builds.
- Around line 6-8: Update the one-shot export service’s volume/command
configuration so the copy operation can write to /host_dist despite Dockerfile’s
USER appuser setting. Configure the container with a matching host UID/GID or
run this export command as an appropriate user, while preserving the existing
dist extraction and success output behavior.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d1cd4942-168b-48dc-9013-8d179a0c7fcf

📥 Commits

Reviewing files that changed from the base of the PR and between d216444 and efc6570.

📒 Files selected for processing (1)
  • docker-compose.yml

Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
@sonarqubecloud

Copy link
Copy Markdown

@khushigoel44-afk

khushigoel44-afk commented Jul 23, 2026

Copy link
Copy Markdown
Author

Hey @shouri123
I have updated the PR, and now it's ready for your review.

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

Labels

gssoc:approved GSSoC: PR approved and scored gssoc Official GSSoC contribution issue type:config Type: Configuration change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Add docker-compose.yml for simplified local development environment setup

1 participant