launcher: agterm overlay always covers the whole session; allow a floating panel size
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 83/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Área
- cli, developer-experience
Línea de trabajo
Empieza con .claude-plugin/skills/revdiff/scripts/launch-revdiff.sh y plugins/codex/skills/revdiff/scripts/launch-revdiff.sh; después compara cómo cada uno construye comandos de superposición de agterm para toda la sesión y para paneles específicos. Actualiza ambas copias del launcher y los documentos de configuración indicados; app/plugin_exit_code_test.go es el punto de entrada sugerido para las pruebas. El trabajo estará listo cuando la opción de tamaño llegue a las superposiciones de toda la sesión y el comportamiento siga siendo el mismo cuando no esté configurada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Go
- Estrellas
- 928
- Forks
- 96
- Merge medio
- 2 d 9 h
- PR fusionados (30 d)
- 25
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de umputun/revdiff
-
Support horizontal scrollAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
umputun/revdiff#334 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Changes inside git submodules are not shown, only the "Subproject commit" pointerPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 64/100
umputun/revdiff#350 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Review artifacts are keyed by repository basename and wall-clock time, so reviews can read or overwrite each otherPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
umputun/revdiff#331 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 64/100
umputun/revdiff#324 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
pi extension: herdr never shows the agent as blocked while a revdiff review is openPosiblemente ocupada @nicdumz la tomó hace 64 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
Los mantenedores suelen responder en 1 día
Todos los issues de umputun/revdiff
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
grafana/tempo-operator#1691 ·
Los mantenedores suelen responder en 1 día
-
fix: presentation.go with mcp-table.go: the message for each MCP server comes backPosiblemente ocupada @mge1512 la tomó hoy. Abiertobug
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
mark3labs/kit#242 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Project submission: Sno StationAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
slavakurilyak/awesome-ai-agents#827 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
bug P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
mpyw/declscope#262 · 1 comentario ·
Los mantenedores suelen responder en 1 día