Skip to content

ci(nightly-link-fix): stop the agent from running out of turns - #164

Merged
abdonpijpelink merged 4 commits into
mainfrom
nightly-link-fix-turns
Sep 28, 2026
Merged

abdonpijpelink merged 4 commits into
mainfrom
nightly-link-fix-turns

Conversation

@abdonpijpelink

Copy link
Copy Markdown
Member

The nightly link fix job failed on 9/27 (run) and 9/28 because the agent hit its 40-turn limit. The 9/26 run succeeded, but it used 38 turns.

There are two causes:

  • It redoes work that's already fixed. fix: update 8 broken link(s) found by nightly link check #163 already fixes the 8 broken links, but it's still open. The agent only checked for an existing PR right before opening one, so each night it investigated the same links again.
  • Denied tool calls. The prompt asks the agent to verify URLs with curl and to run git log inside ./landing_page, but the allowlist permitted neither. The 9/27 run had 14 denied calls and the 9/28 run had 15.

Changes

One commit per change:

  1. Check for an open fix PR (gh pr list + gh pr diff) before investigating. If it covers every broken URL, stop. If it covers only some, investigate the rest.
  2. Allow curl -s -o /dev/null -w ... and git -C landing_page log/show, and tell the agent in the prompt to use git -C instead of cd.
  3. Raise --max-turns from 40 to 50.

Merging #163 separately will clear the current 404s.

🤖 Generated with Claude Code

abdonpijpelink and others added 3 commits September 28, 2026 09:53
The agent only checked for an existing PR right before opening one, so
it re-investigated links an open PR already fixed and ran out of turns.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The prompt told the agent to verify URLs with curl and to run git log
inside ./landing_page, but the allowlist permitted neither, so each run
wasted turns on denied calls. Allow the exact curl check and
git -C landing_page log/show, and point the prompt at git -C.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for qdrant-skills ready!

Name Link
🔨 Latest commit b041b4f
🔍 Latest deploy log https://app.netlify.com/projects/qdrant-skills/deploys/6aba2364a52e28000832289d
😎 Deploy Preview https://deploy-preview-164--qdrant-skills.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@abdonpijpelink
abdonpijpelink merged commit 56f8fef into main Sep 28, 2026
5 checks passed
@abdonpijpelink
abdonpijpelink deleted the nightly-link-fix-turns branch September 28, 2026 09:29
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