Skip to content

Add failing tests for #729: BUG_STEP_TIMEOUTS mismatch#731

Draft
prompt-driven-github[bot] wants to merge 1 commit intomainfrom
fix/issue-729
Draft

Add failing tests for #729: BUG_STEP_TIMEOUTS mismatch#731
prompt-driven-github[bot] wants to merge 1 commit intomainfrom
fix/issue-729

Conversation

@prompt-driven-github
Copy link
Copy Markdown
Contributor

Summary

Adds failing tests that detect the BUG_STEP_TIMEOUTS key mismatch reported in #729.

Test Files

  • Unit tests: tests/test_agentic_bug_orchestrator.py (5 new tests)

Prompt Files

  • Prompt fix: pdd/prompts/agentic_bug_orchestrator_python.prompt (updated to 12-step numbering)

What This PR Contains

  • 5 failing unit tests that reproduce the reported bug
  • Tests verify that BUG_STEP_TIMEOUTS keys match steps_config integer keys 1–12
  • Tests confirm step 6 (Root Cause) currently gets 340s instead of the correct 600s
  • Prompt file updated to 12-step workflow to prevent future regen regressions
  • Tests are verified to fail on current code and will pass once the bug is fixed

Root Cause

BUG_STEP_TIMEOUTS dict uses the stale 5.5 numbering scheme (10 entries with keys 1–5, 5.5, 6–10) while steps_config uses 12 integer keys (1–12). PR #863 regenerated the file from a prompt that hadn't been updated, overwriting the correct 12-step mapping. Step 6 (Root Cause Analysis) gets 340s instead of 600s, causing timeout failures on complex issues.

Next Steps

  1. Restore the correct 1–12 integer-keyed BUG_STEP_TIMEOUTS mapping in pdd/agentic_bug_orchestrator.py
  2. Verify all 5 unit tests pass
  3. Run full test suite
  4. Mark PR as ready for review

Fixes #729


Generated by PDD agentic bug workflow

- 5 unit tests that verify BUG_STEP_TIMEOUTS keys match steps_config 1-12
- Tests confirm step 6 (Root Cause) gets 340s instead of correct 600s
- Update orchestrator prompt to use 12-step numbering to prevent regen regressions

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

BUG_STEP_TIMEOUTS mismatch: orchestrator regen reverted correct 12-step numbering to stale 5.5 scheme

1 participant