feat: add site rename endpoint by vigneshrajsb · Pull Request #261 · GoodRxOSS/lifecycle

vigneshrajsb · 2026-10-12T23:28:45Z

Description

Adds PATCH /api/v2/sites/{siteId} so a site owner can rename a site.
Only the display name changes. The site ID and content URL stay the same.
Names are trimmed and must be 1-255 characters, with no control or text-direction characters. The same check now applies when creating a site. A rename is audited, and an unchanged name is a no-op.

Added service and route tests; the sites tests, lint, and the new OpenAPI operation (renameSite) check out.

Matches the existing owner-only rules. expectedAccessRevision is optional and a rename does not bump it.
tsc reports existing errors in unrelated workspace files; none are in this change.

vigneshrajsb added 4 commits October 12, 2026 17:28

feat(sites): add rename endpoint

b20f921

fix(sites): document rename 404 and cover invalid json

c4b783e

fix(sites): register rename route in the policy manifest

2bfbb34

fix(sites): reject control characters in site names

fa4ef75

vigneshrajsb force-pushed the site-rename branch from 18d6071 to fa4ef75 Compare October 13, 2026 00:29

vigneshrajsb marked this pull request as ready for review October 13, 2026 00:44

vigneshrajsb requested a review from a team as a code owner October 13, 2026 00:44

vigneshrajsb merged commit 8933b58 into main Oct 13, 2026
5 checks passed