Add ACP to agent-canvas <https://github.com/OpenHa...
# proj-agent-canvas
s
Add ACP to agent-canvas https://github.com/OpenHands/agent-canvas/pull/416 • mirrors OpenHands/OpenHands: ◦ Condenser / LLM is greyed out when ACP enabled ◦ no llm Profile for now with ACP • what's still missing for ACP ◦ UX for Codex file injection ◦ UI - show agent type in conversation bar ◦ detect pre-existing claude code / codex conversations in local machine ◦ Cloud: conversation persistence / secrets file persistences etc @limited-student-10747
m
I think we might have a merge conflict here. I've been working on a PR by @chilly-france-44439, and we are also using the Agent menu (see attached image). Since my PR is quite small, I can just wait for you to merge yours first, and then I'll update my PR.
s
ah yes! Go ahead an merge! I already solved that same conflict in OpenHands/OpenHands 😛
m
I still have to fix a couple of things... So this time you can go first 😉
s
OK! ...if you could review 😛
m
Yes... on it
Accepted. 👍
❤️ 1
s
for resolving the merge conflict... I think "Agent" belongs on top of "LLM Profiles" And then just merge the two agent pages: when openhands is enabled then show the subagent UI, else ACP show the ACP options (same as OpenHands/OpenHands)
👍 1