Job c72cf5be-55c7-4772-b577-d77ad648ed55

{"command": "cd /project && git status && git log --oneline -10 && git branch -a"}

On branch issue-53-greg
nothing to commit, working tree clean
b75b64d (HEAD -> issue-53-greg) Fix excessive vertical space in job view blocks
10c3dce (origin/main, main) Better Docker/git inter.
315f993 Add docker image build.
5717258 Merge pull request 'forgejo: repository allowlist with wildcard patterns' (#40) from issue-39-greg into main
823ae62 fix(forgejo): resolve test helper collision after merging main
bccf930 Merge branch 'main' into issue-39-greg
1365ea8 Merge pull request 'Pass the agent's git credential to the sandbox, clone inside it' (#49) from issue-48-greg into main
7c60789 Remove the git tool and Context.Git per review
c3d3720 Pass the agent's git credential to the sandbox, clone inside it
f083eae Merge pull request 'Add pr:review event so agents react to PR reviews' (#37) from issue-36-greg into main
* issue-53-greg
  main
  remotes/origin/main

{"command": "cd /project && git remote get-url origin && git diff main..issue-53-greg --stat"}

https://code.stdio.dk/abrander/zoo.git
 internal/web/templates.go |  4 ++++
 internal/web/web.go       | 12 +++++++++++-
 internal/web/web_test.go  | 47 +++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 62 insertions(+), 1 deletion(-)

{"command": "cd /project && git diff main..issue-53-greg"}

diff --git a/internal/web/templates.go b/internal/web/templates.go
index dd63248..c64e61b 100644
--- a/internal/web/templates.go
+++ b/internal/web/templates.go
@@ -968,6 +968,9 @@ const templates = `
             if (reasoningBody) reasoningBody.textContent += ev.text;
             break;
           case "reasoning_end":
+            // Model output routinely starts/ends with newlines; drop them
+            // so the finished block matches the static (trimmed) view.
+            if (reasoningBody) reasoningBody.textContent = reasoningBody.textContent.trim();
             reasoningBody = null;
             break;
           case "content_start":
@@ -977,6 +980,7 @@ const templates = `
             if (contentBody) contentBody.textContent += ev.text;
             break;
           case "content_end":
+            if (contentBody) contentBody.textContent = contentBody.textContent.trim();
             contentBody = null;
             break;
           case "tool":
diff --git a/internal/web/web.go b/internal/web/web.go
index eeec747..55ab97f 100644
--- a/internal/web/web.go
+++ b/internal/web/web.go
@@ -7,6 +7,7 @@ import (
 	"fmt"
 	"html/template"
 	"net/http"
+	"strings"
 	"sync"
 	"time"
 
@@ -210,7 +211,16 @@ func buildBlocks(logs []store.LogLine) []block {
 	for _, l := range logs {
 		switch l.Stream {
 		case "reasoning", "content":
-			blocks = append(blocks, block{Kind: l.Stream, Text: l.Line})
+			// Model output routinely starts/ends with newlines. The block
+			// body renders with white-space: pre-wrap, so those would show
+			// up as visible blank lines inflating the block's height. Trim
+			// them for display (internal newlines are kept) and drop
+			// blocks that are nothing but whitespace.
+			text := strings.TrimSpace(l.Line)
+			if text == "" {
+				continue
+			}
+			blocks = append(blocks, block{Kind: l.Stream, Text: text})
 
 		case "tool":
 			var entry store.ToolLogEntry
diff --git a/internal/web/web_test.go b/internal/web/web_test.go
index 88974a8..ded09b2 100644
--- a/internal/web/web_test.go
+++ b/internal/web/web_test.go
@@ -104,6 +104,53 @@ func TestJobsAndDetail(t *testing.T) {
 	}
 }
 
+// TestJobDetailTrimsBlockWhitespace verifies that reasoning/content
+// blocks are rendered without the leading/trailing newlines the model
+// routinely emits: the block body uses white-space: pre-wrap, so those
+// would show up as visible blank lines inflating the block's height
+// (issue #53). Internal newlines must be preserved.
+func TestJobDetailTrimsBlockWhitespace(t *testing.T) {
+	s, st := testServer(t)
+	ctx := context.Background()
+
+	if err := st.CreateJob(ctx, store.Job{ID: "job-1", EventKind: "issue:new", Agent: "leon", Owner: "acme", Repo: "widgets", IssueIndex: 1}); err != nil {
+		t.Fatal(err)
+	}
+	if err := st.MarkJobFinished(ctx, "job-1", store.JobSucceeded, ""); err != nil {
+		t.Fatal(err)
+	}
+	if err := st.AppendLog(ctx, "job-1", "content", "\nNow the remaining tool callers:\n\n\n"); err != nil {
+		t.Fatal(err)
+	}
+	// A block that is nothing but whitespace should not render at all.
+	if err := st.AppendLog(ctx, "job-1", "content", "   \n\t  "); err != nil {
+		t.Fatal(err)
+	}
+	if err := st.AppendLog(ctx, "job-1", "reasoning", "  \ninner\nlines\nkept\n  "); err != nil {
+		t.Fatal(err)
+	}
+
+	r := httptest.NewRecorder()
+	s.Handler().ServeHTTP(r, httptest.NewRequest(http.MethodGet, "/jobs/job-1", nil))
+	if r.Code != http.StatusOK {
+		t.Fatalf("expected 200, got %d: %s", r.Code, r.Body.String())
+	}
+
+	body := r.Body.String()
+	if !strings.Contains(body, `<div class="block-body">Now the remaining tool callers:</div>`) {
+		t.Fatalf("expected trimmed content block, got: %s", body)
+	}
+	if strings.Contains(body, "<div class=\"block-body\">\n") {
+		t.Fatalf("block body still starts with a newline: %s", body)
+	}
+	if got := strings.Count(body, `class="block block-content"`); got != 1 {
+		t.Fatalf("expected exactly one content block (whitespace-only one dropped), got %d: %s", got, body)
+	}
+	if !strings.Contains(body, "inner\nlines\nkept") {
+		t.Fatalf("expected internal newlines to be preserved, got: %s", body)
+	}
+}
+
 func TestAuthGate(t *testing.T) {
 	st, err := store.Open(filepath.Join(t.TempDir(), "zoo.db"))
 	if err != nil {

{"command": "cd /project && go build ./... && go test ./... 2>&1 | tail -20", "timeout": 300}

go: downloading github.com/docker/docker v28.5.2+incompatible
go: downloading github.com/hashicorp/hcl/v2 v2.24.0
go: downloading codeberg.org/mvdkleijn/forgejo-sdk/forgejo/v2 v2.2.0
go: downloading modernc.org/sqlite v1.56.0
go: downloading github.com/google/uuid v1.6.0
go: downloading github.com/docker/go-connections v0.8.1
go: downloading github.com/docker/go-units v0.5.0
go: downloading github.com/moby/docker-image-spec v1.3.1
go: downloading github.com/opencontainers/image-spec v1.1.1
go: downloading github.com/containerd/errdefs v1.0.0
go: downloading github.com/containerd/errdefs/pkg v0.3.0
go: downloading github.com/distribution/reference v0.6.0
go: downloading github.com/opencontainers/go-digest v1.0.0
go: downloading github.com/pkg/errors v0.9.1
go: downloading go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.70.0
go: downloading go.opentelemetry.io/otel/trace v1.45.0
go: downloading go.opentelemetry.io/otel v1.45.0
go: downloading github.com/42wim/httpsig v1.2.3
go: downloading github.com/go-fed/httpsig v1.1.0
go: downloading github.com/hashicorp/go-version v1.7.0
go: downloading golang.org/x/crypto v0.39.0
go: downloading github.com/felixge/httpsnoop v1.1.0
go: downloading go.opentelemetry.io/otel/metric v1.45.0
go: downloading github.com/agext/levenshtein v1.2.1
go: downloading github.com/apparentlymart/go-textseg/v15 v15.0.0
go: downloading github.com/mitchellh/go-wordwrap v1.0.1
go: downloading github.com/zclconf/go-cty v1.16.3
go: downloading github.com/go-logr/logr v1.4.4
go: downloading golang.org/x/sys v0.47.0
go: downloading github.com/google/go-cmp v0.7.0
go: downloading github.com/go-logr/stdr v1.2.2
go: downloading go.opentelemetry.io/auto/sdk v1.2.1
go: downloading github.com/cespare/xxhash/v2 v2.3.0
go: downloading golang.org/x/text v0.40.0
go: downloading modernc.org/libc v1.74.4
go: downloading github.com/dustin/go-humanize v1.0.1
go: downloading modernc.org/mathutil v1.7.1
go: downloading modernc.org/memory v1.11.0
go: downloading github.com/remyoudompheng/bigfft v0.0.0-20230129092748-24d4a6f8daec
?   	github.com/abrander/zoo/cmd/zoo	[no test files]
ok  	github.com/abrander/zoo/internal/agentrun	0.004s
ok  	github.com/abrander/zoo/internal/config	0.006s
ok  	github.com/abrander/zoo/internal/forgejo	1.054s
ok  	github.com/abrander/zoo/internal/livelog	0.052s
ok  	github.com/abrander/zoo/internal/llm	0.003s
ok  	github.com/abrander/zoo/internal/scheduler	2.687s
ok  	github.com/abrander/zoo/internal/store	1.733s
ok  	github.com/abrander/zoo/internal/tools	0.003s
ok  	github.com/abrander/zoo/internal/web	1.018s

{"command": "cd /project && git fetch origin && git checkout main && git pull origin main && git log --oneline -3"}

Switched to branch 'main'
Your branch is up to date with 'origin/main'.
From https://code.stdio.dk/abrander/zoo
 * branch            main       -> FETCH_HEAD
Already up to date.
10c3dce (HEAD -> main, origin/main, origin/HEAD) Better Docker/git inter.
315f993 Add docker image build.
5717258 Merge pull request 'forgejo: repository allowlist with wildcard patterns' (#40) from issue-39-greg into main

{"command": "cd /project && git merge --no-ff issue-53-greg -m \"Merge pull request 'Fix excessive vertical space in job view blocks (#53)' (#55) from issue-53-greg into main\"; echo \"exit: $?\"; git status"}

Already up to date.
exit: 0
On branch main
Your branch is ahead of 'origin/main' by 2 commits.
  (use "git push" to publish your local commits)

nothing to commit, working tree clean

{"command": "cd /project && git log --oneline -5 && git push origin main; echo \"push exit: $?\""}

ee6c5af (HEAD -> main) Merge pull request 'Fix excessive vertical space in job view blocks (#53)' (#55) from issue-53-greg into main
b75b64d (issue-53-greg) Fix excessive vertical space in job view blocks
10c3dce (origin/main, origin/HEAD) Better Docker/git inter.
315f993 Add docker image build.
5717258 Merge pull request 'forgejo: repository allowlist with wildcard patterns' (#40) from issue-39-greg into main
Enumerating objects: 1, done.
Counting objects: 100% (1/1)
Counting objects: 100% (1/1), done.
Writing objects: 100% (1/1)
Writing objects: 100% (1/1), 288 bytes | 288.00 KiB/s, done.
Total 1 (delta 0), reused 0 (delta 0), pack-reused 0 (from 0)
remote: 
remote: Forgejo: Not allowed to push to protected branch main
To https://code.stdio.dk/abrander/zoo.git
 ! [remote rejected] main -> main (pre-receive hook declined)
error: failed to push some refs to 'https://code.stdio.dk/abrander/zoo.git'
push exit: 1

{"command": "cd /project && git checkout main && git reset --hard origin/main && git log --oneline -3 && git status"}

Already on 'main'
Your branch is ahead of 'origin/main' by 2 commits.
  (use "git push" to publish your local commits)
HEAD is now at 10c3dce Better Docker/git inter.
10c3dce (HEAD -> main, origin/main, origin/HEAD) Better Docker/git inter.
315f993 Add docker image build.
5717258 Merge pull request 'forgejo: repository allowlist with wildcard patterns' (#40) from issue-39-greg into main
On branch main
Your branch is up to date with 'origin/main'.

nothing to commit, working tree clean

{"body": "Thanks for the approval, @abrander! 🎉\n\nThere was no inline feedback to address, so no code changes were needed. Before attempting the merge I re-verified the branch: `go build ./...` and `go test ./...` pass cleanly (including the new `TestJobDetailTrimsBlockWhitespace`).\n\nI did try to merge as requested, but the push was rejected:\n\n```\nremote: Forgejo: Not allowed to push to protected branch main\n```\n\nMy credentials don't have push access to the protected `main` branch, so I can't complete the merge myself. Could you hit the merge button on #55 (or add the agent account to the branch-protection allowlist for `main` if you'd like me to be able to do this going forward)? The branch is ready to go — `issue-53-greg` @ `b75b64d`, approved and green."}

Comment posted