Skip to content

docs: Correct tmux and mux design guidance - #681

Merged
Reddimus merged 2 commits into
mainfrom
docs/current-mux-guidance-43
Oct 28, 2026
Merged

Reddimus merged 2 commits into
mainfrom
docs/current-mux-guidance-43

Conversation

@Reddimus

@Reddimus Reddimus commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

The tmux and mux-daemon proposals described removed code as current functionality and gave incorrect protocol examples. Clarify what ships today, correct control-mode framing and byte decoding, and remove unsupported implementation-size claims. The tmux proposal is shorter and links to the protocol references.

Validation: isolated native tmux 3.8 protocol fixture; protocol-owner source checks; full just gauntlet on the combined source. This changes two design documents only.

@github-actions github-actions Bot added the docs label Oct 27, 2026
@greptile-apps

greptile-apps Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5 Tier: apex

[Low impact] The documentation changes appear safe to merge; no concrete new defect was found.

Summary

Corrects two design proposals without changing shipped code.

  • The mux proposal now separates shipped control tools from future daemon work.
  • The tmux proposal now gives byte-based protocol guidance for future integration.

Checked current source and the parser-removal release note. No runtime checks were run because only documents changed. Existing GitHub review threads were unavailable because authentication was missing; no previous findings were supplied.

Reviews (2) · Last reviewed commit: "docs: Integrate current main into mux gu..."

@Reddimus
Reddimus changed the base branch from feat/focused-media-context-menu to main October 27, 2026 20:27
@Reddimus
Reddimus merged commit 6cffa8d into main Oct 28, 2026
17 checks passed
@Reddimus
Reddimus deleted the docs/current-mux-guidance-43 branch October 28, 2026 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant