feat(cli): execute commands without file paths

CHANGELOG.md

@@ -15,6 +15,15 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
 ### CLI
 
+#### Enhancements
+
+- Biome now executes commands (lint, format, check and ci) on the working directory by default. [#2266](https://github.com/biomejs/biome/issues/2266) Contributed by @unvalley
+
+  ```diff
+  - biome check .
+  + biome check    # You can run the command without the path
+  ```
+
 ### Configuration
 
 ### Editors

crates/biome_cli/src/execute/traverse.rs

@@ -17,6 +17,7 @@ use crossbeam::channel::{unbounded, Receiver, Sender};
 use rustc_hash::FxHashSet;
 use std::sync::atomic::AtomicU32;
 use std::{
+    env::current_dir,
     ffi::OsString,
     panic::catch_unwind,
     path::{Path, PathBuf},
@@ -32,17 +33,28 @@ pub(crate) fn traverse(
     execution: &Execution,
     session: &mut CliSession,
     cli_options: &CliOptions,
-    inputs: Vec<OsString>,
+    mut inputs: Vec<OsString>,
 ) -> Result<(TraversalSummary, Vec<Error>), CliDiagnostic> {
     init_thread_pool();
-    if inputs.is_empty()
-        && execution.as_stdin_file().is_none()
-        && !cli_options.no_errors_on_unmatched
-    {
-        return Err(CliDiagnostic::missing_argument(
-            "<INPUT>",
-            format!("{}", execution.traversal_mode),
-        ));
+
+    if inputs.is_empty() {
+        match execution.traversal_mode {
+            TraversalMode::Check { .. }
+            | TraversalMode::Lint { .. }
+            | TraversalMode::Format { .. }
+            | TraversalMode::CI { .. } => match current_dir() {
+                Ok(current_dir) => inputs.push(current_dir.into_os_string()),
+                Err(err) => return Err(CliDiagnostic::io_error(err)),
+            },
+            _ => {
+                if execution.as_stdin_file().is_none() && !cli_options.no_errors_on_unmatched {
+                    return Err(CliDiagnostic::missing_argument(
+                        "<INPUT>",
+                        format!("{}", execution.traversal_mode),
+                    ));
+                }
+            }
+        }
     }
 
     let (interner, recv_files) = PathInterner::new();

crates/biome_cli/tests/commands/check.rs

@@ -2897,3 +2897,28 @@ fn print_json_pretty() {
         result,
     ));
 }
+
+#[test]
+fn lint_error_without_file_paths() {
+    let mut fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Path::new("check.js");
+    fs.insert(file_path.into(), LINT_ERROR.as_bytes());
+
+    let result: Result<(), biome_cli::CliDiagnostic> = run_cli(
+        DynRef::Borrowed(&mut fs),
+        &mut console,
+        Args::from([("check"), ""].as_slice()),
+    );
+
+    assert!(result.is_err(), "run_cli returned {result:?}");
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "lint_error_without_file_paths",
+        fs,
+        console,
+        result,
+    ));
+}

crates/biome_cli/tests/commands/ci.rs

@@ -1025,3 +1025,28 @@ A = 0;
         result,
     ));
 }
+
+#[test]
+fn does_formatting_error_without_file_paths() {
+    let mut fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Path::new("ci.js");
+    fs.insert(file_path.into(), UNFORMATTED.as_bytes());
+
+    let result = run_cli(
+        DynRef::Borrowed(&mut fs),
+        &mut console,
+        Args::from([("ci"), ""].as_slice()),
+    );
+
+    assert!(result.is_err(), "run_cli returned {result:?}");
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "does_formatting_error_without_file_paths",
+        fs,
+        console,
+        result,
+    ));
+}

crates/biome_cli/tests/commands/format.rs

@@ -3585,3 +3585,30 @@ fn print_json_pretty() {
         result,
     ));
 }
+
+#[test]
+fn format_without_file_paths() {
+    let mut fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Path::new("format.js");
+    fs.insert(file_path.into(), UNFORMATTED.as_bytes());
+
+    let result: Result<(), biome_cli::CliDiagnostic> = run_cli(
+        DynRef::Borrowed(&mut fs),
+        &mut console,
+        Args::from([("format"), ""].as_slice()),
+    );
+
+    assert!(result.is_err(), "run_cli returned {result:?}");
+
+    assert_file_contents(&fs, file_path, UNFORMATTED);
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "format_without_file_paths",
+        fs,
+        console,
+        result,
+    ));
+}

