fix local renaming bug

Author: kammeows

client/modules/IDE/components/jump-to-definition.js

@@ -9,8 +9,7 @@ export function jumpToDefinition(pos) {
   const token = cm.getTokenAt(pos);
   const tokenType = token.type;
 
-  if (!tokenType || !['variable', 'def'].some((t) => tokenType.includes(t)))
-    return;
+  if (!tokenType || tokenType === 'def') return;
 
   const varName = token.string;
   const ast = getAST(cm);

client/modules/IDE/components/rename-variable.js

@@ -108,7 +108,10 @@ function startRenaming(cm, ast, fromPos, newName, oldName) {
         (!isShadowedLocally &&
           thisScopeVars.hasOwnProperty(oldName) === {} &&
           baseContext === 'global') ||
-        (baseContext === 'global' && !thisScopeVars.hasOwnProperty(oldName));
+        (baseContext === 'global' && !thisScopeVars.hasOwnProperty(oldName)) ||
+        (isDeclaredGlobally &&
+          !thisScopeVars.hasOwnProperty(oldName) &&
+          !isDeclaredInBaseScope);
 
       const shouldRenameGlobalVar =
         thisContext === 'global' && !isDeclaredInBaseScope;