@ansible/ansible-language-server - npm Package Compare versions

Comparing version 0.3.0 to 0.4.0

CHANGELOG.md

@@ -9,2 +9,11 @@ <!-- markdownlint-disable no-duplicate-heading -->
 
+## [0.4.0] - 2021-11-25
+
+### Bugfixes
+
+* Prevented throwing an unhandled exception caused by undefined linter
+  arguments settings (#142) @ssbarnea
+* Implemented opening standalone Ansible files that have no workspace
+  associated (#140) @ganeshrn
+
 ## [0.3.0] - 2021-11-18
@@ -11,0 +20,0 @@

out/server/src/services/ansibleLint.js

@@ -46,3 +46,3 @@ "use strict";
     doValidate(textDocument) {
-        var _a, _b;
+        var _a, _b, _c;
         return __awaiter(this, void 0, void 0, function* () {
@@ -58,5 +58,5 @@ let diagnostics = new Map();
             else {
-                let linterArguments = settings.ansibleLint.arguments;
+                let linterArguments = (_a = settings.ansibleLint.arguments) !== null && _a !== void 0 ? _a : '';
                 // Determine linter config file
-                let ansibleLintConfigPath = (_b = (_a = linterArguments.match(/(?:^|\s)-c\s*(?<sep>[\s'"])(?<conf>.+?)(?:\k<sep>|$)/)) === null || _a === void 0 ? void 0 : _a.groups) === null || _b === void 0 ? void 0 : _b.conf;
+                let ansibleLintConfigPath = (_c = (_b = linterArguments.match(/(?:^|\s)-c\s*(?<sep>[\s'"])(?<conf>.+?)(?:\k<sep>|$)/)) === null || _b === void 0 ? void 0 : _b.groups) === null || _c === void 0 ? void 0 : _c.conf;
                 if (!ansibleLintConfigPath) {
@@ -63,0 +63,0 @@ // Config file not provided in arguments -> search for one mimicking the

out/server/src/services/workspaceManager.js

@@ -21,2 +21,4 @@ "use strict";
 const settingsManager_1 = require("./settingsManager");
+const path = require("path");
+const vscode_uri_1 = require("vscode-uri");
 /**
@@ -66,2 +68,15 @@ * Holds the overall context for the whole workspace.
         }
+        /* *
+        * If control reaches at this point it indicates an individual file is
+        * opened in client without any workspace.
+        * Set the workspace to directory of the file pointed by uri.
+        */
+        const documentFolderPathParts = uri.split(path.sep);
+        documentFolderPathParts.pop();
+        const workspaceFolder = {
+            'uri': documentFolderPathParts.join(path.sep),
+            'name': documentFolderPathParts[documentFolderPathParts.length - 1]
+        };
+        this.connection.console.log(`workspace folder explicitly set to ${vscode_uri_1.URI.parse(workspaceFolder.uri).path}`);
+        return workspaceFolder;
     }
@@ -68,0 +83,0 @@ handleWorkspaceChanged(event) {

package.json

@@ -7,3 +7,3 @@ {
   "license": "MIT",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "contributors": [
@@ -69,2 +69,3 @@ {
     "mocha": "^8.4.0",
+    "nyc": "^15.1.0",
     "prettier": "^2.4.1",
@@ -78,6 +79,8 @@ "rimraf": "^3.0.2",
     "compile": "tsc -p .",
+    "coverage": "nyc report --reporter=text-lcov > out/coverage.lcov",
     "lint": "npm ci && pre-commit run -a",
     "prepack": "npm ci && npm run compile",
+    "preversion": "bin/version-bump-allowed",
     "watch": "tsc --watch -p .",
-    "test": "mocha --require ts-node/register './test/**/*.ts'",
+    "test": "nyc -a mocha --require ts-node/register './test/**/*.ts'",
     "check-dependencies": "node ./scripts/check-dependencies.js"
@@ -84,0 +87,0 @@ },

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

533378

increased by0.44%

Number of package files

120

increased by1.69%

Lines of code

5044

increased by0.3%

Worsened metrics

Dev dependency count

18

increased by5.88%

No dependency changes