V3.8.6 (#190)

* better editor mgmt

* more doc mgmt refresh/close

* added release notes to pipeline

* 3.8.6

.github/workflows/main.yml

@@ -66,6 +66,9 @@ jobs:
 
       - name: get extension version
         run: echo "PACKAGE_VERSION=$(node -p "require('./package.json').version")" >> $GITHUB_ENV
+
+      - name: capture latest release notes
+        run: echo "RELEASE_NOTE=$(node src/scripts/changelog.js)" >> $GITHUB_ENV
 
       - name: create upload artifacts
         uses: actions/upload-artifact@v2
@@ -87,7 +90,7 @@ jobs:
         with:
           tag_name: v${{ env.PACKAGE_VERSION }}
           release_name: ${{ env.VSIX_NAME }}
-          body: See [CHANGE LOG](https://github.com/f5devcentral/vscode-f5-chariot/blob/main/README.md) for details.
+          body: ${{env.RELEASE_NOTE}} See [CHANGE LOG](https://github.com/f5devcentral/vscode-f5/blob/main/README.md) for full details.
           draft: false
           prerelease: false
 

package.json

@@ -3,7 +3,7 @@
 	"displayName": "The F5 Extension",
 	"description": "Supercharge your F5 automation development",
 	"publisher": "F5DevCentral",
-	"version": "3.8.5",
+	"version": "3.8.6",
 	"keywords": [
 		"F5",
 		"F5Networks",

src/cfgExploreCore.ts

@@ -186,14 +186,20 @@ export function cfgExplore(context: ExtensionContext) {
         // flip switch and refresh details
         if(cfgProvider.xcDiag){
             cfgProvider.xcDiag = false;
+            ext.xcDiag.enabled = false;
             console.log('xc diag updatediagnostics disable');
+            if(ext.xcDiag.lastDoc) {
+                // clear the last editor diags
+                ext.xcDiag.updateDiagnostic(ext.xcDiag.lastDoc);
+            }
         } else {
             cfgProvider.xcDiag = true;
 
             // was having errors about functions undefined, so, make sure everything is loaded as we turn this on
             // if (ext.xcDiag.updateDiagnostic === undefined) {
                 console.log('xc diag updatediagnostics enable');
                 ext.xcDiag = new XcDiag(context);
+                ext.xcDiag.enabled = true;
             // }
         }
         cfgProvider.refresh();
@@ -230,6 +236,7 @@ export function cfgExplore(context: ExtensionContext) {
             diagTag = true;
         }
 
+        // provide the text and "IF" xc diagnostics "CAN" be applied
         cfgProvider.render(text, diagTag);
     }));
 

src/scripts/changelog.js

@@ -0,0 +1,30 @@
+let fs = require('fs');
+const path = require('path');
+
+const args = process.argv.slice(2);
+
+// src/scripts/changelog.js
+const text = fs.readFileSync(path.join('CHANGELOG.md'), "utf-8");
+
+
+// trying to take arg for version...
+// if(args[0]) {
+//     const rx2 = new RegExp(`## \[${args[0]}` + "\].+([\\s\\w\\>\\'\\-\\/]+)", 'g');
+//     const rx3 = /## \[\d.\d.\d\].+([\s\w\>\'\-\/]+)(?=---)/;
+
+//     const b2 = rx2.exec(text);
+
+//     const b3 = b2;
+//     // const rx = /## \[3.8.5\].+([\s\w\>\'\-\/]+)/;
+// }
+
+// currently just returns the most recent or first match
+const rx = /## \[\d.\d.\d\].+([\s\w\>\'\-\/]+)(?=---)/;
+
+const body = rx.exec(text);
+
+return console.log(body[0]);
+
+// debugger;
+
+

src/tmosXcDiag.ts

@@ -13,7 +13,8 @@ import {
     Position,
     Range,
     TextDocument,
-    Uri
+    Uri,
+    workspace
 } from "vscode";
 
 import { logger } from "./logger";
@@ -30,8 +31,10 @@ export type xcDiagRule = {
 
 
 export class XcDiag {
+    public enabled: boolean = false;
+    public lastDoc: TextDocument | undefined = undefined;
 
-    diagXC: DiagnosticCollection;
+    public diagXC: DiagnosticCollection;
 
     settingsFileLocation: string;
     rules: xcDiagRule[];
@@ -45,6 +48,14 @@ export class XcDiag {
         this.settingsFileLocation = path.join(context.extensionPath, 'diagRules', 'tmosXcRules.json');
         this.rules = this.loadRules();
 
+        context.subscriptions.push(
+            workspace.onDidChangeTextDocument(e => this.updateDiagnostic(e.document))
+        );
+
+        context.subscriptions.push(
+            workspace.onDidCloseTextDocument(doc => this.diagXC.delete(doc.uri))
+        );
+
     }
 
     loadRules() {
@@ -217,14 +228,27 @@ export class XcDiag {
     }
 
     updateDiagnostic(doc: TextDocument) {
-        // clear current diags dispalyed
-        this.diagXC.clear();
 
-        // get the text from the doc/editor and feed through xc diagnostics
-        const diags = this.getDiagnostic(doc.getText());
+        if(doc.fileName === 'app.conf') {
+
+            // clear current diags in this class
+            this.diagXC.clear();
+            // clear the current diags in the doc/editor
+            this.diagXC.delete(doc.uri);
+
+            if(this.enabled) {
 
-        // pubish the diags to document
-        this.diagXC.set(doc.uri, diags);
+                // capture the doc we are working with
+                this.lastDoc = doc;
+
+                // get the text from the doc/editor and feed through xc diagnostics
+                const diags = this.getDiagnostic(doc.getText());
+
+                // pubish the diags to document
+                this.diagXC.set(doc.uri, diags);
+            }
+
+        }
     }
 }