fix(activity-log): strip PR-description block from reasoning trail #49

Merged
ffaerber merged 2 commits from fix/activity-log-strip-prdesc into main 2026-07-04 13:25:30 +02:00
Owner

The activity log repeated the BEGIN_PR_DESCRIPTION block that is already shown as the PR description (e.g. homelab PR #117). This strips it from the reasoning trail and drops a now-empty entry. Verified with a sample event stream.

The activity log repeated the BEGIN_PR_DESCRIPTION block that is already shown as the PR description (e.g. homelab PR #117). This strips it from the reasoning trail and drops a now-empty entry. Verified with a sample event stream.
ffaerber added 1 commit 2026-07-04 13:12:33 +02:00
The 'tool calls & reasoning' activity log dumped each text part verbatim, including the agent's
BEGIN_PR_DESCRIPTION..END_PR_DESCRIPTION block — which is already published as the PR description, so
it appeared twice (e.g. homelab PR #117). Strip that block from text parts, and drop a text entry
that becomes empty after stripping.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ffaerber added 1 commit 2026-07-04 13:24:39 +02:00
Batches the remaining comment-noise cleanups on top of the activity-log fix:
- Resume comment no longer re-posts the full PR description (## Summary/## Changes) — the PR body and
  diff already carry it; the comment just links the PR (the reasoning trail shows what the run did).
- Strip a leading self-header the model sometimes emits ("🤖 **@pm**" on its own line, sometimes
  twice) before we prepend our own, so headers aren't doubled.
- Also drop the BEGIN_PR_DESCRIPTION block from the plain reply text, and simplify the activity
  block's summary (it repeated "🤖 **@name** — activity log").

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ffaerber merged commit cb11a6b2d1 into main 2026-07-04 13:25:30 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gitea/agents#49