Repository navigation
fix: complete V3 native flows and harden matched evaluation - #326
Draft
EvanProgramming wants to merge 16 commits into
Draft
EvanProgramming wants to merge 16 commits into
EvanProgramming wants to merge 16 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
This branch has not been deployed
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.
Native Plan/Auto turns still depended on textual controls, and native execution could lose task identity, miss context costs, or report unsuccessful work as completed. Interrupted mutations and background processes also lacked sufficient cancellation and restart evidence.
This draft is stacked on #325 and contains the V3 finalization repairs:
Validation on clean signed candidate
469d76e7c6b758f4c12e2181d673ef0ab6191388: 1,374 full local tests passed (two Linux-only sandbox skips on macOS), with 530 tracked-file hashes unchanged. Check/lint/docs, Go tests, real PTY TUI and agent/subagent acceptance, retained wheel/sdist installations outside the checkout, and Docker persistence passed. GitHub verified all 16 commits. Exact-candidate six-environment CI and >=1,800-second soaks are running in 42292145829; Windows baseline/CPU/SQLite/combined controls are running in 42292145962. Historical load controls completed 80 passing observations; these do not establish the original timeout's cause.Release remains blocked. Same-process child actor authentication and detached macOS descendant ownership are unproven; full campaign preflight stops before credential lookup or paid requests. R260 remains unattributed. Current routing benefit ablations, the live 180-row comparison, all 17 passing signed records and same-SHA main integration are outstanding. Historical DeepSeek exposure re-audits to CNY 0.582914960007; no external DeepSeek calls were added. No tags, draft/public Releases, package publication or preview publisher were performed.
Related: #227–#268 and #319. Keep this draft and main integration blocked until the formal requirements are satisfied.