Skip to content

fix: update GitHub repo references after rename#340

Open
dinex-dev wants to merge 1 commit into
masterfrom
fix/publish-repo-rename
Open

fix: update GitHub repo references after rename#340
dinex-dev wants to merge 1 commit into
masterfrom
fix/publish-repo-rename

Conversation

@dinex-dev
Copy link
Copy Markdown
Member

@dinex-dev dinex-dev commented Jun 3, 2026

What

Updates two project-level files to reflect the GitHub repo rename from requestly/requestly-desktop-apprequestly/http-interceptor-desktop-app:

File Change
package.json build.publish.repo requestly-desktop-apphttp-interceptor-desktop-app — consumed by electron-builder's publish step to target the GitHub Releases API
package.json repository.url Same rename — npm metadata, IDE tooling
README.md 3 GitHub-hosted asset URLs + 1 git clone instruction

Net 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_TOKEN PAT (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.json publish.repo is set to requestly-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.
  • The local git remote get-url origin still points at the old URL. GitHub's git transport handles redirects transparently (git push/pull work), so this is cosmetic on developer machines. Each contributor can update locally with git remote set-url origin https://github.com/requestly/http-interceptor-desktop-app.git.

Verification + next steps

  1. Merge this PR.
  2. Regenerate the PUBLISH_TOKEN PAT (Settings → Developer settings → Personal access tokens) with repo scope (or fine-grained: Contents: Read+Write on this repo). Update the PUBLISH_TOKEN secret here.
  3. Re-dispatch the Release Desktop App workflow. Expected: passes both auth gates (WIF for GCP + GitHub PAT for releases), creates the draft release.

Related

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Updated repository references and asset URLs in project documentation.
  • Chores

    • Updated GitHub publishing configuration and repository metadata in project configuration.

…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>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4a9a3e29-c24d-4396-9b98-07bb4f54f8af

📥 Commits

Reviewing files that changed from the base of the PR and between 32118cb and b02d9de.

📒 Files selected for processing (2)
  • README.md
  • package.json

Walkthrough

This PR updates all references from the requestly-desktop-app repository to the renamed http-interceptor-desktop-app repository. Changes span the README documentation, including logo asset URLs and the development setup clone command, as well as the package.json configuration for Electron Builder publishing and the repository metadata field. No functional code changes or new logic is introduced.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: updating GitHub repository references after a project rename from requestly-desktop-app to http-interceptor-desktop-app.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/publish-repo-rename

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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