@lamentis/naome 1.3.12 → 1.3.13

package/Cargo.lock

@@ -76,7 +76,7 @@ checksum = "f8ca58f447f06ed17d5fc4043ce1b10dd205e060fb3ce5b979b8ed8e59ff3f79"
 
 [[package]]
 name = "naome-cli"
-version = "1.3.12"
+version = "1.3.13"
 dependencies = [
  "naome-core",
  "serde_json",
@@ -84,7 +84,7 @@ dependencies = [
 
 [[package]]
 name = "naome-core"
-version = "1.3.12"
+version = "1.3.13"
 dependencies = [
  "serde",
  "serde_json",

package/crates/naome-cli/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "naome-cli"
-version = "1.3.12"
+version = "1.3.13"
 edition.workspace = true
 license.workspace = true
 repository.workspace = true

package/crates/naome-core/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "naome-core"
-version = "1.3.12"
+version = "1.3.13"
 edition.workspace = true
 license.workspace = true
 repository.workspace = true

package/crates/naome-core/src/architecture/rules/external.rs

@@ -73,6 +73,7 @@ fn is_standard_library_dependency(language: Option<&str>, package: &str) -> bool
         Some("javascript") | Some("typescript") => NODE_BUILTINS.contains(&root),
         Some("python") => PYTHON_STDLIB.contains(&root),
         Some("rust") => matches!(root, "std" | "core" | "alloc" | "proc_macro" | "test"),
+        Some("swift") => SWIFT_APPLE_FRAMEWORKS.contains(&root),
         _ => false,
     }
 }
@@ -183,3 +184,61 @@ const PYTHON_STDLIB: &[&str] = &[
     "xml",
     "zipfile",
 ];
+
+const SWIFT_APPLE_FRAMEWORKS: &[&str] = &[
+    "Accelerate",
+    "AppIntents",
+    "AppKit",
+    "ARKit",
+    "AVFoundation",
+    "CloudKit",
+    "Combine",
+    "CoreData",
+    "CoreFoundation",
+    "CoreBluetooth",
+    "CoreGraphics",
+    "CoreImage",
+    "CoreLocation",
+    "CoreML",
+    "CoreMotion",
+    "CoreNFC",
+    "CoreSpotlight",
+    "CoreTelephony",
+    "CoreText",
+    "CryptoKit",
+    "Dispatch",
+    "EventKit",
+    "Foundation",
+    "GameKit",
+    "HealthKit",
+    "LocalAuthentication",
+    "MapKit",
+    "MessageUI",
+    "Metal",
+    "MetalKit",
+    "NaturalLanguage",
+    "Network",
+    "Observation",
+    "PassKit",
+    "PencilKit",
+    "Photos",
+    "QuartzCore",
+    "RealityKit",
+    "SafariServices",
+    "SceneKit",
+    "Security",
+    "SpriteKit",
+    "StoreKit",
+    "Swift",
+    "SwiftData",
+    "SwiftUI",
+    "ThreadNetwork",
+    "UIKit",
+    "UniformTypeIdentifiers",
+    "UserNotifications",
+    "Vision",
+    "WatchKit",
+    "WebKit",
+    "WidgetKit",
+    "XCTest",
+];

package/crates/naome-core/src/architecture/scan/graph_builder.rs

@@ -5,7 +5,7 @@ use std::path::Path;
 use serde_json::json;
 
 use super::imports;
-use super::FileFact;
+use super::{FileFact, ManifestFact};
 use crate::architecture::config::ArchitectureConfig;
 use crate::architecture::model::{ArchitectureEdgeKind, ArchitectureGraph, ArchitectureNodeKind};
 
@@ -16,13 +16,16 @@ pub(super) fn build_path_graph(
     root: &Path,
     files: Vec<String>,
     config: &ArchitectureConfig,
+    manifests: &[ManifestFact],
 ) -> (ArchitectureGraph, BTreeMap<String, FileFact>) {
     let mut graph = ArchitectureGraph::default();
     let mut file_facts = BTreeMap::new();
+    let mut emitted_external_nodes = BTreeSet::new();
     let file_set = files.iter().cloned().collect::<BTreeSet<_>>();
 
     push_repository_and_policy_nodes(&mut graph, config);
     push_directories(&mut graph, &files);
+    push_manifests(&mut graph, manifests, &mut emitted_external_nodes);
 
     for path in files {
         let content = fs::read_to_string(root.join(&path)).unwrap_or_default();
@@ -32,7 +35,7 @@ pub(super) fn build_path_graph(
         file_facts.insert(path, fact);
     }
 
-    push_imports(&mut graph, &file_facts);
+    push_imports(&mut graph, &file_facts, &mut emitted_external_nodes);
     (graph, file_facts)
 }
 
@@ -121,44 +124,117 @@ fn push_file(graph: &mut ArchitectureGraph, fact: &FileFact) {
     push_membership_edges(graph, path, "context", &fact.contexts);
 }
 
-fn push_imports(graph: &mut ArchitectureGraph, file_facts: &BTreeMap<String, FileFact>) {
-    let mut emitted_target_nodes = BTreeSet::new();
+fn push_manifests(
+    graph: &mut ArchitectureGraph,
+    manifests: &[ManifestFact],
+    emitted_external_nodes: &mut BTreeSet<String>,
+) {
+    for manifest in manifests {
+        let package_id = manifest_package_id(manifest);
+        emit::push_node_with_metadata(
+            graph,
+            &package_id,
+            ArchitectureNodeKind::Package,
+            &manifest.package_name,
+            Some(manifest.path.clone()),
+            None,
+            manifest.confidence,
+            &manifest.extractor,
+            json!({
+                "path": manifest.path,
+                "ecosystem": manifest.ecosystem,
+                "packageName": manifest.package_name
+            }),
+        );
+        emit::push_edge(
+            graph,
+            facts::parent_node_id(&manifest.path)
+                .as_deref()
+                .unwrap_or("repository:."),
+            &package_id,
+            ArchitectureEdgeKind::Contains,
+            "contains",
+            Some(manifest.path.clone()),
+        );
+        for dependency in &manifest.dependencies {
+            let external_id = format!("external:{}", dependency.name);
+            push_external_node(
+                graph,
+                emitted_external_nodes,
+                &external_id,
+                &dependency.name,
+                dependency.confidence,
+                &manifest.extractor,
+                json!({
+                    "manifestPath": manifest.path,
+                    "ecosystem": manifest.ecosystem,
+                    "dependencyKind": dependency.dependency_kind,
+                    "version": dependency.version
+                }),
+            );
+            emit::push_edge_with_metadata(
+                graph,
+                &package_id,
+                &external_id,
+                ArchitectureEdgeKind::DependsOn,
+                "depends_on",
+                Some(manifest.path.clone()),
+                None,
+                None,
+                dependency.confidence,
+                &manifest.extractor,
+                json!({
+                    "specifier": dependency.name,
+                    "manifestPath": manifest.path,
+                    "dependencyKind": dependency.dependency_kind,
+                    "version": dependency.version
+                }),
+            );
+        }
+    }
+}
+
+fn manifest_package_id(manifest: &ManifestFact) -> String {
+    format!(
+        "package:{}:{}",
+        stable_fragment(&manifest.ecosystem),
+        stable_fragment(&manifest.path)
+    )
+}
+
+fn push_imports(
+    graph: &mut ArchitectureGraph,
+    file_facts: &BTreeMap<String, FileFact>,
+    emitted_external_nodes: &mut BTreeSet<String>,
+) {
     for fact in file_facts.values() {
         for import in &fact.imports {
             let target_id = match &import.target {
                 super::ImportTarget::File(path) => format!("file:{path}"),
                 super::ImportTarget::ExternalDependency(name) => {
                     let id = format!("external:{name}");
-                    if emitted_target_nodes.insert(id.clone()) {
-                        emit::push_node_with_metadata(
-                            graph,
-                            &id,
-                            ArchitectureNodeKind::ExternalDependency,
-                            name,
-                            None,
-                            None,
-                            import.confidence,
-                            &import.extractor,
-                            json!({ "specifier": import.specifier, "resolvedAs": "external" }),
-                        );
-                    }
+                    push_external_node(
+                        graph,
+                        emitted_external_nodes,
+                        &id,
+                        name,
+                        import.confidence,
+                        &import.extractor,
+                        json!({ "specifier": import.specifier, "resolvedAs": "external" }),
+                    );
                     id
                 }
                 super::ImportTarget::Unknown(specifier) => {
                     let id = format!("unknown-import:{}", stable_fragment(specifier));
-                    if emitted_target_nodes.insert(id.clone()) {
-                        emit::push_node_with_metadata(
-                            graph,
-                            &id,
-                            ArchitectureNodeKind::ExternalDependency,
-                            specifier,
-                            None,
-                            None,
-                            import.confidence,
-                            &import.extractor,
-                            json!({ "specifier": specifier, "resolvedAs": "unknown" }),
-                        );
-                    }
+                    push_external_node(
+                        graph,
+                        emitted_external_nodes,
+                        &id,
+                        specifier,
+                        import.confidence,
+                        &import.extractor,
+                        json!({ "specifier": specifier, "resolvedAs": "unknown" }),
+                    );
                     id
                 }
             };
@@ -179,6 +255,30 @@ fn push_imports(graph: &mut ArchitectureGraph, file_facts: &BTreeMap<String, Fil
     }
 }
 
+fn push_external_node(
+    graph: &mut ArchitectureGraph,
+    emitted_external_nodes: &mut BTreeSet<String>,
+    id: &str,
+    label: &str,
+    confidence: f32,
+    extractor: &str,
+    raw_origin: serde_json::Value,
+) {
+    if emitted_external_nodes.insert(id.to_string()) {
+        emit::push_node_with_metadata(
+            graph,
+            id,
+            ArchitectureNodeKind::ExternalDependency,
+            label,
+            None,
+            None,
+            confidence,
+            extractor,
+            raw_origin,
+        );
+    }
+}
+
 fn push_membership_edges(
     graph: &mut ArchitectureGraph,
     path: &str,

package/crates/naome-core/src/architecture/scan/imports/extractors/swift.rs

@@ -0,0 +1,48 @@
+use super::{is_comment_only, raw, RawImport};
+
+pub(super) fn extract(content: &str) -> Vec<RawImport> {
+    let mut imports = Vec::new();
+    for (index, line) in content.lines().enumerate() {
+        let trimmed = line.trim();
+        if is_comment_only(trimmed) {
+            continue;
+        }
+        let Some(import_value) = swift_import_value(trimmed) else {
+            continue;
+        };
+        if let Some(specifier) = swift_import_specifier(import_value) {
+            imports.push(raw(specifier, index, line, 0.88, "import:swift"));
+        }
+    }
+    imports
+}
+
+fn swift_import_value(line: &str) -> Option<&str> {
+    let mut remaining = line.trim();
+    while remaining.starts_with('@') {
+        let Some((_, after_attribute)) = remaining.split_once(' ') else {
+            return None;
+        };
+        remaining = after_attribute.trim_start();
+    }
+    remaining.strip_prefix("import ")
+}
+
+fn swift_import_specifier(value: &str) -> Option<String> {
+    let mut parts = value.split_whitespace();
+    let first = parts.next()?;
+    let module = if matches!(
+        first,
+        "class" | "struct" | "enum" | "protocol" | "func" | "var" | "typealias"
+    ) {
+        parts.next()?
+    } else {
+        first
+    };
+    module
+        .split('.')
+        .next()
+        .map(str::trim)
+        .filter(|value| !value.is_empty())
+        .map(ToString::to_string)
+}

package/crates/naome-core/src/architecture/scan/imports/extractors.rs

@@ -1,6 +1,8 @@
 use super::language_for_path;
 use crate::architecture::model::SourceRange;
 
+mod swift;
+
 #[derive(Debug, Clone)]
 pub(super) struct RawImport {
     pub specifier: String,
@@ -15,6 +17,7 @@ pub(super) fn extract_raw_imports(path: &str, content: &str) -> Vec<RawImport> {
         Some("rust") => extract_rust(content),
         Some("python") => extract_python(content),
         Some("go") => extract_go(content),
+        Some("swift") => swift::extract(content),
         _ => Vec::new(),
     }
 }

package/crates/naome-core/src/architecture/scan/imports/resolver.rs

@@ -35,6 +35,11 @@ pub(super) fn resolve_import(
             return ImportTarget::File(path);
         }
     }
+    if from_path.ends_with(".swift") {
+        if let Some(path) = resolve_swift_module_import(from_path, specifier, repository_files) {
+            return ImportTarget::File(path);
+        }
+    }
     external_target(specifier)
 }
 
@@ -225,6 +230,40 @@ fn go_mod_module(root: &Path, go_mod_path: &str) -> Option<String> {
     })
 }
 
+fn resolve_swift_module_import(
+    from_path: &str,
+    specifier: &str,
+    repository_files: &BTreeSet<String>,
+) -> Option<String> {
+    if specifier.contains('/') || specifier.contains('.') {
+        return None;
+    }
+    let sources_root = swift_sources_root(from_path)?;
+    let target_prefix = format!("{sources_root}{specifier}/");
+    let conventional_file = format!("{target_prefix}{specifier}.swift");
+    if repository_files.contains(&conventional_file) {
+        return Some(conventional_file);
+    }
+    repository_files
+        .iter()
+        .find(|path| path.starts_with(&target_prefix) && path.ends_with(".swift"))
+        .cloned()
+}
+
+fn swift_sources_root(from_path: &str) -> Option<String> {
+    if from_path.starts_with("Sources/") {
+        return Some("Sources/".to_string());
+    }
+    if from_path.starts_with("Tests/") {
+        return Some("Sources/".to_string());
+    }
+    if let Some((prefix, _)) = from_path.split_once("/Sources/") {
+        return Some(format!("{prefix}/Sources/"));
+    }
+    let (prefix, _) = from_path.split_once("/Tests/")?;
+    Some(format!("{prefix}/Sources/"))
+}
+
 fn resolve_candidates(
     candidate: &str,
     repository_files: &BTreeSet<String>,
@@ -286,10 +325,11 @@ fn extensions_for_language(language: Option<&str>) -> &'static [&'static str] {
         Some("rust") => &["", ".rs"],
         Some("python") => &["", ".py"],
         Some("go") => &["", ".go"],
+        Some("swift") => &["", ".swift"],
         Some("typescript") => &["", ".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs"],
         Some("javascript") => &["", ".js", ".jsx", ".mjs", ".cjs", ".ts", ".tsx"],
         _ => &[
-            "", ".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs", ".rs", ".py", ".go",
+            "", ".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs", ".rs", ".py", ".go", ".swift",
         ],
     }
 }

package/crates/naome-core/src/architecture/scan/imports.rs

@@ -70,6 +70,7 @@ pub(super) fn language_for_path(path: &str) -> Option<&'static str> {
         "rs" => Some("rust"),
         "py" => Some("python"),
         "go" => Some("go"),
+        "swift" => Some("swift"),
         _ => None,
     }
 }

