From e6bfce193c7f5e80421835003a40748df5a53634 Mon Sep 17 00:00:00 2001
From: Andrew Casey <andrew.casey@microsoft.com>
Date: Wed, 27 Sep 2017 10:40:12 -0700
Subject: [PATCH] Add additional TODO about insertion positions

---
 src/services/refactors/extractSymbol.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/services/refactors/extractSymbol.ts b/src/services/refactors/extractSymbol.ts
index 517fd558569..75b21e809ed 100644
--- a/src/services/refactors/extractSymbol.ts
+++ b/src/services/refactors/extractSymbol.ts
@@ -1110,6 +1110,7 @@ namespace ts.refactor.extractSymbol {
     }
 
     // TODO (acasey): need to dig into nested statements
+    // TODO (acasey): don't insert before pinned comments, directives, or triple-slash references
     function getNodeToInsertConstantBefore(maxPos: number, scope: Scope): Node {
         const children = getStatementsOrClassElements(scope);
         Debug.assert(children.length > 0); // There must be at least one child, since we extracted from one.
@@ -1126,6 +1127,7 @@ namespace ts.refactor.extractSymbol {
             }
         }
 
+        Debug.assert(prevChild !== undefined);
         return prevChild;
     }