docs: refresh UI path snapshot for API keys Settings rename - #605
Merged
Conversation
The Warp client renamed Settings > Cloud platform > Oz Cloud API Keys to API keys. Refresh valid_paths.json, migrate doc references, and keep the old label as a deprecated alias for validation.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Author
|
This PR was generated with Warp. Comment   |
hongyi-chen
marked this pull request as ready for review
August 23, 2026 20:41
hongyi-chen
enabled auto-merge (squash)
August 23, 2026 20:41
Contributor
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
![warp-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Overview
This PR refreshes the UI reference snapshot and docs copy for the Settings rename from Oz Cloud API Keys to API keys.
Concerns
- The added deprecated alias does not cover the historical full umbrella path, so old
Settings > Cloud platform > Oz Cloud API Keysreferences will not migrate through the intended alias path. - The terminology holdout list now includes a settings-label example that no longer contains
Oz, which makes the guidance harder to apply.
Verdict
Found: 0 critical, 1 important, 1 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
This was referenced Aug 23, 2026
- Add a deprecated_subpages alias map on the Cloud platform umbrella entry in valid_paths.json so the historical full path "Settings > Cloud platform > Oz Cloud API Keys" migrates deterministically to "Settings > Cloud platform > API keys", instead of falling into the umbrella-match branch and returning an unfixed fuzzy mismatch. - Add a self-test regression covering both the bare and full legacy path forms. - Remove the stale "Settings > Cloud platform > API keys" example from the "What still says Oz" section of terminology.md, since that label no longer contains "Oz". Co-Authored-By: Warp <agent@warp.dev>
hongyi-chen
approved these changes
Aug 23, 2026
warp-factories Bot
added a commit
that referenced
this pull request
Aug 24, 2026
The Warp client renamed Settings > Cloud platform > Oz Cloud API Keys to API keys. Refresh valid_paths.json, migrate doc references, and keep the old label as a deprecated alias for validation. Co-authored-by: warp-factories[bot] <243557089+warp-factories[bot]@users.noreply.github.com>
rachaelrenk
added a commit
that referenced
this pull request
Aug 24, 2026
Resolves conflicts with the tone overhaul (#606), which landed after this branch was approved and touched the same drafting surface. Templates (conceptual, faq, guide-page, procedural, reference): keep this branch's rewritten scaffolds and add #606's [BREVITY] line. #606 added that line to the pre-rewrite templates, so git could not reconcile the two. Main's verbose [AEO GUIDANCE] blocks stay dropped -- this branch replaced them with the one-line [AEO] pointer, which was part of the reviewed rewrite, not a loss from the merge. procedural.md also adopts main's corrected API keys path (Settings > Cloud platform > API keys, from #605). This branch still had the stale Settings > Platform. draft_docs: keep both rule sets. Main's stricter callout rule and its two new bullets (AI-ism buzzwords, user-visible model) plus this branch's link-text rule, which is the more precise one -- it names the per-type closing section and bans "Further reading" / "See also". Verified: no conflict markers, no duplicated checklistVerified: no conflict markers, no duplicated checkliste-check tests, 21/21 factory-noun tests. No src/content pages differ from main. Co-Authored-By: Warp <agent@warp.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
valid_paths.jsonfromwarpdotdev/warpafter the Settings UI rename of Oz Cloud API Keys → API keys.Validation
python3 .agents/skills/validate_ui_refs/validate_ui_refs.py --self-test→ PASSpython3 .agents/skills/validate_ui_refs/validate_ui_refs.py --all→ 0 path / command / format issuesRework changes
Settings > Cloud platform > Oz Cloud API Keys(the historical full path) fell into the umbrella-match branch and returned an unfixed fuzzy mismatch instead of migrating toSettings > Cloud platform > API keys. Added adeprecated_subpagesalias map on theCloud platformumbrella entry invalid_paths.jsonand a matching check invalidate_ui_path()so the full legacy path resolves deterministically. Added a self-test regression (_run_self_test) covering both the bare (Settings > Oz Cloud API Keys) and full (Settings > Cloud platform > Oz Cloud API Keys) legacy forms..agents/references/terminology.mdlistedSettings > Cloud platform > API keysunder "What still says Oz," but that label no longer contains "Oz." Removed the stale bullet (no remaining literal Oz Settings label exists to substitute it with; the client'sDisplayimpl forSettingsSectionhas no variant left that renders "Oz").