Skip to content

chore: stop agents waiting on CI after PR - #973

Merged
skevetter merged 5 commits into
mainfrom
agent-no-ci-wait-after-pr
Aug 10, 2026
Merged

skevetter merged 5 commits into
mainfrom
agent-no-ci-wait-after-pr

Conversation

@devsy-app

@devsy-app devsy-app Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

The daily automation prompts instructed agents to wait for and poll CI
status checks after opening the PR (a ~15 min watch loop with a follow-up
fix-commit cycle). In OpenHands Cloud this polling burns the full run
budget and causes timeouts, since CI often runs longer than the agent
sandbox allows.

Remove the "Ensure status checks pass" step from the shared prompt
template and replace it with a short completion note: the PR is the final
step, and the task is complete once the commit is app-signed (verified=true)
and the local checks in the formatting gate passed (task cli:lint:ci +
task cli:test, which mirror the prek pre-commit hooks). The agent stops
and reports the PR URL.

Changes:

  • hack/automations/template.md.tmpl: drop the status-checks STEP block,
    add a one-paragraph completion statement.
  • hack/automations/main.go: remove the now-unused StatusStepNumber render
    field and its PRStepNumber+1 computation.
  • .agents/agents/*/agent.md: regenerated via go run ./hack/automations
    (14 files); go run ./hack/automations -check is clean.

Verification:

  • go test ./hack/automations/ passes.
  • golangci-lint --new-from-rev=origin/main ./hack/automations/ : 0 issues.
  • Cloud automations synced via go run ./hack/automations -sync
    (14 synced, re-run idempotent: 14 ok).

This PR was created by an AI agent on behalf of the user.

The daily automation prompts instructed agents to wait for and poll CI
status checks after opening the PR (a ~15 min watch loop with a follow-up
fix-commit cycle). In OpenHands Cloud this polling burns the full run
budget and causes timeouts, since CI often runs longer than the agent
sandbox allows.

Remove the "Ensure status checks pass" step from the shared prompt
template and replace it with a short completion note: the PR is the final
step, and the task is complete once the commit is app-signed (verified=true)
and the local checks in the formatting gate passed (task cli:lint:ci +
task cli:test, which mirror the prek pre-commit hooks). The agent stops
and reports the PR URL.

Changes:
- hack/automations/template.md.tmpl: drop the status-checks STEP block,
  add a one-paragraph completion statement.
- hack/automations/main.go: remove the now-unused StatusStepNumber render
  field and its PRStepNumber+1 computation.
- .agents/agents/*/agent.md: regenerated via `go run ./hack/automations`
  (14 files); `go run ./hack/automations -check` is clean.

Verification:
- go test ./hack/automations/ passes.
- golangci-lint --new-from-rev=origin/main ./hack/automations/ : 0 issues.
- Cloud automations synced via `go run ./hack/automations -sync`
  (14 synced, re-run idempotent: 14 ok).

This PR was created by an AI agent on behalf of the user.
@netlify

netlify Bot commented Aug 9, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit f4ee146
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a7911fc4bfeae0008fb5b15

@netlify

netlify Bot commented Aug 9, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for images-devsy-sh canceled.

Name Link
🔨 Latest commit f4ee146
🔍 Latest deploy log https://app.netlify.com/projects/images-devsy-sh/deploys/6a7911fc017d0800084ca3ec

@codacy-production

codacy-production Bot commented Aug 9, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 22 complexity · 0 duplication

Metric Results
Complexity 22
Duplication 0

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@skevetter skevetter changed the title fix: stop agents waiting on CI after PR chore: stop agents waiting on CI after PR Aug 9, 2026
devsy-app Bot and others added 4 commits August 9, 2026 18:42
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
@skevetter
skevetter marked this pull request as ready for review August 10, 2026 00:00
@skevetter
skevetter enabled auto-merge (squash) August 10, 2026 00:00
@skevetter
skevetter merged commit 8831075 into main Aug 10, 2026
70 checks passed
@skevetter
skevetter deleted the agent-no-ci-wait-after-pr branch August 10, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant