Skip to content

fix(hir): honor reassigned function declarations - #3344

Merged
proggeramlug merged 1 commit into
PerryTS:mainfrom
andrewtdiz:codex/node-compat-function-decl-reassign
May 30, 2026
Merged

proggeramlug merged 1 commit into
PerryTS:mainfrom
andrewtdiz:codex/node-compat-function-decl-reassign

Conversation

@andrewtdiz

Copy link
Copy Markdown
Contributor

Summary

Scope

This addresses Issue #3149 Bug 2: reassigned function-declaration bindings. The Object(null)/Object() call-form behavior is handled separately in PR #3343.

Verification

  • Baseline exact fixture failed before the fix: Perry kept calling the original function declaration after reassignment
  • ./run_parity_tests.sh --suite node-suite --filter node-suite/globals/function-decl-reassign
  • ./run_parity_tests.sh --suite node-suite --filter node-suite/globals/global-functions
  • cargo check -p perry-hir
  • cargo check -p perry-hir -p perry
  • cargo fmt --all -- --check
  • ./scripts/check_file_size.sh
  • jq empty test-parity/known_failures.json
  • git diff --check origin/main...HEAD && git diff --check

@proggeramlug
proggeramlug merged commit 47cef32 into PerryTS:main May 30, 2026
11 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