fix: bound review context and preserve worker diagnostics - #65
Merged
Conversation
CBEPX
marked this pull request as ready for review
July 2, 2026 14:57
sf-jin-ku
force-pushed
the
fork/long-untracked-context-cap
branch
from
July 17, 2026 10:05
b78145d to
13a60ea
Compare
sf-jin-ku
changed the title
sf-jin-ku
approved these changes
Jul 17, 2026
sf-jin-ku
left a comment
sf-jin-ku
left a comment
Contributor
There was a problem hiding this comment.
Reviewed the amended contributor branch. The original broad change was reduced to two root-cause fixes: a 32 KiB aggregate untracked-context cap and persistent detached-worker diagnostics. Focused unit/integration tests, the full local npm test/E2E suite, lint, typecheck, Hawkman, and Semgrep all pass. Contributor-authored commits are preserved.
This was referenced Jul 17, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Maintainer-scoped version of the original maintenance bundle for v1.3.0. This PR now keeps two independently verified fixes from @CBEPX: (1) cap aggregate untracked review context at 32 KiB and tell reviewers how to inspect omitted files, and (2) redirect detached worker stdout/stderr into the existing job log. The original transfer, MCP diagnostics, Fable/model fallback, rate-limit classifier, mutation-testing, and 1.3.0 metadata bundle was removed because those concerns need separate specifications and focused PRs. The retained commits preserve CBEPX authorship. Verification: lint, typecheck, full unit tests, focused integration tests, and diff check.