📄 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/ResponsePane/ResponsePaneActions/index.js
• packages/bruno-app/src/components/ResponsePane/index.js
• packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js
• tests/response/response-actions.spec.ts

⚙️ CodeRabbit configuration file

tests/**/**.*: Review the following e2e test code written using the Playwright test library. Ensure that:

• Follow best practices for Playwright code and e2e automation

• Try to reduce usage of page.waitForTimeout(); in code unless absolutely necessary and the locator cannot be found using existing expect() playwright calls

• Avoid using page.pause() in code

• Use locator variables for locators

• Avoid using test.only

• Use multiple assertions

• Promote the use of test.step as much as possible so the generated reports are easier to read

• Ensure that the fixtures like the collections are nested inside the fixtures folder

  Fixture Example*: Here's an example of possible fixture and test pair

  .
  ├── fixtures
  │   └── collection
  │       ├── base.bru
  │       ├── bruno.json
  │       ├── collection.bru
  │       ├── ws-test-request-with-headers.bru
  │       ├── ws-test-request-with-subproto.bru
  │       └── ws-test-request.bru
  ├── connection.spec.ts # <- Depends on the collection in ./fixtures/collection
  ├── headers.spec.ts
  ├── persistence.spec.ts
  ├── variable-interpolation
  │   ├── fixtures
  │   │   └── collection
  │   │       ├── environments
  │   │       ├── bruno.json
  │   │       └── ws-interpolation-test.bru
  │   ├── init-user-data
  │   └── variable-interpolation.spec.ts # <- Depends on the collection in ./variable-interpolation/fixtures/collection
  └── subproto.spec.ts
  

Files:

• tests/response/response-actions.spec.ts

Learnt from: sid-bruno
Repo: usebruno/bruno PR: 6266
File: packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js:38-38
Timestamp: 2025-12-02T09:45:31.709Z
Learning: In the ResponseCopy component (packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js), the copy button is intentionally disabled using `!response.data` to prevent copying stream resets which result in empty strings.

Applied to files:

• packages/bruno-app/src/components/ResponsePane/ResponsePaneActions/index.js
• packages/bruno-app/src/components/ResponsePane/index.js
• packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js

Learnt from: CR
Repo: usebruno/bruno PR: 0
File: CODING_STANDARDS.md:0-0
Timestamp: 2025-12-05T20:31:33.005Z
Learning: Applies to **/*.test.{js,jsx,ts,tsx} : Add tests for any new functionality or meaningful changes. If code is added, removed, or significantly modified, corresponding tests should be updated or created

Applied to files:

• tests/response/response-actions.spec.ts

Learnt from: sanish-bruno
Repo: usebruno/bruno PR: 6090
File: tests/scripting/hooks/init-user-data/ui-state-snapshot.json:1-8
Timestamp: 2025-12-16T07:16:08.934Z
Learning: For e2e tests in the bruno repository: Collections that are shared between CLI and UI tests (comprehensive test suites testing core functionality) should be placed in `packages/bruno-tests/` to avoid duplication. The `tests/**/fixtures/collection` pattern should be used for test-specific collections that test particular UI behaviors or are specific to a single test file.

Applied to files:

• tests/response/response-actions.spec.ts

packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js (1)

• ResponseCopy (47-96)

packages/bruno-app/src/components/ResponsePane/index.js (2)

• selectedFormat (38-38)
• selectedTab (39-39)

packages/bruno-app/src/utils/common/index.js (2)

• dataBuffer (284-284)
• dataBuffer (388-388)

packages/bruno-app/src/components/ResponsePane/index.js (2)

• selectedFormat (38-38)
• selectedTab (39-39)

packages/bruno-app/src/components/ResponsePane/QueryResponse/index.js (2)

• selectedFormat (19-19)
• selectedTab (20-20)

packages/bruno-app/src/utils/common/index.js (4)

• dataBuffer (284-284)
• dataBuffer (388-388)
• formatResponse (277-433)
• formatResponse (277-433)

tests/utils/page/actions.ts (6)

• createCollection (717-717)
• createRequest (718-718)
• sendRequest (731-731)
• switchToEditorTab (740-740)
• switchResponseFormat (738-738)
• clickResponseAction (741-741)

packages/bruno-app/src/utils/codemirror/brunoVarInfo.spec.js (1)

• clipboardText (311-311)