Remove redundant vf- prefix from the rule names

Author: touzoku

README.md

@@ -26,9 +26,9 @@ Example:
         "@salesforce/eslint-plugin-visualforce"
     ],
     "rules": {
-      "visualforce/vf-no-atom-expr": "error",
-      "visualforce/vf-no-apex-tags": "error",
-      "visualforce/vf-jsencode": "error"
+      "visualforce/no-atom-expr": "error",
+      "visualforce/no-apex-tags": "error",
+      "visualforce/jsencode": "error"
     }
 }
 ```

dist/index.js

@@ -981,9 +981,9 @@ patchESLint();
 
 module.exports = {
   rules: {
-    'vf-no-atom-expr': __webpack_require__(6),
-    'vf-no-apex-tags': __webpack_require__(5),
-    'vf-jsencode': __webpack_require__(4)
+    'no-atom-expr': __webpack_require__(6),
+    'no-apex-tags': __webpack_require__(5),
+    'jsencode': __webpack_require__(4)
   }
 };
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/eslint-plugin-visualforce",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "An ESLint plugin to extract and lint scripts from VisualForce pages",
   "main": "dist/index.js",
   "scripts": {

src/index.js

@@ -213,8 +213,8 @@ patchESLint()
 
 module.exports = {
   rules: {
-    'vf-no-atom-expr': require('./rules/vf-no-atom-expr'),
-    'vf-no-apex-tags': require('./rules/vf-no-apex-tags'),
-    'vf-jsencode': require('./rules/vf-jsencode'),
+    'no-atom-expr': require('./rules/no-atom-expr'),
+    'no-apex-tags': require('./rules/no-apex-tags'),
+    'jsencode': require('./rules/jsencode'),
   }
 }

src/rules/vf-jsencode.js renamed to src/rules/jsencode.js

@@ -32,12 +32,12 @@ test('Merge field in JS expression context', assert => {
 
   const messages = execute('simple.page', {
     rules: {
-      'visualforce/vf-no-atom-expr': 'error'
+      'visualforce/no-atom-expr': 'error'
     }
   })
 
   assert.equal(messages.length, 1)
-  assert.deepEqual(messages[0], { ruleId: 'visualforce/vf-no-atom-expr',
+  assert.deepEqual(messages[0], { ruleId: 'visualforce/no-atom-expr',
     severity: 2,
     message: 'VisualForce merge fields should only be allowed in strings',
     line: 5,
@@ -54,7 +54,7 @@ test('Autofixing merge fields in JS expression context', assert => {
 
   const result = execute('fix.page', {
     rules: {
-      'visualforce/vf-no-atom-expr': 'error'
+      'visualforce/no-atom-expr': 'error'
     },
     fix: true
   })
@@ -73,11 +73,11 @@ test('<apex:*> tags in Javascript', assert => {
 
   const messages = execute('apex-tags-in-script.page', {
     rules: {
-      'visualforce/vf-no-apex-tags': 'error'
+      'visualforce/no-apex-tags': 'error'
     }
   })
 
-  assert.deepEqual(messages[0], { ruleId: 'visualforce/vf-no-apex-tags',
+  assert.deepEqual(messages[0], { ruleId: 'visualforce/no-apex-tags',
     severity: 2,
     message: 'VisualForce standard components (<apex:*> tags) are not allowed in Javascript',
     line: 7,
@@ -93,7 +93,7 @@ test('JSENCODE of Apex variables', assert => {
 
   const messages = execute('jsencode.page', {
     rules: {
-      'visualforce/vf-jsencode': 'error'
+      'visualforce/jsencode': 'error'
     }
   })
 
@@ -105,7 +105,7 @@ test('JSENCODE of Apex variables', assert => {
     fix: { range: [ 118, 130 ], text: 'JSENCODE(apexVariable)' },
     message: 'JSENCODE() must be applied to all rendered Apex variables',
     nodeType: 'VFELIdentifier',
-    ruleId: 'visualforce/vf-jsencode',
+    ruleId: 'visualforce/jsencode',
     severity: 2,
     source: 'var foo =  \'{! apexVariable }\'',
   })
@@ -116,7 +116,7 @@ test('JSENCODE of Apex variables', assert => {
     fix: { range: [ 190, 205 ], text: 'JSENCODE(taintedVariable)' },
     message: 'JSENCODE() must be applied to all rendered Apex variables',
     nodeType: 'VFELIdentifier',
-    ruleId: 'visualforce/vf-jsencode',
+    ruleId: 'visualforce/jsencode',
     severity: 2,
     source: 'var bar =  "{! IF(LEN(apexVariable)>5, \'bazinga\', taintedVariable) }"',
   })
@@ -127,7 +127,7 @@ test('JSENCODE of Apex variables', assert => {
     fix: { range: [ 253, 259 ], text: 'JSENCODE(result)' },
     message: 'JSENCODE() must be applied to all rendered Apex variables',
     nodeType: 'VFELIdentifier',
-    ruleId: 'visualforce/vf-jsencode',
+    ruleId: 'visualforce/jsencode',
     severity: 2,
     source: 'var baz = "{! CASE(condition, value, result) }"',
   })
@@ -138,7 +138,7 @@ test('JSENCODE of Apex variables', assert => {
     fix: { range: [ 361, 391 ], text: 'JSENCODE($CurrentPage.parameters.retURL)' },
     message: 'JSENCODE() must be applied to all rendered Apex variables',
     nodeType: 'VFELIdentifier',
-    ruleId: 'visualforce/vf-jsencode',
+    ruleId: 'visualforce/jsencode',
     severity: 2,
     source: 'var unsafeSystemVar = \'{! $CurrentPage.parameters.retURL }\'',
   })
@@ -149,7 +149,7 @@ test('JSENCODE of Apex variables', assert => {
     fix: { range: [ 430, 456 ], text: 'JSENCODE(some.array[another[field]])' },
     message: 'JSENCODE() must be applied to all rendered Apex variables',
     nodeType: 'VFELMemberExpression',
-    ruleId: 'visualforce/vf-jsencode',
+    ruleId: 'visualforce/jsencode',
     severity: 2,
     source: 'var selectorExpression = "{! some.array[another[field]] }"',
   })