Add rule to prevent setting context from outside the ContextManager class

Author: EhabY

.eslintrc.json

@@ -23,17 +23,6 @@
 		"import/internal-regex": "^@/"
 	},
 	"overrides": [
-		{
-			"files": ["test/**/*.{ts,tsx}", "**/*.{test,spec}.ts?(x)"],
-			"settings": {
-				"import/resolver": {
-					"typescript": {
-						// In tests, resolve using the test tsconfig
-						"project": "test/tsconfig.json"
-					}
-				}
-			}
-		},
 		{
 			"files": ["*.ts"],
 			"rules": {
@@ -53,6 +42,23 @@
 				]
 			}
 		},
+		{
+			"files": ["test/**/*.{ts,tsx}", "**/*.{test,spec}.ts?(x)"],
+			"settings": {
+				"import/resolver": {
+					"typescript": {
+						// In tests, resolve using the test tsconfig
+						"project": "test/tsconfig.json"
+					}
+				}
+			}
+		},
+		{
+			"files": ["src/core/contextManager.ts"],
+			"rules": {
+				"no-restricted-syntax": "off"
+			}
+		},
 		{
 			"extends": ["plugin:package-json/legacy-recommended"],
 			"files": ["*.json"],
@@ -110,6 +116,13 @@
 					"sublings_only": true
 				}
 			}
+		],
+		"no-restricted-syntax": [
+			"error",
+			{
+				"selector": "CallExpression[callee.property.name='executeCommand'][arguments.0.value='setContext'][arguments.length>=3]",
+				"message": "Do not use executeCommand('setContext', ...) directly. Use the ContextManager class instead."
+			}
 		]
 	}
 }