Skip to content

chore: sync workflow templates - #1516

Closed
stranske wants to merge 1 commit into
mainfrom
sync/workflows-66c434a698c8
Closed

chore: sync workflow templates#1516
stranske wants to merge 1 commit into
mainfrom
sync/workflows-66c434a698c8

Conversation

@stranske

@stranske stranske commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Sync Summary

Files Updated

  • check_deliberate_break.py: Opt-in Gate helper that proves named deliberate-break acceptance tests fail against the base implementation
  • registry.yml: Agent registry - source of truth for agent keys and runner workflow mapping
  • verifier_verdict_json.py: Extracts structured post-merge verifier verdict JSON without trusting diff text
  • pr_verifier.py: PR verifier - validates PR changes against acceptance criteria
  • WORKFLOW_USER_GUIDE.md: Workflow user guide - explains the CI/agent system for repo consumers

Files Skipped

  • .github/workflows/pr-00-gate.yml: Maintains a fully custom Gate workflow; never overwrite (replaces the hard-coded custom_gate_repos list in maint-68).
  • ci.yml: File exists and sync_mode is create_only
  • renovate.json: File exists and sync_mode is create_only
  • cross-repo-smoke.yml: File exists and sync_mode is create_only
  • llm_slots.json: None

Review Checklist

  • CI passes with updated workflows
  • No repo-specific customizations were overwritten

Source: stranske/Workflows
Source SHA: 628a72520398d27262610ce6fda4dbbff7e84303
Template hash: 66c434a698c8
Consumer-sync plan ID: sha256:66c434a698c815ccc2f424a4dd91191523e1a4e63bd21b83111800e2358052db
Sync phase: canary
Sync branch: sync/workflows-66c434a698c8
Consumer repo: stranske/Manager-Database
Manifest: .github/sync-manifest.yml

  Automated sync from stranske/Workflows
  Template hash: 66c434a698c8

  Changes synced from sync-manifest.yml
Copilot AI review requested due to automatic review settings August 4, 2026 02:44
@stranske stranske added sync Automated sync from Workflows automated Automated sync from Workflows labels Aug 4, 2026
@cursor

cursor Bot commented Aug 4, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (7)
  • sync
  • workflow:source-sync
  • workflow:source-maintenance
  • consumer-sync
  • integration-sync
  • workflows-sync
  • template-sync

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: e68a6985-75b1-498d-ada9-b815edebc71a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

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.

🟡 Not ready to approve

The new dependency pin install logic isn’t enforced when a different PyYAML version is already present, and the new LLM content-block extraction can return empty text and burn parsing/repair attempts unnecessarily.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

Syncs consumer-repo workflow template artifacts from stranske/Workflows into stranske/Manager-Database, updating agent/verifier support scripts and the workflow user guide to match the central automation lane behavior.

Changes:

  • Update PR verifier parsing/repair to tolerate provider responses that return content blocks instead of a plain string.
  • Extend deliberate-break Gate helper to install minimal pytest runtime deps (PyYAML) and to return structured timeout/dependency-install failures.
  • Refresh consumer documentation and agent registry details to align with the central Workflows repo.
File summaries
File Description
WORKFLOW_USER_GUIDE.md Updates maintenance-workflow documentation to distinguish central-only workflows from consumer templates.
scripts/langchain/pr_verifier.py Coerces provider response content into JSON text for structured parsing and tightens repair handling.
scripts/check_deliberate_break.py Ensures lightweight pytest runtime deps are available (without touching Gate workflow files) and improves failure classification.
.github/scripts/verifier_verdict_json.py Normalizes needs_attention to avoid truthiness traps from non-bool values.
.github/agents/registry.yml Updates default Codex execution profile model/fallback mapping.
Review details
  • Files reviewed: 5/5 changed files
  • Comments generated: 2
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

Comment thread scripts/check_deliberate_break.py
Comment thread scripts/langchain/pr_verifier.py
@stranske stranske mentioned this pull request Aug 4, 2026
2 tasks
@stranske

stranske commented Aug 4, 2026

Copy link
Copy Markdown
Owner Author

Superseded by source-corrected workflow sync #1518 after Workflows #2908 merged; closing this intermediate generation.

@stranske stranske closed this Aug 4, 2026
@stranske
stranske deleted the sync/workflows-66c434a698c8 branch August 4, 2026 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated Automated sync from Workflows sync Automated sync from Workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants