feat(import): set yml as a default file formal accross all type of imports

packages/bruno-app/src/components/Sidebar/BulkImportCollectionLocation/index.js

@@ -154,7 +154,7 @@ export const BulkImportCollectionLocation = ({
   const [applyToGlobal, setApplyToGlobal] = useState(true);
   const [applyToCollection, setApplyToCollection] = useState(false);
   const [groupingType, setGroupingType] = useState('tags');
-  const [collectionFormat, setCollectionFormat] = useState('bru');
+  const [collectionFormat, setCollectionFormat] = useState('yml');
   const [renamedCollectionNames, setRenamedCollectionNames] = useState({});
   const [renamedEnvironmentNames, setRenamedEnvironmentNames] = useState({});
   const [importIssues, setImportIssues] = useState({});
@@ -585,6 +585,7 @@ export const BulkImportCollectionLocation = ({
       <Modal
         size="md"
         title="Bulk Import"
+        dataTestId="bulk-import-collection-location-modal"
         confirmText={importStarted ? 'Close' : 'Import'}
         confirmDisabled={Boolean(!selectedCollections?.length)}
         handleConfirm={onSubmit}
@@ -878,6 +879,7 @@ export const BulkImportCollectionLocation = ({
                   <select
                     id="format"
                     name="format"
+                    data-testid="collection-format-selector"
                     className="block textbox mt-2 w-full"
                     value={collectionFormat}
                     onChange={(e) => setCollectionFormat(e.target.value)}

packages/bruno-cli/src/utils/collection.js

@@ -539,7 +539,7 @@ const safeWriteFileSync = (filePath, content) => {
  * @param {string} dirPath - The output directory path
  */
 const createCollectionFromBrunoObject = async (collection, dirPath, options = {}) => {
-  const { format = 'bru' } = options;
+  const { format = 'yml' } = options;
   // Create brunoConfig for yml format
   const brunoConfig = {
     version: '1',
@@ -595,7 +595,7 @@ const createCollectionFromBrunoObject = async (collection, dirPath, options = {}
  * @param {"bru"|"yml"} options.format - Current directory path
  */
 const processCollectionItems = async (items = [], currentPath, options = {}) => {
-  const { format = 'bru' } = options;
+  const { format = 'yml' } = options;
   for (const item of items) {
     if (item.type === 'folder') {
       // Create folder

tests/import/bulk-import/yml-as-default-file-format.spec.ts

@@ -0,0 +1,65 @@
+import { test, expect } from '../../../playwright';
+import * as path from 'path';
+import * as fs from 'fs';
+import { closeAllCollections, openBulkImportModal } from '../../utils/page';
+import { buildCommonLocators } from '../../utils/page/locators';
+
+test.describe('Bulk Import default file format', () => {
+  const testDataDir = path.join(__dirname, '../test-data');
+  const filesToImport = [
+    path.join(testDataDir, 'sample-postman.json'),
+    path.join(testDataDir, 'sample-insomnia.json')
+  ];
+
+  test.afterEach(async ({ page }) => {
+    await closeAllCollections(page);
+  });
+
+  test('File Format selector defaults to OpenCollection (YAML)', async ({ page }) => {
+    await openBulkImportModal(page, filesToImport);
+    const locators = buildCommonLocators(page);
+
+    await test.step('File Format defaults to OpenCollection (YAML)', async () => {
+      const formatSelect = locators.import.bulkFormatSelect();
+      await expect(formatSelect).toBeVisible();
+      await expect(formatSelect).toHaveValue('yml');
+      await expect(formatSelect.locator('option')).toHaveText(['OpenCollection (YAML)', 'BRU Format (.bru)']);
+    });
+  });
+
+  test('importing with the default format writes opencollection.yml collections', async ({ page, createTmpDir }) => {
+    await openBulkImportModal(page, filesToImport);
+    const locators = buildCommonLocators(page);
+    const importDir = await createTmpDir('bulk-import-default-format');
+
+    await test.step('Check the File Format defaults to OpenCollection (YAML)', async () => {
+      await expect(locators.import.bulkFormatSelect()).toHaveValue('yml');
+    });
+
+    await test.step('Import with the default format and close the modal', async () => {
+      await locators.import.bulkLocationInput().fill(importDir);
+
+      await expect(locators.import.bulkSubmitButton()).toHaveText('Import');
+      await locators.import.bulkSubmitButton().click();
+
+      await expect(locators.import.bulkSubmitButton()).toHaveText('Close');
+      await locators.import.bulkSubmitButton().click();
+      await expect(locators.import.bulkModal()).toBeHidden();
+    });
+
+    await test.step('Verify each collection is written as opencollection.yml, not bruno.json', async () => {
+      const collectionDirs = fs
+        .readdirSync(importDir, { withFileTypes: true })
+        .filter((entry) => entry.isDirectory())
+        .map((entry) => path.join(importDir, entry.name));
+
+      expect(collectionDirs.length).toBeGreaterThan(0);
+
+      for (const dir of collectionDirs) {
+        const files = fs.readdirSync(dir);
+        expect(files).toContain('opencollection.yml');
+        expect(files).not.toContain('bruno.json');
+      }
+    });
+  });
+});

tests/utils/page/actions.ts

@@ -55,7 +55,7 @@
         await firstCollection.hover();
         await firstCollection.locator('.collection-actions .icon').click({ force: true });
         await expect(removeMenuItem).toBeVisible({ timeout: 2000 });
       }).toPass({ timeout: 15000 });
       await removeMenuItem.click();
 
       // Wait for modal to appear - could be either regular remove or drafts confirmation
@@ -354,7 +354,7 @@
     } else {
       await locators.sidebar.collection(parentName).hover();
       const collectionAction = locators.actions.collectionActions(parentName);
       await expect(collectionAction).toBeVisible({ timeout: 2000 });
       await collectionAction.click();
     }
 
@@ -481,7 +481,7 @@
   await test.step(`Import collection from "${filePath}"`, async () => {
     const locators = buildCommonLocators(page);
 
     await page.getByTestId('collections-header-add-menu').click();
     await page.locator('.tippy-box .dropdown-item').filter({ hasText: 'Import collection' }).click();
 
     // Wait for import modal
@@ -524,6 +524,26 @@
   });
 };
 
+/**
+ * Open the Bulk Import modal by importing multiple files at once.
+ * Selecting more than one file routes the import flow to the Bulk Import modal
+ * (instead of the single-collection location modal).
+ * @param page - The page object
+ * @param filePaths - Absolute paths of the files to import (must be 2 or more)
+ */
+const openBulkImportModal = async (page: Page, filePaths: string[]) => {
+  await test.step('Open the Bulk Import modal', async () => {
+    const locators = buildCommonLocators(page);
+
+    await locators.plusMenu.button().click();
+    await locators.plusMenu.importCollection().click();
+    await expect(locators.import.modal()).toBeVisible();
+
+    await locators.import.fileInput().setInputFiles(filePaths);
+    await expect(locators.import.bulkModal()).toBeVisible();
+  });
+};
+
 /**
  * Remove a specific collection from the sidebar
  * @param page - The page object
@@ -693,7 +713,7 @@
 ) => {
   await test.step(`Add environment variable "${variable.name}"`, async () => {
     const nameInput = page.locator(`input[name="${index}.name"]`);
     await nameInput.waitFor({ state: 'visible' });
     await nameInput.fill(variable.name);
 
     // Wait for the CodeMirror editor in the row to be ready
@@ -2099,6 +2119,7 @@
   deleteRequest,
   deleteCollectionFromOverview,
   importCollection,
+  openBulkImportModal,
   removeCollection,
   createFolder,
   openEnvironmentSelector,

tests/utils/page/locators.ts

@@ -214,6 +214,10 @@ export const buildCommonLocators = (page: Page) => ({
     locationModal: () => page.locator('[data-testid="import-collection-location-modal"]'),
     locationInput: () => page.locator('#collection-location'),
     fileInput: () => page.locator('input[type="file"]'),
+    bulkModal: () => page.getByTestId('bulk-import-collection-location-modal'),
+    bulkFormatSelect: () => page.getByTestId('bulk-import-collection-location-modal').getByTestId('collection-format-selector'),
+    bulkLocationInput: () => page.getByTestId('bulk-import-collection-location-modal').locator('#collection-location'),
+    bulkSubmitButton: () => page.getByTestId('bulk-import-collection-location-modal-submit-btn'),
     envOption: (name: string) => page.locator('.dropdown-item').getByText(name, { exact: true }),
     parsingError: () => page.getByTestId('import-error-message'),
     browseLink: (root?: Locator) => (root ?? page).getByTestId('import-collection-browse-link'),