Repository navigation
docs: make tmux permission diagnostics reproducible - #12350
Closed
austinywang wants to merge 1 commit into
Closed
austinywang wants to merge 1 commit into
austinywang wants to merge 1 commit into
Conversation
|
Warning Review limit reachedNext included review available in 4 minutes. View limit detailsLimit details: You’ve used all 10 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
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. Comment |
Contributor
|
All contributors have signed the CLA ✍️ ✅ |
teamleaderleo
added
area: docs
Collaborator
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #12219, incorporating and crediting @haung921209. Improves diagnostics by comparing exact operations across shell contexts and documenting server inspection.\n\nValidation: git diff --check; Linux shell and isolated tmux checks; python3 scripts/swift_file_length_budget.py. Builds were not run per Linux-only constraints. Localization audit: Markdown only; no catalogs changed.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Note
Low Risk
Markdown-only documentation with no runtime, API, or security behavior changes.
Overview
Adds a Diagnosing protected-folder access errors section to
docs/local-tmux.mdfor macOS-styleOperation not permittedfailures in tmux (e.g. under~/Documents).The guide tells users to capture evidence before restarting cmux or tmux, inspect the running server (
tmux display-message,ps,lsof), query the correct socket (-S,cmux local-tmux list --json), and compare the same absolute-path operations across tmux, cmux, and another terminal using copy-paste POSIX snippets for directory listing and file read. It also lists what to include in bug reports, distinguishes EPERM from EACCES, and links Apple guidance and issue #2866 as an investigation reference—not a single root cause.Reviewed by Cursor Bugbot for commit 0002c27. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Adds a documented procedure for diagnosing
Operation not permittederrors in tmux panes, so failures are recorded before any restart or permission change. The new section covers identifying the server, comparing the exact failing command across shell contexts, and reporting the right data without including private contents.Written for commit 0002c27. Summary will update on new commits.