Skip to content

Document vm-current-context as the default module loader#526

Merged
kriszyp merged 1 commit into
mainfrom
kris/vm-current-context-default
Jun 13, 2026
Merged

Document vm-current-context as the default module loader#526
kriszyp merged 1 commit into
mainfrom
kris/vm-current-context-default

Conversation

@kriszyp

@kriszyp kriszyp commented Jun 11, 2026

Copy link
Copy Markdown
Member

Summary

Update the v5 migration guide for the new default applications.moduleLoader: vm-current-context (HarperFast/harper#1248).

  • Rewrite the VM Module Loader intro to describe shared intrinsics as the default and how scoped values are obtained (import ... from 'harper').
  • Add a Module Loader Modes reference table covering vm-current-context, vm, native, and compartment.
  • Note that the SES-constrained (https-only) fetch applies only in vm mode.

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.

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>
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@kriszyp kriszyp requested a review from Ethan-Arrowood June 11, 2026 04:55
@github-actions github-actions Bot temporarily deployed to pr-526 June 11, 2026 04:58 Inactive
@github-actions

Copy link
Copy Markdown

🚀 Preview Deployment

Your preview deployment is ready!

🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-526

This preview will update automatically when you push new commits.

@kriszyp kriszyp marked this pull request as ready for review June 11, 2026 05:11
@kriszyp kriszyp requested a review from a team as a code owner June 11, 2026 05:11
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@Ethan-Arrowood Ethan-Arrowood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have something updating this information wherever its detailed in the /reference/ section? Not just the release notes?

@kriszyp kriszyp left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@kriszyp kriszyp merged commit 6c60831 into main Jun 13, 2026
7 checks passed
@kriszyp kriszyp deleted the kris/vm-current-context-default branch June 13, 2026 00:30
@github-actions

Copy link
Copy Markdown

🧹 Preview Cleanup

The preview deployment for this PR has been removed.

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