fixed types

src/index.ts

@@ -9,6 +9,7 @@ import { isString, findNodes, isFunction, isShiftNode, isStatement, copy, isArra
 import { SelectorOrNode, Replacer, RefactorError } from './types';
 import { RefactorCommonPlugin } from "./refactor-plugin-common";
 import { RefactorPlugin } from './refactor-plugin';
+import { RefactorUnsafePlugin } from './refactor-plugin-unsafe';
 
 const debug = DEBUG('shift-refactor');
 
@@ -34,6 +35,7 @@ export class RefactorSession {
 
     this._rebuildParentMap(); 
     this.use(RefactorCommonPlugin);
+    this.use(RefactorUnsafePlugin);
   }
 
   use<T extends RefactorPlugin>(Plugin: new(session:RefactorSession) => T) {

src/refactor-plugin-common.ts

@@ -1,11 +1,11 @@
 import debug from "debug";
-import { ClassDeclaration, ComputedMemberAssignmentTarget, ComputedMemberExpression, ComputedPropertyName, FunctionDeclaration, IdentifierExpression, LiteralBooleanExpression, LiteralStringExpression, StaticMemberAssignmentTarget, StaticMemberExpression, StaticPropertyName, VariableDeclarator } from "shift-ast";
-import { Declaration, Reference, Variable } from "shift-scope";
+import { ComputedMemberAssignmentTarget, ComputedMemberExpression, ComputedPropertyName, IdentifierExpression, LiteralBooleanExpression, LiteralStringExpression, StaticMemberAssignmentTarget, StaticMemberExpression, StaticPropertyName, VariableDeclarator } from "shift-ast";
+import { Declaration, Reference } from "shift-scope";
 import { default as isValid } from 'shift-validator';
 import { IdGenerator, MemorableIdGenerator } from "./id-generator";
 import { RefactorPlugin } from "./refactor-plugin";
 import { SelectorOrNode } from "./types";
-import { findNodes, isLiteral, isStatement, renameScope } from "./util";
+import { findNodes, renameScope } from "./util";
 
 declare module "." {
   interface RefactorSession {
@@ -14,7 +14,6 @@ declare module "." {
 }
 
 export class RefactorCommonPlugin extends RefactorPlugin {
-  name = "common";
 
   register() {
     this.session.common = this;
@@ -112,64 +111,4 @@ export class RefactorCommonPlugin extends RefactorPlugin {
     return this;
   }
 
-  massRename(namePairs: string[][]) {
-    namePairs.forEach(([from, to]) => {
-      this.session.lookupVariableByName(from).forEach((lookup: Variable) => this.session._renameInPlace(lookup, to));
-    });
-  }
-
-  removeDeadVariables() {
-    this.session.query('VariableDeclarator, FunctionDeclaration, ClassDeclaration').forEach(
-      (decl: VariableDeclarator | FunctionDeclaration | ClassDeclaration) => {
-        let name = decl instanceof VariableDeclarator ? decl.binding : decl.name;
-        const lookup = this.session.lookupVariable(name);
-
-        const reads = lookup.references.filter((ref: Reference) => {
-          const isRead = ref.accessibility.isRead;
-          const isBoth = ref.accessibility.isReadWrite;
-          if (isBoth) {
-            // if we're an UpdateExpression
-            const immediateParent = this.session.findParent(ref.node);
-            if (!immediateParent) return false;
-            const nextParent = this.session.findParent(immediateParent);
-            if (isStatement(nextParent)) return false;
-            else return true;
-          } else {
-            return isRead;
-          }
-        });
-
-        if (reads.length === 0) {
-          lookup.references.forEach((ref: Reference) => {
-            const node = ref.node;
-            const immediateParent = this.session.findParent(node);
-            if (!immediateParent) return;
-            const contextualParent = this.session.findParent(immediateParent);
-
-            if (['VariableDeclarator', 'FunctionDeclaration', 'ClassDeclaration'].indexOf(immediateParent.type) > -1) {
-              this.session.delete(immediateParent);
-            } else if (immediateParent.type === 'UpdateExpression' && isStatement(contextualParent)) {
-              this.session.delete(contextualParent);
-            } else if (node.type === 'AssignmentTargetIdentifier') {
-              if (immediateParent.type === 'AssignmentExpression') {
-                if (isLiteral(immediateParent.expression)) {
-                  if (isStatement(contextualParent)) {
-                    this.session.delete(contextualParent);
-                  } else {
-                    this.session.replace(immediateParent, immediateParent.expression);
-                  }
-                } else {
-                  this.session.replace(immediateParent, immediateParent.expression);
-                }
-              }
-            }
-          });
-          this.session.delete(decl);
-        }
-      },
-    );
-    if (this.session.autoCleanup) this.session.cleanup();
-    return this;
-  }
-
 }

src/refactor-plugin.ts

@@ -1,7 +1,6 @@
 import { RefactorSession } from ".";
 
 export abstract class RefactorPlugin {
-  abstract name: string;
   session: RefactorSession;
 
   abstract register(): void;