Skip to content

fix(server): allow blob attachments in web UI (#40692) - #173

Merged
DevFlex-AI merged 1 commit into
bolt-builder:devfrom
anomalyco:dev
Aug 5, 2026
Merged

fix(server): allow blob attachments in web UI (#40692)#173
DevFlex-AI merged 1 commit into
bolt-builder:devfrom
anomalyco:dev

Conversation

@DevFlex-AI

@DevFlex-AI DevFlex-AI commented Aug 5, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Please provide a description of the issue, the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the PR.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.


View with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is enabled.

Summary by CodeRabbit

  • Bug Fixes
    • Improved embedded UI security policy support for image and media content loaded from temporary browser sources.
    • Enabled terminal functionality, attachments, and theme preloading to work correctly within the embedded UI.

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

@rexdotsh is attempting to deploy a commit to the adevloper152's projects Team on Vercel.

A member of the Team first needs to authorize it.

@DevFlex-AI
DevFlex-AI merged commit c7fae89 into bolt-builder:dev Aug 5, 2026
13 of 24 checks passed
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0be23fc0-3416-4772-852d-7e755b928650

📥 Commits

Reviewing files that changed from the base of the PR and between 06ef4dc and 23bbc5c.

📒 Files selected for processing (2)
  • packages/opencode/src/server/shared/ui.ts
  • packages/opencode/test/server/httpapi-ui.test.ts

📝 Walkthrough

Walkthrough

The embedded UI Content Security Policy now permits blob: sources for images and media. Tests cover terminal WASM, blob attachments, theme preload CSP, and the updated img-src and connect-src directives.

Changes

UI CSP updates

Layer / File(s) Summary
CSP policy and validation
packages/opencode/src/server/shared/ui.ts, packages/opencode/test/server/httpapi-ui.test.ts
The UI CSP allows blob: URLs for images and media. The embedded UI test validates terminal WASM, blob attachments, theme preload CSP, and blob: in img-src and connect-src.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: thdxr, jlongster

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@deepsource-io

deepsource-io Bot commented Aug 5, 2026

Copy link
Copy Markdown

ImageDeepSource Code Review

We reviewed changes in 06ef4dc...23bbc5c on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

Important

Some issues found as part of this review are outside of the diff in this pull request and aren't shown in the inline review comments due to GitHub's API limitations. You can see those issues on the DeepSource dashboard.

PR Report Card

Overall Grade  Image

Focus Area: Reliability
Security  Image

Reliability  Image

Complexity  Image

Hygiene  Image

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Image Aug 5, 2026 11:10p.m. Review ↗
Shell Image Aug 5, 2026 11:10p.m. Review ↗
Secrets Image Aug 5, 2026 11:10p.m. Review ↗
Docker Image Aug 5, 2026 11:10p.m. Review ↗
Python Image Aug 5, 2026 11:10p.m. Review ↗
CSS Image Aug 5, 2026 11:10p.m. Review ↗
Rust Image Aug 5, 2026 11:10p.m. Review ↗
Ruby Image Aug 5, 2026 11:10p.m. Review ↗
Swift Image Aug 5, 2026 11:10p.m. Review ↗
PHP Image Aug 5, 2026 11:10p.m. Review ↗
Lua Image Aug 5, 2026 11:10p.m. Review ↗
Java Image Aug 5, 2026 11:10p.m. Review ↗
Go Image Aug 5, 2026 11:10p.m. Review ↗
C & C++ Image Aug 5, 2026 11:10p.m. Review ↗
Ansible Image Aug 5, 2026 11:10p.m. Review ↗
Apex Image Aug 5, 2026 11:10p.m. Review ↗
Elixir Image Aug 5, 2026 11:10p.m. Review ↗
Groovy Image Aug 5, 2026 11:10p.m. Review ↗
Objective-C Image Aug 5, 2026 11:10p.m. Review ↗
PowerShell Image Aug 5, 2026 11:10p.m. Review ↗
Terraform Image Aug 5, 2026 11:10p.m. Review ↗
VB.NET Image Aug 5, 2026 11:10p.m. Review ↗
SQL Image Aug 5, 2026 11:10p.m. Review ↗
Scala Image Aug 5, 2026 11:10p.m. Review ↗
Perl Image Aug 5, 2026 11:10p.m. Review ↗
Kotlin Image Aug 5, 2026 11:10p.m. Review ↗
Helm Image Aug 5, 2026 11:10p.m. Review ↗
Erlang Image Aug 5, 2026 11:10p.m. Review ↗
Dart Image Aug 5, 2026 11:10p.m. Review ↗
C# Image Aug 5, 2026 11:10p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

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