fix(reset): default to all components when no pattern provided (#10109)

The `bit reset` command previously required either a component pattern
or the `--all` flag. This was confusing because:
- `--all` means "all components" (not "all versions")
- But the default behavior already resets all local versions of each
component

This change aligns `bit reset` with `bit tag` behavior - running without
arguments now defaults to all components, with a confirmation prompt for
safety.

Author: davidfirst

components/legacy/e2e-helper/e2e-command-helper.ts

@@ -519,7 +519,7 @@ export default class CommandHelper {
     return this.runCmd(`bit reset ${id} ${head ? '--head' : ''} ${flag}`);
   }
   resetAll(options = '') {
-    return this.runCmd(`bit reset ${options} --all`);
+    return this.runCmd(`bit reset ${options} --silent`);
   }
   resetSoft(id: string) {
     return this.runCmd(`bit reset ${id} --soft`);

gilad-import-fork.diff

@@ -1,5 +1,6 @@
 import { BitError } from '@teambit/bit-error';
 import chalk from 'chalk';
+import yesno from 'yesno';
 import type { Command, CommandOptions } from '@teambit/cli';
 import { COMPONENT_PATTERN_HELP } from '@teambit/legacy.constants';
 import type { SnappingMain } from './snapping.main.runtime';
@@ -15,11 +16,12 @@ export default class ResetCmd implements Command {
   ];
   group = 'version-control';
   extendedDescription = `removes local component versions (tags/snaps) that haven't been exported yet.
-by default reverts all local versions. use --head to revert only the latest version.
+if no component-pattern is provided, resets all components (with confirmation prompt).
+by default reverts all local versions of each component. use --head to revert only the latest version.
 useful for undoing mistakes before exporting. exported versions cannot be reset.`;
   alias = '';
   options = [
-    ['a', 'all', 'revert all unexported tags/snaps for all components'],
+    ['a', 'all', 'DEPRECATED. this is now the default behavior when no component-pattern is provided'],
     ['', 'head', 'revert the head tag/snap only (by default, all local tags/snaps are reverted)'],
     ['', 'soft', 'revert only soft-tags (components tagged with --soft flag)'],
     [
@@ -28,6 +30,7 @@ useful for undoing mistakes before exporting. exported versions cannot be reset.
       "revert the tag even if it's used as a dependency. WARNING: components that depend on this tag will be corrupted",
     ],
     ['', 'never-exported', 'reset only components that were never exported'],
+    ['s', 'silent', 'skip confirmation when resetting all components'],
   ] as CommandOptions;
   loader = true;
 
@@ -40,22 +43,21 @@ useful for undoing mistakes before exporting. exported versions cannot be reset.
       head = false,
       force = false,
       soft = false,
+      silent = false,
       neverExported = false,
-    }: { all?: boolean; head?: boolean; force?: boolean; soft?: boolean; neverExported?: boolean }
+    }: { all?: boolean; head?: boolean; force?: boolean; soft?: boolean; silent?: boolean; neverExported?: boolean }
   ) {
     if (neverExported) {
       const compIds = await this.snapping.resetNeverExported();
       return chalk.green(`successfully reset the following never-exported components:\n${compIds.join('\n')}`);
     }
-    if (!pattern && !all) {
-      throw new BitError('please specify a component-pattern or use --all flag');
-    }
-    if (pattern && all) {
-      throw new BitError('please specify either a component-pattern or --all flag, not both');
-    }
     if (soft && head) {
       throw new BitError('please specify either --soft or --head flag, not both');
     }
+    // if no pattern provided, reset all components (with confirmation unless --silent or --all)
+    if (!pattern && !silent && !all) {
+      await this.promptForResetAll();
+    }
     const { results, isSoftUntag } = await this.snapping.reset(pattern, head, force, soft);
     const titleSuffix = isSoftUntag ? 'soft-untagged (are not candidates for tagging any more)' : 'reset';
     const title = chalk.green(`${results.length} component(s) were ${titleSuffix}:\n`);
@@ -64,4 +66,14 @@ useful for undoing mistakes before exporting. exported versions cannot be reset.
     });
     return title + components.join('\n');
   }
+
+  private async promptForResetAll() {
+    this.snapping.logger.clearStatusLine();
+    const ok = await yesno({
+      question: `${chalk.bold('This will reset all local tags/snaps for all components. Would you like to proceed? [yes(y)/no(n)]')}`,
+    });
+    if (!ok) {
+      throw new BitError('the operation has been canceled');
+    }
+  }
 }

scopes/component/snapping/reset-cmd.ts

@@ -161,7 +161,7 @@ export function snapResultOutput(results: SnapResults): string {
 
   const warningsOutput = warnings && warnings.length ? `${chalk.yellow(warnings.join('\n'))}\n\n` : '';
   const snapExplanation = `\n(use "bit export" to push these components to a remote")
-(use "bit reset --all" to unstage all local versions, or "bit reset --head" to only unstage the latest local snap)`;
+(use "bit reset" to unstage all local versions, or "bit reset --head" to only unstage the latest local snap)`;
 
   const compInBold = (id: ComponentID) => {
     const version = id.hasVersion() ? `@${id.version}` : '';