Skip to content

fix: share one 80-char cap for image and table asset filenames - #242

Merged
EricNGOntos merged 2 commits into
mainfrom
fix/wuchengke/unify-asset-filename-truncate
Aug 3, 2026
Merged

fix: share one 80-char cap for image and table asset filenames#242
EricNGOntos merged 2 commits into
mainfrom
fix/wuchengke/unify-asset-filename-truncate

Conversation

@EricNGOntos

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Confirm markdown image rename with a long last_context still succeeds (no File name too long)
  • Confirm table HTML filenames still truncate at 80 chars via sanitize_table_name_from_header
  • Smoke a PDF/markdown parse that produces both images and tables

Made with Cursor

EricNGOntos and others added 2 commits August 3, 2026 15:39
Unify the post-#236 image context truncate with table header naming so
cosmetic asset stems use a single MAX_ASSET_FILE_NAME_CHARS constant.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@EricNGOntos
EricNGOntos merged commit a10c216 into main Aug 3, 2026
6 checks passed
@EricNGOntos
EricNGOntos deleted the fix/wuchengke/unify-asset-filename-truncate branch August 3, 2026 08:23
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