Address PR review feedback

Author: MariusStorhaug

.github/linters/.luacheckrc

@@ -1,4 +1,4 @@
 -- Test data files are Lua data/config files (e.g. WoW SavedVariables format)
 -- that define top-level globals and are not executed as scripts.
-files["**/tests/data/Assignments.lua"].ignore = {""}
-files["**/tests/data/WoWSavedVariables.lua"].ignore = {""}
+files["**/tests/data/Assignments.lua"].ignore = {"111", "112"}
+files["**/tests/data/WoWSavedVariables.lua"].ignore = {"111", "112"}

src/functions/private/Format-LuaKey.ps1

@@ -29,7 +29,14 @@
         if ($Key -match '^[a-zA-Z_][a-zA-Z0-9_]*$' -and $Key -notin $reservedWords) {
             return $Key
         }
-        $escaped = $Key -replace '\\', '\\\\' -replace '"', '\"'
+        $escaped = $Key `
+            -replace '\\', '\\\\' `
+            -replace '"', '\"' `
+            -replace "`n", '\n' `
+            -replace "`r", '\r' `
+            -replace "`t", '\t' `
+            -replace "`b", '\b' `
+            -replace "`f", '\f'
         return "[`"$escaped`"]"
     }
 

src/functions/private/Read-LuaString.ps1

@@ -153,7 +153,11 @@
                                     break
                                 }
                             }
-                            $null = $result.Append([char][int]$numStr)
+                            $byteValue = [int]$numStr
+                            if ($byteValue -gt 255) {
+                                throw "Invalid decimal escape sequence '\$numStr'. Lua decimal escapes must be in the range 0-255."
+                            }
+                            $null = $result.Append([char]$byteValue)
                         } else {
                             # Unknown escape - just pass through
                             $null = $result.Append($nextChar)

src/functions/private/Skip-LuaWhitespace.ps1

@@ -39,14 +39,19 @@
                         $script:luaPos = $eqStart + $eqCount + 1
                         $closePattern = ']' + ('=' * $eqCount) + ']'
                         $closeLen = $closePattern.Length
+                        $foundClosingDelimiter = $false
                         while ($script:luaPos -lt $script:luaString.Length) {
                             if ($script:luaPos + $closeLen - 1 -lt $script:luaString.Length -and
                                 $script:luaString.Substring($script:luaPos, $closeLen) -eq $closePattern) {
                                 $script:luaPos += $closeLen
+                                $foundClosingDelimiter = $true
                                 break
                             }
                             $script:luaPos++
                         }
+                        if (-not $foundClosingDelimiter) {
+                            throw "Unterminated long-bracket comment."
+                        }
                     } else {
                         # Not a long bracket - treat as single-line comment
                         while ($script:luaPos -lt $script:luaString.Length -and