docxdriver 0.4.2__tar.gz → 0.4.3__tar.gz

{docxdriver-0.4.2 → docxdriver-0.4.3}/Cargo.lock

@@ -416,7 +416,7 @@ dependencies = [
 
 [[package]]
 name = "docxdriver-cli"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "clap",
  "docxdriver-core",
@@ -427,7 +427,7 @@ dependencies = [
 
 [[package]]
 name = "docxdriver-core"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "hex",
  "latex2mathml",
@@ -445,7 +445,7 @@ dependencies = [
 
 [[package]]
 name = "docxdriver-mcp"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "axum",
  "cap-std",
@@ -463,14 +463,14 @@ dependencies = [
 
 [[package]]
 name = "docxdriver-monty-repl"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "docxdriver-core",
 ]
 
 [[package]]
 name = "docxdriver-python"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "docxdriver-core",
  "pyo3",
@@ -480,7 +480,7 @@ dependencies = [
 
 [[package]]
 name = "docxdriver-wasm"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "docxdriver-core",
  "js-sys",

{docxdriver-0.4.2 → docxdriver-0.4.3}/Cargo.toml

@@ -3,7 +3,7 @@ resolver = "2"
 members = ["crates/docxdriver-python"]
 
 [workspace.package]
-version = "0.4.2"
+version = "0.4.3"
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/jhnoel/docxdriver"

{docxdriver-0.4.2 → docxdriver-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: docxdriver
-Version: 0.4.2
+Version: 0.4.3
 Summary: Python SDK for the native Rust DOCX engine
 License-Expression: MIT
 Requires-Python: >=3.10
@@ -29,6 +29,7 @@ from docxdriver.plan import Plan, ReplaceText
 doc = Document(Path("contract.docx").read_bytes())
 doc.markup()                      # body with the ids plans address
 doc.comments(); doc.revisions(); doc.styles(); doc.assets()
+doc.bookmarks()                   # same as doc.read(kind="bookmarks")
 doc.find("thirty")
 review = doc.review(Plan(author="Reviewer", ops=[
     ReplaceText(at="0F537164", select="thirty", with_="sixty"),

{docxdriver-0.4.2/crates/docxdriver-python → docxdriver-0.4.3}/README.md

@@ -20,6 +20,7 @@ from docxdriver.plan import Plan, ReplaceText
 doc = Document(Path("contract.docx").read_bytes())
 doc.markup()                      # body with the ids plans address
 doc.comments(); doc.revisions(); doc.styles(); doc.assets()
+doc.bookmarks()                   # same as doc.read(kind="bookmarks")
 doc.find("thirty")
 review = doc.review(Plan(author="Reviewer", ops=[
     ReplaceText(at="0F537164", select="thirty", with_="sixty"),

{docxdriver-0.4.2 → docxdriver-0.4.3}/crates/docxdriver-core/src/api.rs

@@ -748,6 +748,35 @@ pub enum EditOp {
     CommentDelete {
         comment_id: String,
     },
+    /// Bookmark the paragraph at `at`: its whole content, or the `select`
+    /// span. Applied directly in both change modes (Word does not track
+    /// bookmarks). Body paragraphs only.
+    BookmarkAdd {
+        at: ParagraphAddress,
+        name: String,
+        #[serde(default)]
+        select: Option<String>,
+        #[serde(default)]
+        occurrence: Option<u32>,
+    },
+    /// Rename a body bookmark; `new_name` follows the add rules.
+    BookmarkRename {
+        name: String,
+        new_name: String,
+    },
+    /// Re-anchor a body bookmark, keeping its name and `w:id`.
+    BookmarkMove {
+        name: String,
+        at: ParagraphAddress,
+        #[serde(default)]
+        select: Option<String>,
+        #[serde(default)]
+        occurrence: Option<u32>,
+    },
+    /// Remove a body bookmark's start and end markers; the text stays.
+    BookmarkDelete {
+        name: String,
+    },
     RevisionSettle {
         target: RevisionTarget,
         action: RevisionAction,
@@ -1260,7 +1289,11 @@ fn validate_plan(mut p: Plan) -> Result<Plan, Diagnostic> {
 }
 
 fn validation_field(message: &str) -> Option<&'static str> {
-    let field = if message.contains("MathML") {
+    let field = if message.starts_with("new_name ") {
+        "new_name"
+    } else if message.starts_with("name ") {
+        "name"
+    } else if message.contains("MathML") {
         "mathml"
     } else if message.contains("equation must") {
         "equation"
@@ -1566,6 +1599,31 @@ fn normalize_op(op: &mut EditOp) -> Result<(), String> {
                 return Err("set_page_margins requires a margin".into());
             }
         }
+        EditOp::BookmarkAdd {
+            name,
+            select,
+            occurrence,
+            ..
+        } => {
+            crate::commands::bookmarks::validate_new_name(name, "name")?;
+            validate_bookmark_select(select, occurrence)?;
+        }
+        EditOp::BookmarkMove {
+            name,
+            select,
+            occurrence,
+            ..
+        } => {
+            crate::commands::bookmarks::validate_existing_name(name)?;
+            validate_bookmark_select(select, occurrence)?;
+        }
+        EditOp::BookmarkRename { name, new_name } => {
+            crate::commands::bookmarks::validate_existing_name(name)?;
+            crate::commands::bookmarks::validate_new_name(new_name, "new_name")?;
+        }
+        EditOp::BookmarkDelete { name } => {
+            crate::commands::bookmarks::validate_existing_name(name)?;
+        }
         EditOp::CommentAdd {
             text,
             occurrence,
@@ -1620,6 +1678,18 @@ fn normalize_op(op: &mut EditOp) -> Result<(), String> {
     }
     Ok(())
 }
+fn validate_bookmark_select(select: &Option<String>, occurrence: &Option<u32>) -> Result<(), String> {
+    if select.as_deref() == Some("") {
+        return Err("select must be non-empty; omit it to bookmark the whole paragraph".into());
+    }
+    if occurrence == &Some(0) {
+        return Err("occurrence must be positive".into());
+    }
+    if occurrence.is_some() && select.is_none() {
+        return Err("occurrence requires select".into());
+    }
+    Ok(())
+}
 fn diag(code: impl Into<String>, message: impl Into<String>, path: Option<String>) -> Diagnostic {
     Diagnostic {
         code: code.into(),
@@ -1992,6 +2062,16 @@ fn explain_selector_drift(
             at,
             select: Some(select),
             ..
+        }
+        | EditOp::BookmarkAdd {
+            at,
+            select: Some(select),
+            ..
+        }
+        | EditOp::BookmarkMove {
+            at,
+            select: Some(select),
+            ..
         } => (at, vec![select.as_str()]),
         EditOp::ReviseText {
             at: Some(at),
@@ -2098,6 +2178,10 @@ fn operation_name(op: &EditOp) -> &'static str {
         EditOp::CommentReply { .. } => "comment_reply",
         EditOp::CommentSetStatus { .. } => "comment_set_status",
         EditOp::CommentDelete { .. } => "comment_delete",
+        EditOp::BookmarkAdd { .. } => "bookmark_add",
+        EditOp::BookmarkRename { .. } => "bookmark_rename",
+        EditOp::BookmarkMove { .. } => "bookmark_move",
+        EditOp::BookmarkDelete { .. } => "bookmark_delete",
         EditOp::RevisionSettle { .. } => "revision_settle",
     }
 }
@@ -2128,7 +2212,9 @@ fn resolve_typed_aliases(
         | EditOp::InsertTable { at, .. }
         | EditOp::SetSectionBoundary { at, .. }
         | EditOp::FormatParagraph { at, .. }
-        | EditOp::CommentAdd { at, .. } => resolve(at)?,
+        | EditOp::CommentAdd { at, .. }
+        | EditOp::BookmarkAdd { at, .. }
+        | EditOp::BookmarkMove { at, .. } => resolve(at)?,
         EditOp::ReviseText { at: Some(at), .. } => resolve(at)?,
         EditOp::InsertParagraph { at, .. } => {
             if let InsertAnchor::Paragraph(addr) = at {
@@ -2478,6 +2564,11 @@ fn typed_read(input: Option<&[u8]>, kind: &str, view: Option<&str>) -> crate::ou
             "revisions",
             source,
         ),
+        "bookmarks" => typed_listing(
+            crate::commands::bookmarks::list_bookmarks(&state.package, &state.xml, source),
+            "bookmarks",
+            source,
+        ),
         other => crate::outcome::Outcome::error(format!("unknown read kind: {other}")),
     }
 }