From 2792e48119f1cf5fa7d5e6b63369457b0719d4e4 Mon Sep 17 00:00:00 2001 From: Jaw0r3k Date: Sun, 2 Apr 2023 01:21:53 +0200 Subject: [PATCH] docs: describe private properties (#8879) * feat: describe private properties * Update packages/discord.js/src/structures/GuildMember.js Co-authored-by: MrMythicalYT <91077061+MrMythicalYT@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: MrMythicalYT <91077061+MrMythicalYT@users.noreply.github.com> * Apply suggestions from code review --------- Co-authored-by: MrMythicalYT <91077061+MrMythicalYT@users.noreply.github.com> Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com> --- packages/discord.js/src/managers/CachedManager.js | 7 +++++++ packages/discord.js/src/structures/GuildMember.js | 5 +++++ packages/discord.js/src/structures/Presence.js | 12 ++++++++++++ packages/discord.js/typings/index.d.ts | 4 ++++ 4 files changed, 28 insertions(+) diff --git a/packages/discord.js/src/managers/CachedManager.js b/packages/discord.js/src/managers/CachedManager.js index 1058285d3..e0ce0f8a5 100644 --- a/packages/discord.js/src/managers/CachedManager.js +++ b/packages/discord.js/src/managers/CachedManager.js @@ -11,6 +11,13 @@ class CachedManager extends DataManager { constructor(client, holds, iterable) { super(client, holds); + /** + * The private cache of items for this manager. + * @type {Collection} + * @private + * @readonly + * @name CachedManager#_cache + */ Object.defineProperty(this, '_cache', { value: this.client.options.makeCache(this.constructor, this.holds) }); if (iterable) { diff --git a/packages/discord.js/src/structures/GuildMember.js b/packages/discord.js/src/structures/GuildMember.js index d22c174fa..7c2aec177 100644 --- a/packages/discord.js/src/structures/GuildMember.js +++ b/packages/discord.js/src/structures/GuildMember.js @@ -54,6 +54,11 @@ class GuildMember extends Base { */ this.communicationDisabledUntilTimestamp = null; + /** + * The role ids of the member + * @type {Snowflake[]} + * @private + */ this._roles = []; if (data) this._patch(data); } diff --git a/packages/discord.js/src/structures/Presence.js b/packages/discord.js/src/structures/Presence.js index b642f0ab9..1a3cde067 100644 --- a/packages/discord.js/src/structures/Presence.js +++ b/packages/discord.js/src/structures/Presence.js @@ -141,6 +141,12 @@ class Presence extends Base { */ class Activity { constructor(presence, data) { + /** + * The presence of the Activity + * @type {Presence} + * @readonly + * @name Activity#presence + */ Object.defineProperty(this, 'presence', { value: presence }); /** @@ -287,6 +293,12 @@ class Activity { */ class RichPresenceAssets { constructor(activity, assets) { + /** + * The activity of the RichPresenceAssets + * @type {Activity} + * @readonly + * @name RichPresenceAssets#activity + */ Object.defineProperty(this, 'activity', { value: activity }); /** diff --git a/packages/discord.js/typings/index.d.ts b/packages/discord.js/typings/index.d.ts index 282e81b0d..fb06027ce 100644 --- a/packages/discord.js/typings/index.d.ts +++ b/packages/discord.js/typings/index.d.ts @@ -236,6 +236,7 @@ declare module 'node:events' { export class Activity { private constructor(presence: Presence, data?: RawActivityData); + public readonly presence: Presence; public applicationId: Snowflake | null; public assets: RichPresenceAssets | null; public buttons: string[]; @@ -1505,6 +1506,7 @@ export class GuildMemberFlagsBitField extends BitField { export class GuildMember extends PartialTextBasedChannel(Base) { private constructor(client: Client, data: RawGuildMemberData, guild: Guild); + private _roles: Snowflake[]; public avatar: string | null; public get bannable(): boolean; public get dmChannel(): DMChannel | null; @@ -2442,6 +2444,7 @@ export class ReactionEmoji extends Emoji { export class RichPresenceAssets { private constructor(activity: Activity, assets: RawRichPresenceAssets); + public readonly activity: Activity; public largeImage: Snowflake | null; public largeText: string | null; public smallImage: Snowflake | null; @@ -3678,6 +3681,7 @@ export abstract class DataManager extends BaseManager { export abstract class CachedManager extends DataManager { protected constructor(client: Client, holds: Constructable); + private readonly _cache: Collection; private _add(data: unknown, cache?: boolean, { id, extras }?: { id: K; extras: unknown[] }): Holds; }