Improve the linting diagnostics so that the quick fix is more easily …

…triggered.

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to the "sqlfluff" extension will be documented in this file.
 
 Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how to structure this file.
 
+## [0.0.9] - 2022-09-08
+
+- Add the quick fix option to view the SQLFluff documention on the SQLFluff website.
+- Fix the diagnostics so the quick fix can be triggered by any location on the problem word.
+
 ## [0.0.8] - 2022-09-08
 
 - This release rewrites some of the extension code, but should leave the functionality the same.

package.json

@@ -1,7 +1,7 @@
 {
  "name": "vscode-sqlfluff",
  "displayName": "sqlfluff",
- "version": "0.0.8",
+ "version": "0.0.9",
  "description": "A linter and auto-formatter for SQLfluff, a popular linting tool for SQL and dbt.",
  "publisher": "dorzey",
  "icon": "images/icon.png",

src/features/linter.ts

@@ -33,12 +33,20 @@ export class LinterProvider implements Linter {
  if (filePaths) {
  filePaths.forEach((filePath: FilePath) => {
  filePath.violations.forEach((violation: Violation) => {
- const range = new Range(
+ const path = filePath.filepath;
+ const editorPath = normalize(vscode.window.activeTextEditor.document.uri.fsPath);
+ const violationPosition = new vscode.Position(violation.line_no - 1, violation.line_pos - 1);
+ let range = new Range(
  violation.line_no - 1,
- violation.line_pos,
+ violation.line_pos - 1,
  violation.line_no - 1,
- violation.line_pos
+ violation.line_pos - 1
  );
+
+ if (editorPath.includes(path)) {
+ range = vscode.window.activeTextEditor.document.getWordRangeAtPosition(violationPosition) || range;
+ }
+
  const diagnostic = new Diagnostic(
  range,
  violation.description,
@@ -47,11 +55,6 @@ export class LinterProvider implements Linter {
  diagnostic.code = violation.code;
  diagnostic.source = "sqlfluff";
  diagnostics.push(diagnostic);
-
- const hover = new vscode.Hover(
- "[rule L007](https://docs.sqlfluff.com/en/stable/rules.html#sqlfluff.rules.Rule_L007)",
- range
- );
  });
  });
  }
@@ -62,6 +65,7 @@ export class LinterProvider implements Linter {
 }
 
 interface FilePath {
+ filepath: string;
  violations: Array<Violation>;
 }