can't resolve jsmidgen and midi-file-parser

[jsphweid]

I get this error when I try to use this library.

ERROR in ./node_modules/midiconvert/src/Midi.js
Module not found: Error: Can't resolve 'jsmidgen' in '/Users/josephweidinger/git/personal/MidiConvertWrapperAdvanced/node_modules/midiconvert/src'
 @ ./node_modules/midiconvert/src/Midi.js 2:0-35
 @ ./node_modules/midiconvert/src/MidiConvert.js
 @ ./src/main.ts

ERROR in ./node_modules/midiconvert/src/Midi.js
Module not found: Error: Can't resolve 'midi-file-parser' in '/Users/josephweidinger/git/personal/MidiConvertWrapperAdvanced/node_modules/midiconvert/src'
 @ ./node_modules/midiconvert/src/Midi.js 1:0-43
 @ ./node_modules/midiconvert/src/MidiConvert.js
 @ ./src/main.ts

Here are 4 methods to make it work... but they all feel wrong...

1. If I run the commands:

npm install jsmidgen
npm install midi-file-parser

The errors go away.

2. Looking at the package.json, I notice that these seemingly important dependencies are only devDependencies. If I fork the project and make them dependencies, it installs both of these automatically and everything is good.

3. Deleting node_modules/midiconvert/src (forces it to use /build)

4. Deleting in package.json "module": "src/MidiConvert.js",

So it appears the issue is it is using the src dir and not the bundle when you import. It then fails when it tries to compile Midi.js because of the lines:

import * as Decoder from 'midi-file-parser'
import * as Encoder from 'jsmidgen'

In my opinion, it makes sense to list these things as dependencies and not devDependencies. So I'm going to make a PR and you all can decide.

Great library!