Repository navigation
feat(skill): use create_upload_url for local files - #3
Merged
Merged
Conversation
The hosted MCP server gains a free create_upload_url tool (sonilo-api-dashboard#407). The workflow skill told Codex to refuse a local file and ask for a public HTTPS URL, which would leave the new tool unused. Add a "Use a local file" section: upload only a file the user named, PUT it to the returned upload_url, pass file_url as the tool input, and fall back to asking for a public URL when the tool is missing or the PUT cannot be made. Uploading to any third-party host stays prohibited. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Lightsage docs evalsWaiting for the staging docs URL before running evals. Lightsage will start the selected PR evals automatically when GitHub reports a successful docs deployment for this PR. This usually happens within 15 minutes. Commit: |
steven-panxd
pushed a commit
that referenced
this pull request
Oct 9, 2026
The 1.2.0 draft was written from a checkout that predated PR #3 and added its own local-file section. Keep PR #3's "Use a local file" section and routing row as they are, drop the duplicate, and keep only what 1.2.0 adds on top: the generate_video_subtitles rename, the proofread service-key note, and the release-check guards, now matched to PR #3's wording. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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
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.
What
The hosted MCP server is getting a free
create_upload_urltool in sonilo-ai/sonilo-api-dashboard#407. It returns a presigned PUT URL plus afile_urlthat every hosted tool accepts as a media or subtitle URL.The workflow skill currently tells Codex that a local file is not a public URL and to ask for an HTTPS link. With that wording the new tool would go unused in Codex. This PR adds a Use a local file section to
SKILL.md:create_upload_urlwith the filename and exact byte size.upload_url(curl -T), with no authorization header.file_urlas the tool's URL input.If the tool is not connected or the PUT cannot be made, the skill falls back to asking for a public HTTPS URL. Uploading a user's file to any third-party host stays prohibited, and
upload_urlis treated as a credential that is never shown to the user.The routing table and the README's local-file note are updated to match.
Decisions for the reviewer
codex/plugin-v1.1.0, notmain, because that branch holds the current skill and has no open PR of its own. Merging changes the skill text that the 1.1.0 submission was reviewed with, so it may belong in a later version instead.plugin.json, no rebuiltdist/zips, no change to the skill's frontmatterdescriptionor the listing copy.Merge order
Merge after sonilo-api-dashboard#407 is deployed. The section is conditional on
create_upload_urlbeing among the connected tools, so before that it has no effect.Testing
python scripts/check_release.py: PASS (local checks). All strings the check requires are still present.🤖 Generated with Claude Code