Skip to content

Conversation

@0xBigBoss
Copy link
Owner

Summary

  • Fix image URL handling to preserve absolute URLs for better viewing experience
  • Internal images now keep absolute URLs instead of broken relative paths
  • External images continue to work as absolute URLs
  • All images remain viewable when users have internet connectivity

Changes Made

  • Modified src/transform.ts to keep all images as absolute URLs
  • Updated transform logic documentation
  • Added comprehensive test coverage for image handling scenarios
  • All 24 transform tests pass

Test Plan

  • All existing tests pass
  • New image handling tests added and passing
  • Lint checks pass
  • Tested with real DuckDB blog post URL
  • Verified images now show as absolute URLs in markdown output

Before

\![Author Avatar](../../../images/blog/authors/max_gabrielsson.jpg.md)

After

\![Author Avatar](https://duckdb.org/images/blog/authors/max_gabrielsson.jpg)

🤖 Generated with Claude Code

- Internal images now keep absolute URLs instead of broken relative paths
- External images continue to work as absolute URLs
- All images remain viewable when users have internet connectivity
- Internal links still transform to relative .md paths for local navigation
- Fixes issue where images had incorrect .md extensions appended

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@0xBigBoss 0xBigBoss merged commit 4434939 into main Jun 9, 2025
2 of 3 checks passed
@0xBigBoss 0xBigBoss deleted the fix/image-url-handling branch June 9, 2025 17:58
@github-actions github-actions bot mentioned this pull request Jun 9, 2025
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.

2 participants