feat(isolated-declarations): add builder methods to options struct

[DonIsaac]

Allows constructing and modifying options in a way that will not break if we add new properties to IsolatedDeclarationsOptions.

// breaks as soon as a new member is added
let opts = IsolatedDeclarationsOptions { strip_internal: true }
// resilient to future contract changes
let opts = IsolatedDeclarationsOptions::default().strip_internal()

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[DonIsaac]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• feat(isolated-declarations): add builder methods to options struct #6373 👈
• feat(isolated-declarations): impl Default for options #6372
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #6373 will not alter performance

_{Comparing don/10-08-feat_isolated-declarations_add_builder_methods_to_options_struct (ae3e9f0) with don/10-08-feat_isolated-declarations_impl_default_for_options (4210c8b)}

Summary

✅ 29 untouched benchmarks

[Boshen]

// breaks as soon as a new member is added
let opts = IsolatedDeclarationsOptions { strip_internal: true }

I'm leaning towards forcing the breaking change.

And also ... I want to ban builder pattern at some point.