Skip to content

Migrate extension webviews to Svelte 5 #815

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
nobody5050 wants to merge 23 commits into
base: 2027
Choose a base branch
from
Open

Migrate extension webviews to Svelte 5 #815

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
47be06e
Begin move of webviews to svelte
nobody5050 Nov 9, 2025
8c22aa4
Migrate components to svelte 5 runes syntax, minor css fixups, depend…
nobody5050 Nov 9, 2025
f0235da
The rest of it
nobody5050 Dec 29, 2025
0eae2e6
Merge branch '2027' into feat/convert-to-svelte
nobody5050 Dec 29, 2025
7fefc92
Fixes failed lint
nobody5050 Dec 29, 2025
a5a4901
fixups for prettier webpack
nobody5050 Dec 29, 2025
6d61dbf
address review
nobody5050 Dec 29, 2025
9c2a99d
revert npmmirror change to npmjs in package-lock
nobody5050 Feb 9, 2026
0ab5b89
small size components
nobody5050 Feb 20, 2026
c46ba91
more review addressing
nobody5050 Apr 11, 2026
661e47b
convert functions to arrow functions in riolog
nobody5050 Apr 11, 2026
78c595c
bit more cleanup
nobody5050 Apr 11, 2026
4a4e468
Improve Svelte webview integration, dependency state, and project cre…
nobody5050 Apr 11, 2026
04f086a
more review addressing
nobody5050 Apr 11, 2026
e1b693f
switch imports to use CommonJS syntax so we can yeet allowSyntheticDe…
nobody5050 Apr 14, 2026
dc7b12a
kill localeloader
nobody5050 Apr 14, 2026
c6968b7
remove unnecessary optional
nobody5050 Apr 14, 2026
99f49f8
webpack is dead and we have killed him
nobody5050 Apr 14, 2026
ef6d5c5
tiny fixes in riolog from testing
nobody5050 Apr 14, 2026
9fc284a
fix ci
nobody5050 Apr 14, 2026
f7df760
whoops missing dependency
nobody5050 Apr 14, 2026
6c2b869
prettier
nobody5050 Apr 14, 2026
158f90f
Format tsconfig.rollup.extension.json with prettier
nobody5050 Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ jobs:
npm run lint
npm run unittest
npm run gulp
npm run webpack
npm run vscePackage
working-directory: vscode-wpilib
name: Build VSIX
Expand Down
4 changes: 2 additions & 2 deletions .vscode/launch.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"runtimeExecutable": "${execPath}",
"args": ["--extensionDevelopmentPath=${workspaceFolder}/vscode-wpilib"],
"outFiles": ["${workspaceFolder}/vscode-wpilib/out/**/*.js"],
"preLaunchTask": "vscode-web-watch"
"preLaunchTask": "npm: build:dev"
},
{
"name": "Extension Tests",
Expand All @@ -24,7 +24,7 @@
"--extensionTestsPath=${workspaceFolder}/vscode-wpilib/out/test"
],
"outFiles": ["${workspaceFolder}/vscode-wpilib/out/test/**/*.js"],
"preLaunchTask": "vscode-web-watch"
"preLaunchTask": "npm: build:dev"
}
]
}
24 changes: 18 additions & 6 deletions .vscode/tasks.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,30 @@
"version": "2.0.0",
"tasks": [
{
"label": "npm: build:dev",
"type": "npm",
"identifier": "vscode-web-watch",
"script": "webpack-watch",
"script": "build:dev",
"path": "vscode-wpilib/",
"isBackground": true,
"presentation": {
"reveal": "never"
},
"group": {
"kind": "build",
"isDefault": true
},
"presentation": {
"reveal": "never"
}
},
{
"label": "wpilib: watch",
"type": "npm",
"script": "watch",
"path": "vscode-wpilib/",
"isBackground": true,
"runOptions": {
"runOn": "folderOpen"
},
"presentation": {
"reveal": "never"
},
"problemMatcher": []
}
]
Expand Down
1 change: 0 additions & 1 deletion vscode-wpilib/.vscodeignore
View file
Open in desktop
Copy link
Copy Markdown
Member

@Gold856 Gold856 Apr 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rollup files should be ignored.

Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
.vscode-test/**
node_modules
.gitignore
webpack.config.js
gulpfile.js
i18n/**/package.i18n.json
locale/**
Expand Down
2 changes: 1 addition & 1 deletion vscode-wpilib/eslint.config.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ export default defineConfig([
'out/',
'resources/',
'.eslintrc.js',
'webpack.config.js',
'rollup.*.config.mjs',
'gulpfile.js',
'dist',
]),
Expand Down
4 changes: 4 additions & 0 deletions vscode-wpilib/gulpfile.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,10 @@ gulp.task('clean', async () => {
globSync(['package.nls.*.json', '*.vsix']).map(async (file) => await rm(file));
});

gulp.task('clean-webviews', async () => {
await rm('resources/dist/', { force: true, recursive: true });
});

gulp.task('build', gulp.series('clean', 'i18n-compile', 'i18n-additional'));

gulp.task('default', gulp.series('build'));
Loading
Loading