Fix next issue

codetracer-python-recorder/src/trace_filter/engine.rs: 

Signed-off-by: Tzanko Matev <[email protected]>

Author: tzanko-matev

codetracer-python-recorder/src/trace_filter/engine.rs

@@ -384,7 +384,7 @@ impl ScopeContext {
                 let stripped_owned = stripped.to_path_buf();
                 let better = match &best_match {
                     Some((_, current)) => {
-                        stripped_owned.components().count() >= current.components().count()
+                        stripped_owned.components().count() < current.components().count()
                     }
                     None => true,
                 };
@@ -622,6 +622,54 @@ mod tests {
         })
     }
 
+    #[test]
+    fn project_filter_beats_builtin_source_selection() -> RecorderResult<()> {
+        const BUILTIN_LABEL: &str = "builtin-default";
+        const BUILTIN_FILTER: &str =
+            include_str!("../../resources/trace_filters/builtin_default.toml");
+
+        let project = tempdir().expect("project");
+        let filters_dir = project.path().join(".codetracer");
+        fs::create_dir_all(&filters_dir).unwrap();
+        let filter_path = filters_dir.join("trace-filter.toml");
+        write_filter(
+            &filter_path,
+            r#"
+            [scope]
+            default_exec = "skip"
+            default_value_action = "allow"
+
+            [[scope.rules]]
+            selector = "pkg:my_app.sec"
+            exec = "trace"
+            "#,
+        );
+
+        let code_path = project.path().join("my_app").join("sec.py");
+        fs::create_dir_all(code_path.parent().unwrap()).unwrap();
+        fs::File::create(&code_path).unwrap();
+        let code_path = code_path.to_string_lossy().to_string();
+
+        let config = TraceFilterConfig::from_inline_and_paths(
+            &[(BUILTIN_LABEL, BUILTIN_FILTER)],
+            &[filter_path.clone()],
+        )?;
+
+        Python::with_gil(|py| -> RecorderResult<()> {
+            let module = load_module(py, "my_app.sec", &code_path, "def sec():\n    return 1\n")?;
+            let code_obj = get_code(&module, "sec")?;
+            let wrapper = CodeObjectWrapper::new(py, &code_obj);
+
+            let engine = TraceFilterEngine::new(config.clone());
+            let resolution = engine.resolve(py, &wrapper)?;
+
+            assert_eq!(resolution.exec(), ExecDecision::Trace);
+            assert_eq!(resolution.module_name(), Some("my_app.sec"));
+            assert_eq!(resolution.relative_path(), Some("my_app/sec.py"));
+            Ok(())
+        })
+    }
+
     #[test]
     fn caches_resolution_and_applies_value_patterns() -> RecorderResult<()> {
         let (config, file_path) = filter_with_pkg_rule(