Skip to content

Add docker support - #30

Open
DeyanM1 wants to merge 1 commit into
joan-code6:mainfrom
DeyanM1:add-docker-support
Open

DeyanM1 wants to merge 1 commit into
joan-code6:mainfrom
DeyanM1:add-docker-support

Conversation

@DeyanM1

@DeyanM1 DeyanM1 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

added Docker container support to lanis-ui:

Run it

From the project folder, on any machine with Docker + Docker Compose installed:

docker compose up --build

To run it in the background:

docker compose up --build -d

To stop it:

docker compose down

@appwrite

appwrite Bot commented Aug 26, 2026

Copy link
Copy Markdown

Lanis

Project ID: 698337a60017b46df380

Sites (2)
Site Status Logs Preview QR
 LANiS Appwrite Preview
lanis-ui-appwrite-preview
Queued Queued Authorize Preview URL QR Code
 lanis_ui
6a180889003c768d3660
Queued Queued Authorize Preview URL QR Code

Tip

Trigger functions via HTTP, SDKs, events, webhooks, or scheduled cron jobs

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4d41c4ab-60ac-477b-b218-be5614aba381


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.

@greptile-apps

greptile-apps Bot commented Aug 26, 2026

Copy link
Copy Markdown

Confidence Score: 3/5

The PR should not merge until container startup preserves developer file ownership and rebuilt images reliably use updated dependencies.

The default Compose workflow can create a root-owned .env in the host repository and can continue using stale node_modules after dependency manifests are rebuilt.

Files Needing Attention: Dockerfile, docker-compose.yml, and dockerentry.sh

Important Files Changed

Filename Overview
Dockerfile Builds the Node/Chromium development image, but leaves runtime startup as root, causing host-mounted files created by the entrypoint to be root-owned.
docker-compose.yml Publishes the correct development port and enables live source mounting, but its persistent node_modules volume can mask dependencies from rebuilt images.
dockerentry.sh Initializes .env and launches the requested command, but neither handles non-root ownership nor synchronizes the mounted dependencies.
.dockerignore Excludes local dependencies, Git metadata, environment data, and Docker metadata from the build context.

Fix all with Greploop Fix All in Codex

Reviews (1): Last reviewed commit: "Add docker support" | Re-trigger Greptile

Comment thread Dockerfile
# Adjust this if your dev server / app runs on a different port
EXPOSE 3000

ENTRYPOINT ["/entrypoint.sh"]

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Root-owned environment file

When Compose starts without an existing .env, the root entrypoint writes the file through the bind-mounted repository, causing the host-side .env to be root-owned and potentially uneditable or undeletable without elevated privileges.

Fix in Codex

Comment thread docker-compose.yml
- "3000:3000" # host:container — change the left side if 3000 is taken on your server
volumes:
- ./:/app # live-mounts your project so code changes apply without rebuilding
- /app/node_modules # keeps container's node_modules from being overwritten by the mount

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Stale dependency volume

When a dependency manifest changes and the documented docker compose up --build command recreates the service, the existing /app/node_modules volume masks the dependencies installed in the rebuilt image, causing missing-module failures or execution against stale package versions.

Fix in Codex

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