docs: update guides for SDK changes - #4274
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Claude finished @sdkrelease[bot]'s task in 1m 53s —— View job Docs review — Looks good
The PR changes one sentence in The version is correct. I pulled the published # composio 0.20.0 (published) — utils/sensitive_file_upload_paths.py
def _normalize_path_segments(file_path: t.Union[str, Path]) -> t.List[str]:
p = Path(file_path).expanduser()
resolved = p.resolve() # only the resolved path is ever checkedversus the tree on this branch, where # 0.21.0 — python/composio/utils/sensitive_file_upload_paths.py:36
def _normalize_path_segments(file_path) -> tuple[list[str], list[str]]:
"""Return resolved and written segments so neither can hide a denied name."""So 0.21.0 is genuinely the first Python release with dual-path matching, and "Earlier versions check only the symlink-resolved path" is accurate for 0.20.0 and below. The unchanged TypeScript half still holds. No API-usage, missing-step, or ordering issues; the page is existing content, so no index or One non-blocking note, since it applies to both halves of the sentence and not to this diff specifically: neither version is published yet (PyPI latest is |
d6abfb1 to
4b7e2b5
Compare
|
Claude finished @sdkrelease[bot]'s task in 2m 43s —— View job Docs review — one suggested fix
The force-push replaced the earlier one-line version edit with a new callout. Current diff: a 100 MiB download-cap callout in Every behavioral claim in the callout checks out. I read the implementation rather than trusting the wording:
Also correct to omit any mention of tuning the cap: Dependency realignment is clean. All ten bumps match npm The callout needs a version floorIssue: the download cap is not in any installable release, and unlike the callout 15 lines below it, this one states no version floor — so it reads as describing current behavior.
A reader on either published version who downloads a 200 MB file gets it saved, not an error. This is the same gap I noted last time, but it now sits next to a callout that does declare floors ( Fix — append one sentence: <Callout type="info">
Automatic downloads are capped at **100 MiB** per file, because the download URL comes from the tool response and an oversized body would otherwise exhaust memory or fill the disk. A larger file fails instead of being saved — Python raises `ResponseTooLargeError`, TypeScript throws an error naming the byte count and the limit — and the partial download is discarded. To handle bigger files, leave automatic file handling off and fetch the `s3url` from the raw tool response yourself. The cap requires `@composio/core` 0.18.1 or later (TypeScript) and a `composio` release newer than 0.21.0 (Python); earlier versions download without a size limit.
</Callout>The TS number is pinned by the changeset; fill in the Python one when that release is cut, or keep the "newer than 0.21.0" phrasing the neighbouring callout already uses for the same reason. No other issues: no wrong API usage, no missing steps, no ordering problems, and the page is existing content so no |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
4b7e2b5 to
42dbdbc
Compare
42dbdbc to
abcbfa4
Compare
abcbfa4 to
4386527
Compare
4386527 to
89d1668
Compare
89d1668 to
7e19caa
Compare
7e19caa to
0000450
Compare
0000450 to
25381f8
Compare
25381f8 to
aa0b2c5
Compare
aa0b2c5 to
9cc3449
Compare
|
Claude finished @sdkrelease[bot]'s task in 4m 29s —— View job Docs review — two fixes suggested
The force-push replaced the download-cap callout with a triggers change. Current diff: What checks out. 1. The filter is documented but silently ignored on the published TypeScript releaseIssue: No Python needs no floor: I pulled the published Fix — add a floor sentence, matching the phrasing style used for the other version callouts on the site:
2. The callout's V1/V2 claim doesn't hold for the
|
9cc3449 to
d0bc6e5
Compare
Summary
Automated docs update triggered by SDK source changes on
next.that reference changed APIs or features.
@composio/*dependencies were realigned to their latestpublished releases so Twoslash snippets and example apps validate
against versions users can actually install.
Review checklist
will fail — wait for the release instead of working around it
Generated by Claude Code via GitHub Actions.