Commit 5db218b
committed
fix(docker): copy all declared workspace package.json files; wire ENTRYPOINT to CLI
Two unrelated breakages that prevented the documented `docker build .`
followed by `docker run docx-corpus <subcommand>` workflow from working
for any user.
1. Workspaces: the COPY block only copied apps/cli/package.json, but the
root package.json declares apps/cdx-filter and apps/site as workspaces
too. bun install aborted with "Workspace not found" for both. Added
the missing COPY lines so the image faithfully mirrors the project's
declared workspaces. (Mirrors reality rather than stubbing them out -
keeps the build container honest with the source repo.)
2. Entrypoint: README documents `docker run ... docx-corpus scrape
--batch 100`, but the entrypoint was `tail -f /dev/null` which
ignored args. Switched to ENTRYPOINT bun run /app/apps/cli/index.ts
with CMD --help so the documented invocation works and no-args runs
show usage.
Verified the failing bun install step by replicating the Dockerfile's
COPY+install sequence in a temp directory: 1809 packages installed
cleanly. Verified the CLI's --help argument produces the documented
usage output.1 parent a2a1f5f commit 5db218b
1 file changed
Lines changed: 9 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
| 12 | + | |
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
15 | 16 | | |
16 | 17 | | |
| 18 | + | |
17 | 19 | | |
| 20 | + | |
18 | 21 | | |
19 | | - | |
| 22 | + | |
20 | 23 | | |
21 | 24 | | |
22 | 25 | | |
| |||
40 | 43 | | |
41 | 44 | | |
42 | 45 | | |
43 | | - | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
0 commit comments