graphmark 0.3.2__tar.gz → 0.3.3__tar.gz

{graphmark-0.3.2 → graphmark-0.3.3}/CHANGELOG.md

@@ -1,6 +1,32 @@
 # CHANGELOG
 
 
+## v0.3.3 (2026-07-25)
+
+### Bug Fixes
+
+- **resolve**: [[note.md]] resolves like [[Note]]
+  ([#105](https://github.com/cdcoonce/graphmark/pull/105),
+  [`916da52`](https://github.com/cdcoonce/graphmark/commit/916da52465cc3c5295a9440ae86af1ee6065e1d5))
+
+Obsidian accepts an explicit .md extension, so [[Note.md]] and [[Note]] are the same link. graphmark
+  resolved only the second: the normalizer turned "Note.md" into the key "note md" while the file's
+  catalog key is "note", so the link never matched and was reported as broken. The path-suffix
+  branch was worse — it appends ".md" itself, so [[folder/note.md]] searched for folder/note.md.md.
+
+Strip a case-insensitive trailing ".md" after alias/anchor stripping and before both resolution
+  branches. A missing [[Nowhere.md]] stays unresolved and an ambiguous stem stays ambiguous; a title
+  merely ending in the word "MD" is untouched.
+
+Parity-safe: no frozen fixture uses a .md-style link, so every expected.json is unchanged (81 oracle
+  tests green). The change only adds resolutions that previously failed — it never removes one.
+
+Also updates the test added in #101 that documented this gap as current behavior; it now asserts the
+  fix.
+
+Closes #104
+
+
 ## v0.3.2 (2026-07-25)
 
 ### Bug Fixes

{graphmark-0.3.2 → graphmark-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: graphmark
-Version: 0.3.2
+Version: 0.3.3
 Summary: Deterministic knowledge-graph analysis for markdown / [[wikilink]] vaults.
 Project-URL: Homepage, https://github.com/cdcoonce/graphmark
 Project-URL: Repository, https://github.com/cdcoonce/graphmark

{graphmark-0.3.2 → graphmark-0.3.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "graphmark"
-version = "0.3.2"
+version = "0.3.3"
 description = "Deterministic knowledge-graph analysis for markdown / [[wikilink]] vaults."
 readme = "README.md"
 requires-python = ">=3.11"

{graphmark-0.3.2 → graphmark-0.3.3}/src/graphmark/graph.py

@@ -84,6 +84,11 @@ class NormalizeResolver:
         display = display.split("|")[0]
         # Strip anchor: "Note#Section" → "Note"
         display = display.split("#")[0]
+        # Obsidian accepts an explicit extension, so "Note.md" is the same link as "Note".
+        # Stripped before both branches below: the path-suffix branch appends ".md" itself,
+        # and the bare branch would otherwise normalize to the key "note md".
+        if display.lower().endswith(".md"):
+            display = display[: -len(".md")]
 
         if "/" in display:
             # Path-suffix resolution: find unique rel_path ending with "display.md"

{graphmark-0.3.2 → graphmark-0.3.3}/tests/test_graph.py

@@ -190,3 +190,50 @@ class TestVaultGraphBuild:
         assert degree("personal/gamma.md") == 2
         assert degree("reference/island.md") == 0
         assert degree("reference/stub.md") == 0
+
+
+class TestMarkdownExtensionInLinks:
+    """Obsidian treats [[Note.md]] and [[Note]] as the same link; so must the resolver."""
+
+    def setup_method(self):
+        self.resolver = NormalizeResolver()
+
+    def _catalog(self, *rel_paths: str) -> dict[str, list[str]]:
+        return build_catalog([_doc(p) for p in rel_paths])
+
+    def test_bare_link_with_md_extension_resolves(self):
+        catalog = self._catalog("brain/alpha.md")
+        assert self.resolver.resolve("alpha.md", catalog) == "brain/alpha.md"
+
+    def test_extension_match_is_case_insensitive(self):
+        catalog = self._catalog("brain/alpha.md")
+        assert self.resolver.resolve("Alpha.MD", catalog) == "brain/alpha.md"
+
+    def test_md_extension_with_alias_resolves(self):
+        catalog = self._catalog("brain/alpha.md")
+        assert self.resolver.resolve("alpha.md|the first note", catalog) == "brain/alpha.md"
+
+    def test_md_extension_with_anchor_resolves(self):
+        catalog = self._catalog("brain/alpha.md")
+        assert self.resolver.resolve("alpha.md#Section", catalog) == "brain/alpha.md"
+
+    def test_path_style_link_with_md_extension_resolves(self):
+        catalog = self._catalog("brain/alpha.md", "personal/beta.md")
+        assert self.resolver.resolve("brain/alpha.md", catalog) == "brain/alpha.md"
+
+    def test_missing_target_with_md_extension_stays_unresolved(self):
+        catalog = self._catalog("brain/alpha.md")
+        assert self.resolver.resolve("nowhere.md", catalog) is None
+
+    def test_ambiguous_stem_stays_ambiguous_with_the_extension(self):
+        catalog = {"note": ["dir1/note.md", "dir2/note.md"]}
+        assert self.resolver.resolve("note.md", catalog) is None
+
+    def test_extensionless_links_are_unaffected(self):
+        catalog = self._catalog("brain/alpha.md")
+        assert self.resolver.resolve("alpha", catalog) == "brain/alpha.md"
+
+    def test_a_title_ending_in_md_as_a_word_is_not_stripped(self):
+        # "Doctor MD" ends with "md" but not ".md" — it must not be truncated.
+        catalog = self._catalog("org/Doctor MD.md")
+        assert self.resolver.resolve("Doctor MD", catalog) == "org/Doctor MD.md"

{graphmark-0.3.2 → graphmark-0.3.3}/tests/test_unresolved.py

@@ -204,13 +204,13 @@ class TestNonMarkdownTargets:
         assert graph.out_links["a.md"] == set()
 
     def test_md_targets_are_exempt_from_the_rule(self, tmp_path):
-        # .md is explicitly excluded from the non-note rule, so a link ending in .md keeps
-        # whatever the resolver decides. graphmark does not currently resolve a trailing
-        # ".md" (an Obsidian-compatibility gap tracked separately) — the point here is only
-        # that this rule does not silently swallow it.
+        # .md is excluded from the non-note rule, so a link ending in .md is left to the
+        # resolver — which now resolves it (a trailing ".md" is stripped, matching Obsidian).
         _write(tmp_path, "a.md", "See [[b.md]].\n")
         _write(tmp_path, "b.md", "Target.\n")
-        assert _build(tmp_path).unresolved == {"a.md": ["b.md"]}
+        graph = _build(tmp_path)
+        assert graph.unresolved == {}
+        assert graph.out_links["a.md"] == {"b.md"}
 
     def test_missing_md_target_is_still_unresolved(self, tmp_path):
         _write(tmp_path, "a.md", "See [[Nowhere.md]].\n")