Skip to content

fix(plugin): print exact DocPull setup command #139

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
zacharyr0th merged 1 commit into from
Sep 5, 2026
+7 −5
Merged

fix(plugin): print exact DocPull setup command #139

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions docs/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Fixed
- Print only the exact pinned `pipx install` command when the plugin launcher
cannot execute DocPull.

## [6.5.5] - 2026-09-04

### Fixed
Expand Down
4 changes: 1 addition & 3 deletions plugin/scripts/launch-docpull-mcp.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ const child = spawn("docpull", ["mcp", ...process.argv.slice(2)], {

child.on("error", (error) => {
if (error.code === "ENOENT" || error.code === "EACCES") {
console.error(
"DocPull is not installed or is not executable. Run: pipx install 'docpull[mcp]==6.5.5'",
);
console.error("pipx install 'docpull[mcp]==6.5.5'");
process.exitCode = 127;
return;
}
Expand Down
4 changes: 2 additions & 2 deletions tests/test_ci_policy.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,7 @@ def test_plugin_launcher_reports_actionable_setup_when_docpull_is_missing(
)

assert proc.returncode == 127
assert "pipx install 'docpull[mcp]==6.5.5'" in proc.stderr
assert proc.stderr == "pipx install 'docpull[mcp]==6.5.5'\n"

(tmp_path / "docpull").mkdir()
proc = subprocess.run( # nosec B603
Expand All @@ -345,7 +345,7 @@ def test_plugin_launcher_reports_actionable_setup_when_docpull_is_missing(
)

assert proc.returncode == 127
assert "pipx install 'docpull[mcp]==6.5.5'" in proc.stderr
assert proc.stderr == "pipx install 'docpull[mcp]==6.5.5'\n"


def test_codex_plugin_default_prompt_fits_host_limit() -> None:
Expand Down
Loading