discord.js 15.0.0-dev.1735819501-9fea0698a → 15.0.0-dev.1735905892-bacc08b45

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "discord.js",
-  "version": "15.0.0-dev.1735819501-9fea0698a",
+  "version": "15.0.0-dev.1735905892-bacc08b45",
   "description": "A powerful library for interacting with the Discord API",
   "main": "./src/index.js",
   "types": "./typings/index.d.ts",
@@ -60,10 +60,10 @@
     "lodash.snakecase": "4.1.1",
     "tslib": "^2.8.1",
     "undici": "6.21.0",
-    "@discordjs/rest": "^2.4.0",
-    "@discordjs/util": "^1.1.1",
     "@discordjs/formatters": "^0.5.0",
-    "@discordjs/ws": "^2.0.0"
+    "@discordjs/rest": "^2.4.0",
+    "@discordjs/ws": "^2.0.0",
+    "@discordjs/util": "^1.1.1"
   },
   "devDependencies": {
     "@favware/cliff-jumper": "^4.1.0",
@@ -81,8 +81,8 @@
     "turbo": "^2.3.3",
     "typescript": "~5.5.4",
     "@discordjs/api-extractor": "^7.38.1",
-    "@discordjs/scripts": "^0.1.0",
-    "@discordjs/docgen": "^0.12.1"
+    "@discordjs/docgen": "^0.12.1",
+    "@discordjs/scripts": "^0.1.0"
   },
   "engines": {
     "node": ">=20"

package/src/errors/ErrorCodes.js

@@ -129,6 +129,9 @@
  * @property {'BulkBanUsersOptionEmpty'} BulkBanUsersOptionEmpty
 
  * @property {'PollAlreadyExpired'} PollAlreadyExpired
+
+ * @property {'PermissionOverwritesTypeMandatory'} PermissionOverwritesTypeMandatory
+ * @property {'PermissionOverwritesTypeMismatch'} PermissionOverwritesTypeMismatch
  */
 
 const keys = [
@@ -258,6 +261,9 @@ const keys = [
   'BulkBanUsersOptionEmpty',
 
   'PollAlreadyExpired',
+
+  'PermissionOverwritesTypeMandatory',
+  'PermissionOverwritesTypeMismatch',
 ];
 
 // JSDoc for IntelliSense purposes

package/src/errors/Messages.js

@@ -144,6 +144,11 @@ const Messages = {
   [DjsErrorCodes.BulkBanUsersOptionEmpty]: 'Option "users" array or collection is empty',
 
   [DjsErrorCodes.PollAlreadyExpired]: 'This poll has already expired.',
+
+  [DjsErrorCodes.PermissionOverwritesTypeMandatory]: '"overwrite.type" is mandatory if "overwrite.id" is a Snowflake',
+  [DjsErrorCodes.PermissionOverwritesTypeMismatch]: expected =>
+    `"overwrite.id" is a ${expected.toLowerCase()} object, ` +
+    `but "overwrite.type" is defined and not equal to OverwriteType.${expected}`,
 };
 
 module.exports = Messages;

package/src/structures/PermissionOverwrites.js

@@ -160,7 +160,7 @@ class PermissionOverwrites extends Base {
    * @property {GuildMemberResolvable|RoleResolvable} id Member or role this overwrite is for
    * @property {PermissionResolvable} [allow] The permissions to allow
    * @property {PermissionResolvable} [deny] The permissions to deny
-   * @property {OverwriteType} [type] The type of this OverwriteData
+   * @property {OverwriteType} [type] The type of this OverwriteData (mandatory if `id` is a Snowflake)
    */
 
   /**
@@ -171,21 +171,31 @@ class PermissionOverwrites extends Base {
    */
   static resolve(overwrite, guild) {
     if (overwrite instanceof this) return overwrite.toJSON();
-    if (typeof overwrite.id === 'string' && overwrite.type in OverwriteType) {
-      return {
-        id: overwrite.id,
-        type: overwrite.type,
-        allow: PermissionsBitField.resolve(overwrite.allow ?? PermissionsBitField.DefaultBit).toString(),
-        deny: PermissionsBitField.resolve(overwrite.deny ?? PermissionsBitField.DefaultBit).toString(),
-      };
+
+    const id = guild.roles.resolveId(overwrite.id) ?? guild.client.users.resolveId(overwrite.id);
+    if (!id) {
+      throw new DiscordjsTypeError(ErrorCodes.InvalidType, 'overwrite.id', 'GuildMemberResolvable or RoleResolvable');
+    }
+
+    if (overwrite.type !== undefined && (typeof overwrite.type !== 'number' || !(overwrite.type in OverwriteType))) {
+      throw new DiscordjsTypeError(ErrorCodes.InvalidType, 'overwrite.type', 'OverwriteType', true);
     }
 
-    const userOrRole = guild.roles.cache.get(overwrite.id) ?? guild.client.users.cache.get(overwrite.id);
-    if (!userOrRole) throw new DiscordjsTypeError(ErrorCodes.InvalidType, 'parameter', 'User nor a Role');
-    const type = userOrRole instanceof Role ? OverwriteType.Role : OverwriteType.Member;
+    let type;
+    if (typeof overwrite.id === 'string') {
+      if (overwrite.type === undefined) {
+        throw new DiscordjsTypeError(ErrorCodes.PermissionOverwritesTypeMandatory);
+      }
+      type = overwrite.type;
+    } else {
+      type = overwrite.id instanceof Role ? OverwriteType.Role : OverwriteType.Member;
+      if (overwrite.type !== undefined && type !== overwrite.type) {
+        throw new DiscordjsTypeError(ErrorCodes.PermissionOverwritesTypeMismatch, OverwriteType[type]);
+      }
+    }
 
     return {
-      id: userOrRole.id,
+      id,
       type,
       allow: PermissionsBitField.resolve(overwrite.allow ?? PermissionsBitField.DefaultBit).toString(),
       deny: PermissionsBitField.resolve(overwrite.deny ?? PermissionsBitField.DefaultBit).toString(),

package/typings/index.d.mts

@@ -6519,13 +6519,23 @@ export interface MultipleShardSpawnOptions {
   timeout?: number;
 }
 
-export interface OverwriteData {
+export interface BaseOverwriteData {
   allow?: PermissionResolvable;
   deny?: PermissionResolvable;
   id: GuildMemberResolvable | RoleResolvable;
   type?: OverwriteType;
 }
 
+export interface OverwriteDataWithMandatoryType extends BaseOverwriteData {
+  type: OverwriteType;
+}
+
+export interface OverwriteDataWithOptionalType extends BaseOverwriteData {
+  id: Exclude<GuildMemberResolvable | RoleResolvable, Snowflake>;
+}
+
+export type OverwriteData = OverwriteDataWithMandatoryType | OverwriteDataWithOptionalType;
+
 export type OverwriteResolvable = PermissionOverwrites | OverwriteData;
 
 export type PermissionFlags = Record<keyof typeof PermissionFlagsBits, bigint>;

package/typings/index.d.ts

@@ -6519,13 +6519,23 @@ export interface MultipleShardSpawnOptions {
   timeout?: number;
 }
 
-export interface OverwriteData {
+export interface BaseOverwriteData {
   allow?: PermissionResolvable;
   deny?: PermissionResolvable;
   id: GuildMemberResolvable | RoleResolvable;
   type?: OverwriteType;
 }
 
+export interface OverwriteDataWithMandatoryType extends BaseOverwriteData {
+  type: OverwriteType;
+}
+
+export interface OverwriteDataWithOptionalType extends BaseOverwriteData {
+  id: Exclude<GuildMemberResolvable | RoleResolvable, Snowflake>;
+}
+
+export type OverwriteData = OverwriteDataWithMandatoryType | OverwriteDataWithOptionalType;
+
 export type OverwriteResolvable = PermissionOverwrites | OverwriteData;
 
 export type PermissionFlags = Record<keyof typeof PermissionFlagsBits, bigint>;