rgapi 0.1.4__tar.gz → 0.1.6__tar.gz

{rgapi-0.1.4 → rgapi-0.1.6}/.github/workflows/ci.yml

@@ -7,45 +7,20 @@ on:
   pull_request:
 
 jobs:
-  test:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v6
-      - run: cargo test
-
-  linux:
-    needs: test
-    runs-on: ubuntu-latest
+  build:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+    runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v6
       - uses: PyO3/maturin-action@v1
         with:
           args: --release --out dist -i python3.10 -i python3.11 -i python3.12 -i python3.13
           manylinux: auto
-          before-script-linux: |
-            python3.13 -m pip install 'fastship>=0.0.13'
-            python3.13 -m fastship.rs_prep --release
-      - uses: actions/upload-artifact@v4
-        with:
-          name: wheels-linux
-          path: dist
-
-  macos:
-    needs: test
-    runs-on: macos-latest
-    steps:
-      - uses: actions/checkout@v6
-      - uses: actions/setup-python@v6
-        with:
-          python-version: '3.13'
-      - run: python -m pip install 'fastship>=0.0.13'
-      - run: python -m fastship.rs_prep --release
-      - uses: PyO3/maturin-action@v1
-        with:
-          args: --release --out dist -i python3.10 -i python3.11 -i python3.12 -i python3.13
       - uses: actions/upload-artifact@v4
         with:
-          name: wheels-macos
+          name: wheels-${{ matrix.os }}
           path: dist
 
   sdist:
@@ -63,7 +38,7 @@ jobs:
 
   publish:
     if: startsWith(github.ref, 'refs/tags/v')
-    needs: [linux, macos, sdist]
+    needs: [build, sdist]
     runs-on: ubuntu-latest
     permissions:
       id-token: write

{rgapi-0.1.4 → rgapi-0.1.6}/Cargo.lock

@@ -276,7 +276,7 @@ checksum = "d6f6ff9a378485b298a5286656da665ba74413d36db0979633275d2e708145d4"
 
 [[package]]
 name = "rgapi"
