Repository navigation
Read discussed plans and admit subscriber estimates with own-product reviews - #615
Merged
Merged
Conversation
Preserve all 15 original Writer, Tester, and Engineer commits, including the request-changes correction. Relocate the unchanged regression into its existing parent ownership directory to meet the ten-file limit; focused checks and architecture policy pass.
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.
Outcome
Expose the authored executionPlan through authorized proposal reads. Admit each subscriber's actual estimating contribution into its discussed unestimated item while preserving peers. Required reviews default to the contributor's own reviewing profile and budget in a separate execution; archived profile rows do not block current readiness.
Work authority
Contributor mode (select one):
Plan
Preserve actual accepted source products; repair the earliest draft/own-estimate/default-review/readiness boundaries; verify the exact changed candidate through the existing owner suites and coded scenes before adoption. Simulation upstream parity is separate real group proposal f7cc50fb under Deployment834.
Changes and commits
All15 original agent commits retained under integration2b91e2d5. Coordinator974ec2aa removes accidental peer-budget matching and skips archived readiness profiles. Coordinator88d26760 safely admits first claims and denies stale scope, peer takeover and missing own estimates, pinning SDK83fa5a5d and canonical Platformcddfafc2. e8e5cac updates the original native Decision graph expectation to the human's own-product review default, adding exact own estimate authority, distinct execution identity, reviewing kind and review-estimate assertions.
Verification
29focused cases/fivefiles PASS, including original SQL/official HTTP/native Git estimate read-back and current-profile custody. Layout checks and build PASS locally. Focused development uses the existing SDK source alias and is not runtime artifact custody or live agent proof.
Prior88e652e7 passed required Verify40126144557. Changed88d26760 Verify40134663953 failed its native prerequisite:161passed/1failed across34files; the failed assertion required the obsolete opposite subscriber as reviewer. Corrected expectation retains and strengthens custody assertions. Verify40135877924 passed the complete owner verify:direct step; coded scenes recorded13passed/1failed/1044blocked. Failure was zero matched cases for an obsolete contribution test title, not a passing scene. AST inspection reproduced all three renamed contribution bindings as unmatched. Commit680057e7 binds those same verifier IDs to current case names; all three resolve and all four graph cases PASS. Fresh complete Verify40139338169 Pending; failed evidence and all remaining criteria retained. Failed observations remain retained. SDK exact-head Verify40133193601 PASS supplies the matching checked SDK83fa artifact.
Risk and rollback
No running service, issued authority, retained product or unknown usage changed. Hold adoption until complete checks pass; keep SDK/API/Agent candidates coherent. Before adoption rollback is withholding this candidate. After an authorized adoption restore prior checked APIc8e18e22 through supported development selection/rebuild, preserving retained custody and configuration.
Completion summary
Real source products plus bootstrap repairs are reviewable; complete changed-candidate verification and runtime adoption remain pending. No final merge, new implementation workday, local-upstream integration, public release or complete MCP workflow proof claimed.
Submission checklist
Retained-history repair on the current candidate
Real MCP publication of Deployment f7cc50fb v2 failed proposal_contributions_unverified after growing the API team. Native accepted bb82b9bd read-back identifies the newly subscribed API Releaser as missing; its retained graph has20stale/6blocked nodes, no running workday. The draft remains governance v2. TreeDX may already hold the unbound next revision because signal projection runs after content commit; preserve that source and retry the unchanged operation only after recovery. No successful draft publication is claimed. API Architect send-f0ce50a9bdf0edce4b59afe139122c86 authored the bounded history-isolation proposal at discussion-messages/agent-objective-planning/response-676fdd2f7d55c7cec200d28a.mdx; this coordinator repair is not an acting-agent delivery or approval.
cc2d050 adds only seven production lines: reuse the existing exact-revision frozen-history path for materialized, non-current accepted contributions without active assignments. Active, unmaterialized and missing-callback authority still fails. Exact original Decision/estimates/products/review cycles/claims/usage remain unchanged; this does not authorize replaying the old component or implement accepted-cohort snapshots.
Focused RED reproduced one inactive-history failure while three denial cases passed. After repair,22cases/3files pass with TREESEED_SDK_SOURCE_ROOT=/tmp/codex-sdk-schedule496 and vitest.control-plane.config.ts, including original SQL, official HTTP reader and real Git. Native extension adds subscriber drift, an unestimated next draft, unchanged historical snapshots, moved/denied source failures, and active/unmaterialized/no-callback denials. Four exact bindings extend the existing coded scene; the native read-back binding is preserved. npm run lint (file/architecture policies and strict build) and git diff --check pass. Initial local native invocation omitted the control-plane config and used the old installed SDK; that failure is retained, not a product pass.
680057e owner verification passed in40139338169; its coded scenes were still running when this replacement was pushed. Those results do not qualify cc2d050. Fresh complete owner/native prerequisites, coded scenes, runtime adoption and the unchanged native draft retry are pending for this exact replacement. No live origin, full simulated release or acceptance pass is claimed.
Current exact-candidate result
Verify40142447990 completed SUCCESS on cc2d050: complete owner verify:direct, original native prerequisites, PostgreSQL consumer checks, pack, and the complete coded golden component scenes all passed. Earlier failed/cancelled observations above remain historical and are not reused as qualification for this candidate. The current source was adopted for ordinary coaching through the supported development rebuild; native unchanged draft retry bound existing commit2106938b9cf0808f54fc1af3feb668e94d43d707 with changedPaths[], and the real Deployment group reached four verified estimates and native approval of v14 / Decision68de92c6-c5b4-49d7-94e4-abc8e83d858d. This does not prove acting implementation, live origin transport or full simulated release. Ready for protected staging merge after current required submission checks pass.