Skip to content

ci: authenticate arduino/setup-protoc on ci.yml too#60

Merged
Nic-dorman merged 1 commit into
mainfrom
ci/setup-protoc-token-on-ci
May 14, 2026
Merged

ci: authenticate arduino/setup-protoc on ci.yml too#60
Nic-dorman merged 1 commit into
mainfrom
ci/setup-protoc-token-on-ci

Conversation

@Nic-dorman
Copy link
Copy Markdown
Collaborator

Summary

Mirrors #31's release.yml fix on ci.yml. Without `repo-token`, `arduino/setup-protoc@v3` issues its release-list API call anonymously and shares the ~60/hr per-IP quota with every other runner on the same egress IP. release.yml has been authed since #31 (2026-04-28); ci.yml never was, so the same flake just bit PR #57's post-merge CI run (#25447450352) on the API contract tests job — exact `arduino/setup-protoc@v3 ... API rate limit exceeded` repro of #31's investigation.

PR #57 itself was already merged before its post-merge CI completed (auto-merge triggered on the previous run that did pass), so no code on main is affected. This patch just stops the flake from biting future PRs.

Change

Adds `repo-token: ${{ secrets.GITHUB_TOKEN }}` to both `arduino/setup-protoc@v3` invocations in `.github/workflows/ci.yml`:

  • `Check (antd)` job (line 33)
  • `API contract tests (antd-rust)` job (line 75)

Mirrors release.yml's pattern verbatim. No semantic change — just shifts the action onto our authenticated 5,000/hr token quota.

Test plan

  • CI runs on this PR (the same ci.yml that's being patched)

🤖 Generated with Claude Code

Mirrors PR #31's release.yml fix — without `repo-token`, the action
issues the GitHub release-list API call anonymously and shares the
~60/hr per-IP quota with every other runner on the same egress IP.
release.yml has had the auth since 2026-04-28; ci.yml didn't, and
PR #57's post-merge CI run flaked here on 2026-05-06 (run 25447450352
job 'API contract tests (antd-rust)' — exact `arduino/setup-protoc@v3
... API rate limit exceeded` message reproduced from PR #31's
investigation).

Adds `repo-token: \${{ secrets.GITHUB_TOKEN }}` to both occurrences
in ci.yml — the Check (antd) job at line 33 and the API contract tests
job at line 75. No semantic change to either workflow; just shifts
the action onto our 5,000/hr token quota.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Nic-dorman Nic-dorman merged commit f2fa310 into main May 14, 2026
3 checks passed
@Nic-dorman Nic-dorman deleted the ci/setup-protoc-token-on-ci branch May 14, 2026 10:54
Nic-dorman added a commit that referenced this pull request May 14, 2026
Cuts v0.7.1 atop v0.7.0. Primarily refreshes the upstream `ant-core`
pin to the `ant-cli-v0.2.3` release tag (no API change for antd
consumers). Bundles a substantial round of cross-SDK example/build
fixes, dispatcher improvements, and CI/release workflow hardening.

## antd

- chore(antd): bump ant-core to v0.2.3 (#85)

## SDK example/build fixes

- fix(antd-php): use cost-estimate fields in example 02 (#74)
- fix(antd-elixir): print cost-estimate fields in examples (#75)
- fix(antd-lua): add missing discover module to rockspec (#76)
- fix(antd-kotlin): make put-response cost optional + ship gradle wrapper (#77)
- fix(antd-zig): pass payment_mode to dataPutPublic/dataPutPrivate (#79)
- fix(antd-java): make examples runnable via gradle :examples subproject (#80)
- fix(antd-zig): align stdlib API to declared 0.14.x minimum (#82)
- fix(antd-swift): port to Linux + populate cost-estimate fields (#87)

## ant-dev (developer CLI)

- fix(ant-dev): clean up orphan anvil/antnode and stale node identities on stop (#81)
- fix(ant-dev): tooling cluster — flag alias, sys.executable, anvil preflight, README (#83)
- feat(ant-dev): expand `ant dev example` to dispatch all 15 SDKs (#84)
- fix(ant-dev): dispatcher swift no-skip + lua LUA_PATH wrap (#86)
- feat(ant-dev): expose --preset flag on `ant dev start` (default: small) (#88)

## CI / release

- ci: authenticate arduino/setup-protoc on ci.yml too (#60)
- feat(release): publish antd-linux-arm64 artifact (#89)

## Validation

15/15 SDKs round-tripped end-to-end against a daemon built from this
commit on a Linux dev box (Ubuntu 24.04, 0.7.1 atop ant-core v0.2.3).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

1 participant