Repository navigation
Sonilo plugin 1.2.0: generate_video_subtitles and local-file upload - #4
Draft
steven-panxd wants to merge 21 commits into
Draft
steven-panxd wants to merge 21 commits into
steven-panxd wants to merge 21 commits into
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>
feat(skill): use create_upload_url for local files
…pload The hosted server no longer lists `proofread`; the subtitle tool is `generate_video_subtitles`, with `proofread` kept only as the service key that get_account_services reports. In fresh Codex sessions the 1.1.0 skill still picked the right tool 8 times out of 9, and gave up once because `proofread` was missing. The hosted server also gained `create_upload_url`, so a file on the user's machine can be used. The skill now uploads a file the user named to the one-time URL Sonilo returns, keeps that URL out of replies, and asks for a public HTTPS link where the upload cannot run. Rules for user-supplied URLs are unchanged. plugin.json also picks up the listing copy that was sitting uncommitted in the working tree (long description and first starter prompt), since the 1.2.0 wording builds on it. check_release.py guards against routing to the retired name and checks the upload rules. See docs/REVISION_1.2.0_NOTES.md for validation and what still has to change in the portal before this is submitted. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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>
…dy in this branch) origin/main's tree is identical to 5a434ca, an ancestor of this branch, so main contributes no content. Record the merge so the PR is mergeable. 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.
Draft. Do not merge or package until the 1.1.0 review at OpenAI has a decision.
Why
Two hosted MCP changes landed after the 1.1.0 package was submitted for review:
generate_video_subtitles.proofreadis no longer in the hosted tool list, and OpenAI's scan no longer shows it. The package in review still routes toproofread.create_upload_urllets the hosted server take a file from the user's machine. The skill guidance for it was merged intocodex/plugin-v1.1.0in feat(skill): use create_upload_url for local files #3, after the package had been submitted, so the package in review does not contain it.What is in this PR
Against
mainthis PR carries #3 (the "Use a local file" section, unchanged) plus the 1.2.0 work on top of it:generate_video_subtitles. The skill notes that its availability and trial allowance are still reported under theproofreadservice key.references/dubbing.md: an edited.srtor.vtton the user's machine can be uploaded and used as the dubbing script.references/examples.md: tool name, one local-file example.plugin.json: version 1.2.0. The long description no longer says media must be public HTTPS links. It also picks up the listing copy that had been uncommitted (long description and first starter prompt).scripts/check_release.py: paid-tool list, a guard against routing to the retired name, and guards for the upload rules.README.md: feature list, example, data-sent and local-command statements.docs/REVISION_1.2.0_NOTES.md: release notes, validation, and what must change in the portal.Rules for user-supplied URLs are unchanged.
Validation
python3 scripts/check_release.pyand--livepass.generate_video_subtitles; 1 stopped becauseproofreadwas missing.generate_video_subtitles.create_upload_url, could not upload, started nothing, asked for a public HTTPS link.video_to_musicalthough a finished video was requested; one run, not investigated.http://URL: refused with no tool call.Not validated: a complete run to a delivered result, the Codex desktop app, ChatGPT web, the packaged ZIP, and the portal Skill scan.
Before submitting to OpenAI
docs/LISTING_COPY_1.1.0.jsonstill carry the old long-description sentence.generate_video_subtitlesandcreate_upload_url, and theproofreadentry removed.🤖 Generated with Claude Code