-version = "0.1.4"
+version = "0.1.6"
 dependencies = [
  "globset",
  "grep-matcher",

{rgapi-0.1.4 → rgapi-0.1.6}/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rgapi"
-version = "0.1.4"
+version = "0.1.6"
 edition = "2021"
 license = "MIT OR Apache-2.0"
 description = "Python API for ripgrep-style file walking and searching"
@@ -19,8 +19,7 @@ grep-matcher = ">=0.1.8"
 grep-regex = ">=0.1.14"
 grep-searcher = ">=0.1.16"
 ignore = ">=0.4.26"
-pyo3 = { version = ">=0.28", optional = true }
+pyo3 = ">=0.28"
 
 [features]
-extension-module = ["pyo3", "pyo3/extension-module"]
-pyo3 = ["dep:pyo3"]
+extension-module = ["pyo3/extension-module"]

{rgapi-0.1.4 → rgapi-0.1.6}/DEV.md

@@ -20,7 +20,7 @@ The public Python API lives in `python/rgapi/__init__.py`. The extension module
 ship-rs-test
 ```
 
-Run `cargo fmt --check` and `cargo check` for Rust-only edits. Run `chkstyle` after Python edits once tests pass.
+All tests are Python (`tests/`), run against the built extension; there are no `cargo test` unit tests. For a fast local loop use `maturin develop && pytest -q`. Run `cargo fmt --check` and `cargo check` for Rust-only edits. Run `chkstyle` after Python edits once tests pass.
 
 ## Release
 

{rgapi-0.1.4 → rgapi-0.1.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rgapi
-Version: 0.1.4
+Version: 0.1.6
 Classifier: Programming Language :: Rust
 Classifier: Programming Language :: Python :: Implementation :: CPython
 Requires-Dist: fastship>=0.0.12 ; extra == 'dev'

{rgapi-0.1.4 → rgapi-0.1.6}/pyproject.toml

@@ -23,6 +23,9 @@ Homepage = "https://github.com/AnswerDotAI/rgapi"
 Repository = "https://github.com/AnswerDotAI/rgapi"
 Issues = "https://github.com/AnswerDotAI/rgapi/issues"
 
+[project.entry-points.fastaudit_safe_native]
+rgapi = "rgapi"
+
 [tool.maturin]
 features = ["extension-module"]
 python-source = "python"

{rgapi-0.1.4 → rgapi-0.1.6}/src/lib.rs

@@ -3,7 +3,6 @@
 mod search;
 mod walk;
 
-#[cfg(feature = "pyo3")]
 mod python;
 
 pub use search::{

{rgapi-0.1.4 → rgapi-0.1.6}/src/search.rs

@@ -490,53 +490,3 @@ pub(crate) fn spans_for(
         .map_err(|e| SearchError::Message(e.to_string()))?;
     Ok(spans)
 }
-#[cfg(test)]
-mod tests {
-    use super::*;
-    #[test]
-    fn context_rows_keep_kind_enum_and_public_strings() {
-        let matcher = compile_regex("TODO", None, true).unwrap();
-        let lines = search_text(
-            "memory.txt".to_string(),
-            "alpha\nTODO here\nomega\n",
-            matcher,
-            1,
-            1,
-        )
-        .unwrap();
-        let got: Vec<_> = lines
-            .iter()
-            .map(|line| (line.kind.as_str(), line.line_number, line.line.as_str()))
-            .collect();
-        assert_eq!(
-            got,
-            vec![
-                ("before", 1, "alpha"),
-                ("match", 2, "TODO here"),
-                ("after", 3, "omega")
-            ]
-        );
-        assert_eq!(lines[1].matches, vec![MatchSpan { start: 0, end: 4 }]);
-    }
-    fn temp_path(name: &str) -> std::path::PathBuf {
-        std::env::temp_dir().join(format!("rgapi-{}-{name}", std::process::id()))
-    }
-    #[test]
-    fn search_path_skips_binary_files() {
-        let path = temp_path("binary.dat");
-        std::fs::write(&path, b"TODO before\n\0TODO after\n").unwrap();
-        let matcher = compile_regex("TODO", None, true).unwrap();
-        let lines = search_path(&path, "binary.dat".to_string(), matcher, 0, 0).unwrap();
-        let _ = std::fs::remove_file(&path);
-        assert!(lines.is_empty());
-    }
-    #[test]
-    fn search_path_skips_invalid_utf8_files() {
-        let path = temp_path("invalid.txt");
-        std::fs::write(&path, b"TODO\xff\n").unwrap();
-        let matcher = compile_regex("TODO", None, true).unwrap();
-        let lines = search_path(&path, "invalid.txt".to_string(), matcher, 0, 0).unwrap();
-        let _ = std::fs::remove_file(&path);
-        assert!(lines.is_empty());
-    }
-}

{rgapi-0.1.4 → rgapi-0.1.6}/src/walk.rs

@@ -297,43 +297,3 @@ fn build_path_re(pattern: Option<&str>) -> Result<Option<RegexMatcher>, RgApiErr
 fn re_match(matcher: &RegexMatcher, rel: &str) -> bool {
     matcher.is_match(rel.as_bytes()).unwrap_or(false)
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::path::Path;
-
-    #[test]
-    fn basename_globs_match_nested_paths() {
-        let glob = build_globs(&["*.py".to_string()]).unwrap().unwrap();
-        assert!(glob.is_match(Path::new("app.py")));
-        assert!(glob.is_match(Path::new("src/app.py")));
-        assert!(!glob.is_match(Path::new("src/app.rs")));
-    }
-
-    #[test]
-    fn path_filters_include_exclude_and_regex_paths() {
-        let empty = Vec::new();
-        let filters = PathFilters::new(
-            &["*.py".to_string()],
-            &["skip.py".to_string()],
-            Some(r"src/"),
-            Some(r"test_"),
-            &empty,
-            None,
-        )
-        .unwrap();
-        assert!(filters.path_allowed("src/app.py"));
-        assert!(!filters.path_allowed("src/skip.py"));
-        assert!(!filters.path_allowed("src/app.rs"));
-        assert!(!filters.path_allowed("app.py"));
-        assert!(!filters.path_allowed("src/test_app.py"));
-    }
-
-    #[test]
-    fn relative_paths_use_slashes() {
-        let root = Path::new("project");
-        let path = root.join("src").join("app.py");
-        assert_eq!(rel_path(root, &path), "src/app.py");
-    }
-}

{rgapi-0.1.4 → rgapi-0.1.6}/tests/test_rgapi.py

@@ -121,6 +121,14 @@ def test_rg_returns_structured_matches_context_and_relative_paths(tmp_path):
     assert str(stream) == repr(stream)
 
 
+def test_search_path_skips_binary_and_invalid_utf8(tmp_path):
+    (tmp_path / "bin.dat").write_bytes(b"TODO before\n\0TODO after\n")
+    (tmp_path / "bad.txt").write_bytes(b"TODO\xff\n")
+    matcher = compile("TODO")
+    assert search_path(matcher, tmp_path / "bin.dat") == []
+    assert search_path(matcher, tmp_path / "bad.txt") == []
+
+
 def test_rg_keyboard_interrupt_cancels(tmp_path):
     (tmp_path / "big.txt").write_text("alpha beta gamma\n" * 1_000_000)
     timer = threading.Timer(0.001, _thread.interrupt_main)