fix(docker): logger import#4988
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview In the deps stage, the Dockerfile now creates the logger package directory and copies Reviewed by Cursor Bugbot for commit d2eeadb. Configure here. |
Greptile SummaryThis PR fixes a missing
Confidence Score: 5/5Safe to merge — the change adds the missing logger package to the db Docker image, directly fixing a build-time import failure introduced by tx-tripwire.ts. The fix correctly mirrors the existing pattern for utils and tsconfig: package.json is copied in the deps stage for bun install, and the full source tree is copied in the runner stage before packages/db to preserve layer-cache ordering. No logic changes and no other files are affected. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[base: oven/bun:1.3.13-alpine] --> B[deps stage]
B --> C["mkdir packages/db, logger, tsconfig, utils"]
C --> D[Copy package.json files for each package]
D --> E["bun install - resolves @sim/logger and chalk"]
E --> F[runner stage]
F --> G[COPY node_modules from deps]
G --> H[COPY packages/tsconfig]
H --> I[COPY packages/utils]
I --> J["COPY packages/logger - NEW"]
J --> K[COPY packages/db - changes most frequently]
K --> L[WORKDIR /app/packages/db]
style J fill:#90ee90,stroke:#228B22
Reviews (1): Last reviewed commit: "fix(docker): logger import" | Re-trigger Greptile |
Summary
Logger import missing for new tripwire runtime support
Type of Change
Testing
N/A
Checklist