fixes: OpenAPI Spec Viewer: throws resolver errors for $ref schemas

[arpit-bruno]

Description

OpenAPI 3.1 specs were failing to render in the packaged app, every internal $ref throws "Could not resolve reference: Evaluation failed on URI".
In Dev mode and 3.0 specs are working fine.

Problem

1. Resolver: swagger-client's 3.1 resolver checks the scheme of document.baseURI before dereferencing any refs. The packaged app loads over file:// so the check fails for every ref.
2. Error panel: the panel rendered with raw swagger-ui styles (unreadable heading colors in dark theme), and the long resolver URIs overflowed the panel. Also on window resize the panel was not responsive.

Fix

1. Report an http base to swagger with a small override next to the SwaggerUI mount.

Root cause (resolver):
https://github.com/swagger-api/swagger-js/blob/v3.36.1/src/resolver/apidom/reference/dereference/strategies/openapi-3-1-swagger-client/visitors/dereference.js#L519
the condition is always true, the else branch meant to resolve unknown-scheme bases (like file://) vand the catch-block is isURL-gated, so non-http bases have no path to resolve and hence the error is thrown.

Screenshots

Screen.Recording.2026-07-19.at.4.38.39.PM.mov

Note
The real fix needs to be done in swagger-js , I will raise a PR there for the resolver's scheme gate. Once that lands and swagger-ui-react picks it up, this overriding fix can be deleted.

2. Themed the panel with the app's danger tokens, long tokens now wraps and the panel is responsive

How to replicate the error panel:

openapi: 3.1.0
info:
  title: Long Error Test
  version: 1.0.0
paths:
  /pets:
    get:
      summary: List pets
      responses:
        '200':
          description: ok
          content:
            application/json:
              schema:
                $ref: 'file:///a/very/long/unresolvable/path/that/does/not/exist/index.html#/components/schemas/Pet'

1. Open this spec (API Specs → Open API Spec),
2. then expand GET /pets .

Before :

After:

REF: BRU-3939

Summary by CodeRabbit

• Bug Fixes
  • Improved Swagger error panel styling for clearer visibility and better readability across themes.
  • Fixed Swagger rendering in packaged applications by enabling internal OpenAPI references to resolve correctly.
  • Improved handling of long error messages and constrained content widths for more consistent layouts.

[coderabbitai]

Walkthrough

The Swagger renderer now supports packaged app execution over file: URLs and applies themed styling to its errors panel.

Changes

Swagger renderer

Layer / File(s)	Summary
Packaged URL base handling packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/index.js	The renderer replaces non-HTTP(S) document.baseURI values with http://localhost/ before Swagger UI initialization.
Themed Swagger error presentation packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/StyledWrapper.js	The errors panel uses themed danger colors, spacing, borders, overflow handling, constrained widths, and theme text colors.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: utkarsh-bruno

Poem

Swagger paths cross packaged skies,
While themed errors neatly rise.
Borders frame each warning bright,
Text now matches the chosen light.
$ref finds home through every flight.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the OpenAPI Spec Viewer resolver errors for $ref schemas, which is the main change.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

🧹 Nitpick comments (1)

packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/index.js (1)

17-22: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win

Scope the document.baseURI override to Swagger resolution.

SpecViewer is the only renderer source that renders Swagger, but this module redefines global document.baseURI for the whole Electron renderer. If any other renderer path later needs to rely on the real base URI, that behavior will change. Move this into Swagger’s dependency/config, or use an explicit resolver/base option instead of mutating the global document.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/index.js`
around lines 17 - 22, Remove the global document.baseURI mutation from the
module-level initialization and scope the localhost fallback to Swagger
resolution in SpecViewer. Pass an explicit resolver or base-URI option through
Swagger’s dependency/configuration path, preserving the existing fallback only
for Swagger without affecting other renderers.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/index.js`:
- Around line 17-22: Remove the global document.baseURI mutation from the
module-level initialization and scope the localhost fallback to Swagger
resolution in SpecViewer. Pass an explicit resolver or base-URI option through
Swagger’s dependency/configuration path, preserving the existing fallback only
for Swagger without affecting other renderers.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 14a3aa68-d43b-4d1b-b5b3-9223f46779c1

📥 Commits

Reviewing files that changed from the base of the PR and between 1c2ec7c and 5be517b.

📒 Files selected for processing (2)

• packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/StyledWrapper.js
• packages/bruno-app/src/components/ApiSpecPanel/Renderers/Swagger/index.js

[arpit-bruno]

Ref: swagger-api/swagger-client#4258