Skip to content

Commit 021b7c8

Browse files
1 parent 1a08aab commit 021b7c8

6 files changed

Lines changed: 49 additions & 49 deletions

File tree

Source/Apply/Pipeline.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ const LandDisableUIFixes =
4848

4949
if (LandDisableUIFixes) {
5050
console.log(
51-
"[Output/Pipeline] DisableUIFixes=true skipping 2 UI transforms (InjectMacTitlebarOffsetCSS, InjectPartZIndexCSS)",
51+
"[Output/Pipeline] DisableUIFixes=true - skipping 2 UI transforms (InjectMacTitlebarOffsetCSS, InjectPartZIndexCSS)",
5252
);
5353
}
5454

Source/Plugin/Transform/Inject/Webview/Debug/Logging.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ const Plugin: TransformPlugin = {
106106
}
107107

108108
// Patch content/event handlers
109-
// Patch the REAL content handler (line ~958 the async one that
109+
// Patch the REAL content handler (line ~958 - the async one that
110110
// actually processes extension HTML, NOT the unloadMonitor at ~375).
111111
const RealContentHandler = "hostMessaging.onMessage('content', async";
112112
if (Next.includes(RealContentHandler)) {
@@ -115,7 +115,7 @@ const Plugin: TransformPlugin = {
115115
Next = Next.replace(RealContentHandler, LogRealContent);
116116
}
117117

118-
// Patch after toContentHtml(data) returns log processed HTML
118+
// Patch after toContentHtml(data) returns - log processed HTML
119119
// characteristics so we can verify the content has a root element
120120
// and module script reference.
121121
const ContentProcessed = "const newDocument = toContentHtml(data);";
@@ -125,7 +125,7 @@ const Plugin: TransformPlugin = {
125125
Next = Next.replace(ContentProcessed, LogContentProcessed);
126126
}
127127

128-
// Also keep the old unloadMonitor log it's harmless and the
128+
// Also keep the old unloadMonitor log - it's harmless and the
129129
// data it receives (confirmBeforeClose) is still useful context.
130130
const OnMessageContent = "hostMessaging.onMessage('content'";
131131
if (Next.includes(OnMessageContent)) {

Source/Plugin/Transform/Inject/Webview/Debug/RuntimeDiagnostics.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ const DiagnosticCode = `
3535
3636
DEBUG_WV('RUNTIME_DIAG_BOOT');
3737
38-
// Cross-frame message listener logs postMessage events between
38+
// Cross-frame message listener - logs postMessage events between
3939
// inner iframe and extension host (limited to avoid spam)
4040
var _msgCount = 0;
4141
window.addEventListener('message', function(e) {

Source/Plugin/Transform/Inject/Webview/Debug/RuntimeDiagnosticsInner.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ const Plugin: TransformPlugin = {
279279
DI('VSCODE_API_CHECK_EARLY_ERR', { error: String(e) });
280280
}
281281
282-
// Deferred check at 3s after module scripts have executed
282+
// Deferred check at 3s - after module scripts have executed
283283
setTimeout(function() {
284284
try {
285285
DI('VSCODE_API_CHECK_LATE', {

Source/Run.sh

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,17 @@ fi
66

77
# shellcheck disable=SC2154
88
case "$Dependency" in
9-
"Microsoft/VSCode")
10-
Build="out"
11-
;;
9+
"Microsoft/VSCode")
10+
Build="out"
11+
;;
1212

13-
"CodeEditorLand/Editor")
14-
Build="Source"
15-
;;
13+
"CodeEditorLand/Editor")
14+
Build="Source"
15+
;;
1616

17-
*)
18-
exit 1
19-
;;
17+
*)
18+
exit 1
19+
;;
2020
esac
2121

2222
Build "Source/**/*.{ts,json}" \

Source/prepublishOnly.sh

Lines changed: 34 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -6,42 +6,42 @@ fi
66

77
# shellcheck disable=SC2154
88
case "$Dependency" in
9-
"Microsoft/VSCode")
10-
# Always use `out/` (dev tsc) as the input tree, NOT `out-build/`
11-
# (gulp release tree).
12-
#
13-
# `out-build/` carries the upstream-mangled bytes (`$Hp`, `$ZXb`,
14-
# `$4e`, ...) but is missing every gulp-only artefact: the
15-
# workbench bundler entries (`workbench.web.main.js`,
16-
# `workbench.web.main.internal.js`, `workbench.desktop.main.js`)
17-
# never reach `out-build/` because gulp emits them through a
18-
# separate AMD-optimiser step Output does not run. Mixing the
19-
# two trees produces an incoherent shipped artefact: Output's
20-
# release `Target/Microsoft/VSCode/vs/code/electron-browser/
21-
# workbench/workbench.js` is mangled, but the supplement-copied
22-
# siblings from `out/` are unmangled, and every cross-tree
23-
# `import { $Hp } from './telemetry.js'` blows up at runtime
24-
# because the unmangled file does not export `$Hp`.
25-
#
26-
# `out/` is unmangled but COMPLETE - every file VS Code's source
27-
# tree references is present. Output's transforms operate on
28-
# readable identifiers; consumers (Sky's Static/Application copy
29-
# AND Sky's bundled Vite walk) see a single self-consistent
30-
# universe of bytes; Vite/Rollup re-mangles for the bundled
31-
# profile, browsers gzip the unbundled profile. The shipped
32-
# artefact is a few hundred KB larger but works in every profile
33-
# without the mangling-truth class of patches the previous
34-
# `out-build/` path required.
35-
Build="out"
36-
;;
9+
"Microsoft/VSCode")
10+
# Always use `out/` (dev tsc) as the input tree, NOT `out-build/`
11+
# (gulp release tree).
12+
#
13+
# `out-build/` carries the upstream-mangled bytes (`$Hp`, `$ZXb`,
14+
# `$4e`, ...) but is missing every gulp-only artefact: the
15+
# workbench bundler entries (`workbench.web.main.js`,
16+
# `workbench.web.main.internal.js`, `workbench.desktop.main.js`)
17+
# never reach `out-build/` because gulp emits them through a
18+
# separate AMD-optimiser step Output does not run. Mixing the
19+
# two trees produces an incoherent shipped artefact: Output's
20+
# release `Target/Microsoft/VSCode/vs/code/electron-browser/
21+
# workbench/workbench.js` is mangled, but the supplement-copied
22+
# siblings from `out/` are unmangled, and every cross-tree
23+
# `import { $Hp } from './telemetry.js'` blows up at runtime
24+
# because the unmangled file does not export `$Hp`.
25+
#
26+
# `out/` is unmangled but COMPLETE - every file VS Code's source
27+
# tree references is present. Output's transforms operate on
28+
# readable identifiers; consumers (Sky's Static/Application copy
29+
# AND Sky's bundled Vite walk) see a single self-consistent
30+
# universe of bytes; Vite/Rollup re-mangles for the bundled
31+
# profile, browsers gzip the unbundled profile. The shipped
32+
# artefact is a few hundred KB larger but works in every profile
33+
# without the mangling-truth class of patches the previous
34+
# `out-build/` path required.
35+
Build="out"
36+
;;
3737

38-
"CodeEditorLand/Editor")
39-
Build="Source"
40-
;;
38+
"CodeEditorLand/Editor")
39+
Build="Source"
40+
;;
4141

42-
*)
43-
exit 1
44-
;;
42+
*)
43+
exit 1
44+
;;
4545
esac
4646

4747
Build "Source/**/*.{ts,json}" \

0 commit comments

Comments
 (0)