Refactor "move to function in module scope" moves function past the following variable declaration

TypeScript Version: 3.1.0-dev.20180807

Search Terms: refactor function module

Code

import "fs";

function foo() {
	const bar = 1; // select this line
}

let flag = false;
function doSomethingWithFlag() {
}

1. select the line with the comment
2. choose “extract to function in module scope”

Expected behavior:

import "fs";

function newFunction() {
	return 1;
}

function foo() {
	const bar = newFunction();
}

let flag = false;
function doSomethingWithFlag() {
}

or

import "fs";

function foo() {
	const bar = newFunction();
}

function newFunction() {
	return 1;
}

let flag = false;
function doSomethingWithFlag() {
}

Actual behavior:

import "fs";

function foo() {
	const bar = newFunction();
}

let flag = false;
function newFunction() {
	return 1;
}

function doSomethingWithFlag() {
}

[jessetrinity]

@AlCalzone was there more context when you originally opened this issue that would help us out?

[AlCalzone]

For me it was just the placement of the function. The refactor works, but it seems unlogical where the function is placed. In the above example, newFunction and foo belong together and flag and doSomethingWithFlag logically belong together. After the refactor, this is not immediately clear, especially if there is more than one variable declaration between the functions.