migrate `stable-hash` to `stable-hash-x`

[JounQin]

https://github.com/shuding/stable-hash is out of maintainance for a long time, https://github.com/lukeed/dequal https://github.com/un-ts/stable-hash-x could be considered as an alternative.

cc @43081j @SukkaW

[petercat-assistant]

It looks like you're suggesting a migration from stable-hash to dequal/lite due to the former being out of maintenance. This is a valid consideration, especially if dequal/lite offers similar functionality and is actively maintained.

If you need assistance with implementing this migration in the un-ts/eslint-plugin-import-x project, I can help you with the following steps:

1. Identify Usage: Find where stable-hash is used in the codebase.
2. Replace with dequal/lite: Update the code to use dequal/lite instead.
3. Testing: Ensure that the new implementation works as expected by running tests.

If you would like me to assist with any specific part of this process, please let me know!

[43081j]

we're currently using this for creating cache keys it seems:

eslint-plugin-import-x/src/utils/resolve.ts

Lines 141 to 148 in 3bc48fc

	const cacheKey =
	sourceDir +
	',' +
	childContextHashKey +
	',' +
	memoizedHash +
	',' +
	modulePath

but this doesn't seem sensible for memory usage either (storing an arbitrarily sized string as a cache key)

i wonder if we can be smarter and use a WeakMap or something

[JounQin]

Oh, yeah, dequal/lite seems can not meet our requirements in this case then.

i wonder if we can be smarter and use a WeakMap or something

That would be great if we found.

[SukkaW]

but this doesn't seem sensible for memory usage either (storing an arbitrarily sized string as a cache key)

@43081j We can't store explicit strings like sourceDir and modulePath directly in WeakMap anyway. And even if we find a way (by creating objects), this will only hurt performance even more.

could be considered as an alternative.

@JounQin dequal is not actively maintained either, last release 3 years ago.

[JounQin]

Maybe we should just fork stable-hash. 🤦‍♂️

[SukkaW]

Maybe we should just fork stable-hash. 🤦‍♂️

Let's just ask @shuding politely instead.

[JounQin]

Let's just ask @shuding politely instead.

I tried to contact @shuding and once successfully.

I've emailed him again yestoday.

[shuding]

Hey sorry for the long delay - I just cut a new release for stable-hash. Lmk what other PRs I should be looking into!

[JounQin]

@shuding Thanks for the new release, while I still want to ask, are you going to actively maintain that package? 😂

[JounQin]

@shuding The version of stable-hash is still in 0.0.x, so every single version is considered as BREAKING CHANGE and the upgrade command of package managers will not upgrade it, that'll result duplicate versions installed. I really don't want to waste time on this anymore...

[JounQin]

close via #373