Skip to content

fix(node:os): add priority methods - #1892

Closed
andrewtdiz wants to merge 1 commit into
PerryTS:mainfrom
andrewtdiz:codex/node-os-priority-methods
Closed

fix(node:os): add priority methods#1892
andrewtdiz wants to merge 1 commit into
PerryTS:mainfrom
andrewtdiz:codex/node-os-priority-methods

Conversation

@andrewtdiz

@andrewtdiz andrewtdiz commented May 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add node:os getPriority/setPriority runtime helpers backed by platform priority calls
  • validate Int32/range arguments and surface Node-shaped priority errors
  • route priority methods through HIR, native dispatch table, and captured-method dispatch

Verification

  • cargo fmt --all --check
  • cargo check -p perry-runtime -p perry-hir -p perry-codegen
  • ./scripts/check_file_size.sh
  • ./scripts/regen_api_docs.sh
  • ./run_parity_tests.sh --suite node-suite --module os --filter priority
  • ./run_parity_tests.sh --suite node-suite --module os --filter methods

@andrewtdiz
andrewtdiz force-pushed the codex/node-os-priority-methods branch from 0424b35 to 68053ea Compare May 27, 2026 01:18
@andrewtdiz
andrewtdiz force-pushed the codex/node-os-priority-methods branch from 68053ea to eb4d711 Compare May 27, 2026 04:08
@andrewtdiz

Copy link
Copy Markdown
Contributor Author

Superseded by #1922, rebased on current main after #1890 landed.

@andrewtdiz andrewtdiz closed this May 27, 2026
@andrewtdiz
andrewtdiz deleted the codex/node-os-priority-methods branch May 27, 2026 05:00
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.

1 participant