Promote IFluidDevtools to beta#25695
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR promotes the IFluidDevtools interface and related components from alpha to beta stability level to support devtools integration in Office-Bohemia.
- Updated API stability tags from
@alphato@betafor devtools interfaces - Added
IFluidDevtoolsinterface to the beta API surface - Updated alpha API report to reflect the promotion
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/tools/devtools/devtools-core/src/IFluidDevtools.ts | Changed JSDoc tag from @alpha to @beta |
| packages/tools/devtools/devtools-core/src/ContainerRuntimeDevtools.ts | Changed JSDoc tag from @alpha to @beta for ContainerRuntimeProps |
| packages/tools/devtools/devtools-core/src/ContainerDevtools.ts | Changed JSDoc tag from @alpha to @beta for ContainerDevtoolsProps |
| packages/tools/devtools/devtools-core/api-report/devtools-core.beta.api.md | Added IFluidDevtools interface to beta API surface |
| packages/tools/devtools/devtools-core/api-report/devtools-core.alpha.api.md | Updated IFluidDevtools tag from @alpha to @beta |
alexvy86
left a comment
There was a problem hiding this comment.
We need to also make initializeDevtools and/or tryGetFluidDevtools beta, right? Exposing just the interfaces doesn't really let applications do much, since they are not designed to be implemented externally.
.changeset/few-plants-strive.md
Outdated
| --- | ||
| Promote IFluidDevtools APIs from alpha to beta | ||
|
|
||
| `IFluidDevtools` may now be imported from `/beta`. |
There was a problem hiding this comment.
Have you verified that devtools-core's package.json includes an export for the beta level? If not, these APIs still won't be importable via /beta. I can help you make the necessary changes if not / you have questions :)
There was a problem hiding this comment.
Thanks for pointing out! Didn't know adding beta field in package.json is necessary.
I've referenced pacakges/dds/tree and added new esm & cjs config files and the new fields in the package.json.
packages/tools/devtools/devtools-core/src/ContainerRuntimeDevtools.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
…/FluidFramework into devtools/bump-to-beta
alexvy86
left a comment
There was a problem hiding this comment.
Left a few more comments. I also think API report regeneration is needed? Some of the changes in the ts files aren't reflected in the API reports (readonly on some properties).
Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
…ools.ts Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
|
🔗 Found some broken links! 💔 Run a link check locally to find them. See linkcheck output |
#### Description This PR promotes `IFluidDevtools` interface from `alpha` to `beta` for integration into partner repos. --------- Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
Description
This PR promotes
IFluidDevtoolsinterface fromalphatobetafor integration into partner repos.