fix: Add PYTHONPATH and Phase 3 workflows to Workflows repo#694
fix: Add PYTHONPATH and Phase 3 workflows to Workflows repo#694
Conversation
- Added missing PYTHONPATH env var to agents-issue-optimizer.yml (Phase 1, 2, dedup) - Copied agents-capability-check.yml, agents-decompose.yml, agents-dedup.yml from template - Adapted all workflows for Workflows repo (removed self-checkout, fixed paths) - Fixed agents-auto-label.yml checkout and paths - Created SHORT_TERM_PLAN.md and SHORT_TERM_PLAN_SUMMARY.md Fixes: agents:optimize failing with 'No module named tools' error on issue #691
Automated Status SummaryHead SHA: ab2f414
Coverage Overview
Coverage Trend
Top Coverage Hotspots (lowest coverage)
Updated automatically; will refresh on subsequent CI/Docker completions. Keepalive checklistScopePart of Phase 3 workflow rollout validation per langchain-post-code-rollout.md Tasks
Acceptance criteria
|
🤖 Keepalive Loop StatusPR #694 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
There was a problem hiding this comment.
Pull request overview
This PR fixes critical issues preventing agent workflows from running in the Workflows repository itself by adding missing PYTHONPATH environment variables and copying Phase 3 workflows from the consumer template. The changes adapt these workflows to work directly in the Workflows repo by removing self-checkout logic and adjusting paths.
Key changes:
- Added PYTHONPATH environment variable to agents-issue-optimizer.yml phases to fix ModuleNotFoundError
- Copied and adapted three Phase 3 workflows (capability-check, decompose, dedup) to work in the Workflows repo
- Fixed agents-auto-label.yml to remove unnecessary self-checkout logic
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/agents-issue-optimizer.yml | Added PYTHONPATH to three workflow steps to fix Python module import errors |
| .github/workflows/agents-capability-check.yml | New workflow to check if agents can complete tasks before assignment |
| .github/workflows/agents-decompose.yml | New workflow to break down large issues into actionable sub-tasks |
| .github/workflows/agents-dedup.yml | New workflow to detect potential duplicate issues using semantic similarity |
| .github/workflows/agents-auto-label.yml | Removed self-checkout logic and updated to Python 3.11 for Workflows repo |
| docs/plans/SHORT_TERM_PLAN_SUMMARY.md | New planning document with 2-week execution summary |
| docs/plans/SHORT_TERM_PLAN.md | New detailed planning document for Phase 3 completion |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Remove ../ prefix from file paths in capability-check, decompose, and dedup workflows. Consumer template uses ../ because workflows-repo is a subdirectory, but Workflows repo itself doesn't need the prefix.
- Add agents-capability-check.yml to workflow docs - Add agents-decompose.yml to workflow docs - Add agents-dedup.yml to workflow docs Fixes remaining test failures in PR #694
Per Copilot review comment - this was missing PYTHONPATH which could cause the same ModuleNotFoundError this PR is fixing. All other workflows in this PR have PYTHONPATH set, this one was overlooked. Co-authored-by: copilot-pull-request-reviewer
- agents-capability-check.yml - agents-decompose.yml - agents-dedup.yml Fixes test_canonical_workflow_names_match_expected_mapping failure
|
Status | ✅ no new diagnostics |
Automated Status Summary
Scope
Part of Phase 3 workflow rollout validation per langchain-post-code-rollout.md
Tasks
Acceptance criteria
Head SHA: d91670b
Latest Runs: ❔ in progress — Gate
Required: gate: ❔ in progress