overleaf-comments-export 0.22.0__tar.gz → 0.22.1__tar.gz

{overleaf_comments_export-0.22.0 → overleaf_comments_export-0.22.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: overleaf-comments-export
-Version: 0.22.0
+Version: 0.22.1
 Summary: Export comment threads and tracked changes from an Overleaf project to Markdown + JSON, optimized for AI-agent consumption.
 Author-email: Shivang Gupta <47645533+Mangluu@users.noreply.github.com>
 License: MIT

{overleaf_comments_export-0.22.0 → overleaf_comments_export-0.22.1}/overleaf_comments_export/anchors.py

@@ -35,9 +35,23 @@ def resolve_anchor(
     If text[offset:offset+len(anchored_text)] matches anchored_text, we trust it.
     Otherwise, search +/- search_window characters for the anchored text and
     re-anchor. If still not found, return the original offset's coords and
-    mark stale=True."""
+    mark stale=True.
+
+    A comment can also be attached to a position rather than to a range, and
+    Overleaf sends those with no anchored text at all. Nothing has moved and
+    there is nothing to check, so they are not stale. They used to fall
+    through to the stale branch because both searches need text to look for:
+    on one real paper that flagged 74 of 131 comments as pointing at text
+    that had changed when not one of them did, which buries the few that
+    genuinely have."""
     text = doc.text
     n = len(anchored_text)
+
+    if n == 0:
+        # No text to verify against, and no evidence the position is wrong.
+        safe = min(max(offset, 0), max(0, len(text) - 1))
+        line, col = offset_to_line_col(doc.line_starts, safe)
+        return safe, line, col, False
     if n > 0 and 0 <= offset <= len(text) - n and text[offset : offset + n] == anchored_text:
         line, col = offset_to_line_col(doc.line_starts, offset)
         return offset, line, col, False

{overleaf_comments_export-0.22.0 → overleaf_comments_export-0.22.1}/overleaf_comments_export.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: overleaf-comments-export
-Version: 0.22.0
+Version: 0.22.1
 Summary: Export comment threads and tracked changes from an Overleaf project to Markdown + JSON, optimized for AI-agent consumption.
 Author-email: Shivang Gupta <47645533+Mangluu@users.noreply.github.com>
 License: MIT

{overleaf_comments_export-0.22.0 → overleaf_comments_export-0.22.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "overleaf-comments-export"
-version = "0.22.0"
+version = "0.22.1"
 description = "Export comment threads and tracked changes from an Overleaf project to Markdown + JSON, optimized for AI-agent consumption."
 readme = "README.md"
 license = { text = "MIT" }

{overleaf_comments_export-0.22.0 → overleaf_comments_export-0.22.1}/tests/test_anchors_sections.py

@@ -104,3 +104,47 @@ def test_nearest_heading_before_first_returns_none():
     starts = build_line_starts(text)
     hs = find_headings(text, starts)
     assert nearest_heading(hs, 1) is None
+
+
+def test_a_comment_on_a_position_is_not_stale():
+    """Overleaf lets a comment attach to a point rather than a selection, and
+    sends those with no anchored text. Nothing has moved and there is nothing
+    to check against, so calling them stale is a false alarm. One real paper
+    had 74 of 131 comments flagged that way when none of them had moved, which
+    hides the ones that genuinely did."""
+    from overleaf_comments_export.anchors import build_line_starts, resolve_anchor
+    from overleaf_comments_export.model import DocText
+
+    text = "\\section{Method}\nWe crossed three sensory environments.\n"
+    doc = DocText(doc_id="d", pathname="p.tex", text=text,
+                  line_starts=build_line_starts(text), headings=[])
+
+    at = text.index("three")
+    offset, line, col, stale = resolve_anchor(doc, at, "")
+    assert stale is False, "an empty anchor was reported as stale"
+    assert offset == at, "the position was moved even though nothing was wrong"
+    assert line == 2
+
+
+def test_an_empty_anchor_beyond_the_file_is_still_bounded():
+    from overleaf_comments_export.anchors import build_line_starts, resolve_anchor
+    from overleaf_comments_export.model import DocText
+
+    text = "short\n"
+    doc = DocText(doc_id="d", pathname="p.tex", text=text,
+                  line_starts=build_line_starts(text), headings=[])
+    offset, _, _, stale = resolve_anchor(doc, 9999, "")
+    assert 0 <= offset < len(text)
+    assert stale is False
+
+
+def test_text_that_really_moved_is_still_stale():
+    """The fix must not stop reporting the ones that matter."""
+    from overleaf_comments_export.anchors import build_line_starts, resolve_anchor
+    from overleaf_comments_export.model import DocText
+
+    text = "alpha beta gamma\n"
+    doc = DocText(doc_id="d", pathname="p.tex", text=text,
+                  line_starts=build_line_starts(text), headings=[])
+    _, _, _, stale = resolve_anchor(doc, 0, "text that is gone")
+    assert stale is True