Skip to content

fix: agents-issue-optimizer checkout Workflows repo for scripts - #634

Merged
stranske merged 4 commits into
mainfrom
fix/issue-optimizer-checkout
Jan 7, 2026
Merged

stranske merged 4 commits into
mainfrom
fix/issue-optimizer-checkout

Conversation

@stranske

@stranske stranske commented Jan 7, 2026

Copy link
Copy Markdown
Owner

Source: Issue #184

Automated Status Summary

Scope

Address unmet acceptance criteria from PR #183.

Original scope:

  • Scope section missing from source issue.

Context for Agent

Related Issues/PRs

References

Tasks

  • Tasks section missing from source issue.

Acceptance criteria

  • Acceptance criteria section missing from source issue.

Head SHA: 4d878df
Latest Runs: ❔ in progress — Gate
Required: gate: ❔ in progress

Workflow / Job Result Logs
Agents PR meta manager ❔ in progress View run
Auto-label Dependabot PRs ⏭️ skipped View run
CI Autofix Loop ✅ success View run
Copilot code review ❔ in progress View run
Gate ❔ in progress View run
Health 40 Sweep ✅ success View run
Health 44 Gate Branch Protection ✅ success View run
Health 45 Agents Guard ✅ success View run
Health 50 Security Scan ✅ success View run
Maint 52 Validate Workflows ✅ success View run
PR 11 - Minimal invariant CI ✅ success View run
Selftest CI ✅ success View run
Validate Sync Manifest ✅ success View run

…re mode

- List available models in descriptions (GitHub Models and OpenAI)
- Set compare mode defaults: gpt-4o (GitHub) + o1-mini (OpenAI)
- Compare mode uses high-quality models from different providers by default
- Updated compare mode defaults to use gpt-5 from both providers
- Reordered model lists to show GPT-5 series first (current quality models)
- GPT-5 is the current high-quality model, not o1-mini or gpt-4o
- Set compare mode to use gpt-4o (GitHub) vs gpt-5.2 (OpenAI)
- Add gpt-5.2 to OpenAI model list in descriptions
- Verified gpt-5.2 works with OpenAI API
The consumer repo workflow was checking out the local repo but then
trying to run scripts/langchain/* from Workflows. Fix by explicitly
checking out stranske/Workflows repo where the scripts exist.

Also updated langchain-post-code-rollout.md with current status.
Copilot AI review requested due to automatic review settings January 7, 2026 05:05
@github-actions

github-actions Bot commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

Automated Status Summary

Head SHA: f00a5ee
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / Enforce agents workflow protections
Required: core tests (3.11): ⏳ pending, core tests (3.12): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job Result Logs
(no jobs reported) ⏳ pending

Coverage Overview

  • Coverage history entries: 1

Coverage Trend

Metric Value
Current 92.21%
Baseline 85.00%
Delta +7.21%
Minimum 70.00%
Status ✅ Pass

Top Coverage Hotspots (lowest coverage)

File Coverage Missing
scripts/workflow_health_check.py 62.6% 28
scripts/classify_test_failures.py 62.9% 37
scripts/ledger_validate.py 65.3% 63
scripts/mypy_return_autofix.py 82.6% 11
scripts/ledger_migrate_base.py 85.5% 13
scripts/fix_cosmetic_aggregate.py 92.3% 1
scripts/coverage_history_append.py 92.8% 2
scripts/workflow_validator.py 93.3% 4
scripts/update_autofix_expectations.py 93.9% 1
scripts/pr_metrics_tracker.py 95.7% 3
scripts/generate_residual_trend.py 96.6% 1
scripts/build_autofix_pr_comment.py 97.0% 2
scripts/aggregate_agent_metrics.py 97.2% 0
scripts/fix_numpy_asserts.py 98.1% 0
scripts/sync_test_dependencies.py 98.3% 1

Updated automatically; will refresh on subsequent CI/Docker completions.


Keepalive checklist

Scope

No scope information available

Tasks

  • No tasks defined

Acceptance criteria

  • No acceptance criteria defined

@stranske
stranske merged commit 22ef579 into main Jan 7, 2026
98 checks passed
@stranske
stranske deleted the fix/issue-optimizer-checkout branch January 7, 2026 05:07
@github-actions

github-actions Bot commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

🤖 Keepalive Loop Status

PR #634 | Agent: Codex | Iteration 0/5

Current State

Metric Value
Iteration progress [----------] 0/5
Action wait (missing-agent-label)
Disposition skipped (transient)
Gate success
Tasks 0/2 complete
Keepalive ❌ disabled
Autofix ❌ disabled

🔍 Failure Classification

| Error type | infrastructure |
| Error category | resource |
| Suggested recovery | Confirm the referenced resource exists (repo, PR, branch, workflow, or file). |

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the agents-issue-optimizer workflow to correctly checkout the Workflows repository where the required scripts are located, and adds comprehensive model management tooling to keep LLM model lists up-to-date.

Key changes:

  • Fixes the checkout action in agents-issue-optimizer.yml to explicitly reference the stranske/Workflows repository
  • Adds enhanced model descriptions and compare mode defaults to agents-verifier.yml
  • Introduces a new shell script (update_model_list.sh) to fetch current model availability from OpenAI and GitHub Models APIs
  • Adds MODEL_MANAGEMENT.md documentation for model update processes
  • Updates project status in langchain-post-code-rollout.md to reflect Phase 1 & 2 deployment completion

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
templates/consumer-repo/.github/workflows/agents-issue-optimizer.yml Fixes missing scripts issue by checking out stranske/Workflows repository explicitly
templates/consumer-repo/.github/workflows/agents-verifier.yml Expands model descriptions with specific provider options and adds default models for compare mode
scripts/update_model_list.sh New utility script to query and display available models from OpenAI and GitHub Models APIs
docs/MODEL_MANAGEMENT.md New documentation guide for maintaining model lists and updating workflow configurations
docs/plans/langchain-post-code-rollout.md Updates deployment status, validation results, and remaining tasks for LangChain rollout

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +69 to +70
repository: stranske/Workflows
ref: main

Copilot AI Jan 7, 2026

Copy link

Choose a reason for hiding this comment

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

The repository is hardcoded to 'stranske/Workflows'. This creates a tight coupling to a specific user's fork and prevents the workflow from being reusable across different organizations or repository structures. Consider using a configurable repository reference through repository variables or workflow inputs to make this more maintainable and portable.

Suggested change
repository: stranske/Workflows
ref: main
repository: ${{ vars.WORKFLOWS_REPOSITORY != '' && vars.WORKFLOWS_REPOSITORY || 'stranske/Workflows' }}
ref: ${{ vars.WORKFLOWS_REF != '' && vars.WORKFLOWS_REF || 'main' }}

Copilot uses AI. Check for mistakes.
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