v5.1.1 (#476)

* Remove nested validations

Submodule validates itself on every commit anyway

* Update prettierignore for ahk2

* Remove lint-staged

More trouble than it's worth :)

* Remove husky for realsies

* Update known issues

* bump to 5.1.1 for testing

* Launch with env var, fixup build

* Migrate to @vscode/test-cli

* Move to correct branch

* Fix global recognition

* Use extensions for snippets instead (specifically github PRs)

* Update changelog

.prettierignore

@@ -1,11 +1,38 @@
+# ahk2
+ahk2/client/src/browserClientMain.ts
+ahk2/client/src/extension.ts
+ahk2/client/src/test/extension.test.ts
+ahk2/client/src/test/formattingProvider.test.ts
+ahk2/client/src/test/utils.ts
+ahk2/server/src/ahkProvider.ts
+ahk2/server/src/browserServerMain.ts
+ahk2/server/src/codeActionProvider.ts
+ahk2/server/src/colorProvider.ts
+ahk2/server/src/commandProvider.ts
+ahk2/server/src/common.ts
+ahk2/server/src/completionProvider.ts
+ahk2/server/src/constants.ts
+ahk2/server/src/definitionProvider.ts
+ahk2/server/src/formattingProvider.ts
+ahk2/server/src/hoverProvider.ts
+ahk2/server/src/Lexer.ts
+ahk2/server/src/localize.ts
+ahk2/server/src/PEFile.ts
+ahk2/server/src/referencesProvider.ts
+ahk2/server/src/renameProvider.ts
+ahk2/server/src/scriptrunner.ts
+ahk2/server/src/semanticTokensProvider.ts
+ahk2/server/src/server.ts
+ahk2/server/src/signatureProvider.ts
+ahk2/server/src/symbolProvider.ts
+
 # Folders
 .idea/
 .@vscode/test-electron/
 .vscode-test/
 dist
 node_modules
 out
-ahk2
 
 # Files
 .vscode/settings.json
@@ -14,3 +41,4 @@ language/*.tmLanguage.json
 *.log
 *.txt
 *.vsix
+

.vscode-test.mjs

@@ -0,0 +1,5 @@
+// https://github.com/microsoft/vscode-test-cli
+import { defineConfig } from '@vscode/test-cli';
+export default defineConfig({
+    files: ['out/src/**/*.test.js'],
+});

.vscode/dev.code-snippets

@@ -35,10 +35,4 @@
         ],
         "description": "Generate tests for Mocha.",
     },
-    "Issue link (ahkpp)": {
-        "scope": "markdown",
-        "prefix": "[#",
-        "body": ["[#$1](https://github.com/mark-wiemer-org/ahkpp/issues/$1)"],
-        "description": "Link an ahkpp issue",
-    },
 }

.vscode/extensions.json

@@ -0,0 +1,13 @@
+{
+    "recommendations": [
+        "aaron-bond.better-comments",
+        "dbaeumer.vscode-eslint",
+        "github.copilot",
+        "github.copilot-chat",
+        "github.vscode-pull-request-github",
+        "eamodio.gitlens",
+        "esbenp.prettier-vscode",
+        "simonsiefke.svg-preview",
+        "meganrogge.template-string-converter"
+    ]
+}

.vscode/launch.json

@@ -11,7 +11,11 @@
             "request": "launch",
             "args": ["--extensionDevelopmentPath=${workspaceRoot}"],
             "outFiles": ["${workspaceRoot}/out/**/*.js"],
-            "preLaunchTask": "npm: build:dev"
+            // "preLaunchTask": "npm: build:dev",
+            "env": {
+                "VSCODE_AHK_SERVER_PATH": "", // default path works for debug as well
+                "SYNTAXES_PATH": "" // default path works for debug as well
+            }
         },
         {
             "name": "Test Extension",

Changelog.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 5.1.1 - 2024-08-27 🧪
+
+> 🧪 means this is a [pre-release](https://code.visualstudio.com/updates/v1_63#_pre-release-extensions)!
+
+-   Fix global function recognition ([#472](https://github.com/mark-wiemer-org/ahkpp/issues/472))
+
 ## 5.1.0 - 2024-08-22 🧪
 
 > 🧪 means this is a [pre-release](https://code.visualstudio.com/updates/v1_63#_pre-release-extensions)!

docs/FullV2Integration.md

@@ -129,15 +129,13 @@ New, added:
 These issues will be resolved before a full release
 
 -   Several [commands](#commands) are duplicated. The plan is to retain the current visual style.
--   Globals like `MsgBox` and `ExitApp` are not recognized in any file, e.g. [compile.ahk2](../demos/manualTests/compile.ahk2)
--   `common.ts#loadahk2` needs work
 -   Needs more tests!! Many features have not been tested at all, hence the pre-release!
 -   [Release pipeline](../.github/workflows/deploy.yml) has been changed to push pre-releases. Definitely fix this for the final release 😉
+-   Settings description name placeholders are not replaced
 
 ### Low priority
 
 These issues will be backlogged and resolved after a full release
 
 -   App is bloated, including source map files that can be removed. See [ahk2/webpack.config.js](../ahk2/webpack.config.js)
 -   No newline at end of file when formatting (inconsistent with v1 and industry standards)
--   Format check is [failing in the pipeline](https://github.com/mark-wiemer-org/ahkpp/actions/runs/10503320004/job/29096451649?pr=467)