Cannot use parseFile if the TypeScript compiler moduleResolution is set to bundler

[lowvisiondave]

Is there an existing issue for this?

• I have searched the existing issues

music-metadata version

11.0.0

Current Behavior

TypeScript throw an error when doing the following:

import { parseFile } from "music-metadata";

Because node_modules/music-metadata/lib/core.d.ts does not include a type for parseFile.

Adding // @ts-expect-error allows the application to build successfully, so it seems to just a types issue.

Reproduction repository

Expected Behavior

I would expect the types to match the functions being exported.

Attached audio sample?

• I have provided sufficient information to reproduce the issue

[Borewit]

Thanks for creating a reproduction repository @lowvisiondave !

The problem you are facing is because the node condition, a condition exports, is not correctly set when your project imports music-metadata.

Because node_modules/music-metadata/lib/index.d.ts does not include a type for parseFile.

It certainly does, it is right here:

music-metadata/lib/index.ts

Line 40 in c824bb9

export async function parseFile(filePath: string, options: IOptions = {}): Promise<IAudioMetadata> {

But you are not importing this file, you are importing the compiled version of core.ts, and that does indeed not have parseFile() included.

The reason your conditional export is not working, is most likely related to the fact you are using, "moduleResolution": "bundler", in your tsconfig.json. This module resolution, does not seem to set the node condition, maybe because it is designed for browser usage, I can only speculate why it done like that.

What I fear neither works is this (almost literally taken from TypeScript documentation)

{
  "compilerOptions": {
    "....": "",
    "moduleResolution": "bundler",
    "customConditions": ["node"],
     "....": "",
  }
}

Identical issue on file-type: sindresorhus/file-type#738

I will investigate possible workarounds, to be continued....

[Borewit]

Adding the "customConditions": ["node"] solves the issue, evidence: lowvisiondave/music-metadata-typescript-repro#1

[lowvisiondave]

Interesting, I've never come across that before. Thanks for the explanation!

It might be worth adding a note to the documentation, since I experienced this on a fresh Next.js project and it was discouraging me from using your library.

[net]

Also having this problem. Can this be made to just work out of the box? I've never encountered this issue with any other library before.

[Borewit]

Also having this problem. Can this be made to just work out of the box?

The problem is in your TypeScript compiler is not including Node specific functionality, as it believes by setting bundler as your module-resolution, you are not going to need that.

What do you want me to do about that?

I've never encountered this issue with any other library before.

And which of those libraries have Node.js specific functionality, and nicely integrate browser platforms?

Raise the issue where it comes from, at Next.js (suggest to include Node condition by default) or for this issue: microsoft/TypeScript#61357.