breadboard-ai / breadboard

A library for prototyping generative AI applications.
Apache License 2.0
207 stars 30 forks source link

Only allow attaching ports to "config" ports in "Advanced" view #3720

Open dglazkov opened 2 weeks ago

dglazkov commented 2 weeks ago

We've now seen a bunch of instances where BB novices would accidentally attach wires to configured ports, and it results in confusion.

We should make them not show the port "bubble", so that it's clear which ports are meant to be wired and which ones are configured. For example, in the component below:

image

Only "Context in" and "Context out" should have port bubbles, and the rest should have some other visual treatment that implies that they aren't wireable. I am not sure what it looks like though ✨

When "Showing advanced ports", we can make them bubbles, so that every port can be both configured and wired.

@donsbach @paullewis would love your thoughts on the "what it looks like" bits

dglazkov commented 2 weeks ago

@billbyrne also