fix: Prepend leading slash to docname in path2doc

Co-authored-by: MiMoCode <mimo@xiaomi.com>

Author: SilverRainZ

src/sphinxnotes/recentupdate/__init__.py

@@ -198,7 +198,7 @@ def get_git_revisions(
 def path2doc(repo: Repo, env: BuildEnvironment, blob_path: str) -> str | None:
     """Convert a git repo-relative blob path to a Sphinx document name."""
     docname = env.path2doc(path.join(repo.working_dir, blob_path))
-    return docname if (docname and not path.isabs(docname)) else None
+    return '/' + docname if (docname and not path.isabs(docname)) else None
 
 
 def collect_revisions(

tests/test_path2doc.py

@@ -0,0 +1,50 @@
+import unittest
+from unittest.mock import MagicMock
+
+from sphinxnotes.recentupdate import path2doc
+
+
+def _make_repo(working_dir='/srv/docs'):
+    repo = MagicMock()
+    repo.working_dir = working_dir
+    return repo
+
+
+def _make_env(path2doc_impl):
+    env = MagicMock()
+    env.path2doc = path2doc_impl
+    return env
+
+
+class TestPath2Doc(unittest.TestCase):
+    def test_root_doc_gets_leading_slash(self):
+        """env.path2doc returns 'index' -> result should be '/index'."""
+        repo = _make_repo('/srv/docs')
+        env = _make_env(lambda p: 'index')
+        self.assertEqual(path2doc(repo, env, 'index.rst'), '/index')
+
+    def test_nested_doc_gets_leading_slash(self):
+        """env.path2doc returns 'subdir/page' -> result should be '/subdir/page'."""
+        repo = _make_repo('/srv/docs')
+        env = _make_env(lambda p: 'subdir/page')
+        self.assertEqual(path2doc(repo, env, 'subdir/page.rst'), '/subdir/page')
+
+    def test_returns_none_when_env_returns_none(self):
+        """env.path2doc returns None (not a doc file) -> should return None."""
+        repo = _make_repo('/srv/docs')
+        env = _make_env(lambda p: None)
+        self.assertIsNone(path2doc(repo, env, 'image.png'))
+
+    def test_returns_none_when_env_returns_abs_path(self):
+        """env.path2doc returns an absolute path (e.g. excluded file) -> should return None."""
+        repo = _make_repo('/srv/docs')
+        env = _make_env(lambda p: '/excluded/file')
+        self.assertIsNone(path2doc(repo, env, 'excluded/file'))
+
+    def test_passes_joined_path_to_env(self):
+        """blob_path is joined with repo.working_dir before calling env.path2doc."""
+        repo = _make_repo('/srv/docs')
+        env = MagicMock()
+        env.path2doc = MagicMock(return_value=None)
+        path2doc(repo, env, 'subdir/page.rst')
+        env.path2doc.assert_called_once_with('/srv/docs/subdir/page.rst')