forked from DonJayamanne/pythonVSCode
-
Notifications
You must be signed in to change notification settings - Fork 1
Expand file tree
/
Copy pathimportSortProvider.ts
More file actions
57 lines (56 loc) · 2.77 KB
/
importSortProvider.ts
File metadata and controls
57 lines (56 loc) · 2.77 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
'use strict';
import * as child_process from 'child_process';
import * as fs from 'fs';
import * as path from 'path';
import * as vscode from 'vscode';
import { PythonSettings } from '../common/configSettings';
import { getTempFileWithDocumentContents, getTextEditsFromPatch } from '../common/editor';
import { captureTelemetry } from '../telemetry';
import { FORMAT_SORT_IMPORTS } from '../telemetry/constants';
// tslint:disable-next-line:completed-docs
export class PythonImportSortProvider {
@captureTelemetry(FORMAT_SORT_IMPORTS)
public async sortImports(extensionDir: string, document: vscode.TextDocument): Promise<vscode.TextEdit[]> {
if (document.lineCount === 1) {
return [];
}
// isort does have the ability to read from the process input stream and return the formatted code out of the output stream.
// However they don't support returning the diff of the formatted text when reading data from the input stream.
// Yes getting text formatted that way avoids having to create a temporary file, however the diffing will have
// to be done here in node (extension), i.e. extension cpu, i.e. less responsive solution.
const importScript = path.join(extensionDir, 'pythonFiles', 'sortImports.py');
const tmpFileCreated = document.isDirty;
const filePath = tmpFileCreated ? await getTempFileWithDocumentContents(document) : document.fileName;
const settings = PythonSettings.getInstance(document.uri);
const pythonPath = settings.pythonPath;
const isort = settings.sortImports.path;
const args = settings.sortImports.args.join(' ');
let isortCmd = '';
if (typeof isort === 'string' && isort.length > 0) {
if (isort.indexOf(' ') > 0) {
isortCmd = `"${isort}" "${filePath}" --diff ${args}`;
} else {
isortCmd = `${isort} "${filePath}" --diff ${args}`;
}
} else {
if (pythonPath.indexOf(' ') > 0) {
isortCmd = `"${pythonPath}" "${importScript}" "${filePath}" --diff ${args}`;
} else {
isortCmd = `${pythonPath} "${importScript}" "${filePath}" --diff ${args}`;
}
}
// tslint:disable-next-line:promise-must-complete
return await new Promise<vscode.TextEdit[]>((resolve, reject) => {
child_process.exec(isortCmd, (error, stdout, stderr) => {
if (tmpFileCreated) {
fs.unlink(filePath);
}
if (error || (stderr && stderr.length > 0)) {
reject(error ? error : stderr);
} else {
resolve(getTextEditsFromPatch(document.getText(), stdout));
}
});
});
}
}