chore: mark non-public APIs as @internal

[jelbourn]

R: @mmalerba

Disables the stripInternal TypeScript option (which omits @internal symbols from the .d.ts files) and uses the @internal JsDoc to mark classes as non-public. This will tell our docs generation that this isn't a public API that needs to be documented.

This is mainly for symbols where we don;t want to start them with an underscore (such as for class names) or for symbols that already have an underscored version (rare in our codebase).

[kara]

LGTM

[mmalerba]

Do we want to mark the Error classes @internal? I would think we'd want those in the documentation so people know what kind of errors to expect

[jelbourn]

@mmalerba

My thinking:

• You never want to preemptively see the errors in the docs
• There's nothing to do with the error classes- they're just containers for the string messages.
• I think the docs are better off saying what is supported instead of listing the errors that can be thrown.

[mmalerba]

Ok, sounds good

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}