Skip to content

fix(ci): run docs-sync kilo sessions with --auto - #12657

Closed
iscekic wants to merge 1 commit into
mainfrom
fix/docs-sync-auto-permissions
Closed

fix(ci): run docs-sync kilo sessions with --auto#12657
iscekic wants to merge 1 commit into
mainfrom
fix/docs-sync-auto-permissions

Conversation

@iscekic

@iscekic iscekic commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Why

The daily docs-sync run 30433266378 triaged 293 PRs but chunk 3 failed both attempts: the agent asked to run git log --oneline -30 ... | head -40, and headless kilo run auto-rejects every permission ask (nobody is there to answer), so the chunk produced no JSON and 25 PRs were degraded to unclassified.

Fix

Pass --auto (auto-approve all permissions, built for autonomous/pipeline usage, CLI ≥ v7.4.0) to every docs-sync kilo run invocation:

  • .github/docs-sync/lib.mjs — appended in the shared runKilo helper, covers triage + edit
  • .github/workflows/docs-sync.yml — the verify-fix step invokes kilo run directly

--auto also approves tracked Task subagent asks and skips the headless-root mark that denies subagent permissions outright, so agents spawned by the bot get their tools too.

Test plan

  • Re-run the docs-sync workflow (gh workflow run docs-sync.yml) and confirm no auto-rejecting lines in the triage/edit steps

…g auto-rejected

Headless kilo run auto-rejects every permission ask (e.g. git log), so
triage chunks burned both retries and produced nothing (run 30433266378).
Pass --auto in the shared runKilo helper and the verify-fix step.
@iscekic
iscekic requested a review from jeanduplessis July 29, 2026 14:22
@kilo-code-bot

kilo-code-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • .github/docs-sync/lib.mjs
  • .github/workflows/docs-sync.yml

Reviewed by claude-sonnet-5 · Input: 38 · Output: 8K · Cached: 1M

Review guidance: REVIEW.md from base branch main

@iscekic
iscekic enabled auto-merge (squash) July 29, 2026 14:27
@iscekic
iscekic disabled auto-merge July 29, 2026 14:28
@iscekic

iscekic commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

Duplicate of #12605

@iscekic iscekic marked this as a duplicate of #12605 Jul 29, 2026
@iscekic iscekic closed this Jul 29, 2026
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