Skip to content

test: verify scoped project instructions reach plan workers - #1906

Merged
santoshkumarradha merged 3 commits into
devfrom
fix/plan-workers-standing-orders
Oct 23, 2026
Merged

santoshkumarradha merged 3 commits into
devfrom
fix/plan-workers-standing-orders

Conversation

@santoshkumarradha

Copy link
Copy Markdown
Member

Problem and current behavior

#1549 remained open even though the production fix landed in #1627 (commit ac84606f4). Current plan workers receive the applicable project instructions through the saved memory-rule context. This PR changes tests only and verifies that behavior through the actual terminal and provider request path.

The scripted endpoint checks both the root worker and checker openings. Each must contain the current project's explicit instruction and exclude an unrelated project's instruction. The generated artifact depends on the instruction actually received; the checker verifies it without rewriting it.

Verification

  • make pr-ready passed; all selected tests passed on their first run.
  • Scripted actual-terminal workflow passed: both plan assignments received the correct scoped rule, excluded the foreign rule, completed the plan and verified the actual artifact.
  • Live OpenRouter workflow passed: inspect saved instructions, delegate a two-file task, finish the plan and execute the recorded verification command successfully. All 32 model-call receipts used exactly deepseek/deepseek-v4.1-flash.
  • Four existing focused regressions for rule forwarding also passed.

Examples and artifacts use mock project instructions only. No production behavior or user-facing feature is added.

Closes #1549

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.

session: plan-born task worker briefs omit standing project orders

1 participant