Skip to content

Describe seconv MCP container track defaults accurately - #106

Draft
Blackspirits wants to merge 1 commit into
upl/review-base-mcp-ad51from
fix/mcp-track-description-ad51
Draft

Describe seconv MCP container track defaults accurately#106
Blackspirits wants to merge 1 commit into
upl/review-base-mcp-ad51from
fix/mcp-track-description-ad51

Conversation

@Blackspirits

Copy link
Copy Markdown
Owner

Summary

Upstream PR SubtitleEdit#14129 describes convert_subtitle.trackNumbers as:

Default: every text track.

That is narrower than the actual shared converter behavior. With no track filter, the container loader also processes supported image subtitle tracks:

  • Matroska PGS;
  • Matroska/MP4 VobSub;
  • other image-based paths handled by the shared loader.

Those tracks can trigger OCR unless timeCodesOnly is enabled. An MCP client relying on the current description can therefore incorrectly assume that an unfiltered container conversion is text-only and cheap.

This change updates only the tool parameter description to state that the default is every supported subtitle track, including supported image tracks that may require OCR.

Validation

AI assistance: ChatGPT was used to compare MCP tool descriptions against the shared container loader.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant