fix(tag-dropdown): exclude downstream blocks in loops and parallel siblings#3312
fix(tag-dropdown): exclude downstream blocks in loops and parallel siblings#3312waleedlatif1 merged 8 commits intostagingfrom
Conversation
…blings from reference picker
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryFixes tag dropdown to exclude downstream blocks in loops and parallel siblings. Previously, blocks inside loops could reference downstream blocks in the same loop, and blocks inside parallels could reference sibling blocks that run simultaneously. The fix consistently applies across both frontend tag picker and copilot tools by only adding loop/parallel container IDs rather than all child nodes. Also removes unused Confidence Score: 5/5
Important Files Changed
Last reviewed commit: f31ac10 |
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Outdated
Show resolved
Hide resolved
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Outdated
Show resolved
Hide resolved
|
@greptile |
|
@cursor review |
…ForWorkflow method
|
@cursor review |
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Outdated
Show resolved
Hide resolved
|
@cursor review |
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Outdated
Show resolved
Hide resolved
|
@greptile |
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Show resolved
Hide resolved
…es in parallel branches
|
@cursor review |
|
@cursor review |
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Show resolved
Hide resolved
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Show resolved
Hide resolved
…s for blocks inside parallel
|
@cursor review |
|
@greptile |
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes.ts
Show resolved
Hide resolved
…m accessible references
|
@greptile |
|
@cursor review |
Summary
<reference picker) showing downstream blocks when editing a block inside a loopancestorIds)Type of Change
Testing
Tested manually
Checklist