Skip to content

add HA and OC local apps#2116

Open
merveenoyan wants to merge 5 commits into
mainfrom
add-oc
Open

add HA and OC local apps#2116
merveenoyan wants to merge 5 commits into
mainfrom
add-oc

Conversation

@merveenoyan
Copy link
Copy Markdown
Contributor

@merveenoyan merveenoyan commented Apr 22, 2026

adding logos to moon shortly


Note

Low Risk
Mostly additive snippet/predicate logic and test coverage for new local-app entries; low risk beyond potential UX gating/snippet formatting regressions.

Overview
Adds two new local app integrations, OpenClaw and Hermes (keyed as openclaw and hermes-agent), including copy/paste setup snippets for running them against a local OpenAI-compatible server.

Refactors the existing Pi integration by extracting a shared getLocalServerStep (MLX vs llama.cpp) and centralizing the “tool-calling local agent” eligibility check into isToolCallingLocalAgentModel, which is now shared by Pi/OpenClaw/Hermes. Expands tests to cover the new apps (GGUF + MLX cases) and asserts all three apps share the same displayOnModelPage predicate.

Reviewed by Cursor Bugbot for commit 66539fd. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread packages/tasks/src/local-apps.ts Outdated
Comment thread packages/tasks/src/local-apps.ts Outdated
Comment thread packages/tasks/src/local-apps.ts Outdated
Comment thread packages/tasks/src/local-apps.ts Outdated
Comment on lines +530 to +531
"# Install OpenClaw by running:",
"# npm install -g openclaw@latest",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these could belong in a setup phase, for symmetry with Pi (and uncomment the npm line).

Image

Comment thread packages/tasks/src/local-apps.ts Outdated
Comment on lines +560 to +561
"# Install Hermes by running:",
"# curl -fsSL https://hermes-agent.nousresearch.com/install.sh | bash",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same, I'd make this a setup phase.

Copy link
Copy Markdown
Collaborator

@gary149 gary149 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if not urgent would like to test it a bit before merging (when I'm back next week)

Co-authored-by: Pedro Cuenca <pedro@huggingface.co>
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 134a12f. Configure here.

Comment thread packages/tasks/src/local-apps.ts
@merveenoyan
Copy link
Copy Markdown
Contributor Author

@pcuenca moved to setup
@gary149 there's two ways of doing installation, one is onboarding interactively which doesn't require coding (it's better imho for inference providers but not for local), the other one is non-interactive for which you get llama-server up and setup like this or add the endpoint to your config

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants