Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

launcher: agterm overlay always covers the whole session; allow a floating panel size

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#401 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Một pull request liên quan đã được merge.

  • #404 của @umputun — đã merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
83/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go, shell
Lĩnh vực
cli, developer-experience

Hướng nghiên cứu

Bắt đầu với .claude-plugin/skills/revdiff/scripts/launch-revdiff.sh và plugins/codex/skills/revdiff/scripts/launch-revdiff.sh, sau đó so sánh cách mỗi script tạo lệnh phủ agterm cho toàn phiên và cho từng pane. Cập nhật cả hai bản launcher và các tài liệu cấu hình được liệt kê; app/plugin_exit_code_test.go là điểm bắt đầu được đề xuất cho phần kiểm thử. Công việc hoàn tất khi tùy chọn kích thước có hiệu lực với các lớp phủ toàn phiên và hành vi vẫn giữ nguyên khi tùy chọn chưa được đặt.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

What is the problem?

In agterm the launcher always runs agtermctl session overlay open "$REVDIFF_CMD" --block with no --size-percent, so revdiff covers the whole agent session. While the review is open I cannot see the agent's chat: its last reply, the question it asked, or the context of the change I am annotating. To read it I have to close revdiff and open it again. REVDIFF_POPUP_WIDTH / REVDIFF_POPUP_HEIGHT work for tmux, zellij and wezterm, but the agterm branch ignores them, and REVDIFF_AGTERM_PANE=1 scopes the overlay to the agent's own pane, which still hides the chat.

How does this solve it?

agterm already supports a floating overlay: agtermctl session overlay open <cmd> --size-percent N. At 70% the chat stays visible around the panel, and on my window revdiff still gets ~190 columns, enough for a 120-character code line with the file tree and line numbers. Only the agterm branch needs a change: append --size-percent "$REVDIFF_AGTERM_SIZE_PERCENT" to the session-wide overlay open when the variable is set (not to the --pane open, which agterm does not combine with --size-percent). Unset keeps today's full-session overlay, so nothing changes for current users.

Size of the change: two lines in each launcher copy (.claude-plugin/skills/revdiff/scripts/launch-revdiff.sh and plugins/codex/skills/revdiff/scripts/launch-revdiff.sh). One line appends --size-percent to AGTERM_OPEN when the open is not pane-scoped. The other adds it to the session-wide fallback open after a refused pane overlay. The rest is one line per place that documents REVDIFF_AGTERM_PANE today (README, both SKILL.md and references/config.md, site/docs.html), plus a case in app/plugin_exit_code_test.go if you want it covered.

Proposed name: REVDIFF_AGTERM_SIZE_PERCENT (an integer percent), next to the existing REVDIFF_AGTERM_PANE. Reusing REVDIFF_POPUP_WIDTH would also work, but its values are tmux-style (90%, columns) and agterm takes one percent for both axes, so a separate variable seems clearer.

Today I get this with a user-layer launch-revdiff.sh override that puts an agtermctl shim first on PATH and appends --size-percent ${REVDIFF_AGTERM_SIZE_PERCENT:-70}. With the variable in the launcher, the shim goes away. I can send the PR if this fits.

Ngôn ngữ chính
Go
Star
928
Fork
96
Merge trung bình
2 ngày 3 giờ
Pull request đã merge (30 ngày)
28

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của umputun/revdiff

Tất cả issue của umputun/revdiff

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.