Use .ts extension in relative source imports

[eemeli]

This change adopts the --rewriteRelativeImportExtensions option introduced in Typescript 5.7, and changes all relative paths to use the .ts extension.

Due to rollup/plugins#1820, a post-compilation transform is required to get the CLI tool to work right.

An ESLint rule is added to ensure that the .ts extension is always there; this may be of interest to others as well:

yaml/eslint.config.mjs

Lines 41 to 51 in 18e6112

	'no-restricted-imports': [
	'error',
	{
	patterns: [
	{
	regex: '^\\..*(?<!\\.ts)$',
	message: 'Relative imports must use .ts extension.'
	}
	]
	}
	],

Together with explicit type on imports & exports, this change fixes #301 and makes the library work directly in Deno. A really simple Deno smoke test is included. It would be nice to get the full test suite to run on Deno as well, as discussed in #301 (comment)

Ping @justinmchase & @jeff-hykin: Would you have time to see if this works as expected?

[jeff-hykin]

Yeah thanks for the ping! If I remeber once I'm at my PC I'll try it out

[eemeli]

Also needed to add an afterDeclarations transform to drop the .ts extensions from the imports in the .d.ts files, in order to retain compatibility with older TS versions.

[eemeli]

@jeff-hykin Any time for a trial run yet?

I'm looking here for feedback from some actual Deno user to confirm that this does indeed provide a fix for #301, before just presuming so on my own initiative.

[justinmchase]

Looks like its working to me, I cloned the repo, checked out the branch and wrote a little script.

test script

terminal results

[justinmchase]

Also, worth mentioning, but Deno has made a new package repo https://jsr.io which may be worth targeting instead of / in addition to deno land.

It's not much different in terms of what you publish, just done with a github action.

Here is a pretty minimal example repo I created for reference: https://github.com/justinmchase/type/blob/main/.github/workflows/publish.yml

[jeff-hykin]

Sorry it took a really long time, but aside from one caveat, I can confirm it works in deno natively! Thank you!

// caveat: `import YAML`  (from readme) does *not* work, but the following does 
import * as YAML from 'https://raw.githubusercontent.com/eemeli/yaml/refs/heads/main/src/index.ts'
YAML.parse('3.14159') // 3.14159
YAML.parse('[ true, false, maybe, null ]\n') // [ true, false, 'maybe', null ]

I ran hacked versions of the tests, and they seem to pass too