Skip to content

[dev] [Marfuen] mariano/fix-screenshot-overlay-font-rendering#2659

Merged
Marfuen merged 2 commits intomainfrom
mariano/fix-screenshot-overlay-font-rendering
Apr 24, 2026
Merged

[dev] [Marfuen] mariano/fix-screenshot-overlay-font-rendering#2659
Marfuen merged 2 commits intomainfrom
mariano/fix-screenshot-overlay-font-rendering

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Apr 24, 2026

This is an automated pull request to merge mariano/fix-screenshot-overlay-font-rendering into dev.
It was created by the [Auto Pull Request] action.


Summary by cubic

Fixes broken text rendering in the screenshot overlay by installing system fonts in the API images and using DejaVu Sans in the SVG. Overlay text now renders correctly in production instead of square boxes.

  • Bug Fixes
    • Install fontconfig and fonts-dejavu-core in apps/api Dockerfiles to provide DejaVu Sans at runtime (node:*-slim has no fonts).
    • Set SVG font to "DejaVu Sans", sans-serif in screenshot-overlay.ts to match available fonts and avoid .notdef boxes from librsvg/sharp.

Written for commit d191dd1. Summary will update on new commits.

…uction

The node:*-slim base images ship with zero fonts installed, so
librsvg falls back to .notdef glyphs and the overlay rendered as
□ □ □ boxes in production. Install fontconfig + fonts-dejavu-core
in both Dockerfiles and use "DejaVu Sans" explicitly in the SVG.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Apr 24, 2026 1:27pm
comp-framework-editor Ready Ready Preview, Comment Apr 24, 2026 1:27pm
portal Ready Ready Preview, Comment Apr 24, 2026 1:27pm

Request Review

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Requires human review: This PR modifies Dockerfiles for production images. Changes to infrastructure and deployment configuration should be reviewed by a human to ensure build stability.

@Marfuen Marfuen merged commit 1d87427 into main Apr 24, 2026
15 checks passed
@Marfuen Marfuen deleted the mariano/fix-screenshot-overlay-font-rendering branch April 24, 2026 13:30
claudfuen pushed a commit that referenced this pull request Apr 24, 2026
## [3.32.1](v3.32.0...v3.32.1) (2026-04-24)

### Bug Fixes

* **browserbase:** install DejaVu Sans so overlay text renders in production ([#2659](#2659)) ([1d87427](1d87427))
* **device-agent:** dedicated long-lived session per install (CS-280) ([48fd4ab](48fd4ab))
* fix issue with document underscore when creating framework ([05f44eb](05f44eb))
@claudfuen
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.32.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants