fix: stabilize extension turns, transcript hydration, and reasoning controls - #1431
Open
effortprogrammer wants to merge 13 commits into
Open
fix: stabilize extension turns, transcript hydration, and reasoning controls#1431effortprogrammer wants to merge 13 commits into
effortprogrammer wants to merge 13 commits into
Conversation
Fixes code-yeongyu#1329. Run and apply before_agent_start for idle custom-trigger turns, retain final admission, and prevent provider calls after cancellation during an awaited hook.
Fixes code-yeongyu#1330. Detect bounded numeric prose drift, retain tool-progress and code-fence guards, and wait for hooked recovery turns before printing their final result (code-yeongyu#1329).
Fixes code-yeongyu#1076. Keep incomplete renders on the initial visible boundary while warming caches, preserve live appends and resize, and publish full history once ready.
Fixes code-yeongyu#891. Constrain 58 built-in catalog entries to implemented controls, preserve graded models, remove the stale Xiaomi override, and regenerate only selected provider artifacts with validated selectors.
Integrate origin/main at 09611c6 and retain all four fixes. Align the remaining Xiaomi characterization with its documented disabled-thinking request.
Integrate upstream and fork main at 89a06b0. Preserve the four fixes, regenerate the retained Moonshot capability maps, and assert that upstream-retired model rows stay absent.
Preserve latest overflow and SDK recovery changes, Astra context caps, and the four verified issue fixes. Regenerate conflicting OpenCode data and retain both sets of tracker entries.
Include the upstream RPC watchdog ownership repair and current workspace tooling. Preserve both transcript and report-bug tracker entries. Standard quality, full build, and 58 related regressions passed before this integration commit.
Close each debugger only after its own resume acknowledgement and preserve the two-session fixed-port handoff assertions. Include the erasable-syntax correction in this verified increment.
Preserve the trust, image-compaction and script updates while restoring the correct tracker section boundaries. Quality, full build and 45 related tests passed with one existing platform skip.
Document the verified reasoning controls, extension-turn hooks and atomic transcript hydration under Unreleased without changing released history or weakening policy gates.
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
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.
Summary
Fixes #1329, #1076, and #891. Addresses the numeric-counter near-repeat case in #1330 with progress and code-safety guards.
before_agent_startfor idle custom-trigger turns. Preserve final admission, cancel held hooks without a ghost provider call, and wait for deferred recovery before print-mode output.The fork was synchronized with upstream
89a06b08, and that commit is merged here as758ee7a0. Upstream-retired Moonshot models were not restored: the affected capability table now covers 52 retained entries, with explicit tests for the retired rows.Verification
bun run checkand the complete workspace build passed on the latest integration.One local full-workspace run on macOS/Node 26 recorded failures in 12 coding-agent files, dominated by test/hook timeouts and a child-process completion failure. The same 12 files then passed 109 tests in an isolated single-worker run on unchanged code, without increasing timeouts or skipping tests. The broad-run log is retained; it is not being presented as a full-suite pass. Linux/Node 24 PR CI is a separate verification gate.
Reproducible drivers are included under
.agents/skills/senpi-qa/scripts/scenarios/:The last driver requires Bun >=1.4 on PATH (or selected through
SENPI_QA_BUN) and the skill-localnode-ptydependency.Maintainer note
CONTRIBUTING.mdreservesCHANGELOG.mdentries for maintainers, so no contributor changelog edit is included. Exact nearestchanges.mdcoverage is present. The runtime-change changelog gate needs the corresponding maintainer entry or explicit maintainer policy handling; this user-facing change is not being mislabeled asno-changelog.Capability QA proves Senpi's catalog, UI, persistence, and request controls. It does not claim live account entitlement or provider availability for every retained model.
Summary by cubic
Fixes #1329, #1076, #891, and the numeric near-repeat case in #1330. Idle custom-trigger turns now run
before_agent_start, print mode waits for recovery turns, and hydrated transcript history publishes atomically instead of exposing partial prefixes.Behavior changes
send_custom_messageturns apply admission hooks and cancel without a ghost provider call.generate-models.tssupports--providersfor provider-scoped regeneration, leaving unselected files byte-identical.Verification and rollout
.agents/skills/senpi-qa/scripts/scenarios/plus regression tests inpackages/aiandpackages/coding-agent.changes.mdcoverage is included.Written for commit 762d19a. Summary will update on new commits.