Fix idle zsh PR probes triggering chpwd hooks

[teamleaderleo]

Idle zsh PR polling now uses quiet cd, so it stops firing user chpwd hooks and inflating zoxide rankings. The probe still runs gh in the repository and caches the returned PR.

Closes #3191.

Validation: the test-only commit reproduces the unwanted hook execution locally. After the fix, all 6 PR watcher tests, the stale-cwd shell tests, first-prompt spawn tests, zsh syntax check, and git diff --check pass. The test-only CI run skipped the macos-shell lane, so its green guards are not regression-test evidence.

No user-facing strings changed; localization audit found no strings to translate.

Marshmallow g1 🎐
run: run_cx_bl_bugs_a_old_20261001
session: cx-bl-bugs-a-oldest

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 54 seconds.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: beee5338-2898-45e9-9b48-a559317aa9af

📥 Commits

Reviewing files that changed from the base of the PR and between 8b8762a and bdd5460.

📒 Files selected for processing (2)

• Resources/shell-integration/cmux-zsh-integration.zsh
• tests/test_shell_pr_watch_private_state.py

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

Autopilot is currently an internal CodeRabbit preview.

---

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.}

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[cubic-dev-ai]

All reported issues were addressed across 1 file

_{You’re at about 90% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.}

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

1 issue found across 2 files (changes from recent commits).

_{You’re at about 90% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.}

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="Resources/shell-integration/cmux-zsh-integration.zsh">

<violation number="1" location="Resources/shell-integration/cmux-zsh-integration.zsh:1655">
P2: `cd -q` (chpwd suppression) was added in zsh 5.9. On older zsh the option is rejected, and because `2>/dev/null` swallows the error the `&& gh pr view ...` chain never runs, so the PR probe silently returns `1` forever and the badge stops updating/never appears — the same silent-failure class of regression this file avoids elsewhere by falling back for zselect/jobstates. Confirm the zsh floor (cmux ships on macOS 14+, whose system zsh is 5.9, so this only bites custom/older zsh builds) or gate the option with `is-at-least 5.9` and use a plain `cd` fallback for older zsh. Note a plain-`cd` fallback re-runs chpwd hooks on shells < 5.9, so if old zsh is out of support, documenting the requirement is the simpler fix.</violation>
</file>

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

P2: cd -q (chpwd suppression) was added in zsh 5.9. On older zsh the option is rejected, and because 2>/dev/null swallows the error the && gh pr view ... chain never runs, so the PR probe silently returns 1 forever and the badge stops updating/never appears — the same silent-failure class of regression this file avoids elsewhere by falling back for zselect/jobstates. Confirm the zsh floor (cmux ships on macOS 14+, whose system zsh is 5.9, so this only bites custom/older zsh builds) or gate the option with is-at-least 5.9 and use a plain cd fallback for older zsh. Note a plain-cd fallback re-runs chpwd hooks on shells < 5.9, so if old zsh is out of support, documenting the requirement is the simpler fix.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. At Resources/shell-integration/cmux-zsh-integration.zsh, line 1655:

<comment>`cd -q` (chpwd suppression) was added in zsh 5.9. On older zsh the option is rejected, and because `2>/dev/null` swallows the error the `&& gh pr view ...` chain never runs, so the PR probe silently returns `1` forever and the badge stops updating/never appears — the same silent-failure class of regression this file avoids elsewhere by falling back for zselect/jobstates. Confirm the zsh floor (cmux ships on macOS 14+, whose system zsh is 5.9, so this only bites custom/older zsh builds) or gate the option with `is-at-least 5.9` and use a plain `cd` fallback for older zsh. Note a plain-`cd` fallback re-runs chpwd hooks on shells < 5.9, so if old zsh is out of support, documenting the requirement is the simpler fix.</comment>

<file context>
@@ -1652,7 +1652,7 @@ _cmux_report_pr_for_path() {
     [[ -n "$err_file" ]] || return 1
     gh_output="$(
-        builtin cd "$repo_path" 2>/dev/null \
+        builtin cd -q "$repo_path" 2>/dev/null \
             && gh pr view "$branch" \
                 "${gh_repo_args[@]}" \
</file context>

[github-actions]

Merge receipt for bdd5460cc5: every check was green at merge (14 verified; 19 skipped by policy). Full suite runs on main after merge.