rome · ematipico · Oct 17, 2022 · Oct 12, 2022 · Oct 14, 2022 · Oct 17, 2022
diff --git a/crates/rome_diagnostics_categories/src/categories.rs b/crates/rome_diagnostics_categories/src/categories.rs
@@ -60,6 +60,7 @@ define_dategories! {
     "lint/nursery/useKeyWithClickEvents": "https://rome.tools/docs/lint/rules/useKeyWithClickEvents",
     "lint/nursery/noRestrictedGlobals": "https://rome.tools/docs/lint/rules/noRestrictedGlobals",
     "lint/nursery/noPositiveTabindex": "https://rome.tools/docs/lint/rules/noPositiveTabindex",
+    "lint/nursery/noConstAssign": "https://rome.tools/docs/lint/rules/noConstAssign",
     "lint/nursery/noExplicitAny": "https://rome.tools/docs/lint/rules/noExplicitAny",
     "lint/style/noNegationElse": "https://rome.tools/docs/lint/rules/noNegationElse",
     "lint/style/noShoutyConstants": "https://rome.tools/docs/lint/rules/noShoutyConstants",

diff --git a/crates/rome_js_analyze/src/semantic_analyzers/nursery.rs b/crates/rome_js_analyze/src/semantic_analyzers/nursery.rs
diff --git a/crates/rome_js_analyze/src/semantic_analyzers/nursery/no_const_assign.rs b/crates/rome_js_analyze/src/semantic_analyzers/nursery/no_const_assign.rs
@@ -0,0 +1,93 @@
+use crate::semantic_services::Semantic;
+use rome_analyze::context::RuleContext;
+use rome_analyze::{declare_rule, Rule, RuleDiagnostic};
+use rome_console::markup;
+use rome_js_syntax::{JsIdentifierAssignment, JsVariableDeclaration};
+use rome_rowan::{AstNode, TextRange};
+
+declare_rule! {
+    /// Prevents from having `const` variables being re-assigned.
+    ///
+    /// Trying to assign a value to a `const` will cause an `TypeError` when the code is executed.
+    ///
+    /// ## Examples
+    ///
+    /// ### Invalid
+    ///
+    /// ```js,expect_diagnostic
+    /// const a = 1;
+    /// a = 4;
+    /// ```
+    ///
+    /// ```js,expect_diagnostic
+    /// const a = 2;
+    /// a += 1;
+    /// ```
+    ///
+    /// ```js,expect_diagnostic
+    /// const a = 1;
+    /// ++a;
+    /// ```
+    ///
+    /// ```js,expect_diagnostic
+    /// const a = 1, b = 2;
+    ///
+    /// a = 2;
+    /// ```
+    ///
+    /// ### Valid
+    ///
+    /// ```js
+    /// const a = 10;
+    /// let b = 10;
+    /// b = 20;
+    /// ```
+    ///
+    pub(crate) NoConstAssign {
+        version: "10.0.0",
+        name: "noConstAssign",
+        recommended: false,
+    }
+}
+
+impl Rule for NoConstAssign {
+    type Query = Semantic<JsIdentifierAssignment>;
+    type State = TextRange;
+    type Signals = Option<Self::State>;
+    type Options = ();
+
+    fn run(ctx: &RuleContext<Self>) -> Self::Signals {
+        let node = ctx.query();
+        let model = ctx.model();
+
+        let declared_binding = model.declaration(node)?;
+        if let Some(variable_declaration) = declared_binding
+            .syntax()
+            .ancestors()
+            .find_map(|ancestor| JsVariableDeclaration::cast_ref(&ancestor))
+        {
+            if variable_declaration.is_const() {
+                return Some(declared_binding.syntax().text_trimmed_range());
+            }
+        }
+
+        None
+    }
+
+    fn diagnostic(ctx: &RuleContext<Self>, state: &Self::State) -> Option<RuleDiagnostic> {
+        let node = ctx.query();
+        let name = node.name_token().ok()?;
+        let name = name.text_trimmed();
+        Some(
+            RuleDiagnostic::new(
+                rule_category!(),
+                node.syntax().text_trimmed_range(),
+                markup! {"Can't assign "<Emphasis>{name}</Emphasis>" because it's a constant"},
+            )
+            .secondary(
+                state,
+                markup! {"This is where the variable is defined as constant"},
+            ),
+        )
+    }
+}
diff --git a/crates/rome_js_analyze/tests/specs/nursery/noConstAssign.js b/crates/rome_js_analyze/tests/specs/nursery/noConstAssign.js
@@ -0,0 +1,11 @@
+const a = 1;
+a = 2;
+
+const b = 2, c = 43;
+b = 4;
+++b;
+b += 45;
+b--;
+function f() {
+    b++;
+}
diff --git a/crates/rome_js_analyze/tests/specs/nursery/noConstAssign.js.snap b/crates/rome_js_analyze/tests/specs/nursery/noConstAssign.js.snap
@@ -0,0 +1,160 @@
+---
+source: crates/rome_js_analyze/tests/spec_tests.rs
+expression: noConstAssign.js
+---
+# Input
+```js
+const a = 1;
+a = 2;
+
+const b = 2, c = 43;
+b = 4;
+++b;
+b += 45;
+b--;
+function f() {
+    b++;
+}
+```
+
+# Diagnostics
+```
+noConstAssign.js:2:1 lint/nursery/noConstAssign ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Can't assign a because it's a constant
+
+    1 │ const a = 1;
+  > 2 │ a = 2;
+      │ ^
+    3 │ 
+    4 │ const b = 2, c = 43;
+
+  i This is where the variable is defined as constant
+
+  > 1 │ const a = 1;
+      │       ^
+    2 │ a = 2;
+    3 │ 
+
+
+```
+
+```
+noConstAssign.js:5:1 lint/nursery/noConstAssign ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Can't assign b because it's a constant
+
+    4 │ const b = 2, c = 43;
+  > 5 │ b = 4;
+      │ ^
+    6 │ ++b;
+    7 │ b += 45;
+
+  i This is where the variable is defined as constant
+
+    2 │ a = 2;
+    3 │ 
+  > 4 │ const b = 2, c = 43;
+      │       ^
+    5 │ b = 4;
+    6 │ ++b;
+
+
+```
+
+```
+noConstAssign.js:6:3 lint/nursery/noConstAssign ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Can't assign b because it's a constant
+
+    4 │ const b = 2, c = 43;
+    5 │ b = 4;
+  > 6 │ ++b;
+      │   ^
+    7 │ b += 45;
+    8 │ b--;
+
+  i This is where the variable is defined as constant
+
+    2 │ a = 2;
+    3 │ 
+  > 4 │ const b = 2, c = 43;
+      │       ^
+    5 │ b = 4;
+    6 │ ++b;
+
+
+```
+
+```
+noConstAssign.js:7:1 lint/nursery/noConstAssign ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Can't assign b because it's a constant
+
+    5 │ b = 4;
+    6 │ ++b;
+  > 7 │ b += 45;
+      │ ^
+    8 │ b--;
+    9 │ function f() {
+
+  i This is where the variable is defined as constant
+
+    2 │ a = 2;
+    3 │ 
+  > 4 │ const b = 2, c = 43;
+      │       ^
+    5 │ b = 4;
+    6 │ ++b;
+
+
+```
+
+```
+noConstAssign.js:8:1 lint/nursery/noConstAssign ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Can't assign b because it's a constant
+
+     6 │ ++b;
+     7 │ b += 45;
+   > 8 │ b--;
+       │ ^
+     9 │ function f() {
+    10 │     b++;
+
+  i This is where the variable is defined as constant
+
+    2 │ a = 2;
+    3 │ 
+  > 4 │ const b = 2, c = 43;
+      │       ^
+    5 │ b = 4;
+    6 │ ++b;
+
+
+```
+
+```
+noConstAssign.js:10:5 lint/nursery/noConstAssign ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Can't assign b because it's a constant
+
+     8 │ b--;
+     9 │ function f() {
+  > 10 │     b++;
+       │     ^
+    11 │ }
+
+  i This is where the variable is defined as constant
+
+    2 │ a = 2;
+    3 │ 
+  > 4 │ const b = 2, c = 43;
+      │       ^
+    5 │ b = 4;
+    6 │ ++b;
+
+
+```
+
+
diff --git a/crates/rome_service/src/configuration/linter/rules.rs b/crates/rome_service/src/configuration/linter/rules.rs
diff --git a/editors/vscode/configuration_schema.json b/editors/vscode/configuration_schema.json
@@ -511,6 +511,16 @@
             }
           ]
         },
+        "noConstAssign": {
+          "anyOf": [
+            {
+              "$ref": "#/definitions/RuleConfiguration"
+            },
+            {
+              "type": "null"
+            }
+          ]
+        },
         "noDangerouslySetInnerHtml": {
           "anyOf": [
             {

diff --git a/npm/backend-jsonrpc/src/workspace.ts b/npm/backend-jsonrpc/src/workspace.ts
diff --git a/rome.json b/rome.json
@@ -14,6 +14,7 @@
 				"noNewSymbol": "error",
 				"noVoidElementsWithChildren": "error",
 				"noDangerouslySetInnerHtmlWithChildren": "error",
+				"noConstAssign": "error",
 				"noAutofocus": "error",
 				"useKeyWithClickEvents": "error"
 			}

diff --git a/website/playground/src/romeWorker.ts b/website/playground/src/romeWorker.ts
@@ -122,6 +122,7 @@ self.addEventListener("message", async (e) => {
 							noUselessFragments: "error",
 							noVoidElementsWithChildren: "error",
 							noDangerouslySetInnerHtmlWithChildren: "error",
+							noConstAssign: "error",
 						},
 					},
 				};

diff --git a/website/src/docs/lint/rules/index.md b/website/src/docs/lint/rules/index.md