Skip to content

Fix ruff exclude and document Merge Sync PRs workflow#662

Merged
stranske merged 3 commits intomainfrom
fix/add-faiss-dependency
Jan 8, 2026
Merged

Fix ruff exclude and document Merge Sync PRs workflow#662
stranske merged 3 commits intomainfrom
fix/add-faiss-dependency

Conversation

@stranske
Copy link
Copy Markdown
Owner

@stranske stranske commented Jan 8, 2026

Source: Issue #122

Automated Status Summary

Scope

  • Context / problem:
  • - The Automated Status Summary in PR bodies currently only shows workflow run results
  • - When the CLI-based Codex (via reusable-codex-run.yml) completes iterations, there's no visibility into:
  • - What tasks Codex completed in each round
  • - The final message/output from Codex
  • - How many files were changed
  • - Whether commits were pushed successfully
  • - This makes it hard to track CLI Codex progress vs the UI version
  • - The keepalive loop evaluation outputs (iteration count, tasks remaining, etc.) are logged but not surfaced to the PR summary
  • Goal:
  • - Capture CLI Codex outputs and integrate them into the Automated Status Summary
  • - Provide visibility into Codex iteration progress and outcomes
  • - Show what changed in each round

Context for Agent

Design Decisions & Constraints

  • When the Keepalive loop runs Codex via workflow_run trigger:
  • Updated keepalive_loop.js to return prRef from PR data
  • Updated agents-keepalive-loop.yml to pass pr_ref to the Codex workflow
  • This will fix the failing Keepalive runs on PR #103 where Codex was making changes but failing to push them.
  • | Keepalive | ❌ disabled |

Related Issues/PRs

References

Tasks

  • Update reusable-codex-run.yml to emit structured outputs:
  • Add output for final-message from Codex action
  • Add output for files-changed (count of modified files)
  • Add output for commits-pushed (boolean)
  • Write iteration summary to GITHUB_STEP_SUMMARY
  • Create new section in PR body for CLI Codex status:
  • Add <!-- codex-cli-status:start --> / <!-- codex-cli-status:end --> markers
  • Show last iteration number and outcome
  • Show tasks completed this round
  • Show link to workflow run logs
  • Update agents_pr_meta_update_body.js to populate the new section:
  • Fetch latest keepalive loop run results
  • Extract Codex outputs from workflow artifacts or step summaries
  • Format and insert into PR body
  • Update keepalive_loop.js to pass iteration context to the summary:
  • Include current iteration number in output
  • Include tasks remaining count
  • Include estimated rounds to completion
  • Add tests for the new integration:
  • Test output extraction from workflow runs
  • Test PR body section formatting
  • Test edge cases (no Codex runs, failed runs, etc.)

Acceptance criteria

  • CLI Codex iterations are visible in the PR body Automated Status Summary
  • Each iteration shows: round number, tasks attempted, outcome, and link to logs
  • The summary updates automatically after each keepalive loop run
  • Existing UI Codex tracking (if any) continues to work
  • Head SHA: 2fc7361
  • Latest Runs: ✅ success — Gate
  • Required: gate: ✅ success
  • | Workflow / Job | Result | Logs |
  • |----------------|--------|------|
  • | Agents PR meta manager | ❔ in progress | View run |
  • | CI Autofix Loop | ✅ success | View run |
  • | Gate | ✅ success | 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 | ❌ failure | View run |
  • | PR 11 - Minimal invariant CI | ✅ success | View run |
  • | Selftest CI | ✅ success | View run |
  • Head SHA: ce6ef18
  • Latest Runs: ✅ success — Gate
  • Required: gate: ✅ success
  • | Workflow / Job | Result | Logs |
  • |----------------|--------|------|
  • | Agents PR meta manager | ❔ in progress | View run |
  • | CI Autofix Loop | ✅ success | View run |
  • | Copilot code review | ❔ in progress | View run |
  • | Gate | ✅ success | 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 | ❌ failure | View run |
  • | PR 11 - Minimal invariant CI | ✅ success | View run |
  • | Selftest CI | ✅ success | View run |

Head SHA: e9487bc
Latest Runs: ✅ success — Gate
Required: gate: ✅ success

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 ✅ success 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

Copilot AI review requested due to automatic review settings January 8, 2026 14:14
@stranske stranske temporarily deployed to agent-high-privilege January 8, 2026 14:14 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 8, 2026

Automated Status Summary

Head SHA: eefab34
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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 8, 2026

🤖 Keepalive Loop Status

PR #662 | 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/57 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). |

@stranske stranske merged commit 6525c50 into main Jan 8, 2026
113 checks passed
@stranske stranske deleted the fix/add-faiss-dependency branch January 8, 2026 14:16
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes the ruff CI configuration to properly exclude .venv directories and documents the proper workflow for merging sync PRs.

  • Changed ruff from --exclude to --extend-exclude to preserve default exclusions including .venv
  • Added documentation in CLAUDE.md for using the "Merge Sync PRs" workflow after syncing
  • Fixed string literal escaping and API response access patterns in the agents-verify-to-issue workflow template

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/reusable-10-ci-python.yml Updated ruff to use --extend-exclude instead of --exclude to preserve default exclusions
CLAUDE.md Added section documenting the Merge Sync PRs workflow with usage instructions
templates/consumer-repo/.github/workflows/agents-verify-to-issue.yml Fixed string escaping from \\n to \n and corrected API response property access from issue.data.* to issue.*

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

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