feat(vscode): add problem matcher to highlight errors after build (#437)

Author: i582

client/src/build-system.ts

@@ -62,6 +62,13 @@ export class BlueprintTaskProvider implements TaskProviderBase {
             panel: vscode.TaskPanelKind.Dedicated,
             focus: true,
         }
+
+        const settings = vscode.workspace.getConfiguration("tact")
+        const useProblemMatcher = settings.get<boolean>("linters.useProblemMatcher") ?? false
+        if (useProblemMatcher) {
+            task.problemMatchers = ["$tact"]
+        }
+
         return task
     }
 }
@@ -118,6 +125,12 @@ export class TactTemplateTaskProvider implements TaskProviderBase {
             focus: true,
         }
 
+        const settings = vscode.workspace.getConfiguration("tact")
+        const useProblemMatcher = settings.get<boolean>("linters.useProblemMatcher") ?? false
+        if (useProblemMatcher) {
+            task.problemMatchers = ["$tact"]
+        }
+
         return task
     }
 }

package.json

@@ -121,7 +121,10 @@
                 "command": "tact.build",
                 "title": "Build Project",
                 "category": "Tact",
-                "icon": "$(gear)"
+                "icon": "$(gear)",
+                "problemMatcher": [
+                    "$tact"
+                ]
             },
             {
                 "command": "tact/getTypeAtPosition",
@@ -436,6 +439,11 @@
                         "default": false,
                         "description": "Enable Tact compiler analysis"
                     },
+                    "tact.linters.useProblemMatcher": {
+                        "type": "boolean",
+                        "default": false,
+                        "description": "Highlight errors in editor after build"
+                    },
                     "tact.linters.misti.enable": {
                         "type": "boolean",
                         "default": false,
@@ -486,6 +494,26 @@
                 "type": "tact-template-test",
                 "properties": {}
             }
+        ],
+        "problemMatchers": [
+            {
+                "name": "tact",
+                "owner": "tact",
+                "fileLocation": [
+                    "relative",
+                    "${workspaceFolder}"
+                ],
+                "pattern": [
+                    {
+                        "regexp": "^Error:\\s+(.+):(\\d+):(\\d+):\\s+(.+)$",
+                        "file": 1,
+                        "line": 2,
+                        "column": 3,
+                        "message": 4
+                    }
+                ],
+                "severity": "error"
+            }
         ]
     },
     "dependencies": {

server/src/utils/settings.ts

@@ -68,6 +68,7 @@ export interface TactSettings {
             enable: boolean
             binPath: string
         }
+        useProblemMatcher: boolean
     }
 }
 
@@ -137,6 +138,7 @@ const defaultSettings: TactSettings = {
             enable: false,
             binPath: "npx misti",
         },
+        useProblemMatcher: false,
     },
 }
 
@@ -241,6 +243,8 @@ function mergeSettings(vsSettings: Partial<TactSettings>): TactSettings {
                 enable: vsSettings.linters?.misti.enable ?? defaultSettings.linters.misti.enable,
                 binPath: vsSettings.linters?.misti.binPath ?? defaultSettings.linters.misti.binPath,
             },
+            useProblemMatcher:
+                vsSettings.linters?.useProblemMatcher ?? defaultSettings.linters.useProblemMatcher,
         },
     }
 }