TUI: Git-Clone-Progress von bare-Repos zerstört Layout #50

New issue

Closed

opened 2026-07-13 01:37:10 +02:00 by frank · 1 comment

frank commented

2026-07-13 01:37:10 +02:00

Owner

Problem

Beim ersten Bare-Clone eines Repos (z. B. unter forge agent watch mit TUI) schreibt git clone --bare seinen Progress direkt auf das Terminal. Die Ausgabe landet am unteren Bildschirmrand, überlappt mit der TUI-Footer-Zeile (Filter · [/] scroll) und verschiebt das gesamte Layout.

Beobachteter Output (Ende des Bildschirms):

Cloning into bare repository '/Users/frank/Documents/forge/frank/forgecli.git'...lter · [/] scroll
                                                                                 remote: Enumerating objects: 1779, done.
remote: Counting objects: 100% (1779/1779), done.
remote: Compressing objects: 100% (1237/1237), done.
remote: Total 1779 (delta 791), reused 1274 (delta 487), pack-reused 0 (from 0)
Receiving objects: 100% (1779/1779), 2.94 MiB | 20.48 MiB/s, done.             Receiving objects:  93% (1655/1779)
Resolving deltas: 100% (791/791), done.                           Resolving deltas:   0% (0/791)

Erwartetes Verhalten

Git-Progress (Clone/Fetch) muss innerhalb der Agent-TUI bleiben (Log-Panel), analog zu anderen Agent-/Git-/CI-Ausgaben.
Carriage-Return-Progress (Receiving objects: xx%) darf die TUI nicht corrupt machen (Zeilen überschreiben / Layout verschieben).
Ohne TUI (--no-tui) darf Progress weiterhin auf stderr sichtbar sein.

Vermutete Ursache

internal/gitutil/git.go → clone() / CloneBare() leitet Stdout/Stderr fest auf os.Stdout/os.Stderr um.

EnsureRepoRoot in internal/agent/gitflow.go hat zwar einen log io.Writer und schreibt dort cloning bare …, übergibt den Writer aber nicht an CloneBare. Dadurch umgeht der eigentliche git clone --bare-Progress die TUI komplett.

Fix-Ideen

CloneBare/clone um optionalen io.Writer für Progress erweitern (oder Stdout/Stderr setzbar machen).
In EnsureRepoRoot den vorhandenen log-Writer an den Clone durchreichen.
Sicherstellen, dass RunLogWriter CR-Progress korrekt als einzelne Log-Zeilen behandelt (Test TestRunLogWriterCarriageReturn prüfen/erweitern).
Gleiches Muster für andere Git-Aufrufe prüfen, die noch direkt aufs Terminal schreiben.

Repro

Bare-Cache für ein Repo löschen (oder neues Repo ohne lokalen Cache watchen).
forge agent watch im interaktiven Terminal starten (TUI aktiv).
Beim ersten Clone den Progress beobachten — Layout bricht.

## Problem Beim ersten Bare-Clone eines Repos (z. B. unter `forge agent watch` mit TUI) schreibt `git clone --bare` seinen Progress direkt auf das Terminal. Die Ausgabe landet am unteren Bildschirmrand, überlappt mit der TUI-Footer-Zeile (`Filter · [/] scroll`) und verschiebt das gesamte Layout. Beobachteter Output (Ende des Bildschirms): ``` Cloning into bare repository '/Users/frank/Documents/forge/frank/forgecli.git'...lter · [/] scroll remote: Enumerating objects: 1779, done. remote: Counting objects: 100% (1779/1779), done. remote: Compressing objects: 100% (1237/1237), done. remote: Total 1779 (delta 791), reused 1274 (delta 487), pack-reused 0 (from 0) Receiving objects: 100% (1779/1779), 2.94 MiB | 20.48 MiB/s, done. Receiving objects: 93% (1655/1779) Resolving deltas: 100% (791/791), done. Resolving deltas: 0% (0/791) ``` ## Erwartetes Verhalten - Git-Progress (Clone/Fetch) muss **innerhalb** der Agent-TUI bleiben (Log-Panel), analog zu anderen Agent-/Git-/CI-Ausgaben. - Carriage-Return-Progress (`Receiving objects: xx%`) darf die TUI nicht corrupt machen (Zeilen überschreiben / Layout verschieben). - Ohne TUI (`--no-tui`) darf Progress weiterhin auf stderr sichtbar sein. ## Vermutete Ursache `internal/gitutil/git.go` → `clone()` / `CloneBare()` leitet `Stdout`/`Stderr` fest auf `os.Stdout`/`os.Stderr` um. `EnsureRepoRoot` in `internal/agent/gitflow.go` hat zwar einen `log io.Writer` und schreibt dort `cloning bare …`, übergibt den Writer aber **nicht** an `CloneBare`. Dadurch umgeht der eigentliche `git clone --bare`-Progress die TUI komplett. ## Fix-Ideen 1. `CloneBare`/`clone` um optionalen `io.Writer` für Progress erweitern (oder Stdout/Stderr setzbar machen). 2. In `EnsureRepoRoot` den vorhandenen `log`-Writer an den Clone durchreichen. 3. Sicherstellen, dass `RunLogWriter` CR-Progress korrekt als einzelne Log-Zeilen behandelt (Test `TestRunLogWriterCarriageReturn` prüfen/erweitern). 4. Gleiches Muster für andere Git-Aufrufe prüfen, die noch direkt aufs Terminal schreiben. ## Repro 1. Bare-Cache für ein Repo löschen (oder neues Repo ohne lokalen Cache watchen). 2. `forge agent watch` im interaktiven Terminal starten (TUI aktiv). 3. Beim ersten Clone den Progress beobachten — Layout bricht.

frank added the

agent:ready

label

2026-07-13 01:37:27 +02:00

agent was assigned by frank

2026-07-13 01:37:33 +02:00

frank removed the

agent:ready

label

2026-07-13 01:38:33 +02:00

frank commented

2026-07-13 01:38:39 +02:00

Author

Owner

🤖 forge agent started

Agent: cursor-agent
Model: (default)
Mode: pr
Trigger: assignee=agent

TUI: Git-Clone-Progress von bare-Repos zerstört Layout — Beim ersten Bare-Clone eines Repos (z. B. unter forge agent watch mit TUI) schreibt git clone --bare seinen Progress direkt auf das Terminal. Die Ausgabe landet am unteren Bildschirmrand,…

🤖 **forge agent started** - Agent: `cursor-agent` - Model: `(default)` - Mode: `pr` - Trigger: `assignee=agent` TUI: Git-Clone-Progress von bare-Repos zerstört Layout — Beim ersten Bare-Clone eines Repos (z. B. unter `forge agent watch` mit TUI) schreibt `git clone --bare` seinen Progress direkt auf das Terminal. Die Ausgabe landet am unteren Bildschirmrand,…

frank added the

agent:working

label

2026-07-13 01:38:39 +02:00

~~frank referenced this issue 2026-07-13 01:38:40 +02:00~~

fix(agent): route bare git clone progress into TUI log panel #51

frank referenced this issue

2026-07-13 01:38:41 +02:00

fix(agent): route bare git clone progress into TUI log panel #51

frank referenced this issue

2026-07-13 01:38:49 +02:00

fix(agent): route bare git clone progress into TUI log panel #51