Skip to content

chore: bump deno stack to deno_core 0.412 / deno_ast 0.53 - #167

Merged
Patrick Kelly (pk8189) merged 1 commit into
portofcontext:mainfrom
BezotCorp:modernize-deno-stack
Oct 5, 2026
Merged

Patrick Kelly (pk8189) merged 1 commit into
portofcontext:mainfrom
BezotCorp:modernize-deno-stack

Conversation

@remi-bezot

Copy link
Copy Markdown
Contributor

Description

Update the Deno runtime stack from deno_core 0.381 / V8 145 / temporal_rs 0.1 to deno_core 0.412 / V8 150 / temporal_rs 0.2.

This removes the Temporal/ICU incompatibility that forces downstream consumers to pin older ICU versions.

Related Issues

N/A

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Test coverage improvement

Changes Made

  • Updated deno_core to 0.412, deno_ast to 0.53.3, deno_resolver to 0.90, and node_resolver to 0.97.
  • Updated the resulting runtime stack to V8 150 and temporal_rs 0.2.
  • Adapted the required Deno API changes: #[op2(async)] becomes #[op2], and PollEventLoopOptions::pump_v8_message_loop is no longer present.
  • Added the change under UNRELEASED in CHANGELOG.md.

Testing

The full Rust test suite passes with the updated stack, including code mode execution, TypeScript transpilation and type checking, session server execution tests, and doctests.

cargo clippy -- -D warnings was also run. It currently fails on pre-existing lint issues in pctx_codegen and pctx_config; the affected source lines are unchanged by this PR.

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed

Test Commands Run

cargo test
cargo fmt --all --check
cargo clippy -- -D warnings

Breaking Changes

  • This PR includes breaking changes
  • Migration guide updated in CHANGELOG.md

No PCTX public API changes are intended.

Documentation

  • Code comments added/updated
  • README.md updated (if needed)
  • CHANGELOG.md updated under "Unreleased"
  • API documentation updated

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Screenshots/Videos

N/A

Additional Context

The dependency update was intentionally kept narrow. PCTX required only a small number of source-level API adaptations for the newer Deno stack.

For downstream consumers, the main benefit is moving off temporal_rs 0.1, removing the transitive ICU compatibility constraint that required older ICU pins.

Moves to v8 150 and temporal_rs 0.2 so downstream workspaces can use
icu_calendar 2.3. op2(async) becomes op2 (async ops are eager by default)
and PollEventLoopOptions::pump_v8_message_loop is gone.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@pk8189
Patrick Kelly (pk8189) merged commit 4603698 into portofcontext:main Oct 5, 2026
5 checks passed
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