fix: fixes #13, and add more cases (#14)

* fix: fixes #13, and add more cases

* chore: update comment

src/operationUtils.ts

@@ -136,7 +136,7 @@ export function removeRange(range: number[]): Operation {
  * @returns {string} The text of the node
  */
 export function getText(node: Node, source: string): string {
-  const start = node.range[0]
-  const end = node.range[1]
+  const start = node?.range[0]
+  const end = node?.range[1]
   return source.slice(start, end)
 }

vue-transformations/__test__/v-bind-sync.spec.ts

@@ -1,7 +1,7 @@
 import { runTest } from '../../src/testUtils'
 
 runTest(
-  'Convert usage of slot-scope before vue 2.6',
+  'Replace .sync modifiers in v-bind with v-model',
   'v-bind-sync',
   'v-bind-sync',
   'vue',

vue-transformations/__testfixtures__/v-bind-sync/v-bind-sync.input.vue

@@ -1,5 +1,11 @@
 <template>
   <div class="hello">
     <ChildComponent :title.sync="pageTitle" />
+    <ChildComponent v-bind:propName.sync="foo" />
+    <ChildComponent v-bind:[dynamiArg].sync="foo" />
+    <ChildComponent :propName.sync="foo" />
+
+    <!-- incorrect usage, not transform -->
+    <ChildComponent v-bind.sync="foo" />
   </div>
 </template>

vue-transformations/__testfixtures__/v-bind-sync/v-bind-sync.output.vue

@@ -1,5 +1,11 @@
 <template>
   <div class="hello">
     <ChildComponent v-model:title="pageTitle" />
+    <ChildComponent v-model:propName="foo" />
+    <ChildComponent v-model:[dynamiArg]="foo" />
+    <ChildComponent v-model:propName="foo" />
+
+    <!-- incorrect usage, not transform -->
+    <ChildComponent v-bind.sync="foo" />
   </div>
 </template>

vue-transformations/v-bind-sync.ts

@@ -18,10 +18,10 @@ export const transformAST: VueASTTransformation = context => {
 
 export default wrap(transformAST)
 /**
- * search slot attribute nodes
+ * search v-bind attribute nodes
  *
  * @param context
- * @returns slot attribute nodes
+ * @returns v-bind attribute nodes
  */
 function findNodes(context: any): Node[] {
   const { file } = context