Skip to content

fix: disambiguate TypeScript and MPEG-TS routing#3345

Open
huangruiteng wants to merge 1 commit into
volcengine:mainfrom
huangruiteng:codex/fix-3266-typescript-media-routing
Open

fix: disambiguate TypeScript and MPEG-TS routing#3345
huangruiteng wants to merge 1 commit into
volcengine:mainfrom
huangruiteng:codex/fix-3266-typescript-media-routing

Conversation

@huangruiteng

Copy link
Copy Markdown
Contributor

Summary

  • prefer the code/text path for local .ts files and repository TypeScript files
  • preserve MPEG-TS routing when HTTP metadata or the generated video namespace identifies media
  • normalize UnderstandingAPI media source_format values so TreeBuilder keeps media resources under their media namespace

Testing

  • pytest tests/parse/test_directory_parser_routing.py tests/parse/test_parser_router.py tests/parse/test_media_type_routing.py tests/storage/test_typescript_summary_routing.py -q --no-cov (47 passed)
  • ruff check on changed Python files
  • git diff --check

Fixes #3266

@huangruiteng

Copy link
Copy Markdown
Contributor Author

@qin-ctx 请协助 review:本 PR 修复 #3266,改动是「fix: disambiguate TypeScript and MPEG-TS routing」。必要性、验证与风险已写在 PR 描述中,谢谢!

@huangruiteng

Copy link
Copy Markdown
Contributor Author

@qin-ctx @MaojiaSheng #3283#3345 都在修复 #3266,且当前 CI 均绿、均可合并,需要维护者指定唯一 canonical。

我的判断是优先保留 #3283:它用有界内容探测区分 TypeScript 与 MPEG-TS,保留本地 MPEG-TS,并把规则抽象为可扩展的歧义后缀处理;#3345 更小,但更依赖来源元数据 / 资源命名空间,可能把本地 MPEG-TS 固定走代码路径。

请确认保留哪一条,并由 maintainer 关闭另一条;我不会自行合并或关闭。

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

[Bug]: TypeScript files are prioritised as media files

1 participant