Post-merge-review: template-no-negated-comparison: document name-clash; drop non-standard 'ne'

docs/rules/template-no-negated-comparison.md

@@ -2,11 +2,15 @@
 
 <!-- end auto-generated rule header -->
 
+> **Note**: This rule is NOT a port of upstream `ember-template-lint`'s `no-negated-condition`.
+> Upstream's rule prefers `not-eq` over `if/else`; this rule bans `not-eq` (and similar) entirely.
+> These are opposite goals.
+
 Disallows negated comparison operators in templates.
 
 ## Rule Details
 
-Use positive comparison operators with `{{unless}}` instead of negated comparison operators like `not-eq` or `ne`.
+Use positive comparison operators with `{{unless}}` instead of negated comparison operators like `not-eq`.
 
 ## Examples
 
@@ -20,14 +24,6 @@ Examples of **incorrect** code for this rule:
 </template>
 ```
 
-```gjs
-<template>
-  {{#if (ne this.a this.b)}}
-    Not equal
-  {{/if}}
-</template>
-```
-
 Examples of **correct** code for this rule:
 
 ```gjs

lib/rules/template-no-negated-comparison.js

@@ -21,7 +21,7 @@ module.exports = {
         if (
           node.path &&
           node.path.type === 'GlimmerPathExpression' &&
-          (node.path.original === 'not-eq' || node.path.original === 'ne')
+          node.path.original === 'not-eq'
         ) {
           context.report({
             node,

tests/lib/rules/template-no-negated-comparison.js

@@ -29,6 +29,12 @@ ruleTester.run('template-no-negated-comparison', rule, {
     `<template>
       <div></div>
     </template>`,
+    // `ne` is not a standard Ember/ember-truth-helpers helper; the rule must not flag it.
+    `<template>
+      {{#if (ne this.a this.b)}}
+        Not equal
+      {{/if}}
+    </template>`,
   ],
 
   invalid: [
@@ -46,20 +52,6 @@ ruleTester.run('template-no-negated-comparison', rule, {
         },
       ],
     },
-    {
-      code: `<template>
-        {{#if (ne this.a this.b)}}
-          Not equal
-        {{/if}}
-      </template>`,
-      output: null,
-      errors: [
-        {
-          message: 'Use positive comparison operators instead of negated ones.',
-          type: 'GlimmerSubExpression',
-        },
-      ],
-    },
     {
       code: `<template>
         <div class={{if (not-eq this.state "active") "inactive"}}></div>