File existence checks does not work on exports of non-js files

[freshgum-bubbles]

As of current, publint doesn't seem to support Bun's export field entry.

For example, the following raises no error:

{
  "exports": {
    ".": {
      "bun": "./lufhwuh.js"
    }
  }
}

I'd be more than happy to send in a PR if you'd like.

(It's also worth noting that Bun supports TypeScript out-of-the-box, so "bun": "./src/index.ts" is possible.)

^{P.S. I'm loving publint so far. It's found two pretty painful issues from the start -- thank you so much!}

[bluwy]

Thanks! I wonder what error do you expect from that currently? It seems the missing published files check, and the file extension check seems to be working fine.

About TypeScript support, publint doesn't have much checks for it right now, but it still does check if the file is missing for example.

[freshgum-bubbles]

It seems the missing published files check, and the file extension check seems to be working fine.

Hmm, they don't seem to work on my machine -- whatever I enter into the bun condition goes unnoticed.

About TypeScript support, publint doesn't have much checks for it right now, but it still does check if the file is missing for example.

Roger that :-)

[bluwy]

Would be great if you can share an example here, otherwise I'm not sure what to fix for this issue.

[freshgum-bubbles]

Sorry for the delay -- I haven't got as much time as I usually have this week, but I could get you a repro. next week. In my testing, publint wasn't checking what was in the bun condition, so "bun": "efuhefefouhefu.ts" would pass just fine.

If it helps, the repository in question is https://github.com/freshgum/typedi -- that's where I was able to reproduce the issue :)

[bluwy]

Ah ok I found the issue. On this line:

publint/pkg/src/index.js

Lines 622 to 623 in 7ba9aa3

	// TODO: maybe check .ts in the future
	if (!isFilePathLintable(filePath)) continue

If the path is not lintable (which simply means if it's not .js, .cjs, .mjs), it should continue to check if the file exist like:

publint/pkg/src/index.js

Lines 580 to 588 in 7ba9aa3

	// types. check file existence only
	if (currentPath.includes('types')) {
	const pq = createPromiseQueue()
	for (const filePath of exportsFiles) {
	pq.push(async () => await readFile(filePath, currentPath))
	}
	await pq.wait()
	return
	}

But it wasn't.