Skip to content

Prevent duplicate research idea portfolios - #124

Merged
Yushangjinghong2 merged 1 commit into
mainfrom
fix/research-idea-portfolio-uniqueness
Sep 14, 2026
Merged

Prevent duplicate research idea portfolios#124
Yushangjinghong2 merged 1 commit into
mainfrom
fix/research-idea-portfolio-uniqueness

Conversation

@lbx154

@lbx154 lbx154 commented Sep 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • expose the runtime-owned canonical Idea portfolio path in the Engineer prompt immediately after preparation
  • reject Idea-stage Planner tasks that attempt to own a competing portfolio or tournament under .argus/teams
  • preserve ordinary bounded Team use and avoid false positives for non-portfolio tasks
  • add regression coverage for prompt propagation and the historical duplicate-path contract

Root cause

The research runtime formed the canonical portfolio during library preparation, but only emitted an event. The Planner could still assign a different team path, and the Engineer never received the canonical root in its prompt, so it interpreted the empty alternate path as authorization to form a second portfolio.

Testing

  • 69 passed in the focused vertical/portfolio/team suite
  • Ruff passed on all changed files
  • full suite reached 10660 passed, 61 skipped; the only six failures were missing optional SciencePlots locally
  • after installing the declared plotting dependency, tests/skills/test_paper_chart_style.py: 23 passed
  • git diff --check passed

Expose the runtime-owned canonical portfolio to the Engineer and reject Planner tasks that attempt to create a competing Idea-stage portfolio.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f3fbb53a-0adf-4d28-8b03-e2d941d62608
@Yushangjinghong2
Yushangjinghong2 merged commit 2b9cfc1 into main Sep 14, 2026
5 of 6 checks passed
@lbx154
lbx154 deleted the fix/research-idea-portfolio-uniqueness branch September 14, 2026 12:42
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.

2 participants