package/crates/naome-core/src/architecture/scan/manifest/common.rs

@@ -0,0 +1,102 @@
+use std::path::Path;
+
+use crate::architecture::scan::{ManifestDependency, ManifestFact};
+
+pub(super) enum ManifestKind {
+    PackageJson,
+    CargoToml,
+    PyprojectToml,
+    GoMod,
+    PomXml,
+    BuildGradle,
+    PackageSwift,
+    XcodeProject,
+}
+
+pub(super) fn manifest_kind(path: &str) -> Option<ManifestKind> {
+    match path.rsplit('/').next()? {
+        "package.json" => Some(ManifestKind::PackageJson),
+        "Cargo.toml" => Some(ManifestKind::CargoToml),
+        "pyproject.toml" => Some(ManifestKind::PyprojectToml),
+        "go.mod" => Some(ManifestKind::GoMod),
+        "pom.xml" => Some(ManifestKind::PomXml),
+        "build.gradle" | "build.gradle.kts" => Some(ManifestKind::BuildGradle),
+        "Package.swift" => Some(ManifestKind::PackageSwift),
+        "project.pbxproj" => {
+            if path.contains(".xcodeproj/") {
+                Some(ManifestKind::XcodeProject)
+            } else {
+                None
+            }
+        }
+        _ => None,
+    }
+}
+
+pub(super) fn fact(
+    path: &str,
+    ecosystem: &str,
+    package_name: String,
+    mut dependencies: Vec<ManifestDependency>,
+    confidence: f32,
+) -> ManifestFact {
+    dependencies.sort_by(|left, right| {
+        left.name
+            .cmp(&right.name)
+            .then(left.dependency_kind.cmp(&right.dependency_kind))
+    });
+    dependencies.dedup_by(|left, right| {
+        left.name == right.name && left.dependency_kind == right.dependency_kind
+    });
+    ManifestFact {
+        path: path.to_string(),
+        ecosystem: ecosystem.to_string(),
+        package_name,
+        dependencies,
+        confidence,
+        extractor: format!("manifest:{}", manifest_kind_label(path)),
+    }
+}
+
+pub(super) fn dependency(
+    name: impl Into<String>,
+    dependency_kind: impl Into<String>,
+    version: Option<String>,
+    confidence: f32,
+) -> ManifestDependency {
+    ManifestDependency {
+        name: name.into(),
+        dependency_kind: dependency_kind.into(),
+        version,
+        confidence,
+    }
+}
+
+pub(super) fn fallback_package_name(path: &str, ecosystem: &str) -> String {
+    let parent = Path::new(path)
+        .parent()
+        .map(|path| path.to_string_lossy().replace('\\', "/"))
+        .filter(|path| !path.is_empty())
+        .unwrap_or_else(|| path.to_string());
+    format!("{ecosystem}:{parent}")
+}
+
+pub(super) fn quoted_value(value: &str) -> Option<String> {
+    let trimmed = value.trim();
+    trimmed
+        .strip_prefix('"')
+        .and_then(|value| value.split_once('"').map(|(value, _)| value.to_string()))
+        .or_else(|| {
+            trimmed
+                .strip_prefix('\'')
+                .and_then(|value| value.split_once('\'').map(|(value, _)| value.to_string()))
+        })
+}
+
+pub(super) fn clean_toml_line(line: &str) -> &str {
+    line.split('#').next().unwrap_or_default().trim()
+}
+
+fn manifest_kind_label(path: &str) -> &str {
+    path.rsplit('/').next().unwrap_or(path)
+}

package/crates/naome-core/src/architecture/scan/manifest/parsers/json.rs

@@ -0,0 +1,46 @@
+use serde_json::Value;
+
+use crate::architecture::scan::{ManifestDependency, ManifestFact};
+
+use super::super::common::{dependency, fact, fallback_package_name};
+
+pub(in crate::architecture::scan::manifest) fn package_json(
+    path: &str,
+    content: &str,
+) -> Option<ManifestFact> {
+    let value = serde_json::from_str::<Value>(content).ok()?;
+    let package_name = value
+        .get("name")
+        .and_then(Value::as_str)
+        .map(str::to_string)
+        .unwrap_or_else(|| fallback_package_name(path, "npm"));
+    let mut dependencies = Vec::new();
+    push_dependency_object(&mut dependencies, &value, "dependencies", "runtime");
+    push_dependency_object(&mut dependencies, &value, "devDependencies", "development");
+    push_dependency_object(&mut dependencies, &value, "peerDependencies", "peer");
+    push_dependency_object(
+        &mut dependencies,
+        &value,
+        "optionalDependencies",
+        "optional",
+    );
+    Some(fact(path, "npm", package_name, dependencies, 0.98))
+}
+
+fn push_dependency_object(
+    dependencies: &mut Vec<ManifestDependency>,
+    value: &Value,
+    field: &str,
+    kind: &str,
+) {
+    if let Some(object) = value.get(field).and_then(Value::as_object) {
+        dependencies.extend(object.iter().map(|(name, version)| {
+            dependency(
+                name.clone(),
+                kind,
+                version.as_str().map(str::to_string),
+                0.98,
+            )
+        }));
+    }
+}