meocord 4.1.0 → 4.1.1

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 # meocord
 
+## 4.1.1
+
+### Patch Changes
+
+- [#424](https://github.com/meocord/meocord/pull/424) [`43b9454`](https://github.com/meocord/meocord/commit/43b945447262405f61cced6660cd1d2be881d4b8) Thanks [@l7aromeo](https://github.com/l7aromeo)! - A mock guild's `members.fetch({ user: ids })` now resolves to a collection of each of those members, found in the cache or made the way `members.fetch(id)` makes one. It resolved to an empty collection, so in a test a message command naming two or more members by ID that the guild hasn't cached refused them all as not members of the server, while one such member resolved. Mentioned members were not affected: they come from the message's mentions.
+
+- [#422](https://github.com/meocord/meocord/pull/422) [`22daff2`](https://github.com/meocord/meocord/commit/22daff26ac4a694a8d70eace38a4c038d3cef0ef) Thanks [@l7aromeo](https://github.com/l7aromeo)! - `createMockClient()` is typed as a logged-in client, `DeepMocked<Client<true>>`, as the mock already behaves. Pass it straight to `createMockInteraction`, `createMockChannel`, `createMockUser` or `module.init({ ready: { client } })`: each of these failed to compile without a cast, since an interaction's client is `Client<true>`. `client.user` is no longer nullable, so `client.user!.id` can be `client.user.id`. Remove any `as unknown as Client<true>` cast on the mock.
+
+- [#430](https://github.com/meocord/meocord/pull/430) [`1d0c0cf`](https://github.com/meocord/meocord/commit/1d0c0cf7d9755e5d24a0f218809a2f95fe403ccc) Thanks [@l7aromeo](https://github.com/l7aromeo)! - Mocks from `meocord/testing` now compute what discord.js computes from them, where they returned `undefined` or a placeholder object:
+
+  - **Lookups:** a manager's `resolve()` and `resolveId()` read its cache, so `guild.members.resolve(id)` finds a member given to `createMockGuild`. `guild.members.me` is the bot's member, the cached one or one with @everyone. A guild's roles and channels, and the guilds of a client's messages and interactions, belong where discord.js puts them, and everything in a guild shares its client.
+  - **Ranking and permissions:** `role.comparePositionTo()`, `channel.permissionsFor()`, `member.permissionsIn()` and an interaction's `appPermissions` compute from the roles, the channel's `permissionOverwrites` and the bot's member. Being real, they keep working after `resetAllMocks()`.
+  - **Messages:** `message.member` reads the guild's member cache, so a test that removes the author's member there gets `null`, as discord.js gives for an author it hasn't cached. A default message now caches its author and the author's member, as Discord's message event does. `message.mentions.has(user)` answers from what the content mentions.
+  - **Defaults:** a role is not hoisted, managed or mentionable, with no icon, tags or colours. A member's `joinedTimestamp` is when the mock was made, so `partial` is `false`. A message has no `reference`, `poll` or stickers. `avatarURL()` and `iconURL()` are `null` without an avatar or icon, `displayAvatarURL()` is Discord's default avatar, and a mock client `isReady()`.
+  - **User context menus:** `targetUser` is the user with `targetId`, the client's cached one or one made, and `targetMember` that user's member in a server. A `targetId` given later picks its user, and both stay assignable.
+
+- [#431](https://github.com/meocord/meocord/pull/431) [`b24a0bc`](https://github.com/meocord/meocord/commit/b24a0bca78b72c602718a5a8f496251484f37bfe) Thanks [@l7aromeo](https://github.com/l7aromeo)! - A mock still reads some values discord.js computes as a truthy placeholder: a message's `editable`, `deletable`, `pinnable`, `crosspostable`, `bulkDeletable`, `hasThread` and `partial`, a member's `manageable`, `kickable`, `bannable` and `moderatable`, a role's `editable`, a channel's `viewable`, `manageable` and `deletable` and its thread and voice counterparts, and `partial` on users, channels and reactions. A test that reads one now gets a warning, once per run, that 5.0 computes it as discord.js does, and how to set it on the mock, such as `message.editable = false`, to test either way. A value the test sets is read without a warning.
+
+  `message.thread` is the thread the message's channel caches under the message's id, as discord.js reads it. Without one it is still a placeholder thread, with a warning that 5.0 gives `null` there.
+
+- [#429](https://github.com/meocord/meocord/pull/429) [`961455d`](https://github.com/meocord/meocord/commit/961455d86534be61c2b22739240fa48d53564dd7) Thanks [@l7aromeo](https://github.com/l7aromeo)! - Mocks from `meocord/testing` now pass to one another, and to the testing module, without a cast:
+
+  - **A mock reached through a property that may be `null`** keeps its mock API, such as `message.member?.fetch.mockResolvedValue(member)` or `message.guild?.members.fetch`.
+  - **An overloaded method's mock** takes what any of its overloads takes or resolves to, such as `guild.members.fetch.mockResolvedValue(member)` and `channel.messages.fetch.mockResolvedValue(message)`, where only the list form compiled before.
+  - **`createMockInteraction` given `guild: null`, `member: null` or a DM channel** builds an interaction outside a server, typed so its `guild` and `member` may be `null`.
+  - **`createMockMessage`** is typed as discord.js emits a message, so `module.emit('messageCreate', createMockMessage())` compiles.
+  - **`createMockChannel(ThreadChannel)`** is typed as a public or private thread, as discord.js types every thread, so it fits an interaction's `channel`, a guild's channels and the `threadCreate` event.
+  - **`getResponse(interaction).calls[n].payload`** is typed by the call's `method`: after `call.method === 'reply'`, it is the reply's options. `ResponseCall` itself is unchanged, so code that builds or extends one compiles as before.
+
 ## 4.1.0
 
 ### Highlights

package/dist/cjs/_shared/{controller.decorator-Ds7dj8uZ.cjs → controller.decorator-BywwkU9O.cjs}

@@ -119,7 +119,7 @@ let problem;
     try {
         // Called through a variable so a bundler does not try to resolve the path while building the
         // application; the file is only there once the build is done.
-        const load = module$1.createRequire((typeof document === 'undefined' ? require('u' + 'rl').pathToFileURL(__filename).href : (_documentCurrentScript && _documentCurrentScript.tagName.toUpperCase() === 'SCRIPT' && _documentCurrentScript.src || new URL('_shared/controller.decorator-Ds7dj8uZ.cjs', document.baseURI).href)));
+        const load = module$1.createRequire((typeof document === 'undefined' ? require('u' + 'rl').pathToFileURL(__filename).href : (_documentCurrentScript && _documentCurrentScript.tagName.toUpperCase() === 'SCRIPT' && _documentCurrentScript.src || new URL('_shared/controller.decorator-BywwkU9O.cjs', document.baseURI).href)));
         const loaded = load(compiledPath);
         return loaded.default ?? loaded;
     } catch (error) {
@@ -346,6 +346,12 @@ const warned = new Set();
  */ function warnDeprecatedBehaviour(logger, old, outcome, replacement) {
     warnOnce(logger, `${old} is deprecated; in the next major version (5.0) it ${outcome}. Use ${replacement} instead.`);
 }
+/**
+ * Warns once per run that a mock reads `value` as a placeholder where discord.js computes it, saying what 5.0 gives
+ * and how to set it.
+ */ function warnPlaceholder(logger, value, outcome, how) {
+    warnOnce(logger, `${value} reads a placeholder here; in the next major version (5.0) ${outcome}. ${how}`);
+}
 function warnOnce(logger, text) {
     if (warned.has(text)) return;
     warned.add(text);
@@ -5142,6 +5148,7 @@ exports.useTheme = useTheme;
 exports.userWords = userWords;
 exports.warnDeprecated = warnDeprecated;
 exports.warnDeprecatedBehaviour = warnDeprecatedBehaviour;
+exports.warnPlaceholder = warnPlaceholder;
 exports.withRelease = withRelease;
 exports.withSendableFiles = withSendableFiles;
 exports.withoutFiles = withoutFiles;

package/dist/cjs/_shared/{providers-BSL8vml4.cjs → providers-BMMLp2SN.cjs}

@@ -1,6 +1,6 @@
 'use strict';
 
-var controller_decorator = require('./controller.decorator-Ds7dj8uZ.cjs');
+var controller_decorator = require('./controller.decorator-BywwkU9O.cjs');
 require('reflect-metadata');
 var node_async_hooks = require('node:async_hooks');
 var inversify = require('inversify');

package/dist/cjs/_shared/{sharded-cooldown-store-BGfooXEO.cjs → sharded-cooldown-store-B65iaiAy.cjs}

@@ -1,7 +1,7 @@
 'use strict';
 
 var node_crypto = require('node:crypto');
-var controller_decorator = require('./controller.decorator-Ds7dj8uZ.cjs');
+var controller_decorator = require('./controller.decorator-BywwkU9O.cjs');
 
 /** The `fatal` code of a shard whose app MeoCord refused, which every shard would refuse alike. */ const REFUSED_CODE = 'Refused';
 /** Whether an IPC message is one of MeoCord's own, rather than one of discord.js's. */ function isShardMessage(message) {

package/dist/cjs/_shared/{startup-roots-H1ssJOrz.cjs → startup-roots-lFc69AxC.cjs}

@@ -1,10 +1,10 @@
 'use strict';
 
 var discord_js = require('discord.js');
-var controller_decorator = require('./controller.decorator-Ds7dj8uZ.cjs');
-var providers = require('./providers-BSL8vml4.cjs');
+var controller_decorator = require('./controller.decorator-BywwkU9O.cjs');
+var providers = require('./providers-BMMLp2SN.cjs');
 var enum_index = require('../enum/index.cjs');
-var translateError = require('./translate-error-Zwll3hC2.cjs');
+var translateError = require('./translate-error-BnM8FbSH.cjs');
 
 /**
  * Every customId-pattern route of the given controllers, most specific first, read from metadata alone.

package/dist/cjs/_shared/{translate-error-Zwll3hC2.cjs → translate-error-BnM8FbSH.cjs}

@@ -1,6 +1,6 @@
 'use strict';
 
-var controller_decorator = require('./controller.decorator-Ds7dj8uZ.cjs');
+var controller_decorator = require('./controller.decorator-BywwkU9O.cjs');
 var discord_js = require('discord.js');
 
 /** The longest customId Discord accepts. */ const MAX_CUSTOM_ID_LENGTH = 100;

package/dist/cjs/common/index.cjs

@@ -1,9 +1,9 @@
 'use strict';
 
-var controller_decorator = require('../_shared/controller.decorator-Ds7dj8uZ.cjs');
+var controller_decorator = require('../_shared/controller.decorator-BywwkU9O.cjs');
 require('reflect-metadata');
-var translateError = require('../_shared/translate-error-Zwll3hC2.cjs');
-var shardedCooldownStore = require('../_shared/sharded-cooldown-store-BGfooXEO.cjs');
+var translateError = require('../_shared/translate-error-BnM8FbSH.cjs');
+var shardedCooldownStore = require('../_shared/sharded-cooldown-store-B65iaiAy.cjs');
 require('../enum/index.cjs');
 require('discord.js');
 require('path');

package/dist/cjs/core/index.cjs

@@ -1,14 +1,14 @@
 'use strict';
 
 require('reflect-metadata');
-var providers = require('../_shared/providers-BSL8vml4.cjs');
+var providers = require('../_shared/providers-BMMLp2SN.cjs');
 var inversify = require('inversify');
 var discord_js = require('discord.js');
-var controller_decorator = require('../_shared/controller.decorator-Ds7dj8uZ.cjs');
+var controller_decorator = require('../_shared/controller.decorator-BywwkU9O.cjs');
 require('node:crypto');
-var shardedCooldownStore = require('../_shared/sharded-cooldown-store-BGfooXEO.cjs');
+var shardedCooldownStore = require('../_shared/sharded-cooldown-store-B65iaiAy.cjs');
 var path$1 = require('path');
-var startupRoots = require('../_shared/startup-roots-H1ssJOrz.cjs');
+var startupRoots = require('../_shared/startup-roots-lFc69AxC.cjs');
 var enum_index = require('../enum/index.cjs');
 var node_fs = require('node:fs');
 var path = require('node:path');
@@ -23,7 +23,7 @@ require('dayjs/plugin/utc.js');
 require('dayjs/plugin/timezone.js');
 require('module');
 require('chalk');
-require('../_shared/translate-error-Zwll3hC2.cjs');
+require('../_shared/translate-error-BnM8FbSH.cjs');
 
 function _interopDefaultCompat (e) { return e && typeof e === 'object' && 'default' in e ? e : { default: e }; }
 

package/dist/cjs/decorator/index.cjs

@@ -1,8 +1,8 @@
 'use strict';
 
 require('reflect-metadata');
-var providers = require('../_shared/providers-BSL8vml4.cjs');
-var controller_decorator = require('../_shared/controller.decorator-Ds7dj8uZ.cjs');
+var providers = require('../_shared/providers-BMMLp2SN.cjs');
+var controller_decorator = require('../_shared/controller.decorator-BywwkU9O.cjs');
 var inversify = require('inversify');
 require('node:async_hooks');
 require('../enum/index.cjs');