Skip to content

Ensure latest version of Lemonade is installed on local-ai-use - #154

Open
danielholanda wants to merge 1 commit into
mainfrom
dholanda/ns
Open

Ensure latest version of Lemonade is installed on local-ai-use#154
danielholanda wants to merge 1 commit into
mainfrom
dholanda/ns

Conversation

@danielholanda

@danielholanda danielholanda commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR solves the issue below:

#152

CLI selected by name, not capability. find_cli() searches PATH for lemonade before lemonade-server and uses whichever it finds first. The lemonade-sdk pip package installs both: lemonade (an eval/benchmarking tool with no serve subcommand) and lemonade-server (the REST server). Pip users get the wrong binary, lemonade serve exits without error and without a server, and setup burns 90 s on a health check that can never pass. Fix: probe the candidate's --help for serve/pull subcommands instead of trusting the name.

The actual underlying issue is that Lemonade was targeting a stale version of Lemonade installed in the system. Pip installation has been deprecated for a few months now. This PR ensures that the skill knows what is the right version of Lemonade to install.

In the future, this install module should be shared across all Lemonade skills.

"remove it however it was installed: `sudo apt remove lemonade-server` "
"(apt/PPA) or `pip uninstall lemonade-sdk` (pip)"
)
return "`sudo apt remove lemonade-server`, or the package manager it came from"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Does this make the agent run sudo commands? Sounds risky? Should this prompt a question to the user.

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.

2 participants