Skip to content

chore: remove duplicate, broken upstream_sync workflow - #13

Merged
solderzzc merged 1 commit into
mainfrom
chore/remove-duplicate-broken-upstream-sync
Aug 16, 2026
Merged

chore: remove duplicate, broken upstream_sync workflow#13
solderzzc merged 1 commit into
mainfrom
chore/remove-duplicate-broken-upstream-sync

Conversation

@solderzzc

Copy link
Copy Markdown
Member

Summary

  • This repo has two upstream-sync workflows: upstream-sync.yml (hyphenated, daily, 10/10 recent runs succeeded) and upstream_sync.yml (underscored, every 4h, 10/10 recent runs failed).
  • The underscored one fails at the checkout step itself:
    fatal: unable to access 'https://github.com/SharpAI/mlx-swift/': The requested URL returned error: 403
    
  • It overrides the checkout token with secrets.SWIFTLM_PR_TOKEN (confirmed present on this repo via the API, but rejected here — likely expired or wrong scope; I can't inspect or rotate a secret's value). The working sibling workflow does the same job using only the default GITHUB_TOKEN, which proves that credential is sufficient for this repo's actual sync needs.
  • Rather than repair a credential I can't verify, this removes the redundant duplicate. The hyphenated workflow already covers the same purpose correctly.

Test plan

  • Confirm upstream-sync.yml continues to run successfully on its next scheduled trigger.
  • Confirm the underscored workflow no longer appears in the Actions tab after merge.

upstream-sync.yml (hyphenated) already syncs daily and has succeeded on
every recent run using the default GITHUB_TOKEN. upstream_sync.yml
(underscored) duplicated the same job every 4 hours instead, but has
failed every run with a 403 at checkout:

  fatal: unable to access 'https://github.com/SharpAI/mlx-swift/':
  The requested URL returned error: 403

It overrides the checkout token with SWIFTLM_PR_TOKEN, which is
configured as a secret on this repo but rejected by GitHub for this
operation. Since the working sibling workflow proves the default
token is sufficient for this repo's sync needs, the fix is to drop
the redundant, broken duplicate rather than repair a credential that
can't be inspected or rotated from here.
@solderzzc
solderzzc merged commit ac713a0 into main Aug 16, 2026
7 checks passed
@solderzzc
solderzzc deleted the chore/remove-duplicate-broken-upstream-sync branch August 16, 2026 15:25
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