Skip to content

Bump @types/node from 24.10.11 to 25.2.1#3821

Closed
dependabot[bot] wants to merge 1 commit intomasterfrom
dependabot/npm_and_yarn/types/node-25.2.1
Closed

Bump @types/node from 24.10.11 to 25.2.1#3821
dependabot[bot] wants to merge 1 commit intomasterfrom
dependabot/npm_and_yarn/types/node-25.2.1

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 5, 2026

Bumps @types/node from 24.10.11 to 25.2.1.

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 24.10.11 to 25.2.1.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-version: 25.2.1
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Feb 5, 2026
@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedprismarine-provider-anvil@​2.11.07610010080100
Addedgh-helpers@​1.2.0100100100100100

View full report

plainprince pushed a commit to plainprince/mineflayer that referenced this pull request Feb 8, 2026
- time: Disable daylight cycle before time transition tests to prevent
  time from drifting between /time set and the assertion (e.g. expected
  18000, got 21695). Re-enable only for progression and toggle tests.
- nether: Add 1s wait after chunk loading to give slow CI environments
  time to stabilize before block placement.

These flaky tests were causing spurious CI failures on unrelated PRs
(e.g. PrismarineJS#3821).

Co-authored-by: Cursor <cursoragent@cursor.com>
@plainprince plainprince mentioned this pull request Feb 8, 2026
3 tasks
@plainprince
Copy link
Contributor

The CI failures here are not caused by this PR -- they're pre-existing flaky tests:

  • 1.20.6 time test: Daylight cycle was left enabled during /time set assertions, causing time to drift (expected 18000, got 21695).
  • 1.15.2 nether test: placeBlock times out on slow CI waiting for blockUpdate after chunk loading.

Opened #3827 to fix both. Once that's merged, this PR should pass cleanly.

plainprince pushed a commit to plainprince/mineflayer that referenced this pull request Feb 8, 2026
Bump @types/node to ^25.2.1 (from PrismarineJS#3821) and fix two flaky tests that
were causing spurious CI failures:

- time: disable daylight cycle during /time set assertions to prevent
  drift (e.g. expected 18000, got 21695)
- nether: add stabilization wait after chunk loading on slow CI

Co-authored-by: Cursor <cursoragent@cursor.com>
@plainprince
Copy link
Contributor

Opened #3828 which includes the @types/node bump from this PR along with fixes for the two flaky tests (time and nether) that were causing the CI failures here. Once #3828 is merged, this PR can be closed.

@plainprince
Copy link
Contributor

Just a heads up -- #3828 includes this @types/node bump along with fixes for the flaky tests that were blocking CI here. If #3828 gets merged, this PR can be closed.

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 9, 2026

Superseded by #3829.

@dependabot dependabot bot closed this Feb 9, 2026
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/types/node-25.2.1 branch February 9, 2026 23:35
plainprince pushed a commit to plainprince/mineflayer that referenced this pull request Feb 14, 2026
Bump @types/node to ^25.2.1 (from PrismarineJS#3821) and fix two flaky tests that
were causing spurious CI failures:

- time: disable daylight cycle during /time set assertions to prevent
  drift (e.g. expected 18000, got 21695)
- nether: add stabilization wait after chunk loading on slow CI

Co-authored-by: Cursor <cursoragent@cursor.com>
rom1504 pushed a commit that referenced this pull request Feb 15, 2026
* Bump @types/node from 24.0.6 to 25.2.1 and fix flaky tests

Bump @types/node to ^25.2.1 (from #3821) and fix two flaky tests that
were causing spurious CI failures:

- time: disable daylight cycle during /time set assertions to prevent
  drift (e.g. expected 18000, got 21695)
- nether: add stabilization wait after chunk loading on slow CI

Co-authored-by: Cursor <cursoragent@cursor.com>

* Increase runExample timeout from 30s to 60s

The 30s timeout for example tests covers child process spawn, server
join, 5s stabilization wait, chunk loading, and the actual test. On
slow CI runners this was too tight, causing spurious timeouts (e.g.
exampleBee on 1.19.3).

Co-authored-by: Cursor <cursoragent@cursor.com>

* Replace timeout-based test fixes with robust alternatives

nether: Replace blind 1s wait with polling loop that checks for a
solid block below before placing. Mocha's 90s test timeout is the
backstop.

runExample: Remove withTimeout wrapper (mocha handles overall timeout),
remove 5s setTimeout before saying "loaded" (server command ordering
guarantees the TP is applied before the chat message reaches the child),
remove timeout from child join detection. This eliminates the premature
"Promise timed out" failures on slow CI.

testCommon: Increase base onceWithCleanup timeout from 5s to 20s for
helper functions (gamemode change, teleport, clear inventory, etc.)
to prevent spurious failures on slow CI while still providing useful
error messages.

Co-authored-by: Cursor <cursoragent@cursor.com>

* Fix runExample: wait for child entity + physics settling

The child bot's physics engine needs time to initialize at the
teleported position (ground detection, etc.) before tests that
require jumping (exampleDigger build) can work.

Replace the removed blind 5s setTimeout with:
1. Condition-based poll for child entity at the TP target position
2. waitForTicks(60) for physics engine to settle (~3s game time)

This is more robust than the original setTimeout because the entity
check adapts to server speed and waitForTicks is tied to game ticks.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Simeon Kummer <simeon@hitthecode.de>
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant