improvement(deps): harden dependency graph - #7343
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThe PR consolidates and upgrades dependency families while replacing obsolete wrappers with maintained implementations.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/app/layout.tsx | The sidebar cookie-regex fix now emits the intended whitespace matcher, and the layout selects the appropriate hosted or runtime public-environment script. |
| apps/sim/app/_shell/public-env-script.tsx | Replaces the runtime-env wrapper with escaped inline serialization and request-time environment loading for self-hosted deployments. |
| apps/sim/app/(landing)/demo/components/demo-scheduler/demo-scheduler.tsx | Replaces the Cal React wrapper with a directly configured iframe and origin- and source-validated message handling. |
| apps/docs/components/workflow-preview/workflow-preview.tsx | Migrates the documentation workflow preview to typed @xyflow/react nodes and edges. |
| apps/docs/app/global.css | Contains formatting-only line wrapping with no styling behavior change. |
| bun.lock | Updates the resolved dependency graph to match the coordinated manifest changes. |
Reviews (4): Last reviewed commit: "fix(deps): address review edge cases" | Re-trigger Greptile
8b22db1 to
e01bed8
Compare
There was a problem hiding this comment.
All reported issues were addressed
Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.
Re-trigger cubic
0172374 to
29274cf
Compare
|
@greptile review |
|
@cubic review |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 185 files
Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.
Fix all with cubic | Re-trigger cubic
Summary
Type of Change
Testing
Checklist