Move completion provider to standalone file

Author: wingrunr21

src/providers/completion.ts

@@ -0,0 +1,67 @@
+import * as vscode from 'vscode';
+import { ExtensionContext } from 'vscode';
+import * as cp from 'child_process';
+
+export function registerCompletionProvider(ctx: ExtensionContext) {
+	if (vscode.workspace.getConfiguration('ruby').codeCompletion == 'rcodetools') {
+		const completeCommand = function (args) {
+			let rctCompletePath = vscode.workspace.getConfiguration('ruby.rctComplete').get('commandPath', 'rct-complete');
+			args.push('--interpreter');
+			args.push(vscode.workspace.getConfiguration('ruby.interpreter').get('commandPath', 'ruby'));
+			if (process.platform === 'win32')
+				return cp.spawn('cmd', ['/c', rctCompletePath].concat(args));
+			return cp.spawn(rctCompletePath, args);
+		}
+
+		const completeTest = completeCommand(['--help']);
+		completeTest.on('exit', () => {
+			ctx.subscriptions.push(
+				vscode.languages.registerCompletionItemProvider(
+					/** selector */'ruby',
+					/** provider */{
+						provideCompletionItems: function completionProvider(document, position, token) {
+							return new Promise((resolve, reject) => {
+								const line = position.line + 1;
+								const column = position.character;
+								let child = completeCommand([
+									'--completion-class-info',
+									'--dev',
+									'--fork',
+									'--line=' + line,
+									'--column=' + column
+								]);
+								let outbuf = [],
+									errbuf = [];
+								child.stderr.on('data', (data) => errbuf.push(data));
+								child.stdout.on('data', (data) => outbuf.push(data));
+								child.stdout.on('end', () => {
+									if (errbuf.length > 0) return reject(Buffer.concat(errbuf).toString());
+									let completionItems = [];
+									Buffer.concat(outbuf).toString().split('\n').forEach(function (elem) {
+										let items = elem.split('\t');
+										if (/^[^\w]/.test(items[0])) return;
+										if (items[0].trim().length === 0) return;
+										let completionItem = new vscode.CompletionItem(items[0]);
+										completionItem.detail = items[1];
+										completionItem.documentation = items[1];
+										completionItem.filterText = items[0];
+										completionItem.insertText = items[0];
+										completionItem.label = items[0];
+										completionItem.kind = vscode.CompletionItemKind.Method;
+										completionItems.push(completionItem);
+									}, this);
+									if (completionItems.length === 0)
+										return reject([]);
+									return resolve(completionItems);
+								});
+								child.stdin.end(document.getText());
+							});
+						}
+					},
+					/** triggerCharacters */ ...['.']
+				)
+			)
+		});
+		completeTest.on('error', () => 0);
+	}
+}

src/ruby.ts

@@ -4,13 +4,13 @@ import * as vscode from 'vscode';
 import { Location, ExtensionContext, Position } from 'vscode';
 import * as Locate from './locate/locate';
 import * as path from 'path';
-import * as cp from 'child_process';
 import { LintCollection } from './lint/lintCollection';
 import * as utils from './utils';
 import { registerTaskProvider } from './task/rake';
 import { Config as LintConfig } from './lint/lintConfig';
 import * as debounce from 'lodash/debounce';
 
+import { registerCompletionProvider } from './providers/completion';
 import { registerFormatter } from './providers/formatter';
 import { registerHighlightProvider } from './providers/highlight';
 import { registerIntellisenseProvider } from './providers/intellisense';
@@ -82,67 +82,3 @@ function registerLinters(ctx: ExtensionContext) {
 	// run against all of the current open files
 	vscode.window.visibleTextEditors.forEach(executeLinting);
 }
-
-function registerCompletionProvider(ctx: ExtensionContext) {
-	if (vscode.workspace.getConfiguration('ruby').codeCompletion == 'rcodetools') {
-		const completeCommand = function (args) {
-			let rctCompletePath = vscode.workspace.getConfiguration('ruby.rctComplete').get('commandPath', 'rct-complete');
-			args.push('--interpreter');
-			args.push(vscode.workspace.getConfiguration('ruby.interpreter').get('commandPath', 'ruby'));
-			if (process.platform === 'win32')
-				return cp.spawn('cmd', ['/c', rctCompletePath].concat(args));
-			return cp.spawn(rctCompletePath, args);
-		}
-
-		const completeTest = completeCommand(['--help']);
-		completeTest.on('exit', () => {
-			ctx.subscriptions.push(
-				vscode.languages.registerCompletionItemProvider(
-					/** selector */'ruby',
-					/** provider */{
-						provideCompletionItems: function completionProvider(document, position, token) {
-							return new Promise((resolve, reject) => {
-								const line = position.line + 1;
-								const column = position.character;
-								let child = completeCommand([
-									'--completion-class-info',
-									'--dev',
-									'--fork',
-									'--line=' + line,
-									'--column=' + column
-								]);
-								let outbuf = [],
-									errbuf = [];
-								child.stderr.on('data', (data) => errbuf.push(data));
-								child.stdout.on('data', (data) => outbuf.push(data));
-								child.stdout.on('end', () => {
-									if (errbuf.length > 0) return reject(Buffer.concat(errbuf).toString());
-									let completionItems = [];
-									Buffer.concat(outbuf).toString().split('\n').forEach(function (elem) {
-										let items = elem.split('\t');
-										if (/^[^\w]/.test(items[0])) return;
-										if (items[0].trim().length === 0) return;
-										let completionItem = new vscode.CompletionItem(items[0]);
-										completionItem.detail = items[1];
-										completionItem.documentation = items[1];
-										completionItem.filterText = items[0];
-										completionItem.insertText = items[0];
-										completionItem.label = items[0];
-										completionItem.kind = vscode.CompletionItemKind.Method;
-										completionItems.push(completionItem);
-									}, this);
-									if (completionItems.length === 0)
-										return reject([]);
-									return resolve(completionItems);
-								});
-								child.stdin.end(document.getText());
-							});
-						}
-					},
-					/** triggerCharacters */ ...['.']
-				)
-			)
-		});
-		completeTest.on('error', () => 0);
-	}
-}