Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@@ -0,0 +90,4 @@
+
+## Open Questions
+
+- [x] Where does the reason go: one comment with a closing section, a second comment, a table per finding, or a `<details>` block?

@@ -179,3 +179,3 @@
     parser.add_argument("pr", help="a PR number of the default repository, or the URL of a PR on the configured server")
     parser.add_argument("--attach", action="store_true",
-                        help="post the checked draft and its diagrams to the PR, without a new review")
+                        help="post the checked draft and its diagrams to the PR, without a new review; "

@@ -126,0 +175,4 @@
+            raise
+        # The new body links only the new SVGs.
+        _delete_assets(gitea, same["id"], old)
+        return same.get("html_url", "")

@@ -191,3 +191,3 @@
         self.writes.append("asset")
         comment = next(c for c in self.comments if c.get("id") == comment_id)
-        asset = {"name": name, "size": len(content), "content_type": content_type,
+        asset_id = sum(len(c.get("assets", ())) for c in self.comments) + 1

@@ -135,2 +170,3 @@
+        body, uploaded = _upload(gitea, same["id"], directory, names, body)
         try:
-            gitea.delete_issue_comment(comment["id"])
+            gitea.edit_issue_comment(same["id"], marker + "\n" + body)

@@ -141,0 +174,4 @@
+            _delete_assets(gitea, same["id"], uploaded)
+            raise
+        # The new body links only the new SVGs.
+        _delete_assets(gitea, same["id"], old)

@@ -92,2 +82,2 @@
-        header[1:1] = [""] + block
-    return "\n".join(header) + sep + rest
+    # Only the header holds the draft line; a finding may quote the same words.
+    header = [line for line in head.split("\n") if not line.startswith(DRAFT_LINE)]

@@ -134,1 +171,3 @@
-    except Exception:
+    same = next((c for c in reversed(reports) if REPORT_MARKER.search(c["body"]).group(1) == head), None)
+    if same:
+        old, old_body = gitea.list_comment_assets(same["id"]) or [], same["body"]

@@ -0,0 +59,4 @@
+
+- [test_reviewreport.AttachTest.test_a_second_post_for_the_same_head_replaces_the_report](../../tests/test_reviewreport.py)
+
+#### Scenario#139.03: a failed replacement leaves the old report

@@ -121,1 +164,4 @@
         raise AttachError(f"the draft links SVGs that are not next to it: {', '.join(missing)}")
+    login = (gitea.current_user() or {}).get("login")
+    # Only the own reports: a report of someone else is theirs to replace.
+    reports = [c for c in gitea.list_issue_comments(number) or ()

@@ -141,0 +183,4 @@
+            if current is not None:
+                _delete_assets(gitea, same["id"], _unlinked(uploaded, current))
+            raise
+        # Only the SVGs that the old report linked go; a file that someone added to the comment stays.

@@ -1639,3 +1639,3 @@
 `tools/review-report --attach` posts the checked draft instead, without a new review:
 a comment of the PR with the SVGs as attachments, under the account of the token.
-It refuses a draft that still has its line "> **Draft:** …", is for another head, or is posted already.
+It refuses a draft that still has its line "> **Draft:** …", or is for another head.

@@ -0,0 +106,4 @@
+- `werkrevisor/reviewreport.py`: `confirm` removes the draft line from the header
+  and appends the reason as plain text under "### Assessment of the findings",
+  after the last section of the report and before the usage line.
+- `werkrevisor/reviewreport.py`: `attach` looks for own comments with the marker `<!-- hs-review:<sha> -->`.

@@ -135,0 +179,4 @@
+        own = ASSETS_MARKER.search(same["body"])
+        own = {int(i) for i in own.group(1).split(",") if i} if own else set()
+        old = [a for a in gitea.list_comment_assets(same["id"]) or [] if a.get("id") in own]
+        body, uploaded = _upload(gitea, same["id"], directory, names, body)

@@ -141,0 +191,4 @@
+            if current is not None:
+                _delete_assets(gitea, same["id"], _unlinked(uploaded, current))
+            raise
+        _delete_assets(gitea, same["id"], old)