fix(blocks): allow tool expansion in disabled mode, improve child deploy badge freshness#4002
Conversation
…loy badge freshness
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Tool cards in Reviewed by Cursor Bugbot for commit 95e5a2f. Configure here. |
Greptile SummaryThis PR delivers two focused bug fixes: (1) tool-input panels can now be expanded in disabled/locked (read-only) mode using local component state, without writing back to the Zustand store; and (2) Key changes:
Confidence Score: 5/5Safe to merge — no bugs found, changes are well-scoped and correctly implemented. All three changes are clean, minimal bug fixes. The disabled-mode expansion correctly isolates state to local component state and never writes to the store. The refetchOnMount: 'always' pattern is idiomatic TanStack Query v5 usage for ensuring freshness on mount. No logic errors, security concerns, or regressions identified. No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant U as User
participant TI as ToolInput
participant LS as LocalState (useState)
participant ST as Zustand Store
participant API as /api/workflows/{id}/deploy
U->>TI: clicks expand (disabled mode)
TI->>TI: isPreview || disabled? YES
TI->>LS: setLocalExpanded(toolIndex, toggled)
Note over ST: Store unchanged
U->>TI: clicks expand (edit mode)
TI->>TI: isPreview || disabled? NO
TI->>ST: setStoreValue(tools with isExpanded toggled)
U->>TI: navigates to workflow (component mounts)
TI->>API: fetch with refetchOnMount: 'always'
API-->>TI: fresh deployment status
TI->>TI: display up-to-date badge
Reviews (2): Last reviewed commit: "fix(blocks): allow tool expansion in dis..." | Re-trigger Greptile |
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 95e5a2f. Configure here.
Summary
refetchOnMount: 'always'to child workflow deploy badges so deployment status is fresh on navigationuseDeploymentInfooptions to acceptrefetchOnMountType of Change
Testing
Tested manually
Checklist