fix: use safe-decode-uri-component

package.json

@@ -49,6 +49,7 @@
     "object-hash": "^1.1.8",
     "opentracing": "^0.14.0",
     "rxjs": "^5.5.0",
+    "safe-decode-uri-component": "^1.1.3",
     "semaphore-async-await": "^1.5.1",
     "string-similarity": "^1.1.0",
     "typescript": "2.7.2",

src/test/fs.test.ts

@@ -2,7 +2,7 @@ import * as chai from 'chai'
 import chaiAsPromised = require('chai-as-promised')
 import * as fs from 'mz/fs'
 import * as path from 'path'
-import * as rimraf from 'rimraf'
+import rimraf from 'rimraf'
 import * as temp from 'temp'
 import { LocalFileSystem } from '../fs'
 import { path2uri } from '../util'

src/test/memfs.test.ts

@@ -18,6 +18,14 @@ describe('memfs.ts', () => {
                 sinon.assert.calledOnce(listener)
                 sinon.assert.calledWithExactly(listener, 'file:///foo/bar.txt', undefined)
             })
+            it('should add a URI with `%` character and emit an event', () => {
+                const listener = sinon.spy()
+                const fs = new InMemoryFileSystem('/')
+                fs.on('add', listener)
+                fs.add('file:///foo/%bar.txt')
+                sinon.assert.calledOnce(listener)
+                sinon.assert.calledWithExactly(listener, 'file:///foo/%bar.txt', undefined)
+            })
             it('should add content for a URI and emit an event', () => {
                 const listener = sinon.spy()
                 const fs = new InMemoryFileSystem('/')

src/typings/safe-decode-uri-component.d.ts

@@ -0,0 +1,3 @@
+declare module 'safe-decode-uri-component' {
+    export default function safeDecodeURIComponent(uriComponent: string): string
+}

src/util.ts

@@ -1,5 +1,6 @@
 import { escapePathComponent } from 'fast-json-patch'
 import { Observable } from 'rxjs'
+import safeDecodeURIComponent from 'safe-decode-uri-component'
 import { compareTwoStrings } from 'string-similarity'
 import * as ts from 'typescript'
 import * as url from 'url'
@@ -43,7 +44,7 @@ export function normalizeUri(uri: string): string {
     if (!parts.pathname) {
         return uri
     }
-    const pathParts = parts.pathname.split('/').map(segment => encodeURIComponent(decodeURIComponent(segment)))
+    const pathParts = parts.pathname.split('/').map(segment => encodeURIComponent(safeDecodeURIComponent(segment)))
     // Decode Windows drive letter colon
     if (/^[a-z]%3A$/i.test(pathParts[1])) {
         pathParts[1] = decodeURIComponent(pathParts[1])
@@ -95,7 +96,7 @@ export function uri2path(uri: string): string {
         filePath = filePath.substr(1).replace(/\//g, '\\')
     }
 
-    return decodeURIComponent(filePath)
+    return safeDecodeURIComponent(filePath)
 }
 
 const jstsPattern = /\.[tj]sx?$/

tsconfig.json

@@ -9,6 +9,7 @@
     "sourceMap": true,
     "declaration": true,
     "strictFunctionTypes": false,
+    "esModuleInterop": true,
     "strictPropertyInitialization": false,
     "plugins": [
       {