feat: add ignore_missing_submod option

Configurations.md

@@ -1306,6 +1306,15 @@ If you want to ignore every file under the directory where you put your rustfmt.
 ignore = ["/"]
 ```
 
+## `ignore_missing_submod`
+
+Ignore missing submodule error.
+By default, missing modules will lead to module not found errors and rustfmt won't format anything.
+
+- **Default value**: `false`
+- **Possible values**: `true`, `false`
+- **Stable**: No
+
 ## `imports_indent`
 
 Indent style of imports

src/config/mod.rs

@@ -179,6 +179,7 @@ create_config! {
          or they are left with trailing whitespaces";
     ignore: IgnoreList, IgnoreList::default(), false,
         "Skip formatting the specified files and directories";
+    ignore_missing_submod: bool, false, false, "Ignore missing submodule error";
 
     // Not user-facing
     verbose: Verbosity, Verbosity::Normal, false, "How much to information to emit to the user";
@@ -697,6 +698,7 @@ show_parse_errors = true
 error_on_line_overflow = false
 error_on_unformatted = false
 ignore = []
+ignore_missing_submod = false
 emit_mode = "Files"
 make_backup = false
 "#,

src/formatting.rs

@@ -135,6 +135,7 @@ fn format_project<T: FormatHandler>(
         &context.parse_session,
         directory_ownership.unwrap_or(DirectoryOwnership::UnownedViaBlock),
         !input_is_stdin && !config.skip_children(),
+        config.ignore_missing_submod(),
     )
     .visit_crate(&krate)?
     .into_iter()

src/modules.rs

@@ -62,6 +62,7 @@ pub(crate) struct ModResolver<'ast, 'sess> {
     directory: Directory,
     file_map: FileModMap<'ast>,
     recursive: bool,
+    ignore_missing_submod: bool,
 }
 
 /// Represents errors while trying to resolve modules.
@@ -105,6 +106,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
         parse_sess: &'sess ParseSess,
         directory_ownership: DirectoryOwnership,
         recursive: bool,
+        ignore_missing_submod: bool,
     ) -> Self {
         ModResolver {
             directory: Directory {
@@ -114,6 +116,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
             file_map: BTreeMap::new(),
             parse_sess,
             recursive,
+            ignore_missing_submod,
         }
     }
 
@@ -249,6 +252,14 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
             // mod foo;
             // Look for an extern file.
             self.find_external_module(item.ident, &item.attrs, sub_mod)
+                .or_else(|err| match err.kind {
+                    ModuleResolutionErrorKind::NotFound { file: _ }
+                        if self.ignore_missing_submod =>
+                    {
+                        Ok(None)
+                    }
+                    _ => Err(err),
+                })
         } else {
             // An internal module (`mod foo { /* ... */ }`);
             Ok(Some(SubModKind::Internal(item)))

tests/rustfmt/main.rs

@@ -1,8 +1,8 @@
 //! Integration tests for rustfmt.
 
 use std::env;
-use std::fs::remove_file;
-use std::path::Path;
+use std::fs::{read_to_string, remove_file};
+use std::path::{Path, PathBuf};
 use std::process::Command;
 
 use rustfmt_config_proc_macro::rustfmt_only_ci_test;
@@ -200,3 +200,30 @@ fn rustfmt_emits_error_when_control_brace_style_is_always_next_line() {
     let (_stdout, stderr) = rustfmt(&args);
     assert!(!stderr.contains("error[internal]: left behind trailing whitespace"))
 }
+
+#[test]
+fn ignore_missing_sub_mod_false() {
+    // Ensure that missing submodules cause module not found errors when trying to
+    // resolve submodules with `skip_children=false` and `ignore_missing_submod=false`
+    let args = [
+        "--config=skip_children=false,ignore_missing_submod=false",
+        "tests/source/issue-5609.rs",
+    ];
+    let (_stdout, stderr) = rustfmt(&args);
+    // Module resolution fails because we're unable to find `missing_submod.rs`
+    assert!(stderr.contains("missing_submod.rs does not exist"))
+}
+
+#[test]
+fn ignore_missing_sub_mod_true() {
+    // Ensure that missing submodules don't cause module not found errors when trying to
+    // resolve submodules with `skip_children=false` and `ignore_missing_submod=true`.
+    let args = [
+        "--emit=stdout",
+        "--config=skip_children=false,ignore_missing_submod=true",
+        "tests/source/issue-5609.rs",
+    ];
+    let (stdout, _stderr) = rustfmt(&args);
+    let target = read_to_string(PathBuf::from("tests/target/issue-5609.rs")).unwrap();
+    assert!(stdout.ends_with(&target));
+}

tests/source/issue-5609.rs

@@ -0,0 +1,8 @@
+// rustfmt-ignore_missing_submod: true
+// rustfmt-skip_children: false
+
+mod missing_submod;
+
+fn test() {
+
+}

tests/target/issue-5609.rs

@@ -0,0 +1,6 @@
+// rustfmt-ignore_missing_submod: true
+// rustfmt-skip_children: false
+
+mod missing_submod;
+
+fn test() {}