Skip to content

fix(collector): Preserve integrate patch proposal attribution - #1085

Merged
chaojhou merged 6 commits into
mainfrom
fix/integrate-patch-proposal-attribution
Aug 4, 2026
Merged

fix(collector): Preserve integrate patch proposal attribution#1085
chaojhou merged 6 commits into
mainfrom
fix/integrate-patch-proposal-attribution

Conversation

@meinali-566

Copy link
Copy Markdown
Contributor

Summary

  • carry specialist proposal phase, domain, and provenance through integrate_patch execution, promotion, and resume replay
  • classify delayed serving-specialist KEEPs by proposal ownership instead of the phase active when execution finishes
  • keep true kernel integrate adoptions on the Kernel Agent path and surface ownerless integrate_patch gains as unattributed

Test plan

  • python3 -m pytest -q src/hyperloom/inference_optimizer/tests/test_breakdown*.py src/hyperloom/inference_optimizer/tests/test_reporters*.py src/hyperloom/inference_optimizer/tests/test_sbd_optimizations.py src/hyperloom/inference_optimizer/tests/test_phase_state_plateau.py (170 passed)
  • run the new async producer and resume attribution regressions directly in the local environment
  • python3 -m compileall -q src/hyperloom/inference_optimizer/breakdown src/hyperloom/orchestrator
  • git diff --check

Made with Cursor

Carry proposal ownership through delayed execution and resume so serving-specialist KEEPs are not misclassified by their completion phase.

Co-authored-by: Cursor <cursoragent@cursor.com>
@meinali-566
meinali-566 requested a review from a team as a code owner August 3, 2026 09:56
@meinali-566 meinali-566 changed the title Preserve integrate patch proposal attribution fix: Preserve integrate patch proposal attribution Aug 3, 2026
@meinali-566 meinali-566 changed the title fix: Preserve integrate patch proposal attribution fix(collector): Preserve integrate patch proposal attribution Aug 3, 2026
meinali-566 and others added 3 commits August 3, 2026 10:05
Stamp direct Framework Agent KEEPs with their owning phase so delayed writeback cannot reclassify them by completion context.

Co-authored-by: Cursor <cursoragent@cursor.com>
Require explicit proposal ownership across every collector surface, keep missing owners unattributed, and avoid forwarding metadata that can alter accuracy gating.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keep ownerless integrate_patch gains consistently unattributed across every collector surface instead of guessing from execution timing.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

CI E2E report — ✅ Succeeded

item value
result ✅ Succeeded
model Qwen/Qwen3-0.6B (dense)
resources 1× GPU, TP=1
PR branch fix/integrate-patch-proposal-attribution
commit 79857a486da6b28ff7d7fb416a7f29ec07a92e47
session_id d219800d-afbb-4c32-a0ff-14e51fc4fede
queue → dispatch 1s
run time 153m 47s
total 153m 48s

details

@ZhengGong-amd ZhengGong-amd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

meinali-566 and others added 2 commits August 4, 2026 02:09
Preserve proposal ownership metadata while adopting the latest current-best lift guard and upstream runtime changes.

Co-authored-by: Cursor <cursoragent@cursor.com>
Persist source snapshot manifests and changed-file paths through promotion and resume so downstream consumers can inspect adopted source changes.

Co-authored-by: Cursor <cursoragent@cursor.com>
@chaojhou
chaojhou merged commit a56bf46 into main Aug 4, 2026
25 of 26 checks passed
@chaojhou
chaojhou deleted the fix/integrate-patch-proposal-attribution branch August 4, 2026 02:51
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.

3 participants