Conversation
Greptile SummaryThis PR introduces Tilt as a local development orchestrator, adding a
Confidence Score: 4/5Safe to merge — purely additive tooling config with no impact on the production build or runtime. All bun run script names in the Tiltfile match the ones defined in package.json, the vale guard degrades gracefully, and the VS Code changes are non-breaking. The two minor points are the macOS-only install hint in the warn() call and the undocumented port-sharing constraint between dev and preview. Tiltfile — the brew-specific install hint and the shared port between dev and preview are worth a quick look before merging. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Tiltfile loads] --> B{vale in PATH?}
B -- yes --> C[register check:writing resource]
B -- no --> D[warn + skip check:writing]
subgraph setup["Label: setup"]
E[install\nbun install]
end
subgraph services["Label: services"]
F[dev\nserve: bun dev\nauto_init=true]
G[preview\ncmd: bun run build\nserve: bun preview\nauto_init=false]
end
subgraph build_group["Label: build"]
H[build\nbun run build\nauto_init=false]
end
subgraph checks["Label: checks"]
I[check]
J[check:format]
K[check:lint]
L[check:type]
M[check:links]
C
end
subgraph utils["Label: utils"]
N[fix]
O[fix:format]
P[fix:lint]
end
E --> F
E --> G
E --> H
E --> I
E --> J
E --> K
E --> L
E --> M
E --> N
E --> O
E --> P
F -. port 4321 .- G
Reviews (1): Last reviewed commit: "chore: add tilt" | Re-trigger Greptile |
| ) | ||
|
|
||
| local_resource( | ||
| "build", |
There was a problem hiding this comment.
suggestion(non-blocking): your "build" resource doesnt seem useful, it's never used as a dep and the "preview" resource is already building
You could:
- remove the "build" resource
- or don't build in the "preview" resource and set "build" as a dep of "preview"
There was a problem hiding this comment.
or don't build in the "preview" resource and set "build" as a dep of "preview"
This is what I wanted originally, but build was never running when I ran preview because build has auto_init=False - I'd have to manually run both
If it would be better to auto-init build I can do that, or just remove it and keep the build in preview?
There was a problem hiding this comment.
This behaviour is really weird, I don't know why it does this
You can maybe remove the build and just keep preview, but that's ok actually
This PR configures Tilt in the repo.