fix: remove BOM

mdvorak · Apr 15, 2023 · 23fd4a1 · 23fd4a1
1 parent 7d39025
commit 23fd4a1
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/properties.spec.ts b/src/properties.spec.ts
@@ -17,6 +17,14 @@ describe('parse', () => {
       'extra'
     ])
   })
+
+  it('should remove BOM', () => {
+    const sample = '\ufefffoo=bar\n#test'
+
+    // Test
+    const result = properties.parse(sample)
+    expect(result.lines).toEqual(['foo=bar', '#test'])
+  })
 })
 
 describe('stringify', () => {

diff --git a/src/properties.ts b/src/properties.ts
@@ -42,6 +42,11 @@ export const parse = (contents: string): Properties => {
     lines.pop()
   }
 
+  // Remove BOM from the first line
+  if (lines.length > 0 && lines[0].codePointAt(0) === 0xfeff) {
+    lines[0] = lines[0].slice(1)
+  }
+
   return {lines}
 }
 
@@ -473,7 +478,7 @@ const parseUnicode = (sequence: string, line: number): string => {
   if (!sequence.match(/^0x[\da-fA-F]{4}$/)) {
     throw new Error(`Invalid unicode sequence at line ${line}`)
   }
-  return String.fromCharCode(parseInt(sequence, 16))
+  return String.fromCodePoint(parseInt(sequence, 16))
 }
 
 /**