chore: point README docs links at docs.e2b.dev - #331
Conversation
The docs site moved to its own subdomain. e2b.dev/docs now 308s to docs.e2b.dev/, and the subdomain has no /docs path prefix, so e2b.dev/docs/<path> maps to docs.e2b.dev/<path>. UTM parameters are unchanged and already survived the redirect, so this removes a redirect hop rather than fixing broken attribution. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM — a trivial, mechanical documentation link update.
What was reviewed: the three README link changes (root, js/, python/) swapping e2b.dev/docs for docs.e2b.dev/, confirming UTM query parameters are preserved unchanged; the new changeset file's version bump scope (patch for both packages) matches the two READMEs that ship with those packages.
Extended reasoning...
Overview
The diff touches only three README files and adds one new changeset file. Each README change replaces a single markdown link's URL from https://e2b.dev/docs?... to https://docs.e2b.dev/?..., preserving the UTM query string exactly. The changeset declares a patch bump for @ e2b/code-interpreter and @ e2b/code-interpreter-python, matching the two README files (js and python) that ship with those packages, plus the root README for the monorepo.
Security risks
None. This is a documentation-only change with no code, no build config, no dependency, and no CI changes. The only risk vector would be a malicious redirect target, but docs.e2b.dev is the project's own documented domain and the PR description's redirect explanation is corroborated by the URL structure.
Level of scrutiny
Low. This is exactly the kind of small, mechanical, self-contained change (three one-line URL swaps plus a changeset) that fits the approval guidelines' criteria for low-risk changes needing no deep human review.
Other factors
Verified via git diff that each URL edit is a straightforward substring swap of the domain/path with the UTM parameters byte-for-byte unchanged, matching the PR description's claims. No CODEOWNERS file constraints apply to README/changeset paths, and there's no prior review activity or unaddressed objections in the timeline to consider.
2 participants
Follow-up to #330. The docs site moved to its own subdomain, so the links that PR tagged point at the pre-migration domain.
e2b.dev/docsreturns a 308 todocs.e2b.dev/, and the subdomain has no/docspath prefix, soe2b.dev/docs/<path>maps todocs.e2b.dev/<path>.The UTM query string is preserved across the redirect, so attribution was not broken. This removes the redirect hop.
Three links, one per README. Changeset covers both SDK packages, since
js/README.mdandpython/README.mdship to npm and PyPI.🤖 Generated with Claude Code