From dee674d3e2fc93553149c4c84f868deecb226e97 Mon Sep 17 00:00:00 2001 From: Codeman maintainer Date: Fri, 18 Sep 2026 13:41:45 +0200 Subject: [PATCH] fix(install): point the launcher-only caveat at the thing that resolves it The caveat #429 added ends with "see the docs above", and "the docs above" is CLI_DOCS[$i], which for DeepSeek is the upstream harness repo. Per docs/deepseek-integration.md the harness ships only the web, headless and base profiles, so following that link and running `npm install -g @deepseek-ai/dsh` leaves the reader exactly where the caveat is warning them about: a dsh that cannot drive a pane. What actually resolves it is Codeman's own Run dropdown, which offers "DeepSeek: add a terminal profile..." and installs one in a click. The new wording stays generic for any future launcherProfile entry, since Codeman is the thing being installed at all three call sites. Also flips one word in the generator: the comment said "see installCommandFor below" and that function is defined above it. Co-Authored-By: Claude Opus 5 (1M context) --- install.sh | 2 +- scripts/generate-cli-catalog.mts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/install.sh b/install.sh index 8b5672bc..29b6a13d 100755 --- a/install.sh +++ b/install.sh @@ -587,7 +587,7 @@ cli_catalog_print_install_hints() { elif [[ -n "${CLI_DOCS[$i]}" ]]; then echo -e " ${CLI_LABELS[$i]}: see ${CYAN}${CLI_DOCS[$i]}${NC}" if [[ "${CLI_LAUNCHER_ONLY[$i]}" == "1" ]]; then - echo -e " (its package installs a launcher only — it needs a profile that can drive a pane, see the docs above)" + echo -e " (installs a launcher only: it still needs a terminal profile, and Codeman's Run menu can add one)" fi fi done diff --git a/scripts/generate-cli-catalog.mts b/scripts/generate-cli-catalog.mts index e17df0ec..de0d7934 100644 --- a/scripts/generate-cli-catalog.mts +++ b/scripts/generate-cli-catalog.mts @@ -151,7 +151,7 @@ export function renderInstallShBlock(entries: CliEntry[] = STOCK_CLIS): string { labels.push(shQuote(entry.label)); enabled.push(entry.enabled ? '1' : '0'); // Parallel to CLI_IDS: 1 when this entry's install command installs a launcher rather - // than something that can drive a pane on its own (see installCommandFor below). Purely + // than something that can drive a pane on its own (see installCommandFor above). Purely // derived from discovery.launcherProfile — install.sh's hint printer reads this to add a // caveat instead of hardcoding which id it means. launcherOnly.push(entry.discovery.launcherProfile ? '1' : '0');