feat: Python client interface, tests, bump to v0.4.0 (#2)

Expose Config, UrlEntry, and TomlStorage to Python via PyO3 for
programmatic bookmark management. Add CLI integration tests, config
error-path tests, and Python tests (67 total across Rust + Python).

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: lostmygithubaccount

Cargo.lock

@@ -6,7 +6,10 @@ echo "Running ruff..."
 uv run --only-group dev ruff check .
 uv run --only-group dev ruff format --check .
 
+echo "Building Python bindings for type checking..."
+uv run maturin develop
+
 echo "Running ty..."
-uv run --only-group dev ty check
+uv run ty check
 
 echo "Python checks passed!"

bin/check-py

@@ -1,6 +1,6 @@
 [package]
 name = "dkdc-bookmarks-app"
-version = "0.3.3"
+version = "0.4.0"
 edition = "2024"
 authors = ["Cody <cody@dkdc.dev>"]
 description = "Desktop app for bookmarks (iced)"
@@ -13,7 +13,7 @@ name = "bookmarks_app"
 path = "src/lib.rs"
 
 [dependencies]
-dkdc-bookmarks-core = { version = "0.3.3", path = "../bookmarks-core" }
+dkdc-bookmarks-core = { version = "0.4.0", path = "../bookmarks-core" }
 iced = { version = "0.14", features = ["tokio", "svg"] }
 open = "5"
 png = "0.17"

crates/bookmarks-app/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "dkdc-bookmarks"
-version = "0.3.3"
+version = "0.4.0"
 edition = "2024"
 authors = ["Cody <cody@dkdc.dev>"]
 description = "Bookmarks in your filesystem"
@@ -23,7 +23,10 @@ webapp = ["dep:dkdc-bookmarks-webapp"]
 
 [dependencies]
 anyhow = "1"
-dkdc-bookmarks-core = { version = "0.3.3", path = "../bookmarks-core" }
-dkdc-bookmarks-app = { version = "0.3.3", path = "../bookmarks-app", optional = true }
-dkdc-bookmarks-webapp = { version = "0.3.3", path = "../bookmarks-webapp", optional = true }
+dkdc-bookmarks-core = { version = "0.4.0", path = "../bookmarks-core" }
+dkdc-bookmarks-app = { version = "0.4.0", path = "../bookmarks-app", optional = true }
+dkdc-bookmarks-webapp = { version = "0.4.0", path = "../bookmarks-webapp", optional = true }
 clap = { version = "4.5", features = ["derive"] }
+
+[dev-dependencies]
+tempfile = "3"

crates/bookmarks-cli/Cargo.toml

@@ -0,0 +1,59 @@
+use std::fs;
+use std::io::Write;
+
+use tempfile::tempdir;
+
+const TEST_CONFIG: &str = r#"[urls]
+github = { url = "https://github.com", aliases = ["gh"] }
+dkdc-bookmarks = "https://github.com/lostmygithubaccount/bookmarks"
+
+[groups]
+dev = ["gh", "dkdc-bookmarks"]
+"#;
+
+fn write_config(dir: &std::path::Path) -> std::path::PathBuf {
+    let path = dir.join("bookmarks.toml");
+    let mut f = fs::File::create(&path).unwrap();
+    f.write_all(TEST_CONFIG.as_bytes()).unwrap();
+    path
+}
+
+#[test]
+fn test_print_config() {
+    let dir = tempdir().unwrap();
+    let path = write_config(dir.path());
+    let result = bookmarks::run_cli(["bookmarks", "-f", path.to_str().unwrap()]);
+    assert!(result.is_ok());
+}
+
+#[test]
+fn test_file_not_found() {
+    let result = bookmarks::run_cli(["bookmarks", "-f", "/nonexistent/bookmarks.toml"]);
+    assert!(result.is_err());
+    let err = result.unwrap_err().to_string();
+    assert!(err.contains("not found"), "unexpected error: {err}");
+}
+
+#[test]
+fn test_unknown_bookmark() {
+    let dir = tempdir().unwrap();
+    let path = write_config(dir.path());
+    let result = bookmarks::run_cli(["bookmarks", "-f", path.to_str().unwrap(), "nonexistent"]);
+    assert!(result.is_err());
+}
+
+#[test]
+fn test_local_creates_config() {
+    let dir = tempdir().unwrap();
+    let config_path = dir.path().join("bookmarks.toml");
+    assert!(!config_path.exists());
+
+    // Set cwd to temp dir so --local creates bookmarks.toml there
+    let original_dir = std::env::current_dir().unwrap();
+    std::env::set_current_dir(dir.path()).unwrap();
+    let result = bookmarks::run_cli(["bookmarks", "--local"]);
+    std::env::set_current_dir(original_dir).unwrap();
+
+    assert!(result.is_ok());
+    assert!(config_path.exists());
+}

crates/bookmarks-cli/tests/cli_integration.rs

@@ -1,6 +1,6 @@
 [package]
 name = "dkdc-bookmarks-core"
-version = "0.3.3"
+version = "0.4.0"
 edition = "2024"
 authors = ["Cody <cody@dkdc.dev>"]
 description = "Core library for bookmarks"

crates/bookmarks-core/Cargo.toml

@@ -77,7 +77,7 @@ impl UrlEntry {
     }
 }
 
