Remove multi argument

Author: tonybaloney

backend/src/maintainability.rs

@@ -571,14 +571,14 @@ fn mi_rank(score: f64) -> char {
 }
 
 /// Analyze source code and return MI metrics
-fn analyze_source(source: &str, multi: bool) -> Result<(f64, char), String> {
+fn analyze_source(source: &str) -> Result<(f64, char), String> {
     let parsed = parse_module(source).map_err(|e| e.to_string())?;
 
     // Calculate raw metrics
     let raw = calculate_raw_metrics(source);
 
     // Calculate comment percentage
-    let comment_lines = raw.comments + if multi { raw.multi } else { 0 };
+    let comment_lines = raw.comments + raw.multi;
     let comments_percent = if raw.sloc > 0 {
         (comment_lines as f64 / raw.sloc as f64) * 100.0
     } else {
@@ -608,26 +608,25 @@ fn analyze_source(source: &str, multi: bool) -> Result<(f64, char), String> {
 }
 
 /// Public API for parallel module - returns (MI value, rank string).
-pub fn analyze_source_mi(source: &str, multi: bool) -> (f64, String) {
-    match analyze_source(source, multi) {
+pub fn analyze_source_mi(source: &str) -> (f64, String) {
+    match analyze_source(source) {
         Ok((mi, rank)) => (mi, rank.to_string()),
         Err(_) => (0.0, "C".to_string()),
     }
 }
 
 #[pyfunction]
-#[pyo3(signature = (entries, multi=true))]
+#[pyo3(signature = (entries))]
 pub fn harvest_maintainability_metrics(
     py: Python<'_>,
     entries: Vec<(String, String)>,
-    multi: bool,
 ) -> PyResult<Vec<(String, Py<PyDict>)>> {
     let mut results = Vec::with_capacity(entries.len());
 
     for (name, source) in entries {
         let dict = PyDict::new(py);
 
-        match analyze_source(&source, multi) {
+        match analyze_source(&source) {
             Ok((mi, rank)) => {
                 dict.set_item("mi", mi)?;
                 dict.set_item("rank", rank.to_string())?;

backend/src/parallel.rs

@@ -599,7 +599,6 @@ fn analyze_file(
     include_cyclomatic: bool,
     include_halstead: bool,
     include_maintainability: bool,
-    multi: bool,
 ) -> FileAnalysisResult {
     let raw = if include_raw {
         Some(RawResult {
@@ -651,7 +650,7 @@ fn analyze_file(
     };
 
     let maintainability = if include_maintainability {
-        let (mi, rank) = maintainability::analyze_source_mi(source, multi);
+        let (mi, rank) = maintainability::analyze_source_mi(source);
         Some(MaintainabilityResult {
             total: MaintainabilityTotal { mi, rank },
         })
@@ -686,18 +685,16 @@ fn analyze_file(
 /// # Arguments
 /// * `paths` - List of file paths to analyze
 /// * `operators` - List of operator names to run ("raw", "cyclomatic", "halstead", "maintainability")
-/// * `multi` - Whether to include multi-line strings in MI calculation
 ///
 /// # Returns
 /// A dictionary mapping file paths (and directory paths) to their analysis results.
 /// Directory paths contain aggregated metrics from all files within them.
 #[pyfunction]
-#[pyo3(signature = (paths, operators, multi=true))]
+#[pyo3(signature = (paths, operators))]
 pub fn analyze_files_parallel<'py>(
     py: Python<'py>,
     paths: Vec<String>,
     operators: Vec<String>,
-    multi: bool,
 ) -> PyResult<Bound<'py, PyDict>> {
     let include_raw = operators.iter().any(|o| o == "raw");
     let include_cyclomatic = operators.iter().any(|o| o == "cyclomatic");
@@ -732,7 +729,6 @@ pub fn analyze_files_parallel<'py>(
                         include_cyclomatic,
                         include_halstead,
                         include_maintainability,
-                        multi,
                     );
 
                     (path.clone(), result)

backend/src/storage.rs

@@ -1305,7 +1305,7 @@ impl WilyIndex {
                     };
 
                     let mi = if include_maintainability {
-                        let (mi_val, rank) = maintainability::analyze_source_mi(&content, true);
+                        let (mi_val, rank) = maintainability::analyze_source_mi(&content);
                         Some((mi_val, rank))
                     } else {
                         None

src/wily/backend.pyi

@@ -28,7 +28,6 @@ def iter_filenames(targets: list[str], include_ipynb: bool = False) -> list[str]
 def analyze_files_parallel(
     paths: list[str],
     operators: list[str],
-    multi: bool = False,
 ) -> dict[str, dict[str, Any]]:
     """Analyze files in parallel using Rust/rayon."""
     ...

src/wily/commands/build.py

@@ -86,7 +86,7 @@ def run_operators_parallel(
 
     # Run all operators in parallel on all files using Rust/rayon
     # This also computes directory-level aggregates
-    parallel_results = analyze_files_parallel(file_paths, operator_names, multi=True)
+    parallel_results = analyze_files_parallel(file_paths, operator_names)
 
     # Transform results into the expected format per operator
     results: dict[str, dict[str, Any]] = {name: {} for name in operator_names}