Document vm-current-context as the default module loader#526
Conversation
Harper v5 now defaults applications.moduleLoader to vm-current-context. Rewrite the VM Module Loader section to describe shared intrinsics as the default, add a Module Loader Modes reference, and note that the constrained SES fetch applies only in vm mode. 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! |
🚀 Preview DeploymentYour preview deployment is ready! 🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-526 This preview will update automatically when you push new commits. |
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
Ethan-Arrowood
left a comment
There was a problem hiding this comment.
Do we have something updating this information wherever its detailed in the /reference/ section? Not just the release notes?
kriszyp
left a comment
There was a problem hiding this comment.
Good catch on the migration guide. One follow-up: reference/configuration/options.md line 330 still describes moduleLoader with vm as the default and doesn't mention vm-current-context as an option. Should that be updated in this PR or a follow-on?
Also worth noting: the table in this PR could link to or cross-reference the options page so readers can find the full config reference. Otherwise LGTM — the migration guide update is the right place to explain the behavioral change.
🧹 Preview CleanupThe preview deployment for this PR has been removed. |
Summary
Update the v5 migration guide for the new default
applications.moduleLoader: vm-current-context(HarperFast/harper#1248).import ... from 'harper').vm-current-context,vm,native, andcompartment.fetchapplies only invmmode.Purpose
Keep the migration docs aligned with the changed default so app authors understand the intrinsics-sharing behavior and when to opt back into
vm.Generated with assistance from Claude (Opus 4.7); reviewed by Kris.