Skip to content

Preserve all four sub-agent report headings with None. for empty sections - #934

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-7494-critic-envelope-preserve-headings
Sep 12, 2026
Merged

Preserve all four sub-agent report headings with None. for empty sections#934
TheGreatAxios merged 1 commit into
mainfrom
cl-7494-critic-envelope-preserve-headings

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • The leaf report contract requires all four envelope headings in order, with None. under headings that have nothing to report
  • The report formatter always renders Summary, Findings, Blockers, and Paths, substituting None. for empty Findings, Blockers, and Paths bodies

Verification

  • bun test src/subagent/index.test.ts src/agent/prompts.test.ts passes (80 pass, 0 fail)
  • bun run check passes (6981 pass, 0 fail)

Fixes CL-7494

@linear-code

linear-code Bot commented Sep 12, 2026

Copy link
Copy Markdown

CL-7494

@TheGreatAxios
TheGreatAxios merged commit 14cbcfb into main Sep 12, 2026
9 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-7494-critic-envelope-preserve-headings branch September 12, 2026 05:04
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.

1 participant