Skip to content

chore: sync workflow templates - #1716

Merged
stranske-keepalive[bot] merged 1 commit into
mainfrom
sync/workflows-candidate
Aug 13, 2026
Merged

chore: sync workflow templates#1716
stranske-keepalive[bot] merged 1 commit into
mainfrom
sync/workflows-candidate

Conversation

@stranske

@stranske stranske commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Sync Summary

Files Updated

  • agents-81-gate-followups.yml: Gate followups hub - consolidates keepalive and autofix followups
  • agents-keepalive-sweep.yml: Keepalive sweep (#2267) - periodic level-based resync; dispatches the loop for open agent PRs so silent zero-commit stalls resurface
  • sync_dev_dependencies.py: Syncs dev dependency versions from autofix-versions.env to pyproject.toml
  • issue_format.py: Pure-stdlib validator for AGENT_ISSUE_FORMAT compliance. Single fleet definition of agent-processable; used by agents-issue-format-guard.yml and callable directly by local filers to pre-flight before gh issue create (non-zero exit = unfit). Do not fork per repo.
  • registry.yml: Agent registry - source of truth for agent keys and runner workflow mapping
  • keepalive_loop.js: Core keepalive loop logic
  • keepalive_challenge_due.js: Selects due automation-owned authority challenges for the hourly keepalive sweep
  • error_classifier.js: Classifies CI errors for agent handling
  • path-classifier/ (1 files): Classifies changed PR paths for path-aware expensive job gating
  • AGENT_ISSUE_FORMAT.md: Format specification for agent-compatible issues
  • SETUP_CHECKLIST.md: Consumer repo setup checklist. Includes the default_workflow_permissions=write step in section 3.3.1 that prevents a Gate startup_failure on fresh consumers (see #2157). Synced so it no longer drifts across the fleet.
  • WORKFLOW_USER_GUIDE.md: Workflow user guide - explains the CI/agent system for repo consumers

Files Skipped

  • pr-00-gate.yml: File exists and sync_mode is create_only
  • 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
  • .github/scripts/package.json: Repo installs .github/scripts dependencies from package-lock.json and forbids tracked node_modules
  • .github/scripts/node_modules/minimatch: Repo installs .github/scripts dependencies from package-lock.json and forbids tracked node_modules
  • .github/scripts/node_modules/brace-expansion: Repo installs .github/scripts dependencies from package-lock.json and forbids tracked node_modules
  • .github/scripts/node_modules/balanced-match: Repo installs .github/scripts dependencies from package-lock.json and forbids tracked node_modules
  • llm_slots.json: None

Review Checklist

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

Source: stranske/Workflows
Source SHA: 590ae8053b0ac6ce1f5efd36e9379f240d794e0f
Template hash: 7a99d523d28c
Consumer-sync plan ID: sha256:7a99d523d28c8148ab3219144e30e97258a6127dae3b08a9c6937b5da1f0aa78
Sync phase: canary
Sync branch: sync/workflows-candidate
Consumer repo: stranske/trip-planner
Manifest: .github/sync-manifest.yml

autofix: false

@stranske stranske added sync Automated sync from Workflows automated Automated sync from Workflows sync:delivery-staging Generated delivery is mutable and must not merge labels Aug 13, 2026
@cursor

cursor Bot commented Aug 13, 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.

@netlify

netlify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Deploy Preview for stranske-trip-planner canceled.

Name Link
🔨 Latest commit 35d07df
🔍 Latest deploy log https://app.netlify.com/projects/stranske-trip-planner/deploys/6a7ddd1e1802ef0008783dca

@coderabbitai

coderabbitai Bot commented Aug 13, 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: c0d7a161-566a-49ca-b28f-30881f653a70

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.

@stranske
stranske marked this pull request as ready for review August 13, 2026 04:10
@stranske

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@stranske

Copy link
Copy Markdown
Owner Author

@codex review

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5ab6565868

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/scripts/issue_format.py Outdated
_NODE_SUFFIX = re.compile(r"::.+$") # pytest node ids: tests/x.py::test_y
_ORIGINAL_ISSUE_BLOCK_RE = re.compile(
r"<details\b[^>]*>\s*<summary>Original Issue</summary>\s*"
r"(?P<fence>`{3,})text\s*\n(?P<inner>.*?)\n(?P=fence)\s*</details>",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Accept tilde-fenced provenance blocks

When an existing formatted issue uses a ~~~text archive, this regex leaves the archived body visible even though scripts/langchain/issue_formatter.py:480-483 explicitly recognizes tilde-fenced Original Issue blocks. If that archive contains three stale paths, validate(..., repo_root=...) counts them as live citations and incorrectly marks an otherwise conforming issue as not agent-processable. Accept both supported fence characters in the Workflows source before syncing this validator.

AGENTS.md reference: AGENTS.md:L22-L22

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Confirmed as source-owned. The bounded fix is open in stranske/Workflows#3064 with root/template parity and a tilde-fence regression. This generated PR remains unmergeable; Maint 68 will refresh this stable candidate in place after the source fix merges.

@stranske
stranske marked this pull request as draft August 13, 2026 04:29
@stranske
stranske force-pushed the sync/workflows-candidate branch from 5ab6565 to 63e9dde Compare August 13, 2026 04:29
@stranske
stranske marked this pull request as ready for review August 13, 2026 04:38
@stranske

Copy link
Copy Markdown
Owner Author

@codex review

@stranske

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 63e9ddeef5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@stranske
stranske marked this pull request as draft August 13, 2026 04:53
@stranske
stranske force-pushed the sync/workflows-candidate branch from 63e9dde to 744abed Compare August 13, 2026 04:53
@stranske
stranske marked this pull request as ready for review August 13, 2026 05:01
@stranske

Copy link
Copy Markdown
Owner Author

@codex review

@stranske

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 744abed6bb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +115 to +117
inputs: {
pr_number: String(pr.number),
force_retry: 'true',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Limit forced retries to due authority challenges

On every scheduled sweep, this now sets force_retry for every non-draft agent-labelled PR, not just the dueChallenge case. In agents-81-gate-followups.yml, that flag bypasses non-success Gate states and resets both no-progress counters, so an ordinary stalled PR can rerun despite a failed/cancelled Gate and repeatedly avoid the zero-activity/progress-review stop logic. Preserve the previous normal re-evaluation for ordinary PRs and force only the explicitly due recovery path; make the source-of-truth fix in Workflows before syncing it here.

AGENTS.md reference: AGENTS.md:L65-L77

Useful? React with 👍 / 👎.

Comment on lines +102 to +108
if (dueChallenge) {
challenged += 1;
core.info(
`PR #${pr.number}: independently rechecking due authority claim ` +
`(due ${dueChallenge.dueAt}; ${dueChallenge.nextAction || dueChallenge.key || 'no detail'})`,
);
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Implement an outcome for due authority challenges

When an auth failure creates a due authority claim, this branch only increments a counter and logs it; the following dispatch is identical whether dueChallenge is present or null, and no challenge context or result is passed downstream. A repeated auth failure therefore just recreates challenge_due_at and leaves agent:needs-attention indefinitely, without ever confirming the boundary, recording an exact human action, or applying needs-human. Route due claims through an actual independent challenge/terminal decision in the Workflows source before syncing.

AGENTS.md reference: AGENTS.md:L65-L77

Useful? React with 👍 / 👎.

@stranske

Copy link
Copy Markdown
Owner Author

Source-owned findings accepted and grouped in stranske/Workflows#3066. This generated PR remains blocked; a no-filter Maint 68 canary will refresh this same candidate PR after the source fix merges.

@stranske
stranske marked this pull request as draft August 13, 2026 11:47
@stranske
stranske force-pushed the sync/workflows-candidate branch from 744abed to 09d6c8a Compare August 13, 2026 11:47
@stranske
stranske marked this pull request as ready for review August 13, 2026 11:56
@stranske

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 09d6c8a22b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/sync_dev_dependencies.py
@stranske
stranske marked this pull request as draft August 13, 2026 12:27
@stranske
stranske force-pushed the sync/workflows-candidate branch from 09d6c8a to 64bc1b3 Compare August 13, 2026 12:28
@stranske
stranske marked this pull request as ready for review August 13, 2026 12:36
@stranske

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 64bc1b3855

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/scripts/keepalive_challenge_due.js
Comment thread .github/scripts/keepalive_loop.js Outdated
if (core) core.warning(`Failed to add needs-human label: ${error.message}`);
if (core) core.warning(`Failed to add ${escalationDisposition} routing label: ${error.message}`);
}
if (escalationDisposition === 'automation-retry' && !stop) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Run recovery for automation-owned stop states

For terminal non-auth states such as round-budget-exhausted, selectEscalationDisposition returns automation-retry, but this !stop condition prevents the only recovery dispatch from running. The added agent:retry label cannot resolve this case because evaluation checks the hard iteration limit before choosing work, and each hourly non-forced sweep reaches the same stop again, leaving the PR permanently automation-owned with neither recovery nor a human escalation; route these stop states to an actual bounded fallback in the Workflows source.

AGENTS.md reference: AGENTS.md:L65-L77

Useful? React with 👍 / 👎.

Automated sync from stranske/Workflows
Template hash: 7a99d523d28c

Changes synced from sync-manifest.yml
@stranske
stranske force-pushed the sync/workflows-candidate branch from 1565572 to 35d07df Compare August 13, 2026 15:05
@stranske
stranske marked this pull request as ready for review August 13, 2026 15:13
@agents-workflows-bot agents-workflows-bot Bot added the sync:delivery-ready Generated delivery is sealed to its exact reviewed head label Aug 13, 2026
@stranske-keepalive
stranske-keepalive Bot merged commit 55e1c84 into main Aug 13, 2026
100 of 104 checks passed
@stranske-keepalive stranske-keepalive Bot removed the sync:delivery-staging Generated delivery is mutable and must not merge label Aug 13, 2026
@stranske-keepalive
stranske-keepalive Bot deleted the sync/workflows-candidate branch August 13, 2026 15:34
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:delivery-ready Generated delivery is sealed to its exact reviewed head sync Automated sync from Workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant