Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Chiusa Adatta ai principianti
  • Qualcosa non va?
#401 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Una pull request collegata è già stata integrata.

  • #404 di @umputun — integrata

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
83/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go, shell
Ambito
cli, developer-experience

Direzione di ricerca

Inizia con .claude-plugin/skills/revdiff/scripts/launch-revdiff.sh e plugins/codex/skills/revdiff/scripts/launch-revdiff.sh, poi confronta il modo in cui ciascuno crea i comandi di overlay agterm per l’intera sessione e per i singoli pannelli. Aggiorna entrambe le copie del launcher e i documenti di configurazione indicati; app/plugin_exit_code_test.go è il punto di partenza suggerito per i test. Il lavoro è completo quando l’impostazione facoltativa delle dimensioni viene applicata agli overlay dell’intera sessione e il comportamento resta invariato quando l’impostazione non è definita.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Go
Stelle
928
Fork
96
Merge medio
2g 3h
PR unite (30g)
28

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di umputun/revdiff

Tutte le issue di umputun/revdiff

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.