Skip to content

Commit a6ff7ca

Browse files
committed
Fix lint: replaceAll, replaceText instead of replaceTextRange
1 parent bbe82da commit a6ff7ca

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

.claude/settings.local.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
"permissions": {
3+
"allow": [
4+
"Bash(npx vitest:*)",
5+
"Bash(git add:*)"
6+
]
7+
}
8+
}

lib/rules/template-link-rel-noopener.js

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,13 @@ module.exports = {
4444
if (relAttr && relAttr.value?.type === 'GlimmerTextNode') {
4545
// Strip existing noopener/noreferrer tokens, then re-add in canonical order
4646
// (matches ember-template-lint behavior)
47-
const oldValue = relAttr.value.chars.trim().replace(/\s+/g, ' ');
47+
const oldValue = relAttr.value.chars.trim().replaceAll(/\s+/g, ' ');
4848
const filtered = oldValue
4949
.split(' ')
5050
.filter((t) => t !== 'noopener' && t !== 'noreferrer')
5151
.join(' ');
5252
const newValue = `${filtered} noopener noreferrer`.trim();
53-
return fixer.replaceTextRange(
54-
[relAttr.value.range[0], relAttr.value.range[1]],
55-
`"${newValue}"`
56-
);
53+
return fixer.replaceText(relAttr.value, `"${newValue}"`);
5754
}
5855
// No rel attribute — insert one before the closing >
5956
const sourceCode = context.sourceCode;

0 commit comments

Comments
 (0)