feat: add LSP support for TypeScript/JavaScript via vtsls (desktop only)

CLAUDE.md

@@ -14,6 +14,9 @@
 - No trailing whitespace.
 - Use `const` and `let` instead of `var`.
 
+## Build artifacts — do not hand-edit
+- **`src/cacheManifest.json`** is a generated build artifact (gitignored, produced by `gulpfile.js/index.js`). It lists files + hashes for the service-worker cache. Never hand-edit or commit it — it is regenerated by the build, so edits are overwritten and won't be tracked anyway. When you add/remove/rename source files, just let the build regenerate it.
+
 ## Translations / i18n
 - All user-visible strings must go in `src/nls/root/strings.js` — never hardcode English in source files.
 - Use `const Strings = require("strings");` then `Strings.KEY_NAME`.

gulpfile.js/index.js

@@ -909,7 +909,7 @@ function _renameExtensionConcatAsExtensionJSInDist(extensionName) {
 }
 
 const minifyableExtensions = ["CloseOthers", "CodeFolding", "DebugCommands", "Git",
-    "HealthData", "JavaScriptCodeHints", "JavaScriptRefactoring", "QuickView"];
+    "HealthData", "JavaScriptCodeHints", "JavaScriptRefactoring", "QuickView", "TypeScriptSupport"];
 // extensions that nned not be minified either coz they are single file extensions or some other reason.
 const nonMinifyExtensions = ["CSSAtRuleCodeHints", "CSSCodeHints",
     "CSSPseudoSelectorHints", "DarkTheme", "HandlebarsSupport", "HTMLCodeHints", "HtmlEntityCodeHints",

gulpfile.js/thirdparty-lib-copy.js

@@ -266,7 +266,10 @@ let copyThirdPartyLibs = series(
     renameFile.bind(renameFile, 'node_modules/@xterm/addon-webgl/lib/addon-webgl.js.map',
         'addon-webgl.js.map', 'src/thirdparty/xterm'),
     copyLicence.bind(copyLicence, 'node_modules/@xterm/xterm/LICENSE', 'xterm')
-
+    // vtsls language server (bundled in src-node is not installed in pipline tests as the destop app building
+    // does it for desktop LSP). we ran it once and copied the license here.
+    // copyLicence.bind(copyLicence, 'src-node/node_modules/@vtsls/language-server/LICENSE', 'vtsls'),
+    // copyLicence.bind(copyLicence, 'src-node/node_modules/typescript/LICENSE.txt', 'typescript')
 );
 
 /**

gulpfile.js/validate-build.js

@@ -35,9 +35,12 @@ const LARGE_FILE_LIST_DEV = {
 // Size limits for production/staging builds (in MB)
 const PROD_MAX_FILE_SIZE_MB = 2;
 const PROD_MAX_TOTAL_SIZE_MB = 80;
-// Custom size limits for known large files (size in MB) For staging/production builds
+// Custom size limits for known large files (size in MB) For staging/production builds.
+// Margin policy: keep ~1 MB of headroom over a file's current size for individual files (and ~5 MB
+// for the aggregate/total). Bump a limit only enough to restore that margin when a file legitimately
+// grows - don't pad it large, so unexpected size jumps still get caught.
 const LARGE_FILE_LIST_PROD = {
-    'dist/brackets.js': 10, // this is the full minified file itself renamed in prod
+    'dist/brackets.js': 11, // this is the full minified file itself renamed in prod (~10 MB + 1 MB margin)
     'dist/phoenix/virtualfs.js.map': 3
 };
 

src-node/index.js

@@ -70,6 +70,9 @@ require("./test-connection");
 require("./utils");
 require("./terminal");
 require("./git/cli");
+// Note: "./lsp-client" is intentionally NOT required here. It is lazy-loaded on demand the
+// first time the desktop LSP client is used (via NodeUtils._loadNodeExtensionModule), so the
+// LSP framework adds nothing to node boot time. See src/languageTools/LSPClient.js.
 require("./claude-code-agent");
 function randomNonce(byteLength) {
     const randomBuffer = new Uint8Array(byteLength);