update CDN links for opencollection docs

[naman-bruno]

Description

Contribution Checklist:

• I've used AI significantly to create this pull request
• The pull request only addresses one issue or adds one feature.
• The pull request does not introduce any breaking changes
• I have added screenshots or gifs to help explain the change if applicable.
• I have read the contribution guidelines.
• Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

Summary by CodeRabbit

• Chores
  • Updated assets referenced in generated documentation for improved consistency and performance.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Single file update that modifies external asset references in the GenerateDocumentation component, switching CDN links from core/playground assets to OpenCollection docs assets (core.css → docs.css, playground.umd.js → docs.js).

Changes

Cohort / File(s)	Summary
Documentation CDN Asset References packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js	Updated HTML asset links from core/playground CDN assets to OpenCollection docs assets; replaced core.css with docs.css and playground.umd.js with docs.js

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• add: collection-docs #6583: Directly related—also modifies the GenerateDocumentation component's CDN asset references with the same filename updates.

Suggested reviewers

• bijin-bruno
• lohit-bruno
• helloanoop

Poem

🚀 From core to docs, the assets take flight,
CDN links now shine with OpenCollection's light,
Playground bows out, docs.js takes the stage,
Documentation generation turns a brand new page! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: updating CDN links for OpenCollection documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7105e4e and f00b09c.

📒 Files selected for processing (1)

• packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js

🧰 Additional context used

📓 Path-based instructions (1)

**/*.{js,jsx,ts,tsx}

📄 CodeRabbit inference engine (CODING_STANDARDS.md)

**/*.{js,jsx,ts,tsx}: Use 2 spaces for indentation. No tabs, just spaces
Stick to single quotes for strings. For JSX/TSX attributes, use double quotes (e.g., )
Always add semicolons at the end of statements
No trailing commas
Always use parentheses around parameters in arrow functions, even for single params
For multiline constructs, put opening braces on the same line, and ensure consistency. Minimum 2 elements for multiline
No newlines inside function parentheses
Space before and after the arrow in arrow functions. () => {} is good
No space between function name and parentheses. func() not func ()
Semicolons go at the end of the line, not on a new line
Names for functions need to be concise and descriptive
Add in JSDoc comments to add more details to the abstractions if needed
Add in meaningful comments instead of obvious ones where complex code flow is explained properly

Files:

• packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js

🧠 Learnings (1)

📚 Learning: 2025-12-17T21:41:24.730Z

Learnt from: naman-bruno
Repo: usebruno/bruno PR: 6407
File: packages/bruno-app/src/components/Environments/ConfirmCloseEnvironment/index.js:5-41
Timestamp: 2025-12-17T21:41:24.730Z
Learning: Do not suggest PropTypes validation for React components in the Bruno codebase. The project does not use PropTypes, so reviews should avoid proposing PropTypes and rely on the existing typing/validation approach (e.g., TypeScript or alternative runtime checks) if applicable. This guideline applies broadly to all JavaScript/JSX components in the repo.

Applied to files:

• packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)

• GitHub Check: Unit Tests
• GitHub Check: CLI Tests
• GitHub Check: Playwright E2E Tests
• GitHub Check: SSL Tests - Windows
• GitHub Check: SSL Tests - macOS
• GitHub Check: SSL Tests - Linux

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

# First, verify the file exists and check the context around lines 88-89
fd "GenerateDocumentation" --type f -path "*index.js" | head -5

Repository: usebruno/bruno

Length of output: 226

---

🏁 Script executed:

# Check if CDN URLs are accessible
echo "=== Checking https://cdn.opencollection.com/docs.css ===" 
curl -I -s --max-time 5 https://cdn.opencollection.com/docs.css 2>&1 | head -10

echo ""
echo "=== Checking https://cdn.opencollection.com/docs.js ===" 
curl -I -s --max-time 5 https://cdn.opencollection.com/docs.js 2>&1 | head -10

Repository: usebruno/bruno

Length of output: 768

---

🌐 Web query:

OpenCollection library docs.js window.OpenCollection API constructor

💡 Result:

Do you want (A) a link to the library documentation for a JavaScript API that exposes window.OpenCollection, (B) usage examples for the OpenCollection constructor, or (C) help inspecting window.OpenCollection in your browser (console commands)? Which one?

---

🏁 Script executed:

# Find the file with correct fd syntax
find . -name "index.js" -path "*GenerateDocumentation*" 2>/dev/null | head -5

Repository: usebruno/bruno

Length of output: 155

---

🏁 Script executed:

# Test the ORIGINAL CDN URLs mentioned in the review
echo "=== Checking https://cdn.opencollection.com/core.css ===" 
curl -I -s --max-time 5 https://cdn.opencollection.com/core.css 2>&1 | head -10

echo ""
echo "=== Checking https://cdn.opencollection.com/playground.umd.js ===" 
curl -I -s --max-time 5 https://cdn.opencollection.com/playground.umd.js 2>&1 | head -10

Repository: usebruno/bruno

Length of output: 795

---

🏁 Script executed:

# Read the file if found to understand context
if [ -f "packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js" ]; then
  echo "=== File content around lines 85-105 ==="
  sed -n '85,105p' "packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js"
fi

Repository: usebruno/bruno

Length of output: 736

---

New CDN URLs are not accessible—requests return HTTP 403 Forbidden.

The new URLs (https://cdn.opencollection.com/docs.css and https://cdn.opencollection.com/docs.js) return HTTP 403 errors and cannot be loaded. The original URLs (core.css and playground.umd.js) are working correctly with HTTP 200 responses.

This change will break the generated documentation HTML files. Either restore the original working URLs or provide accessible CDN endpoints for docs.css and docs.js before merging.

🤖 Prompt for AI Agents

In
packages/bruno-app/src/components/Sidebar/Collections/Collection/GenerateDocumentation/index.js
around lines 88-89, the updated CDN URLs
(https://cdn.opencollection.com/docs.css and
https://cdn.opencollection.com/docs.js) return HTTP 403 and will break generated
docs; revert these two lines to the previously working asset URLs (core.css and
playground.umd.js) or replace them with accessible CDN endpoints that return
HTTP 200, and verify by loading the URLs (or running automated fetch/check)
before merging.