Skip to content

fix(#61): add summary length guidance and increase schema limit - #62

Merged
rh-hemartin merged 1 commit into
mainfrom
agent/61-summary-length-guidance
Jul 8, 2026
Merged

fix(#61): add summary length guidance and increase schema limit#62
rh-hemartin merged 1 commit into
mainfrom
agent/61-summary-length-guidance

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Add conciseness guidance to the retro-analysis skill for evidence notes routed into the summary field: one sentence per existing issue, prioritize most impactful evidence if space constrained.

Increase summary maxLength in retro-result.schema.json from 16,384 to 32,768 to provide additional headroom. GitHub supports up to 65,536 chars in issue comments, so this is within platform limits.

Both changes address the overflow risk introduced by PR #59, which routed all evidence for existing issues into summary without length guidance or increased schema capacity.


Closes #61

Post-script verification

  • Branch is not main/master (agent/61-summary-length-guidance)
  • Secret scan passed (gitleaks — 19a59848232667f38610e8c53967cb1b0d2a1d5d..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

Add conciseness guidance to the retro-analysis skill for evidence
notes routed into the summary field: one sentence per existing
issue, prioritize most impactful evidence if space constrained.

Increase summary maxLength in retro-result.schema.json from
16,384 to 32,768 to provide additional headroom. GitHub supports
up to 65,536 chars in issue comments, so this is within platform
limits.

Both changes address the overflow risk introduced by PR #59,
which routed all evidence for existing issues into summary
without length guidance or increased schema capacity.

Closes #61
@fullsend-ai-coder
fullsend-ai-coder Bot requested a review from a team as a code owner July 8, 2026 15:09
@rh-hemartin
rh-hemartin added this pull request to the merge queue Jul 8, 2026
Merged via the queue into main with commit 65dc152 Jul 8, 2026
9 checks passed
@fullsend-ai-retro

fullsend-ai-retro Bot commented Jul 8, 2026

Copy link
Copy Markdown

🤖 Finished Retro · ✅ Success · Started 3:14 PM UTC · Completed 3:19 PM UTC
Commit: 14bfba8 · View workflow run →

@fullsend-ai-retro

Copy link
Copy Markdown

Retro: PR #62 — summary length guidance and schema limit increase

Pipeline performance: excellent. PR #62 is a clean, minimal fix that addressed issue #61 (filed by the retro agent after analyzing PR #59). The full cycle — from retro detecting the overflow risk on PR #59 to PR #62 merging the fix — took 25 minutes, including human review and approval. The coder agent produced a single commit with 2 lines changed across 2 files, implementing both recommended options from the issue (conciseness guidance in the skill + doubling the schema maxLength).

No new proposals. The improvement opportunities identified below are already tracked in existing issues.

Evidence for existing issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(schema): add summary length guidance or increase maxLength to prevent evidence-routing overflow

1 participant