You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Update serialization docs to reflect support for workspace comments #10266
The "Save and Load" doc serialization.mdx claims that workspace comments are not supported by JSON serialization, but support for this was added in #7927
The stale doc should be updated to remove this claim. The following serializer hooks example should also be renamed to something more applicable, since it uses workspace-comments as an example which could be confusing.
Bonus: this page also includes a broken link on the text "built-in priorities". The correct link is here. This link fix can also be included on a PR that fixes this issue.
I’m working on this. I’ll keep it to the documentation scope described here: document built-in workspace-comment JSON serialization, rename the custom serializer example so it no longer implies comments need a plugin, fix the priorities link, and validate the docs lint/format/build.
@zspriggs - I have added a PR (#10628) that does the following
Show from docs that workspace-comment is supported out of the box
Added a custom myPlugin in the example
Updated the example on the clear stage to show what happens to the variables of the myPlugin serializer.
One minor thing I noticed in the part of "When a serializer is triggered, two things happen" -
It should mention that the clear and load does not happen for the triggered serializer, but, clear happens for all in the priority order of smallest to largest, and then load happens in the reverse priority order.
Activity
zspriggs commentedon Aug 13, 2026
Bonus: this page also includes a broken link on the text "built-in priorities". The correct link is here. This link fix can also be included on a PR that fixes this issue.
Johnkothapalli commentedon Aug 13, 2026
I’m working on this. I’ll keep it to the documentation scope described here: document built-in workspace-comment JSON serialization, rename the custom serializer example so it no longer implies comments need a plugin, fix the priorities link, and validate the docs lint/format/build.
ritfiz commentedon Oct 24, 2026
@zspriggs - I have added a PR (#10628) that does the following
One minor thing I noticed in the part of "When a serializer is triggered, two things happen" -
It should mention that the
clearandloaddoes not happen for the triggered serializer, but, clear happens for all in the priority order of smallest to largest, and then load happens in the reverse priority order.docs: update serialization docs to reflect support for workspace comm…