Skip to content

Conversation

@kapetr
Copy link
Contributor

@kapetr kapetr commented Nov 19, 2025

Summary

Fixes the display of agent-generated images in response markdown by rewriting their URIs to load the content through the API using the /api/v1/files/{file_id}/content endpoint.

Linked Issues

Resolves #1592

Documentation

  • No Docs Needed: minor UI fix

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Nov 19, 2025
@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @kapetr, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request resolves a UI bug that prevented agent-generated images from rendering correctly within markdown responses. The fix ensures these images are properly loaded by transforming their internal URIs into API-accessible content URLs, significantly improving the visual presentation of agent interactions.

Highlights

  • Image Display Fix: Addresses an issue where agent-generated images were not displayed correctly within markdown responses in the UI.
  • URI Rewriting Mechanism: Implements a mechanism to rewrite image URIs that originate from platform files, transforming them to load content through the /api/v1/files/{file_id}/content API endpoint.
  • Enhanced URL Transformation: Modifies the urlTransform utility function to intelligently detect and convert platform-specific image URLs, ensuring proper rendering.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@kapetr kapetr requested a review from PetrBulanek November 19, 2025 08:48
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly fixes an issue with displaying agent-generated images in markdown by transforming their URIs to use the file content API endpoint. The implementation is straightforward and effective. I've included one suggestion to enhance the robustness of the code by handling an edge case where the file ID might be missing from the URI.

Comment on lines +17 to +19
if (value.startsWith(PLATFORM_FILE_CONTENT_URL_BASE)) {
return getFileContentUrl(getFileIdFromFilePlatformUrl(value));
}
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve robustness, it's a good practice to check if a valid fileId is extracted before constructing the content URL. If value is just agentstack://, getFileIdFromFilePlatformUrl would return an empty string, leading to an invalid API call like /api/v1/files//content. Adding a check for fileId ensures we only process valid platform file URLs, falling back to the default transform for invalid ones.

  if (value.startsWith(PLATFORM_FILE_CONTENT_URL_BASE)) {
    const fileId = getFileIdFromFilePlatformUrl(value);

    if (fileId) {
      return getFileContentUrl(fileId);
    }
  }

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 19, 2025
@kapetr kapetr merged commit e5d0a72 into main Nov 19, 2025
8 of 9 checks passed
@kapetr kapetr deleted the fix/1592-display-agent-generated-images-in-markdown branch November 19, 2025 12:43
edengilbert pushed a commit to edengilbert/agentstack that referenced this pull request Nov 21, 2025
edengilbert added a commit that referenced this pull request Nov 21, 2025
…1513)

* Add blog post: How to Run a BeeAI Framework Agent as an A2A Server

Signed-off-by: Eden Gilbert <[email protected]>

* Finalize A2A Server blog post after preview edits

Signed-off-by: Eden Gilbert <[email protected]>

* Update a2a-server.mdx

Updated based on gemini feedback

