Skip to content

Call Rush's API at api.phoenix.so - #4046

Open
muqsitnawaz wants to merge 1 commit into
mainfrom
chore/api-phoenix-so
Open

muqsitnawaz wants to merge 1 commit into
mainfrom
chore/api-phoenix-so

Conversation

@muqsitnawaz

Copy link
Copy Markdown
Contributor

api.phoenix.so is now Rush's only API host. api.prix.dev is being retired to a 308 redirect (PHNX-4503), and most HTTP clients drop the Authorization header when they follow a redirect to another host, so callers must name api.phoenix.so directly.

What changed

Every api.prix.dev in code and current docs becomes api.phoenix.so (changelogs keep their history). No route, header or behavior changes; api.phoenix.so serves the identical API today.

  • README.md
  • cli/AGENTS.md
  • cli/docs/cloud.md
  • cli/src/lib/rush-api.ts
  • skills/secrets/SKILL.md

Evidence

Live probe of the route clients call, run 2026-10-27:

GET https://api.phoenix.so/o/x/sessions -> 401
GET https://api.prix.dev/o/x/sessions   -> 401

Tests run in this branch:

 Test Files  1 passed (1)
      Tests  3 passed (3)
   Start at  21:56:00
   Duration  6.80s (transform 10ms, setup 13ms, import 16ms, tests 5.83s, environment 0ms)

Tracking: https://linear.app/getrush/issue/PHNX-4503

api.phoenix.so is Rush's only API host (PHNX-4503); api.prix.dev is being retired to a
redirect, which most clients follow without their Authorization header.
@muqsitnawaz

Copy link
Copy Markdown
Contributor Author

Independent review: APPROVE

Checked the PR head with git grep. The only code change is cli/src/lib/rush-api.ts:1, and the default is now https://api.phoenix.so. RUSH_PROXY_BASE is still an override, but nothing in the repo pins it to prix.dev. Every test that sets it uses a local mock URL. No test asserts the old host. No api.prix.dev is left outside cli/.changelog/*, which is history. The diff has no unrelated changes. CI is green.

Not part of this PR: cli/src/lib/feed-broadcast.ts:149 still builds https://prix.dev/console/sessions/.... That is the retired site, not the API host, and the tests at feed-broadcast.test.ts:232,241,415,427 and send.test.ts:65 assert it. Leave it out of this PR.

This branch has not been deployed

No deployments
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