fix: update GitHub repo references after rename#340
Conversation
…esktop-app The repo was renamed from 'requestly/requestly-desktop-app' to 'requestly/http-interceptor-desktop-app'. GitHub's REST API handles most read redirects, but the redirect is inconsistent across write endpoints (release create/upload), and the stale URL also shows up in error messages — confusing for future debugging. Today's failing release run surfaced this: https://github.com/requestly/http-interceptor-desktop-app/actions/runs/26888111284 (The immediate cause of that failure is an expired PUBLISH_TOKEN PAT returning '401 Bad credentials' — being regenerated separately. This PR removes the latent fragility on the URL side so future failures log useful URLs.) Changes: - package.json build.publish.repo: requestly-desktop-app → http-interceptor-desktop-app (electron-builder publish target). - package.json repository.url: same rename (npm metadata + IDE tooling). - README.md: 3 GitHub-hosted asset URLs + 1 git-clone instruction. GitHub serves these via redirect today; cosmetic + future-proof. After this lands + PUBLISH_TOKEN is refreshed, re-dispatch the Release Desktop App workflow. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
WalkthroughThis PR updates all references from the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What
Updates two project-level files to reflect the GitHub repo rename from
requestly/requestly-desktop-app→requestly/http-interceptor-desktop-app:package.jsonbuild.publish.reporequestly-desktop-app→http-interceptor-desktop-app— consumed by electron-builder's publish step to target the GitHub Releases APIpackage.jsonrepository.urlREADME.mdgit cloneinstructionNet diff: 2 files, +6 / −6.
Why
Today's release dispatch surfaced the rename's residual fragility:
https://github.com/requestly/http-interceptor-desktop-app/actions/runs/26888111284
The error log shows electron-builder hitting
api.github.com/repos/requestly/requestly-desktop-app/releases(the old URL) and getting 401 from GitHub.⚠ The 401 itself is NOT caused by this URL — it's caused by an expired
PUBLISH_TOKENPAT (170+ days old, beyond GitHub's typical PAT lifetime). That has to be regenerated separately in repo Settings → Secrets and Variables → Actions.This PR fixes the adjacent, latent issue: even after the PAT is refreshed, the publish URL still points at the old name. GitHub's REST API redirects most reads, but is inconsistent on write endpoints (release create + asset upload), and the misleading URL in error logs costs the next person's debugging time.
Adjacent rename references not touched (deliberate)
electron-builder-setapp.jsonpublish.repois set torequestly-desktop-app-release— a separate repo (Mac SetApp distribution variant). Whether that one was also renamed is unknown; left alone to keep this PR's scope clean. If it was renamed, a separate PR can mirror this one.git remote get-url originstill points at the old URL. GitHub's git transport handles redirects transparently (git push/pullwork), so this is cosmetic on developer machines. Each contributor can update locally withgit remote set-url origin https://github.com/requestly/http-interceptor-desktop-app.git.Verification + next steps
PUBLISH_TOKENPAT (Settings → Developer settings → Personal access tokens) withreposcope (or fine-grained: Contents: Read+Write on this repo). Update thePUBLISH_TOKENsecret here.Related
🤖 Generated with Claude Code
Summary by CodeRabbit
Documentation
Chores