Skip to content

feat(frontend): add searchable select input#274

Merged
ckakgun merged 8 commits into
heymrun:mainfrom
eryue0220:feat/searchable-select
Jul 2, 2026
Merged

feat(frontend): add searchable select input#274
ckakgun merged 8 commits into
heymrun:mainfrom
eryue0220:feat/searchable-select

Conversation

@eryue0220

Copy link
Copy Markdown
Contributor

Summary

Convert node operation dropdowns to the shared searchable select UI across the remaining E2E-covered node property panels. Which can help user find the operation, especially when the operation list is too long.

UI is in following the pictures:

0272d8622643e521ae842ffe03132b35

@mbakgun

mbakgun commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Thanks for the PR @eryue0220 🙏 The searchable operation select is a genuinely useful UX improvement, especially for nodes with long operation lists like GitHub, Notion, Linear, and S3 👑

Before merging, could you address two small things and one related UX ask? 🤗

Duplicate operation lists. The new grouped lists duplicate the old flat operation lists for Linear, GitHub, ClickHouse, Notion, and S3. The old *OperationOptions are still returned from the controller but no longer used by the node components. Could we either remove the now-unused flat lists or derive one from the other, so there is a single source of truth? That will avoid drift when new operations get added later.

Unrelated E2E change. In canvas-workflow-executions.spec.ts, the curl was changed from http://127.0.0.1:9/e2e-http-error to http://. This looks unrelated to the searchable select work, and it also shifts the test from a failed HTTP call to an invalid URL scenario, which weakens what the test covers. Could you revert it, or explain why it is needed?

Execute node target workflow selector. The Execute node already loads available target workflows into its selector, but it still uses the old plain Select. Could we also move that target workflow selector to the new searchable select pattern? That would make selecting a target workflow easier when the workspace has many workflows.

Everything else looks good on my side. Typecheck, lint, and build all passed on my local merge snapshot. Thanks again for the improvement! 🚀

eryue0220 added 2 commits July 2, 2026 15:02
…to feat/searchable-select

* 'feat/searchable-select' of github.com:eryue0220/heym:
  impl/chat-queue (heymrun#278)
  docs: add plugin authoring guide
  fix: make plugin dependency locking cross-platform
@eryue0220

Copy link
Copy Markdown
Contributor Author

canvas-workflow-executions.spec.ts

Okay, I've updated.

@ckakgun

ckakgun commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Thank you so much 🙏

@ckakgun ckakgun merged commit 993bc38 into heymrun:main Jul 2, 2026
2 checks passed
@eryue0220 eryue0220 deleted the feat/searchable-select branch July 2, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants