Skip to content

Improve tmux snapshot restore and scrollback handling#207

Merged
andyrewlee merged 2 commits intomainfrom
weird-scroll
Apr 8, 2026
Merged

Improve tmux snapshot restore and scrollback handling#207
andyrewlee merged 2 commits intomainfrom
weird-scroll

Conversation

@andyrewlee
Copy link
Copy Markdown
Owner

@andyrewlee andyrewlee commented Apr 8, 2026

Summary

Describe the change and intended behavior.

Quality Checklist

  • Ran make devcheck locally.
  • Ran make lint-strict-new locally for changed code.
  • If UI/rendering changed, ran make harness-presets.
  • If tmux/e2e changed, ran go test ./internal/tmux ./internal/e2e.

Open with Devin

Copy link
Copy Markdown
Owner Author

@andyrewlee andyrewlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 7 additional findings.

Open in Devin Review

@andyrewlee andyrewlee merged commit 0fc2034 into main Apr 8, 2026
3 checks passed
@andyrewlee andyrewlee deleted the weird-scroll branch April 8, 2026 06:07
Megatherium added a commit to Megatherium/amux that referenced this pull request Apr 16, 2026
Merge upstream/main bringing in:
- Remove openclaw CLI subsystem (andyrewlee#204)
- Fix hover wheel pane routing (andyrewlee#205)
- Extract VTerm.SelectedText() dedup
- Extract resetPTYStateLocked dedup
- Improve tmux snapshot restore & scrollback handling (andyrewlee#207)
- Improve file tab reuse & workspace rebind handling (andyrewlee#206)

Conflicts resolved:
- go.mod: kept our yaml.v3 + go-sqlmock deps alongside upstream changes
- internal/cli/cmd_doctor.go: accepted upstream deletion (openclaw removal)

Refactoring:
- Extract prefix command table/matching/execution to app_ui_prefix.go
  to keep app_ui.go under 500-line file length limit

Affected ticket(s): bmx-bh4

💘 Generated with Crush

Assisted-by: GLM-5.1 via Crush <crush@charm.land>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant