Merge pull request #67 from rryam/feature/extract-mlx-to-separate-repo

Extract VecturaMLXKit into separate repository

Author: rudrankriyam

.vscode/launch.json

@@ -20,26 +20,6 @@
             "configuration": "release",
             "preLaunchTask": "swift: Build Release vectura-cli"
         },
-        {
-            "type": "swift",
-            "request": "launch",
-            "args": [],
-            "cwd": "${workspaceFolder:VecturaKit}",
-            "name": "Debug vectura-mlx-cli",
-            "target": "vectura-mlx-cli",
-            "configuration": "debug",
-            "preLaunchTask": "swift: Build Debug vectura-mlx-cli"
-        },
-        {
-            "type": "swift",
-            "request": "launch",
-            "args": [],
-            "cwd": "${workspaceFolder:VecturaKit}",
-            "name": "Release vectura-mlx-cli",
-            "target": "vectura-mlx-cli",
-            "configuration": "release",
-            "preLaunchTask": "swift: Build Release vectura-mlx-cli"
-        },
         {
             "type": "swift",
             "request": "launch",
@@ -60,26 +40,6 @@
             "configuration": "release",
             "preLaunchTask": "swift: Build Release TestNLExamples"
         },
-        {
-            "type": "swift",
-            "request": "launch",
-            "args": [],
-            "cwd": "${workspaceFolder:VecturaKit}",
-            "name": "Debug TestMLXExamples",
-            "target": "TestMLXExamples",
-            "configuration": "debug",
-            "preLaunchTask": "swift: Build Debug TestMLXExamples"
-        },
-        {
-            "type": "swift",
-            "request": "launch",
-            "args": [],
-            "cwd": "${workspaceFolder:VecturaKit}",
-            "name": "Release TestMLXExamples",
-            "target": "TestMLXExamples",
-            "configuration": "release",
-            "preLaunchTask": "swift: Build Release TestMLXExamples"
-        },
         {
             "type": "swift",
             "request": "launch",
@@ -101,4 +61,4 @@
             "preLaunchTask": "swift: Build Release TestExamples"
         }
     ]
-}
+}

AGENTS.md

@@ -1,19 +1,19 @@
 # Repository Guidelines
 
 ## Project Structure & Module Organization
-VecturaKit ships as a Swift package with libraries `VecturaKit` (core storage + hybrid search) and `VecturaMLXKit` (MLX embeddings), plus CLI targets `VecturaCLI` and `VecturaMLXCLI`. CLI entry points sit in `Sources/VecturaCLI` and `Sources/VecturaMLXCLI`; sample executables live in `Sources/TestExamples` and `Sources/TestMLXExamples`. Tests mirror the split under `Tests/VecturaKitTests` and `Tests/VecturaMLXKitTests`; keep features scoped to the relevant module so MLX stays opt-in.
+VecturaKit ships as a Swift package with libraries `VecturaKit` (core storage + hybrid search) and `VecturaNLKit` (NaturalLanguage embeddings), plus CLI target `VecturaCLI`. The CLI entry point sits in `Sources/VecturaCLI`; sample executables live in `Sources/TestExamples` and `Sources/TestNLExamples`. Tests live under `Tests/VecturaKitTests` and `Tests/VecturaNLKitTests`. MLX support has been extracted to a separate repository at [rryam/VecturaMLXKit](https://github.com/rryam/VecturaMLXKit).
 
 ## Build, Test, and Development Commands
 - `swift build` compiles libraries and executables (add `-c release` for performance validation).
-- `swift run vectura-cli mock --db-name demo-db` seeds and exercises the default engine; swap `vectura-mlx-cli` to cover MLX.
+- `swift run vectura-cli mock --db-name demo-db` seeds and exercises the default engine.
 - `swift test` runs the Swift Testing suites; add `--filter SuiteName/TestName` to narrow scope.
 - `swift package update` refreshes dependency pins before release branches or large upgrades.
 
 ## Coding Style & Naming Conventions
 Follow Swift 6 defaults: two-space indentation, trailing commas in multi-line literals, and a 120-character soft wrap. Keep types UpperCamelCase, members lowerCamelCase, and CLI command enums verb-based (`case add`, `case search`). Use the existing async/await APIs, isolate file IO in helpers, and document public entry points with concise `///` comments.
 
 ## Testing Guidelines
-Declare new Swift Testing suites with `@Suite` and `@Test` annotations, keeping test functions `async throws` and cleaning up resources explicitly within each test. Prefer per-test temporary directories over `~/Documents/VecturaKit`, and short-circuit MLX flows when Metal or device libraries are unavailable. Do not introduce new `XCTestCase` subclasses—Swift Testing is the required framework going forward. When touching MLX flows, cover them in `VecturaMLXKitTests` and include at least one CLI invocation; update `codemagic.yaml` whenever the CI matrix or steps change.
+Declare new Swift Testing suites with `@Suite` and `@Test` annotations, keeping test functions `async throws` and cleaning up resources explicitly within each test. Prefer per-test temporary directories over `~/Documents/VecturaKit`. Do not introduce new `XCTestCase` subclasses—Swift Testing is the required framework going forward. Update `codemagic.yaml` whenever the CI matrix or steps change.
 
 ## Commit & Pull Request Guidelines
 Commits stay imperative and scoped (`Add cosine similarity guard`, `Adjust CLI mock seed`). Keep dependency bumps isolated with matching `Package.resolved` updates. Pull requests should outline intent, summarize functional impact, and describe verification steps—attach CLI output when behavior changes. Link issues with `Fixes #<id>` and confirm CI before requesting review.
@@ -22,7 +22,7 @@ Commits stay imperative and scoped (`Add cosine similarity guard`, `Adjust CLI m
 Before opening a PR (or completing an automated change):
 
 - [ ] `swift build` and `swift test` succeed locally without warnings.
-- [ ] Run `swift run vectura-cli mock --db-name qa-db` (and the `vectura-mlx-cli` variant when MLX code changes) to ensure smoke coverage.
+- [ ] Run `swift run vectura-cli mock --db-name qa-db` to ensure smoke coverage.
 - [ ] Persistent storage defaults and dimension negotiation stay intact—verify hybrid search thresholds and expected `VecturaError.dimensionMismatch` behavior.
 - [ ] Public API or CLI flag changes are reflected in `README.md`, `codemagic.yaml`, and this guide.
 - [ ] Remove temporary database artifacts from `~/Documents/VecturaKit` so tests stay deterministic.

Package.swift

@@ -1,4 +1,4 @@
-// swift-tools-version: 6.1
+// swift-tools-version: 6.0
 // The swift-tools-version declares the minimum version of Swift required to build this package.
 
 import PackageDescription
@@ -17,10 +17,6 @@ let package = Package(
       name: "VecturaKit",
       targets: ["VecturaKit"]
     ),
-    .library(
-      name: "VecturaMLXKit",
-      targets: ["VecturaMLXKit"]
-    ),
     .library(
       name: "VecturaNLKit",
       targets: ["VecturaNLKit"]
@@ -29,24 +25,10 @@ let package = Package(
       name: "vectura-cli",
       targets: ["VecturaCLI"]
     ),
-    .executable(
-      name: "vectura-mlx-cli",
-      targets: ["VecturaMLXCLI"]
-    ),
-  ],
-  traits: [
-    .trait(
-      name: "MLX",
-      description: "Enable MLX-based embeddings for GPU-accelerated inference"
-    ),
   ],
   dependencies: [
-    // Always included - lightweight dependencies
     .package(url: "https://github.com/jkrukowski/swift-embeddings.git", from: "0.0.21"),
     .package(url: "https://github.com/apple/swift-argument-parser.git", from: "1.4.0"),
-
-    // MLX dependency - only loaded when MLX trait is enabled via target conditions
-    .package(url: "https://github.com/ml-explore/mlx-swift-lm/", from: "2.30.3"),
   ],
   targets: [
     .target(
@@ -59,17 +41,6 @@ let package = Package(
         .define("ACCELERATE_LAPACK_ILP64"),
       ]
     ),
-    .target(
-      name: "VecturaMLXKit",
-      dependencies: [
-        "VecturaKit",
-        .product(
-          name: "MLXEmbedders",
-          package: "mlx-swift-lm",
-          condition: .when(traits: ["MLX"])
-        ),
-      ]
-    ),
     .target(
       name: "VecturaNLKit",
       dependencies: [
@@ -86,24 +57,10 @@ let package = Package(
         .copy("Resources/mock_documents.json")
       ]
     ),
-    .executableTarget(
-      name: "VecturaMLXCLI",
-      dependencies: [
-        "VecturaKit",
-        .target(name: "VecturaMLXKit", condition: .when(traits: ["MLX"])),
-        .product(name: "ArgumentParser", package: "swift-argument-parser"),
-      ]
-    ),
     .executableTarget(
       name: "TestExamples",
       dependencies: ["VecturaKit"]
     ),
-    .executableTarget(
-      name: "TestMLXExamples",
-      dependencies: [
-        .target(name: "VecturaMLXKit", condition: .when(traits: ["MLX"]))
-      ]
-    ),
     .executableTarget(
       name: "TestNLExamples",
       dependencies: ["VecturaNLKit"]
@@ -112,12 +69,6 @@ let package = Package(
       name: "VecturaKitTests",
       dependencies: ["VecturaKit"]
     ),
-    .testTarget(
-      name: "VecturaMLXKitTests",
-      dependencies: [
-        .target(name: "VecturaMLXKit", condition: .when(traits: ["MLX"]))
-      ]
-    ),
     .testTarget(
       name: "VecturaNLKitTests",
       dependencies: ["VecturaNLKit"]