Confusing Appservice docs: "(the sender_localpart)"

[jaller94]

Three functions in src/appservice/Appservice.ts include "(the sender_localpart)" in their docs description. I don't know what this means and find it to be confusing, even after reading the function's code.

matrix-bot-sdk/src/appservice/Appservice.ts

Lines 346 to 350 in 5c7fbdf

	* Get the application service's "bot" Intent (the sender_localpart).
	* @returns {Intent} The intent for the application service itself.
	*/
	public get botIntent(): Intent {
	return this.getIntentForUserId(this.botUserId);

matrix-bot-sdk/src/appservice/Appservice.ts

Lines 354 to 360 in 5c7fbdf

	* Get the application service's "bot" MatrixClient (the sender_localpart).
	* Normally the botIntent should be used to ensure that the bot user is safely
	* handled.
	* @returns {MatrixClient} The client for the application service itself.
	*/
	public get botClient(): MatrixClient {
	return this.botIntent.underlyingClient;

matrix-bot-sdk/src/appservice/Appservice.ts

Lines 339 to 342 in 5c7fbdf

	* Get the application service's "bot" user ID (the sender_localpart).
	*/
	public get botUserId(): string {
	return this.getUserId(this.registration.sender_localpart);

This last one got me wondering when reading the function description inside of a bot project using the lib. I felt like I needed to look up whether the function will return a user ID or local part.

[jaller94]

Ahhhh, is this possibly to indicate that this function will always use the sender_localpart from the registration file, instead of any user_id override?

If this is the case, IMO it should say "based on the sender_localpart from the registration file" to be more clear.