Skip to content

fix: update default state for advanced options and change default collection format#6366

Merged
bijin-bruno merged 1 commit intousebruno:mainfrom
abhishek-bruno:feat/file-type-default
Dec 9, 2025
Merged

fix: update default state for advanced options and change default collection format#6366
bijin-bruno merged 1 commit intousebruno:mainfrom
abhishek-bruno:feat/file-type-default

Conversation

@abhishek-bruno
Copy link
Member

@abhishek-bruno abhishek-bruno commented Dec 9, 2025 •
edited by coderabbitai bot
Loading

Description

  • Set default state of showAdvanced to true in CreateCollection component.
  • Change default format for collections from 'yml' to 'bru'.

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

  • New Features
    • Advanced options now display by default when creating collections.
    • Default file format for new collections changed to BRU.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 9, 2025 •
edited
Loading

Walkthrough

The CreateCollection component's initial state is modified to show advanced options by default instead of hidden. The default file format selection is changed from YAML ('yml') to BRU ('bru') to align with the advanced options visibility.

Changes

Cohort / File(s) Summary
State Initialization Updates
packages/bruno-app/src/components/Sidebar/CreateCollection/index.js		 Changed showAdvanced initial state from false to true; updated default file format from 'yml' to 'bru'

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

Advanced unlocked from the start, 🚀
BRU takes the leading part,
Defaults reshape with subtle art,
One small file, one daring heart.

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 directly and clearly summarizes the two main changes: setting advanced options to show by default and changing the default collection format from YAML to BRU.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent 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 cf4c896 and c245f36.

📒 Files selected for processing (1)
  • packages/bruno-app/src/components/Sidebar/CreateCollection/index.js (2 hunks)
🧰 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/CreateCollection/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: Playwright E2E Tests
  • GitHub Check: SSL Tests - macOS
  • GitHub Check: SSL Tests - Linux
  • GitHub Check: SSL Tests - Windows
  • GitHub Check: Unit Tests
  • GitHub Check: CLI Tests
🔇 Additional comments (2)
packages/bruno-app/src/components/Sidebar/CreateCollection/index.js (2)

30-30: Verify test and documentation updates for showAdvanced default behavior change.

Changing showAdvanced from false to true by default improves discoverability of the file format selector. The toggle in the dropdown menu (lines 308-317) remains functional.

Ensure tests and documentation reflect this new default behavior where advanced options are now visible by default.

50-50: Unable to verify review comment — file location does not match repository structure.

The specified file path packages/bruno-app/src/components/Sidebar/CreateCollection/index.js could not be found in the repository. Verify the correct file path before this review comment can be evaluated. The original claims about changing the default format from 'yml' to 'bru', test updates, and help text clarity cannot be verified without access to the actual file.

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

@bijin-bruno bijin-bruno merged commit 879d227 into usebruno:main Dec 9, 2025
8 checks passed
This was referenced Feb 1, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bijin-bruno bijin-bruno bijin-bruno approved these changes

@helloanoop helloanoop Awaiting requested review from helloanoop helloanoop is a code owner

@lohit-bruno lohit-bruno Awaiting requested review from lohit-bruno lohit-bruno is a code owner

@naman-bruno naman-bruno Awaiting requested review from naman-bruno naman-bruno is a code owner

Assignees

No one assigned

Labels

size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@abhishek-bruno @bijin-bruno