Deprecate MarkdownEditor, MarkdownViewer, InlineAutocomplete, and related code

[iansan5653]

Per #3604, we will migrate the following draft/experimental APIs from primer/react to the internal GitHub codebase. Since drafts are not semvered we technically don't have to deprecate them, but I think it's still probably a good idea to follow a deprecation process rather than having them just disappear. So this PR deprecates the components, and then in v37 we can ultimately delete the code. This would be an aggressive timeline for other APIs, but again these are all drafts:

• MarkdownEditor
• MardownViewer
• InlineAutocomplete
• hooks/useCombobox
• hooks/useDynamicTextareaHeight
• hooks/useIgnoreKeyboardActionsWhileComposing
• hooks/useSafeAsyncCallback
• hooks/useSyntheticChange
• hooks/useUnifiedFileSelect, useClickFileSelect, useDropFileSelect, usePasteFileSelect

While some internal APIs will be migrated as well, I've focused on public APIs here.

Side note: I also moved InlineAutocomplete into drafts where it belongs. Not sure how it escaped 🤷.

Changelog

New

Changed

• Deprecated MarkdownEditor, MardownViewer, InlineAutocomplete, and related hooks

Removed

Rollout strategy

• Patch release
• Minor release
• Major release; if selected, include a written rollout or migration plan

Testing & Reviewing

Merge checklist

• [ ] Added/updated tests
• Added/updated documentation
• Added/updated previews (Storybook)
• Changes are SSR compatible
• [ ] Tested in Chrome
• [ ] Tested in Firefox
• [ ] Tested in Safari
• [ ] Tested in Edge
• [ ] Integration tests pass at github/github (Learn more about how to run integration tests)

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

[changeset-bot]

🦋 Changeset detected

Latest commit: 42016cf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@primer/react	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

size-limit report 📦

Path	Size
dist/browser.esm.js	105.54 KB (0%)
dist/browser.umd.js	106.13 KB (0%)

[joshblack]

@iansan5653 just wanted to ask, is usage over in dotcom already migrated over to use the internal version of this component or would it be helpful to add linting to help folks move over to the new package?

[iansan5653]

usage over in dotcom already migrated over to use the internal version of this component or would it be helpful to add linting to help folks move over to the new package?

Not yet. I haven't actually copied the code over to dotcom, but when I do so I will add linting accordingly. I have already added rules to lint specifically against importing @primer/react/drafts/MarkdownEditor since we have CommentBox in dotcom (inflight).

[mperrotti]

Assuming we don't want to move these to the deprecated package yet, this looks correct to me.

[iansan5653]

Assuming we don't want to move these to the deprecated package yet, this looks correct to me.

Not sure what we would do here, since they are already in the drafts package. I'm open to moving them if you want.