[Replacement]: `trim-repeated`

[mbtools]

Package to replace

trim-repeated

Suggested replacement(s)

str.replace(new RegExp(`(?:${RegExp.escape(target)}){2,}`, 'g'), target)

Manifest type

micro-utility (tiny utility replaceable with native code or removal)

Rationale

https://npmx.dev/package/trim-repeated

4.7m downloads/week -> use regex instead

also avoid dep on escape-string-regexp

Availability

No response

Code example (optional)

function trimRepeated(str, target) {
  return str.replace(new RegExp(`(?:${RegExp.escape(target)}){2,}`, 'g'), target)
}

[gameroman]

Might need to wait until RegExp.escape is out of Newly available stage

[mbtools]

Maybe... but it's already used here 😉

module-replacements/manifests/native.json

Lines 2055 to 2059 in 4579976

	"escape-string-regexp": {
	"type": "module",
	"moduleName": "escape-string-regexp",
	"replacements": ["RegExp.escape"]
	},

[gameroman]

Maybe... but it's already used here 😉

module-replacements/manifests/native.json

Lines 2055 to 2059 in 4579976

"escape-string-regexp": {
"type": "module",
"moduleName": "escape-string-regexp",
"replacements": ["RegExp.escape"]
},

For that one since it references the webFeatureId we can detect if it is available in specific runtime or not

For micro-utilities we don't have a way to do that yet, but maybe we should @43081j

[joaopedrodcf]

Was checking the link @gameroman and it's already baseline, you mean us waiting for baseline widely available ?

[gameroman]

you mean us waiting for baseline widely available ?

Yes, widely available

Or having a way in micro-utilities to define what native features are being used

[43081j]

are we going further than needed with this?

isn't the replacement to not even have this function in the first place? i.e. trimRepeated(str, '-') becomes str.replaceAll(/(?:-{2,})/g, '-')?

[gameroman]

Looks like most of the usage is coming from old versions of https://npmx.dev/package/filenamify/v/4.3.0