Default applications.moduleLoader to vm-current-context#338
Conversation
Match the core default flip: applications now load in Harper's own
context by default, sharing JavaScript intrinsics. This avoids the
cross-realm compatibility problems (instanceof and other identity
checks failing across the application/Harper boundary) that separate
per-application intrinsics can cause. Scope-specific values remain
available via `import ... from 'harper'` / `require('harper')`.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
Reviewed; one potential blocker found. Correctness:
The PR body states "the core default flip lands via #1248 against core main first", but no submodule bump is included here. This PR should not merge until a companion commit bumps the core submodule pointer to a commit that includes |
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
Summary
Update the harper-pro shipped default
applications.moduleLoaderfromvmtovm-current-contextinstatic/defaultConfig.yaml, matching the core change in HarperFast/harper#1248.Purpose
vm-current-contextruns the application VM loader in Harper's own context so applications share JavaScript intrinsics, avoiding cross-realminstanceof/identity-check failures at the application/Harper boundary.vmmode (separate intrinsics) remains available for anyone who needs it.Where to look
coresubmodule pointer — the core default flip lands via #1248 against coremainfirst.Generated with assistance from Claude (Opus 4.7); reviewed by Kris.