fix: parser treats "Nan" as a number instead of a string (#22)

* fix: parser treats "Nan" as a number instead of a string

Author: neo451

CHANGELOG.md

@@ -25,6 +25,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Fixed typo in `ObsidianPasteImg`'s command description
 - Fixed the case when `opts.attachments` is `nil`.
 - Fixed bug where `ObsidianNewFromTemplate` did not respect `note_id_func`
+- Fixed bug where parser treats "Nan" as a number instead of a string
 
 ## [v3.9.0](https://github.com/epwalsh/obsidian.nvim/releases/tag/v3.9.0) - 2024-07-11
 

lua/obsidian/util.lua

@@ -1341,4 +1341,12 @@ util.buffer_is_empty = function(bufnr)
   end
 end
 
+---Check if a string is NaN
+---
+---@param v any
+---@return boolean
+util.isNan = function(v)
+  return tostring(v) == tostring(0 / 0)
+end
+
 return util

lua/obsidian/yaml/parser.lua

@@ -555,7 +555,7 @@ end
 ---@diagnostic disable-next-line: unused-local
 Parser._parse_number = function(self, i, text)
   local out = tonumber(text)
-  if out == nil then
+  if out == nil or util.isNan(out) then
     return false, nil, nil
   else
     return true, nil, out

test/obsidian/yaml/parser_spec.lua

@@ -30,6 +30,10 @@ describe("Parser class", function()
       return parser:parse_number(str)
     end, "foo")
     assert.is_false(ok)
+    ok, _ = pcall(function(str)
+      return parser:parse_number(str)
+    end, "Nan")
+    assert.is_false(ok)
   end)
 
   it("should parse booleans while trimming whitespace", function()