-#[derive(Debug, Serialize, Deserialize, Default)]
+#[derive(Debug, Clone, Serialize, Deserialize, Default)]
 pub struct Config {
     #[serde(default)]
     pub urls: HashMap<String, UrlEntry>,
@@ -752,4 +752,38 @@ dev = ["gh", "dkdc-bookmarks"]
         assert!(config.delete_alias("nope").is_err());
         assert!(config.delete_group("nope").is_err());
     }
+
+    #[test]
+    fn test_parse_malformed_toml() {
+        assert!(toml::from_str::<Config>("this is not valid { toml").is_err());
+    }
+
+    #[test]
+    fn test_parse_url_wrong_type() {
+        let toml = "[urls]\ngithub = 42";
+        assert!(toml::from_str::<Config>(toml).is_err());
+    }
+
+    #[test]
+    fn test_parse_missing_url_in_full_entry() {
+        let toml = "[urls.gh]\naliases = [\"x\"]";
+        assert!(toml::from_str::<Config>(toml).is_err());
+    }
+
+    #[test]
+    fn test_parse_groups_only_no_urls() {
+        let toml = "[groups]\ndev = [\"gh\"]";
+        let config: Config = toml::from_str(toml).unwrap();
+        assert!(config.urls.is_empty());
+        let warnings = config.validate();
+        assert!(warnings.iter().any(|w| w.contains("gh")));
+    }
+
+    #[test]
+    fn test_parse_extra_sections_ignored() {
+        let toml = "[urls]\ngithub = \"https://github.com\"\n\n[metadata]\nauthor = \"test\"";
+        // Config doesn't use deny_unknown_fields, so extra sections are ignored
+        let result = toml::from_str::<Config>(toml);
+        assert!(result.is_ok());
+    }
 }

crates/bookmarks-core/src/config.rs

@@ -163,4 +163,30 @@ dev = ["gh"]
         let storage = TomlStorage::new(PathBuf::from("/tmp/test.toml"));
         assert_eq!(storage.backend_name(), "toml");
     }
+
+    #[test]
+    fn test_load_nonexistent_file() {
+        let storage = TomlStorage::new(PathBuf::from("/nonexistent/path/bookmarks.toml"));
+        assert!(storage.load().is_err());
+    }
+
+    #[test]
+    fn test_load_malformed_file() {
+        let dir = tempfile::tempdir().unwrap();
+        let path = dir.path().join("bookmarks.toml");
+        fs::write(&path, "this is not valid { toml").unwrap();
+        let storage = TomlStorage::new(path);
+        assert!(storage.load().is_err());
+    }
+
+    #[test]
+    fn test_load_empty_file() {
+        let dir = tempfile::tempdir().unwrap();
+        let path = dir.path().join("bookmarks.toml");
+        fs::write(&path, "").unwrap();
+        let storage = TomlStorage::new(path);
+        let config = storage.load().unwrap();
+        assert!(config.urls.is_empty());
+        assert!(config.groups.is_empty());
+    }
 }