-
-
Notifications
You must be signed in to change notification settings - Fork 39
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Switch to Yarn v2 (Berry) - Setup Yarn (v2) workspaces - Eject webview and upgrade to webpack 5 - Slightly update main webpack config - Update IDE-related files/settings
- Loading branch information
1 parent
4eef87a
commit 73f890e
Showing
32 changed files
with
11,443 additions
and
12,930 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
/.yarn/releases/** binary | ||
/.yarn/plugins/** binary |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -33,27 +33,20 @@ jobs: | |
uses: actions/setup-node@v1 | ||
with: | ||
node-version: 12.x | ||
- name: Install VSCE | ||
run: | | ||
yarn global add vsce | ||
echo "$(yarn global bin)" >> $GITHUB_PATH | ||
- name: Get Yarn cache directory | ||
id: yarn-cache | ||
run: echo "::set-output name=dir::$(yarn cache dir)" | ||
run: echo "::set-output name=dir::$(yarn config get cacheFolder)" | ||
- name: Yarn cache | ||
uses: actions/[email protected] | ||
with: | ||
path: ${{ steps.yarn-cache.outputs.dir }} | ||
key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }} | ||
restore-keys: | | ||
${{ runner.os }}-yarn- | ||
- name: Install dependencies in / | ||
run: yarn --frozen-lockfile | ||
- name: Install dependencies in /webview/ | ||
working-directory: webview | ||
run: yarn --frozen-lockfile | ||
- name: Install dependencies | ||
run: yarn --immutable | ||
- name: Build extension | ||
run: vsce package -o ${{ steps.utils.outputs.artifact_name }} | ||
run: yarn dlx vsce package -o ${{ steps.utils.outputs.artifact_name }} --yarn | ||
- name: Upload a Build Artifact | ||
uses: actions/[email protected] | ||
with: | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,18 @@ | ||
|
||
# Build output | ||
dist | ||
util | ||
node_modules | ||
|
||
# Build artifacts | ||
*.vsix | ||
stats.json | ||
stats.json | ||
|
||
# Yarn | ||
.yarn/* | ||
!.yarn/patches | ||
!.yarn/releases | ||
!.yarn/plugins | ||
!.yarn/sdks | ||
!.yarn/versions | ||
!.yarn/yarn.lock | ||
.pnp.* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,8 @@ | ||
{ | ||
// See https://go.microsoft.com/fwlink/?LinkId=827846 | ||
// for the documentation about the extensions.json format | ||
"recommendations": [ | ||
"eamodio.tsl-problem-matcher" | ||
] | ||
} | ||
// See https://go.microsoft.com/fwlink/?LinkId=827846 | ||
// for the documentation about the extensions.json format | ||
"recommendations": [ | ||
"eamodio.tsl-problem-matcher", | ||
"arcanis.vscode-zipfs" | ||
] | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1,26 @@ | ||
{ | ||
"typescript.tsdk": "node_modules\\typescript\\lib", | ||
"files.exclude": { | ||
"*.vsix": true, | ||
"**/*.lock": true, | ||
"**/node_modules/": true, | ||
"dist/": true, | ||
"util/": true, | ||
"webview/build/": true | ||
} | ||
} | ||
"typescript.tsdk": ".yarn/sdks/typescript/lib", | ||
"files.exclude": { | ||
"*.vsix": true, | ||
"**/*.lock": true, | ||
"**/node_modules/": true, | ||
"dist/": true, | ||
"util/": true, | ||
"webview/build/": true, | ||
".yarn/": true, | ||
".yarnrc.yml": true, | ||
".pnp.*": true, | ||
"LICENSE.txt": true, | ||
"**/.*ignore": true, | ||
"**/.gitattributes": true, | ||
"**/.eslintcache": true, | ||
"**/webpack.config.js": true, | ||
"**/tslint.json": true, | ||
"**/tsconfig.json": true | ||
}, | ||
"search.exclude": { | ||
"**/.yarn": true, | ||
"**/.pnp.*": true | ||
}, | ||
"typescript.enablePromptUseWorkspaceTsdk": true | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# This file is automatically generated by @yarnpkg/sdks. | ||
# Manual changes might be lost! | ||
|
||
integrations: | ||
- vscode |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
#!/usr/bin/env node | ||
|
||
const {existsSync} = require(`fs`); | ||
const {createRequire, createRequireFromPath} = require(`module`); | ||
const {resolve} = require(`path`); | ||
|
||
const relPnpApiPath = "../../../../.pnp.cjs"; | ||
|
||
const absPnpApiPath = resolve(__dirname, relPnpApiPath); | ||
const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath); | ||
|
||
if (existsSync(absPnpApiPath)) { | ||
if (!process.versions.pnp) { | ||
// Setup the environment to be able to require typescript/bin/tsc | ||
require(absPnpApiPath).setup(); | ||
} | ||
} | ||
|
||
// Defer to the real typescript/bin/tsc your application uses | ||
module.exports = absRequire(`typescript/bin/tsc`); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
#!/usr/bin/env node | ||
|
||
const {existsSync} = require(`fs`); | ||
const {createRequire, createRequireFromPath} = require(`module`); | ||
const {resolve} = require(`path`); | ||
|
||
const relPnpApiPath = "../../../../.pnp.cjs"; | ||
|
||
const absPnpApiPath = resolve(__dirname, relPnpApiPath); | ||
const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath); | ||
|
||
if (existsSync(absPnpApiPath)) { | ||
if (!process.versions.pnp) { | ||
// Setup the environment to be able to require typescript/bin/tsserver | ||
require(absPnpApiPath).setup(); | ||
} | ||
} | ||
|
||
// Defer to the real typescript/bin/tsserver your application uses | ||
module.exports = absRequire(`typescript/bin/tsserver`); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
#!/usr/bin/env node | ||
|
||
const {existsSync} = require(`fs`); | ||
const {createRequire, createRequireFromPath} = require(`module`); | ||
const {resolve} = require(`path`); | ||
|
||
const relPnpApiPath = "../../../../.pnp.cjs"; | ||
|
||
const absPnpApiPath = resolve(__dirname, relPnpApiPath); | ||
const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath); | ||
|
||
if (existsSync(absPnpApiPath)) { | ||
if (!process.versions.pnp) { | ||
// Setup the environment to be able to require typescript/lib/tsc.js | ||
require(absPnpApiPath).setup(); | ||
} | ||
} | ||
|
||
// Defer to the real typescript/lib/tsc.js your application uses | ||
module.exports = absRequire(`typescript/lib/tsc.js`); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,157 @@ | ||
#!/usr/bin/env node | ||
|
||
const {existsSync} = require(`fs`); | ||
const {createRequire, createRequireFromPath} = require(`module`); | ||
const {resolve} = require(`path`); | ||
|
||
const relPnpApiPath = "../../../../.pnp.cjs"; | ||
|
||
const absPnpApiPath = resolve(__dirname, relPnpApiPath); | ||
const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath); | ||
|
||
const moduleWrapper = tsserver => { | ||
if (!process.versions.pnp) { | ||
return tsserver; | ||
} | ||
|
||
const {isAbsolute} = require(`path`); | ||
const pnpApi = require(`pnpapi`); | ||
|
||
const isVirtual = str => str.match(/\/(\$\$virtual|__virtual__)\//); | ||
const normalize = str => str.replace(/\\/g, `/`).replace(/^\/?/, `/`); | ||
|
||
const dependencyTreeRoots = new Set(pnpApi.getDependencyTreeRoots().map(locator => { | ||
return `${locator.name}@${locator.reference}`; | ||
})); | ||
|
||
// VSCode sends the zip paths to TS using the "zip://" prefix, that TS | ||
// doesn't understand. This layer makes sure to remove the protocol | ||
// before forwarding it to TS, and to add it back on all returned paths. | ||
|
||
function toEditorPath(str) { | ||
// We add the `zip:` prefix to both `.zip/` paths and virtual paths | ||
if (isAbsolute(str) && !str.match(/^\^zip:/) && (str.match(/\.zip\//) || isVirtual(str))) { | ||
// We also take the opportunity to turn virtual paths into physical ones; | ||
// this makes it much easier to work with workspaces that list peer | ||
// dependencies, since otherwise Ctrl+Click would bring us to the virtual | ||
// file instances instead of the real ones. | ||
// | ||
// We only do this to modules owned by the the dependency tree roots. | ||
// This avoids breaking the resolution when jumping inside a vendor | ||
// with peer dep (otherwise jumping into react-dom would show resolution | ||
// errors on react). | ||
// | ||
const resolved = isVirtual(str) ? pnpApi.resolveVirtual(str) : str; | ||
if (resolved) { | ||
const locator = pnpApi.findPackageLocator(resolved); | ||
if (locator && dependencyTreeRoots.has(`${locator.name}@${locator.reference}`)) { | ||
str = resolved; | ||
} | ||
} | ||
|
||
str = normalize(str); | ||
|
||
if (str.match(/\.zip\//)) { | ||
switch (hostInfo) { | ||
// Absolute VSCode `Uri.fsPath`s need to start with a slash. | ||
// VSCode only adds it automatically for supported schemes, | ||
// so we have to do it manually for the `zip` scheme. | ||
// The path needs to start with a caret otherwise VSCode doesn't handle the protocol | ||
// | ||
// Ref: https://github.com/microsoft/vscode/issues/105014#issuecomment-686760910 | ||
// | ||
case `vscode`: { | ||
str = `^zip:${str}`; | ||
} break; | ||
|
||
// To make "go to definition" work, | ||
// We have to resolve the actual file system path from virtual path | ||
// and convert scheme to supported by [vim-rzip](https://github.com/lbrayner/vim-rzip) | ||
case `coc-nvim`: { | ||
str = normalize(resolved).replace(/\.zip\//, `.zip::`); | ||
str = resolve(`zipfile:${str}`); | ||
} break; | ||
|
||
// Support neovim native LSP and [typescript-language-server](https://github.com/theia-ide/typescript-language-server) | ||
// We have to resolve the actual file system path from virtual path, | ||
// everything else is up to neovim | ||
case `neovim`: { | ||
str = normalize(resolved).replace(/\.zip\//, `.zip::`); | ||
str = `zipfile:${str}`; | ||
} break; | ||
|
||
default: { | ||
str = `zip:${str}`; | ||
} break; | ||
} | ||
} | ||
} | ||
|
||
return str; | ||
} | ||
|
||
function fromEditorPath(str) { | ||
return process.platform === `win32` | ||
? str.replace(/^\^?zip:\//, ``) | ||
: str.replace(/^\^?zip:/, ``); | ||
} | ||
|
||
// Force enable 'allowLocalPluginLoads' | ||
// TypeScript tries to resolve plugins using a path relative to itself | ||
// which doesn't work when using the global cache | ||
// https://github.com/microsoft/TypeScript/blob/1b57a0395e0bff191581c9606aab92832001de62/src/server/project.ts#L2238 | ||
// VSCode doesn't want to enable 'allowLocalPluginLoads' due to security concerns but | ||
// TypeScript already does local loads and if this code is running the user trusts the workspace | ||
// https://github.com/microsoft/vscode/issues/45856 | ||
const ConfiguredProject = tsserver.server.ConfiguredProject; | ||
const {enablePluginsWithOptions: originalEnablePluginsWithOptions} = ConfiguredProject.prototype; | ||
ConfiguredProject.prototype.enablePluginsWithOptions = function() { | ||
this.projectService.allowLocalPluginLoads = true; | ||
return originalEnablePluginsWithOptions.apply(this, arguments); | ||
}; | ||
|
||
// And here is the point where we hijack the VSCode <-> TS communications | ||
// by adding ourselves in the middle. We locate everything that looks | ||
// like an absolute path of ours and normalize it. | ||
|
||
const Session = tsserver.server.Session; | ||
const {onMessage: originalOnMessage, send: originalSend} = Session.prototype; | ||
let hostInfo = `unknown`; | ||
|
||
Object.assign(Session.prototype, { | ||
onMessage(/** @type {string} */ message) { | ||
const parsedMessage = JSON.parse(message) | ||
|
||
if ( | ||
parsedMessage != null && | ||
typeof parsedMessage === `object` && | ||
parsedMessage.arguments && | ||
typeof parsedMessage.arguments.hostInfo === `string` | ||
) { | ||
hostInfo = parsedMessage.arguments.hostInfo; | ||
} | ||
|
||
return originalOnMessage.call(this, JSON.stringify(parsedMessage, (key, value) => { | ||
return typeof value === `string` ? fromEditorPath(value) : value; | ||
})); | ||
}, | ||
|
||
send(/** @type {any} */ msg) { | ||
return originalSend.call(this, JSON.parse(JSON.stringify(msg, (key, value) => { | ||
return typeof value === `string` ? toEditorPath(value) : value; | ||
}))); | ||
} | ||
}); | ||
|
||
return tsserver; | ||
}; | ||
|
||
if (existsSync(absPnpApiPath)) { | ||
if (!process.versions.pnp) { | ||
// Setup the environment to be able to require typescript/lib/tsserver.js | ||
require(absPnpApiPath).setup(); | ||
} | ||
} | ||
|
||
// Defer to the real typescript/lib/tsserver.js your application uses | ||
module.exports = moduleWrapper(absRequire(`typescript/lib/tsserver.js`)); |
Oops, something went wrong.