fix(custom-tool): restore modal body scroll so Save stays reachable

[waleedlatif1]

Summary

• The Edit/Create Agent Tool modal clipped its footer (Save/Update) and could not scroll when a tool had long code or a long schema.
• Root cause: improvement(platform): remove tour, simplify sidebar/header, drop loading skeletons #4354 migrated this modal to ChipModal and changed the body from a scroll region (flex-1 overflow-y-auto) to flex-none overflow-visible so the hand-positioned EnvVar/Tag autocompletes could spill past it. That removed the scroll region, so tall content grew the body past the modal's max-h-[84vh] cap and the overflow-hidden surface clipped the footer.
• Restore ChipModalBody as the scroll region (its documented behavior — header/footer stay pinned), and switch the EnvVar/Tag dropdowns to portaled inputRef caret-anchoring, matching the canonical Function-block editor (code.tsx), so they anchor to the caret in a portal and are never clipped by the scroll boundary.
• No editor height caps, no magic numbers — removes the workaround rather than layering another on top. Net −1 line, one file.

Type of Change

• Bug fix

Testing

Tested manually; tsc, biome, and the tool-input test suite (47 tests) pass.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 1, 2026 5:38pm

[cursor]

PR Summary

Low Risk
Single-file modal layout change; restores documented ChipModalBody scrolling with no auth, data, or API impact.

Overview
Fixes the Edit/Create Agent Tool modal so long schema/code no longer pushes the Next/Save footer out of view.

ChipModalBody no longer uses flex-none overflow-visible (which had opted the body out of scrolling). It now relies on the default body layout (flex-1 + overflow-y-auto), so tall content scrolls inside the modal while the header and footer stay pinned. The inline comment is updated to describe that scroll behavior and that EnvVar/Tag/schema-param menus are expected to use portaled popovers so they are not clipped by the scroll region.

^{Reviewed by Cursor Bugbot for commit d90dde9. Configure here.}

[greptile-apps]

Greptile Summary

This PR fixes the custom tool modal so long content can scroll without hiding the footer. The main changes are:

• Restores the modal body as the scroll region.
• Keeps the header and footer outside the scrolling content.
• Documents that autocomplete menus use portaled caret anchoring.

Confidence Score: 5/5

This looks safe to merge.

• No blocking issues found in the changed code.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/components/custom-tool-modal/custom-tool-modal.tsx	Restores the custom tool modal body to its default scrollable layout so long schema or code content stays within the modal.

_{Reviews (3): Last reviewed commit: "fix(custom-tool): keep dropdown anchors ..." | Re-trigger Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ 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 d90dde9. Configure here.}

[waleedlatif1]

@greptile the schema-param menu anchors to a position:absolute element inside codeEditorRef, which scrolls with the body — the portaled popover tracks it via Floating UI ancestor-scroll autoUpdate. It is not viewport-pinned, so it doesn't drift. Please re-review.

[waleedlatif1]

@cursor review

[cursor]

✅ 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 d90dde9. Configure here.}