fix: update fast-element exports#7488
Closed
janechu wants to merge 4 commits intoreleases/fast-element-v3from
Closed
fix: update fast-element exports#7488janechu wants to merge 4 commits intoreleases/fast-element-v3from
janechu wants to merge 4 commits intoreleases/fast-element-v3from
Conversation
Replace side-effect installers with explicit registration, add kernel types, and move hydration support into the template runtime. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Move non-FASTElement imports onto explicit fast-element subpaths across router, examples, benchmarks, and declarative fixtures. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Update package docs, website content, generated API pages, and size reports to match the new fast-element entrypoint structure. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Re-apply the small set of export-surface and change-file differences that were lost when the branch was rebased onto releases/fast-element-v3. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
ca1b4c5 to
293e972
Compare
Collaborator
Author
|
Too many changes, unclear on the resolution of side effects and dead code. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
📖 Description
@microsoft/fast-elementexportFASTElementonly and publish the remaining APIs on focused subpathsregisterFAST(),registerFASTDebugMessages(), andinstallPolyfills()hooksViewTemplatesupport into the template runtime and remove the legacy hydration installer entrypoints🎫 Issues
👩💻 Reviewer Notes
FASTElementand removes the hydration installer entrypoints instead of preserving them@microsoft/fast-routernow includes atype: nonebeachball change file sonpm run checkchangepasses alongside the existing fast-element entriesreleases/fast-element-v3to bring in fix: remove shared kernel #7487 and clear conflicts📑 Test Plan
npm run buildnpm run prebuild -w sites/websitenpm run testnpm run biome:checknpm run checkchange✅ Checklist
General
$ npm run change