crates/biome_cli/tests/commands/lint.rs

@@ -1966,7 +1966,7 @@ fn group_level_recommended_false_enable_specific() {
     let code = r#"
     function SubmitButton() {
         return <button>Submit</button>;
-    }    
+    }
     "#;
 
     let file_path = Path::new("fix.jsx");
@@ -3246,3 +3246,28 @@ import "lodash";
         result,
     ));
 }
+
+#[test]
+fn should_lint_error_without_file_paths() {
+    let mut fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Path::new("check.js");
+    fs.insert(file_path.into(), LINT_ERROR.as_bytes());
+
+    let result = run_cli(
+        DynRef::Borrowed(&mut fs),
+        &mut console,
+        Args::from([("lint"), ""].as_slice()),
+    );
+
+    assert!(result.is_err(), "run_cli returned {result:?}");
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "lint_error_without_file_paths",
+        fs,
+        console,
+        result,
+    ));
+}

crates/biome_cli/tests/snapshots/main_commands_check/lint_error_without_file_paths.snap

@@ -0,0 +1,70 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: content
+---
+## `check.js`
+
+```js
+for(;true;);
+
+```
+
+# Termination Message
+
+```block
+check ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+
+
+
+```
+
+# Emitted Messages
+
+```block
+check.js:1:1 lint/style/useWhile  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Use a while loop instead of a for loop.
+
+  > 1 │ for(;true;);
+      │ ^^^^^^^^^^^
+    2 │ 
+
+  i Prefer a while loop over a for loop without initialization and update.
+
+  i Safe fix: Use a while loop.
+
+    1   │ - for(;true;);
+      1 │ + while(true);
+    2 2 │   
+
+
+```
+
+```block
+check.js:1:6 lint/correctness/noConstantCondition ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Unexpected constant condition.
+
+  > 1 │ for(;true;);
+      │      ^^^^
+    2 │ 
+
+
+```
+
+```block
+check.js format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Formatter would have printed the following content:
+
+    1 │ for·(;·true;·);
+      │    +  +     +  
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes needed.
+Found 3 errors.
+```

crates/biome_cli/tests/snapshots/main_commands_ci/does_formatting_error_without_file_paths.snap

@@ -0,0 +1,39 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: content
+---
+## `ci.js`
+
+```js
+  statement(  )  
+```
+
+# Termination Message
+
+```block
+ci ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+
+
+
+```
+
+# Emitted Messages
+
+```block
+ci.js format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × File content differs from formatting output
+
+    1   │ - ··statement(··)··
+      1 │ + statement();
+      2 │ + 
+
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes needed.
+Found 1 error.
+```

crates/biome_cli/tests/snapshots/main_commands_format/format_without_file_paths.snap

@@ -0,0 +1,39 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: content
+---
+## `format.js`
+
+```js
+  statement(  )  
+```
+
+# Termination Message
+
+```block
+format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+
+
+
+```
+
+# Emitted Messages
+
+```block
+format.js format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Formatter would have printed the following content:
+
+    1   │ - ··statement(··)··
+      1 │ + statement();
+      2 │ + 
+
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes needed.
+Found 1 error.
+```

crates/biome_cli/tests/snapshots/main_commands_lint/group_level_recommended_false_enable_specific.snap

@@ -23,7 +23,7 @@ expression: content
 
     function SubmitButton() {
         return <button>Submit</button>;
-    }    
+    }
 
 ```
 
@@ -48,7 +48,7 @@ fix.jsx:3:16 lint/a11y/useButtonType ━━━━━━━━━━━━━━
     2 │     function SubmitButton() {
   > 3 │         return <button>Submit</button>;
       │                ^^^^^^^^
-    4 │     }····
+    4 │     }
     5 │     
 
   i The default type of a button is submit, which causes the submission of a form when placed inside a `form` element. This is likely not the behaviour that you want inside a React application.