python-openxml
diff --git a/‎features/cmt-mutations.feature‎
Lines changed: 0 additions & 2 deletions b/‎features/cmt-mutations.feature‎
Lines changed: 0 additions & 2 deletions
diff --git a/‎src/docx/comments.py‎
Lines changed: 12 additions & 1 deletion b/‎src/docx/comments.py‎
Lines changed: 12 additions & 1 deletion
diff --git a/‎src/docx/shared.py‎
Lines changed: 1 addition & 1 deletion b/‎src/docx/shared.py‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎tests/test_comments.py‎
Lines changed: 35 additions & 0 deletions b/‎tests/test_comments.py‎
Lines changed: 35 additions & 0 deletions
@@ -47,14 +47,12 @@ Feature: Comment mutations
      Then run.iter_inner_content() yields a single Picture drawing
 
 
-  @wip
   Scenario: update Comment.author
     Given a Comment object
      When I assign "Jane Smith" to comment.author
      Then comment.author == "Jane Smith"
 
 
-  @wip
   Scenario: update Comment.initials
     Given a Comment object
      When I assign "JS" to comment.initials
 
@@ -116,9 +116,16 @@ def add_paragraph(self, text: str = "", style: str | ParagraphStyle | None = Non
 
     @property
     def author(self) -> str:
-        """The recorded author of this comment."""
+        """Read/write. The recorded author of this comment.
+
+        This field is required but can be set to the empty string.
+        """
         return self._comment_elm.author
 
+    @author.setter
+    def author(self, value: str):
+        self._comment_elm.author = value
+
     @property
     def comment_id(self) -> int:
         """The unique identifier of this comment."""
@@ -133,6 +140,10 @@ def initials(self) -> str | None:
         """
         return self._comment_elm.initials
 
+    @initials.setter
+    def initials(self, value: str | None):
+        self._comment_elm.initials = value
+
     @property
     def timestamp(self) -> dt.datetime | None:
         """The date and time this comment was authored.
 
@@ -328,7 +328,7 @@ def __init__(self, parent: t.ProvidesXmlPart):
         self._parent = parent
 
     @property
-    def part(self):
+    def part(self) -> XmlPart:
         """The package part containing this object."""
         return self._parent.part
 
 
@@ -153,6 +153,14 @@ def and_it_can_add_text_to_the_comment_when_adding_it(self, comments: Comments,
         assert [p.text for p in comment.paragraphs] == ["para 1", "", "para 2"]
         assert all(p._p.style == "CommentText" for p in comment.paragraphs)
 
+    def and_it_sets_the_author_and_their_initials_when_adding_a_comment_when_provided(
+        self, comments: Comments, package_: Mock
+    ):
+        comment = comments.add_comment(author="Steve Canny", initials="SJC")
+
+        assert comment.author == "Steve Canny"
+        assert comment.initials == "SJC"
+
     # -- fixtures --------------------------------------------------------------------------------
 
     @pytest.fixture
@@ -213,6 +221,33 @@ def it_provides_access_to_the_paragraphs_it_contains(self, comments_part_: Mock)
         assert len(paragraphs) == 2
         assert [para.text for para in paragraphs] == ["First para", "Second para"]
 
+    def it_can_update_the_comment_author(self, comments_part_: Mock):
+        comment_elm = cast(CT_Comment, element("w:comment{w:id=42,w:author=Old Author}"))
+        comment = Comment(comment_elm, comments_part_)
+
+        comment.author = "New Author"
+
+        assert comment.author == "New Author"
+
+    @pytest.mark.parametrize(
+        "initials",
+        [
+            # -- valid initials --
+            "XYZ",
+            # -- empty string is valid
+            "",
+            # -- None is valid, removes existing initials
+            None,
+        ],
+    )
+    def it_can_update_the_comment_initials(self, initials: str | None, comments_part_: Mock):
+        comment_elm = cast(CT_Comment, element("w:comment{w:id=42,w:initials=ABC}"))
+        comment = Comment(comment_elm, comments_part_)
+
+        comment.initials = initials
+
+        assert comment.initials == initials
+
     # -- fixtures --------------------------------------------------------------------------------
 
     @pytest.fixture