Signed-off-by: Eden Gilbert <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): group trajectories (#1482)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): proactive access_token refresh to avoid refresh race (#1501)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): runtime envs and feature flags cleanup (#1479)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): render mermaid diagrams and latex syntax in markdown (#1488)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(sdk): utilize platform auth in mcp extension (#1508)

Signed-off-by: Tomas Pilar <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(server): add user info endpoint (#1505)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs: remove redundant ToC from connectors guide (#1519)

Signed-off-by: Jenna Winkler <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): array length checks (#1526)

Signed-off-by: Eden Gilbert <[email protected]>

* feat: initial implementation for connectors management in UI (#1443)

* feat: initial implementation for connectors management in UI

Signed-off-by: Tomas Weiss <[email protected]>

* feat: connectors prep work

Signed-off-by: Tomas Weiss <[email protected]>

* feat: proper error handling

Signed-off-by: Tomas Weiss <[email protected]>

* fix: missing licence headers

Signed-off-by: Tomas Weiss <[email protected]>

* fix: imports

Signed-off-by: Tomas Weiss <[email protected]>

* fix: missing schema

Signed-off-by: Tomas Weiss <[email protected]>

* chore: post rebase adjustments

Signed-off-by: Tomas Weiss <[email protected]>

* fix: proper schema

Signed-off-by: Tomas Weiss <[email protected]>

* fix: code review comments

Signed-off-by: Tomas Weiss <[email protected]>

---------

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs: update deployment guide (#1525)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): make 'add new agent' button admin only (#1527)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs: overhaul (#1540)

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(docs): add guide on connecting CLI to remote servers (#1541)

Signed-off-by: Tomas Pilar <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(server): refresh unmanaged agent cards (#1528)

Signed-off-by: Aleš Kalfas <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs: minor changes (#1545)

Signed-off-by: Matous Havlena <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* chore: docs workflow (#1502)

* chore: docs workflow

Signed-off-by: Tomas Weiss <[email protected]>

* fix: better quoting

Signed-off-by: Tomas Weiss <[email protected]>

* fix: better git habndling

Signed-off-by: Tomas Weiss <[email protected]>

* chore: improve PR template

Signed-off-by: Tomas Weiss <[email protected]>

* chore: provide docs

Signed-off-by: Tomas Weiss <[email protected]>

---------

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* chore: bump up uvicorn timeouts to 5s, make configurable (#1548)

Signed-off-by: Matous Havlena <[email protected]>
Signed-off-by: Jan Pokorný <[email protected]>
Co-authored-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(server): connectors header forwading (#1553)

Signed-off-by: Tomas Pilar <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc1

Signed-off-by: Eden Gilbert <[email protected]>

* docs: add explicit WSL instructions

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix: long operations inside transactions cause db connection exhaustion (#1549)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* ci: add missing mise.lock checksums

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(agentstack-cli): clarify agent statuses (fixes #1535)

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(agentstack-cli): redesign agentstack list output
Fixes #1536

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs(agentstack-cli): add custom help text (#1554)

Signed-off-by: Eden Gilbert <[email protected]>

* docs: improve WSL2 installation instructions

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* Update deployment-guide.mdx (#1573)

Signed-off-by: Jenna Winkler <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* chore: fixing dead links in the docs (#1557)

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* refactor(web): use single framewok-diagram.svg file themeable with CSS variables (#1539)

Signed-off-by: Eden Gilbert <[email protected]>

* refactor(ui): rework Spinner component with CSS only animation, remove lottie-react (#1538)

Signed-off-by: Eden Gilbert <[email protected]>

* fix(agentstack-cli): properly react to misconfigured WSL2

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs(agentstack-cli): clarify WSL install steps

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(agentstack-cli): add `agentstack server switch` alias

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): remove links to docs and support (#1570)

Signed-off-by: Eden Gilbert <[email protected]>

* chore(ui): update "starting the agent" copy (#1571)

Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): messages being cut off on narrower screens (#1546)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): hide offline agents (#1434)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): render Markdown syntax inside trajectories (#1555)

Signed-off-by: Eden Gilbert <[email protected]>

* chore(server): add resource discovery root fallback to connectors (#1587)

Signed-off-by: Tomas Pilar <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): update homepage and sidebar (#1512)

Signed-off-by: Eden Gilbert <[email protected]>

* chore: remove unnecessary __init__.py (#1494)

Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): replace trajectory group with latest content, add docs (#1523)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): mute error toasts on session expiration (#1589)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix: avoid setting up embeddings on RITS

resolves #1578

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat: update beeai-framework

Signed-off-by: Tomas Dvorak <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* Update README.md (#1569)

* Update README.md

Add Radek's agent as reference

Signed-off-by: Jenna Winkler <[email protected]>

* Update README.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jenna Winkler <[email protected]>

* Update README.md

* Update README.md

* Update README.md

Signed-off-by: Jenna Winkler <[email protected]>

---------

Signed-off-by: Jenna Winkler <[email protected]>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Eden Gilbert <[email protected]>

* docs: cleanup (#1586)

* docs: updates

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jenna Winkler <[email protected]>

* Update docs/docs.json

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jenna Winkler <[email protected]>

* Update authenticate-cli-to-server.mdx

Signed-off-by: Jenna Winkler <[email protected]>

* Update README.md

Signed-off-by: Jenna Winkler <[email protected]>

* docs: updates

---------

Signed-off-by: Jenna Winkler <[email protected]>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(helm): peroperly configure agent_registry.sync_period_cron

Fixes #1211

Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(tests): skip heavy docling e2e test (#1595)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat: add validation for agent import URL (#1544)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(agentstack-cli): use better heuristics for determining for adding agents (#1603)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(agentstack-cli): add server info and confirmations to CLI commands (#1590)

Signed-off-by: Eden Gilbert <[email protected]>

* feat(cli): add SingleSelect field support to forms (#1596)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(ui): display agent-generated images in response markdown (#1601)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): render markdown in API error messages (#1593)

Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Petr Bulánek <[email protected]>
Co-authored-by: Petr Bulánek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* docs: add permissions and tokens page (#1576)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Matous Havlena <[email protected]>
Co-authored-by: Matous Havlena <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(server): send proxy errors as a2a errors (#1594)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(tests): use maximize disk space action (#1607)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(tests): switch to custom model again (#1608)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* chore: form extension rework (#1481)

Signed-off-by: Eden Gilbert <[email protected]>

* fix(server): include redirect_uri into connector's token request (#1616)

Signed-off-by: Tomas Pilar <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat: update beeai framework (#1611)

Signed-off-by: Tomas Dvorak <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* fix(cli): agent info command missing auth (#1622)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* feat(ui): redo sidebar and homepage (#1610)

Signed-off-by: Eden Gilbert <[email protected]>

* feat: extending TS sdk (#1609)

* feat: extending TS sdk

Signed-off-by: Tomas Weiss <[email protected]>

* fix: code review comments

Signed-off-by: Tomas Weiss <[email protected]>

* fix: remove GET from the callApi function, still keep the method for extensibility in the future

Signed-off-by: Tomas Weiss <[email protected]>

* chore: cleanup package.json

Signed-off-by: Tomas Weiss <[email protected]>

* chore: testing release pipeline

Signed-off-by: Tomas Weiss <[email protected]>

* chore: install mise

Signed-off-by: Tomas Weiss <[email protected]>

* fix: no git checks

Signed-off-by: Tomas Weiss <[email protected]>

* chore: setup release action

Signed-off-by: Tomas Weiss <[email protected]>

* fix: code review commnets

Signed-off-by: Tomas Weiss <[email protected]>

---------

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc2

Signed-off-by: Eden Gilbert <[email protected]>

* fix(agents): add temporary fallback imports (#1624)

Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc3

Signed-off-by: Eden Gilbert <[email protected]>

* chore: update release sdk to npm instead of pnpm

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc4

Signed-off-by: Eden Gilbert <[email protected]>

* chore: update npm for release

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc5

Signed-off-by: Eden Gilbert <[email protected]>

* chore: add release tag for npm

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc6

Signed-off-by: Eden Gilbert <[email protected]>

* chore: missing repository

Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>

* release: v0.4.2-rc7

Signed-off-by: Eden Gilbert <[email protected]>

* chore: run mise fix

Signed-off-by: Eden Gilbert <[email protected]>

---------

Signed-off-by: Eden Gilbert <[email protected]>
Signed-off-by: Eden Gilbert <[email protected]>
Signed-off-by: Petr Kadlec <[email protected]>
Signed-off-by: Tomas Pilar <[email protected]>
Signed-off-by: Radek Ježek <[email protected]>
Signed-off-by: Jenna Winkler <[email protected]>
Signed-off-by: Tomas Weiss <[email protected]>
Signed-off-by: Aleš Kalfas <[email protected]>
Signed-off-by: Matous Havlena <[email protected]>
Signed-off-by: Jan Pokorný <[email protected]>
Signed-off-by: Tomas Dvorak <[email protected]>
Signed-off-by: Petr Bulánek <[email protected]>
Co-authored-by: Petr Kadlec <[email protected]>
Co-authored-by: Petr Bulánek <[email protected]>
Co-authored-by: Tomas Pilar <[email protected]>
Co-authored-by: Radek Ježek <[email protected]>
Co-authored-by: Jenna Winkler <[email protected]>
Co-authored-by: Tomáš Weiss <[email protected]>
Co-authored-by: Aleš Kalfas <[email protected]>
Co-authored-by: Matous Havlena <[email protected]>
Co-authored-by: Jan Pokorný <[email protected]>
Co-authored-by: Arun Babu Neelicattu <[email protected]>
Co-authored-by: Tomas Dvorak <[email protected]>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI - display agent generated images in markdown

3 participants