Readd removed comments

apps/language-server-web-worker/project.json

@@ -51,6 +51,7 @@
       "dependsOn": ["build"],
       "options": {
         "commands": [
+          // Manually delete all dependencies in package.json because the "externalDependencies": "none" setting in "build" above seems to have no effect on the generated package.json
           "node tools/scripts/delete-dependencies.mjs language-server-web-worker",
           "node tools/scripts/add-package-json-version.mjs language-server-web-worker",
           "node tools/scripts/publish.mjs language-server-web-worker"

apps/vs-code-extension/project.json

@@ -29,6 +29,7 @@
       "configurations": {
         "dev": {
           "esbuildOptions": {
+            // Required, so VS Code's debugger is able to attach when running the extension
             "sourcemap": "linked"
           },
           "minify": false
@@ -58,7 +59,9 @@
       "dependsOn": ["build"],
       "options": {
         "commands": [
+          // Delete vscode dependency in package.json which was generated by Nx
           "node tools/scripts/delete-dependencies.mjs vs-code-extension",
+          // Add an empty license file, so vsce does not complain during packaging
           "touch dist/apps/vs-code-extension/LICENSE",
           "node tools/scripts/add-package-json-version.mjs vs-code-extension",
           "cd dist/apps/vs-code-extension && npx vsce package --out jayvee.vsix"

jest.preset.js

@@ -6,14 +6,5 @@ const nxPreset = require('@nx/jest/preset').default;
 
 module.exports = {
   ...nxPreset,
-  /* TODO: Update to latest Jest snapshotFormat
-   * By default Nx has kept the older style of Jest Snapshot formats
-   * to prevent breaking of any existing tests with snapshots.
-   * It's recommend you update to the latest format.
-   * You can do this by removing snapshotFormat property
-   * and running tests with --update-snapshot flag.
-   * Example: "nx affected --targets=test --update-snapshot"
-   * More info: https://jestjs.io/docs/upgrading-to-jest29#snapshot-format
-   */
   snapshotFormat: { escapeString: true, printBasicPrototype: true },
 };

libs/language-server/project.json

@@ -9,6 +9,7 @@
       "options": {
         "commands": [
           "langium generate -f libs/language-server/langium-config.json",
+          // Workaround until https://github.com/langium/langium/issues/740 is resolved:
           "node tools/scripts/fix-monarch-grammar-escape.mjs",
           "node tools/scripts/language-server/generate-stdlib.mjs"
         ],