Skip to content

Update dependencies (NPM and others)#451

Draft
deribaucourt wants to merge 13 commits intoyoctoproject:stagingfrom
savoirfairelinux:update-npm
Draft

Update dependencies (NPM and others)#451
deribaucourt wants to merge 13 commits intoyoctoproject:stagingfrom
savoirfairelinux:update-npm

Conversation

@deribaucourt
Copy link
Copy Markdown
Member

Run npm update in every package to update the dependencies. Fixes a few security audit warnings.

@deribaucourt deribaucourt force-pushed the update-npm branch 2 times, most recently from 26472ab to 1f52746 Compare July 30, 2025 09:30
@deribaucourt deribaucourt changed the title npm: update npm packages Update dependencies (NPM and others) Jul 30, 2025
@deribaucourt deribaucourt marked this pull request as draft July 30, 2025 13:15
@deribaucourt
Copy link
Copy Markdown
Member Author

@WilsonZiweiWang from my experience, some tests failed due to calling devtool modify. With the updated poky, it seemed to download Uninative and build many recipes to patch sources which is extremly slow.
It looked like if we stay on the current poky version we were unaffected. It would be ok for now.

Maybe I'm wrong and the tests fail for a reason, but that's my thoughts so far.

@WilsonZiweiWang
Copy link
Copy Markdown
Contributor

WilsonZiweiWang commented Sep 19, 2025

@WilsonZiweiWang from my experience, some tests failed due to calling devtool modify. With the updated poky, it seemed to download Uninative and build many recipes to patch sources which is extremly slow. It looked like if we stay on the current poky version we were unaffected. It would be ok for now.

Maybe I'm wrong and the tests fail for a reason, but that's my thoughts so far.

When the integration tests are skipped, the CI can consistently pass if a little more time is given to the unit tests, for example, 10 minutes of timeout.

Successful runs with integration tests skipped:
https://github.com/yoctoproject/vscode-bitbake/actions/runs/17841666750/job/50732404783
https://github.com/yoctoproject/vscode-bitbake/actions/runs/17841666750/job/50734311096
https://github.com/yoctoproject/vscode-bitbake/actions/runs/17841666750/job/50734781920

According to the logs from a previous CI run and my local runs, they are all stuck at the reviewDiagnostics() from the integration tests. See this run where the integration tests spent hours at reviewDiagnostics(). I downloaded the logs in case they expire later:
15_Test Project.txt

@WilsonZiweiWang
Copy link
Copy Markdown
Contributor

While the bitbake is scanning the project (/integration-tests/project-folder), multiple bitbake processes will spawn like the following:

zwang    1479052 12.5  0.4 208992 136632 ?       Sl   15:20   0:01 /usr/bin/python3 /home/zwang/projects/vscode-bitbake/resources/poky/bitbake/bin/bitbake-server decafbad 3 6 /home/zwang/projects/vscode-bitbake/integration-tests/project-folder/build/bitbake-cookerdaemon.log /home/zwang/projects/vscode-bitbake/integration-tests/project-folder/build/bitbake.lock /home/zwang/projects/vscode-bitbake/integration-tests/project-folder/build/bitbake.sock 0 0 None 0

In the meantime, the extension host is repeatedly trying to reconnect to the bitbake server:
image
The last attempt is when calling reviewDiagnostics() as shown in the log file I put in the above comment. Then it hangs forever as the bitbake server process no longer appears.

@deribaucourt
Copy link
Copy Markdown
Member Author

Maybe we need to disable some test parallelization option that is new or reworded?

WilsonZiweiWang and others added 11 commits March 24, 2026 11:21
Run `npm update` in every package to update the dependencies.
Fixes a few security audit warnings.
Run our update-refs scripts to bump all non-nodejs dependencies.
Update the run-test script based on the latest vscode-test sample.
Follow the same pattern as for other settings to avoid potential
mismatches.
With the updated extensions, some diagnostics linger. They are not
harmful, we just need the test to filter out our own expected.
It looks like pylance now sends the diagnostics as pylint, and
require to show and save the document before seding them.
Otherwise it would simply remove the line, which may revert to
the user's command wrapper if they had one.
The latest vscode version doesn't want to fire the OpenDocument
language server event, so we need to deploy full force edits to
trigger the diagnostics.
It was fine except on headless mode.

Also sometimes the diagnostics are sourced as Pylance, sometimes as
Pylint, so we use a broader filter.

See: microsoft/vscode#138924
Run `npm upgrade` in all packages to bump the minor revisions on all
components. Some npm audit warnings are still present so another
ncu round will be required.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants