You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Update proxies/isp.mdx IP rotation section: ISP proxies are static across sessions (not just within a session), only changing in rare ISP-initiated replacement events.
Update the info callout on proxies/overview.mdx to split ISP (cross-session stable) from datacenter (session-stable only).
Test plan
Visual check in Mintlify preview that both callouts render correctly.
Note
Low Risk
Documentation-only wording changes to proxy rotation behavior; no runtime or API changes.
Overview
Updates proxy docs so ISP and datacenter IP behavior match the product: ISP is described as a static exit IP across sessions (only rare ISP replacements), and datacenter as rotating per request, including within one browser session.
On proxies/isp.mdx and proxies/datacenter.mdx, the IP Rotation Behavior sections and cross-links are rewritten to point stable-IP use cases (allowlists, managed auth) at ISP instead of implying datacenter session stability.
The proxies/overview.mdxInfo callout no longer groups datacenter and ISP as session-stable; it now contrasts cross-session static (ISP) vs per-request rotation (datacenter) and when to use ISP, custom BYO, or residential.
Reviewed by Cursor Bugbot for commit 89ec31c. Bugbot is set up for automated code reviews on this repo. Configure here.
ISP proxies are static both within a single browser session and across
separate sessions attached to the same proxy. Update the ISP page and
the overview info callout to reflect this and split datacenter (still
session-stable only) from ISP in the cross-type guidance.
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk assessment: Very Low.
I re-evaluated the synchronized diff directly. The PR only changes copy in proxies/datacenter.mdx, proxies/isp.mdx, and proxies/overview.mdx to clarify datacenter versus ISP proxy IP behavior. There are no code, configuration, infrastructure, auth, permission, build, or prompt-instruction changes, and no CODEOWNERS file is present in the repo.
The PR is already approved, so I’m leaving this assessment as a comment rather than adding another approval.
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk assessment: Very Low.
I evaluated the current diff directly. It only changes explanatory copy in proxies/datacenter.mdx, proxies/isp.mdx, and proxies/overview.mdx to clarify datacenter versus ISP proxy IP behavior. There are no code, configuration, build, infrastructure, auth, permission, or prompt-instruction changes, and no CODEOWNERS file is present in the checkout.
The PR is already approved, so I’m leaving this assessment as a comment rather than adding another approval.
This PR didn't match the auto-monitor filter configured on your GitHub connection:
PRs in the kernel, infra, hypeman, and hypeship repos. kernel is a ~mono repo with many logical services underneath, ensure to focus on the implicated service for the PR
Reason: PR edits documentation files in the proxies section and does not indicate changes to the kernel, infra, hypeman, or hypeship repositories.
To monitor this PR anyway, reply with @firetiger monitor this.
The reason will be displayed to describe this comment to others. Learn more.
Risk assessment: Very Low.
I evaluated the current diff directly. It only changes explanatory copy in proxies/datacenter.mdx, proxies/isp.mdx, and proxies/overview.mdx to clarify datacenter versus ISP proxy IP behavior. There are no code, configuration, build, infrastructure, auth, permission, or prompt-instruction changes, and no CODEOWNERS file is present in the checkout.
The PR is already approved, so I’m leaving this assessment as a comment rather than adding another approval.
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
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.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Summary
proxies/isp.mdxIP rotation section: ISP proxies are static across sessions (not just within a session), only changing in rare ISP-initiated replacement events.proxies/overview.mdxto split ISP (cross-session stable) from datacenter (session-stable only).Test plan
Note
Low Risk
Documentation-only wording changes to proxy rotation behavior; no runtime or API changes.
Overview
Updates proxy docs so ISP and datacenter IP behavior match the product: ISP is described as a static exit IP across sessions (only rare ISP replacements), and datacenter as rotating per request, including within one browser session.
On
proxies/isp.mdxandproxies/datacenter.mdx, the IP Rotation Behavior sections and cross-links are rewritten to point stable-IP use cases (allowlists, managed auth) at ISP instead of implying datacenter session stability.The
proxies/overview.mdxInfo callout no longer groups datacenter and ISP as session-stable; it now contrasts cross-session static (ISP) vs per-request rotation (datacenter) and when to use ISP, custom BYO, or residential.Reviewed by Cursor Bugbot for commit 89ec31c. Bugbot is set up for automated code reviews on this repo. Configure here.