[jdx]

https://entire.io/gh/jdx/mise/trails/561

Several error and warning messages suggested an outdated command, used an old feature name, or gave advice that cannot work. This PR updates them. Only the message text changes; mise behaves the same otherwise.

Remote installs gave advice that cannot work. When mise bootstrap remote installs mise (install_mise) to a directory that is not on the host's login PATH, the warning used to say:

WARN  /home/me/bin/mise is not on host's login PATH; add its directory to PATH or declare [bootstrap.mise_shell_activate] in the bootstrap project

Declaring [bootstrap.mise_shell_activate] does not help. The activation blocks it writes run mise activate … by name, so they also need mise on PATH. The warning now names the directory to add:

WARN  /home/me/bin/mise is not on host's login PATH; add /home/me/bin to PATH

Other messages

• When watchexec is missing, mise watch now suggests mise use -g watchexec, the same command its help shows. It used to suggest watchexec@latest.
• mise cache task now says "No artifact cache entries for …", the feature's current name. It used to say "output cache".

No migration is needed. Scripts that match on the old wording of these messages will need updating.

Validation: e2e tests now check each new message, and each of those tests fails without this change. One is new, e2e/cli/test_watch_missing_watchexec, which runs mise watch with only mise on PATH and a 30 second limit; the others are updated assertions in test_bootstrap_remote and test_task_artifact_cache_inspect. Clippy (-D warnings) and hk checks pass.

AI-assisted — Tool: Claude Code; model: anthropic/claude-opus-5-5; version: claude-code_2-1-289_agent.

🤖 Generated with Claude Code

---

Note

Low Risk
String-only changes to warnings and error hints; scripts that assert on old message text may need updates.

Overview
User-facing message fixes so hints match current behavior and workable fixes—no functional behavior changes beyond wording.

Remote bootstrap: When an installed mise binary isn’t on the host’s login PATH, the warning now tells users to add the parent directory to PATH and drops the misleading [bootstrap.mise_shell_activate] suggestion (activation still needs mise on PATH).

mise watch: Missing watchexec now suggests mise use -g watchexec instead of watchexec@latest, aligned with help text. New e2e covers this with a PATH that only contains mise.

mise cache task: Empty-state copy says “artifact cache” instead of “output cache”; e2e assertion updated accordingly.

Tests: Remote bootstrap e2e captures install output and asserts the new PATH warning (and that mise_shell_activate isn’t mentioned).

^{Reviewed by Cursor Bugbot for commit 4e4d4f7. Bugbot is set up for automated code reviews on this repo. Configure here.}

Summary by CodeRabbit

• Bug Fixes
  • Remote installation guidance now explains that the install directory’s parent must be added to the remote login PATH.
  • The missing watchexec hint now shows the global install command without a version suffix.
  • Empty task-cache messages now refer to “artifact cache entries” for clearer terminology.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)
• Review profile: CHILL
• Plan: Advanced
• Run ID: 64d1deb3-b1b4-4fc7-81de-b0538a593633

📥 Commits

Reviewing files that changed from the base of the PR and between 4a98b02 and 4e4d4f7.

📒 Files selected for processing (6)

• e2e/cli/test_bootstrap_remote
• e2e/cli/test_watch_missing_watchexec
• e2e/tasks/test_task_artifact_cache_inspect
• src/cli/cache/task.rs
• src/cli/watch.rs
• src/system/remote.rs

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The changes update messages for remote bootstrap PATH setup, missing watchexec, and empty task artifact caches. End-to-end tests verify the updated messages and related command behavior.

Changes

Remote Bootstrap PATH Warning

Layer / File(s)	Summary
Update and verify the remote PATH warning src/system/remote.rs, e2e/cli/test_bootstrap_remote	The warning directs users to add the install path’s parent directory to the remote login PATH. The test checks the warning, confirms the executable was installed, and verifies the output does not contain mise_shell_activate.

Missing Watchexec Hint

Layer / File(s)	Summary
Update and verify the watchexec install hint src/cli/watch.rs, e2e/cli/test_watch_missing_watchexec	The install hint uses mise use -g watchexec without @latest. The test checks the exit status and hint with a restricted PATH.

Task Artifact Cache Message

Layer / File(s)	Summary
Update and verify the empty-cache message src/cli/cache/task.rs, e2e/tasks/test_task_artifact_cache_inspect	The empty-cache message and its test now use “artifact cache entries” instead of “output cache entries.”

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: risu729

Merge Risk: ⚪ Minimal · up to 4e4d4

The updated messages and their checks are consistent; no concrete merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 files. (3 skipped: 3…	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main changes to user-facing error and warning hints. It is concise and specific enough for the changeset.

Full details: Docstring Coverage

Explanation

Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 files. (3 skipped: 3 unsupported.)

• Fix all pre-merge checks with AI

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Updates error messages in CLI commands and tests.

The PR appears safe to merge; the earlier watch-test issue is fixed, and no new actionable issues remain.

What we checked:

• Watch hint remains testable: run_with_timeout captures both output streams and returns the child's exit status. The test checks that status and the captured hint.

Summary

Updates the remote-install warning, the missing-watchexec hint, and the artifact-cache message.

• The watch test now uses an isolated PATH and a bounded run.
• Both trust errors keep the working command-reference link. jdx deferred the security-guide link until its section lands.
• No new actionable issues were found.

AI-assisted — Tool: Greptile; model: llmproxy/gpt-6.1-sol; version: unavailable.

_{Reviews (2) · Last reviewed commit: "fix: keep the trust link and isolate the..." · Reviewed by Greptile}

[github-actions]

Instruction counts

benchmark	trend	instructions	Δ	wall (min)	Δ
env	▃█▁▆▇	27,617,353 → 27,619,971	+0.01%	19.52 → 19.69ms	+0.87%
hook-env	▁█▄▃▂	29,598,137 → 29,594,311	-0.01%	20.65 → 20.50ms	-0.70%
ls	▃▄▁█▄	31,404,378 → 31,387,705	-0.05%	21.34 → 21.59ms	+1.15%
registry	█▇▃▄▁	39,991,186 → 39,915,051	-0.19%	17.82 → 18.06ms	+1.35%
startup	▆▇▅█▁	10,954,162 → 10,936,333	-0.16%	12.00 → 12.23ms	+1.93%

No instruction-count regression above 1%.

_{Only instruction counts gate. Wall clock is shown for context — on identical hardware it moves 4-20% run to run.}

_{Measured by tak — instruction-counted CLI benchmarks, stored in this repository's git notes.}

_{4e4d4f7490a9 vs 4a98b02f988b · measured on the runner, not pushed to the history.}