Skip to content

fix(sync): add github.token fallback to prevent auth failures in main… - #19

Merged
iamkayleb merged 1 commit into
mainfrom
claude/check-multi-agent-support-D9Be2
Mar 24, 2026
Merged

iamkayleb merged 1 commit into
mainfrom
claude/check-multi-agent-support-D9Be2

Conversation

@iamkayleb

Copy link
Copy Markdown
Owner

…t-68

The sync workflow fails on every run because REPO_TOKEN is empty when neither OWNER_PR_PAT nor SERVICE_BOT_PAT secrets are configured. This causes gh repo clone to exit with code 4 (auth required).

  • Add github.token as final fallback so clone/read operations succeed
  • Add explicit "Verify cross-repo token" step that fails early with a clear error message when no PAT is available for PR creation
  • This prevents all 11 matrix jobs from failing with the same cryptic "set GH_TOKEN environment variable" error

https://claude.ai/code/session_01FC8XoyssN5v6hQCTtcjoB5

…t-68

The sync workflow fails on every run because REPO_TOKEN is empty when
neither OWNER_PR_PAT nor SERVICE_BOT_PAT secrets are configured. This
causes `gh repo clone` to exit with code 4 (auth required).

- Add `github.token` as final fallback so clone/read operations succeed
- Add explicit "Verify cross-repo token" step that fails early with a
  clear error message when no PAT is available for PR creation
- This prevents all 11 matrix jobs from failing with the same cryptic
  "set GH_TOKEN environment variable" error

https://claude.ai/code/session_01FC8XoyssN5v6hQCTtcjoB5
@github-actions

Copy link
Copy Markdown

⚠️ Action Required: Unable to determine source issue for PR #19. The PR title, branch name, or body must contain the issue number (e.g. stranske#123, branch: issue-123, or the hidden marker ).

@iamkayleb
iamkayleb temporarily deployed to agent-high-privilege March 24, 2026 13:18 — with GitHub Actions Inactive
@iamkayleb
iamkayleb merged commit 088dd26 into main Mar 24, 2026
28 checks passed
@github-actions

Copy link
Copy Markdown

Automated Status Summary

Head SHA: ff96868
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / guard
Required: core tests (3.11): ⏳ pending, core tests (3.12): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job Result Logs
(no jobs reported) ⏳ pending

Coverage Overview

  • Coverage history entries: 1

Coverage Trend

Metric Value
Current 93.12%
Baseline 85.00%
Delta +8.12%
Minimum 70.00%
Status ✅ Pass

Top Coverage Hotspots (lowest coverage)

File Coverage Missing
src/cli_parser.py 81.8% 4
src/percentile_calculator.py 95.0% 1
src/aggregator.py 95.0% 2
src/__init__.py 100.0% 0
src/ndjson_parser.py 100.0% 0

Updated automatically; will refresh on subsequent CI/Docker completions.


Keepalive checklist

Scope

No scope information available

Tasks

  • No tasks defined

Acceptance criteria

  • No acceptance criteria defined

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