Fix YAML parsing issues with unquoted URLs (#348)

Fixes #347.

Author: epwalsh

CHANGELOG.md

@@ -15,6 +15,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Changed API of client methods `Client:find_tags()` and `Client:find_tags_async()`. The return value (or value passed to the callback) is now a list of objects representing the location of tags found. These objects have the following fields: `tag: string`, `path: string|Path`, `line: integer`.
 
+### Fixed
+
+- Fixed a YAML parsing issue with unquoted URLs in an array item.
+
 ## [v2.7.1](https://github.com/epwalsh/obsidian.nvim/releases/tag/v2.7.1) - 2024-01-23
 
 ### Fixed

lua/obsidian/yaml/native.lua

@@ -216,8 +216,16 @@ end
 Parser._try_parse_field = function(self, lines, i, text)
   local line = lines[i]
   text = text and text or util.strip_comments(line.content)
+
   local _, key, value
-  _, _, key, value = string.find(text, "([a-zA-Z0-9_-]+):(.*)")
+
+  -- First look for start of mapping, array, block, etc, e.g. 'foo:'
+  _, _, key = string.find(text, "([a-zA-Z0-9_-]+):$")
+  if not key then
+    -- Then try inline field, e.g. 'foo: bar'
+    _, _, key, value = string.find(text, "([a-zA-Z0-9_-]+): (.*)")
+  end
+
   value = value and util.strip_whitespace(value) or nil
   if value == "" then
     value = nil

test/obsidian/yaml/parser_spec.lua

@@ -201,7 +201,9 @@ describe("Parser class", function()
     local result = parser:parse(table.concat({
       "aliases:",
       ' - "Research project: staged training"',
+      "sources:",
+      " - https://example.com",
     }, "\n"))
-    assert.are_same({ aliases = { "Research project: staged training" } }, result)
+    assert.are_same({ aliases = { "Research project: staged training" }, sources = { "https://example.com" } }, result)
   end)
 end)