fix(docker): fix build context, DB URL absolute path, and stale repo references#35
Merged
RaghavChamadiya merged 2 commits intorepowise-dev:mainfrom Apr 6, 2026
Conversation
- REPOWISE_DB_URL used 3 slashes (sqlite+aiosqlite:///data/wiki.db), which SQLite interprets as a relative path "data/wiki.db" relative to the process CWD. Changed to 4 slashes (////data/wiki.db) so the path is the absolute /data/wiki.db that the volume mount expects. - Remove `2>/dev/null || true` from the COPY instruction — Dockerfile COPY does not support shell redirection; this silently failed to make the instruction optional and was dead syntax. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Dockerfile header comment: update build command to use -f docker/Dockerfile - docker-compose.yml: set build context to project root (context: .., dockerfile: docker/Dockerfile) so COPY paths to packages/ resolve correctly; also fix REPOWISE_DB_URL to use 4 slashes (absolute /data/wiki.db path) - docker/README.md: update build command and fix default DB URL in table - docs/QUICKSTART.md, docs/USER_GUIDE.md: replace remote-URL docker build with git clone + local build (-f docker/Dockerfile), update repo URL from RaghavChamadiya/repowise to repowise-dev/repowise, add -f flag to docker compose command Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Collaborator
|
This is a clean fix for a real regression from the docker/ subdirectory move. The 4-slash absolute path and build context fix are both correct. One small note: removing the '2>/dev/null || true' from the COPY instruction is also correct. Dockerfile COPY does not support shell redirection, so that was dead syntax. Good catch. Merging this first so #26 can rebase on top. |
7 tasks
RaghavChamadiya
approved these changes
Apr 6, 2026
Collaborator
RaghavChamadiya
left a comment
RaghavChamadiya
left a comment
There was a problem hiding this comment.
Clean fix, merging.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docker/Dockerfile: Fix header comment build command to use-f docker/Dockerfiledocker/docker-compose.yml: Fix build context to project root (context: ..,dockerfile: docker/Dockerfile) soCOPY packages/paths resolve correctly; fixREPOWISE_DB_URLfrom///(relative) to////(absolute/data/wiki.db)docker/README.md: Update build command and fix default DB URL in env vars tabledocs/QUICKSTART.md,docs/USER_GUIDE.md: Replace remote-URLdocker buildwith local clone + build; update stale repo URL fromRaghavChamadiya/repowise→repowise-dev/repowise; add-f docker/docker-compose.ymltodocker composecommandRoot causes
These issues appear to be oversights from #14 (moving
Dockerfileanddocker-compose.ymlinto thedocker/subdirectory) — the build context paths and documentation were not updated to reflect the new file locations.docker-compose.ymlhadbuild: .— when run from thedocker/directory, the build context wasdocker/, makingCOPY packages/...fail silently or at build time.sqlite+aiosqlite:///data/wiki.db(3 slashes) is a relative path resolved from the process CWD, not the intended absolute/data/wiki.dbvolume mount.Test plan
docker build -t repowise -f docker/Dockerfile .from project root succeedsdocker compose -f docker/docker-compose.yml upresolvespackages/paths correctly/data/wiki.db(the mounted volume), not a relative path