meocord 4.1.0-beta.8 → 4.1.0-beta.9

package/CHANGELOG.md

@@ -1,5 +1,44 @@
 # meocord
 
+## 4.1.0-beta.9
+
+### Minor Changes
+
+- [#280](https://github.com/meocord/meocord/pull/280) [`c79cd44`](https://github.com/meocord/meocord/commit/c79cd44c4badc87f26a936e57fa1cc33f4cc4136) Thanks [@l7aromeo](https://github.com/l7aromeo)! - A translation's `{params}` are checked against the default catalog's. Until now only its keys were, so a translation that misspelt a param, such as `'Diblokir {usr}.'` for `'Banned {user}.'`, compiled and showed the user `{usr}` as written.
+
+  - **When the code compiles**, `createTranslator` refuses a message that uses a `{param}` its default message doesn't take, and a plural's form may use `{count}` besides. The error names each one, such as `id: ban.done takes no {usr}; the default is "Banned {user}."`. A translation may use the default's params in any order, and leave some out. The compiler reads a message's params only from a catalog whose text it keeps: one made with `defineCatalog`, written with `as const`, or written inline.
+  - **When a test runs**, `expectCompleteCatalog` from `meocord/testing` reports the same mistakes from the catalogs' own strings, so a catalog from a plain variable or a JSON file is checked too. It also reports a `{param}` that a translation of MeoCord's own texts uses and MeoCord's English doesn't take.
+
+  A `{param}` is the same thing in every check and when translating: ASCII letters, digits or `_` between braces. Other text in braces, such as `Wrap text in { and }.`, is the message's own. So a default message with such braces no longer makes `t('wrap')` ask for params it doesn't use. A message may also hold hundreds of params: one with 47 or more failed to compile with "Type instantiation is excessively deep and possibly infinite".
+
+  If your build or a test now fails, rename the param to the one the default message uses. A translation may leave a param out, but it can't add one: the translator only fills the params the default message names.
+
+- [#281](https://github.com/meocord/meocord/pull/281) [`c59ddc2`](https://github.com/meocord/meocord/commit/c59ddc28ab25a123c87fbb4f6c6dd7f394a63efb) Thanks [@l7aromeo](https://github.com/l7aromeo)! - `createMockMessage` takes an `author`, so a test can send several messages as one user. Before, every mock message came from a new person, so a per-user `@Cooldown` or a check on who sent a message couldn't be tested through `dispatch()` without assigning `message.author` by hand.
+
+  ```ts
+  const author = createMockUser()
+  await module.dispatch(createMockMessage({ author, content: '!daily' }))
+  await module.dispatch(createMockMessage({ author, content: '!daily' })) // refused by the cooldown
+  ```
+
+  - The author is cached on the message's client, so a mention of it resolves to the same user.
+  - In a server, the message's `member` is the guild's cached member for that user, such as one given to `createMockGuild({ members })`, or a new member with the author's id, which is then cached. Every message from that author in one server (the same `guild` given to each) has the same member.
+  - `author: client.user` gives a message the bot itself sent.
+  - An interaction's `member` works the same way: for a `user` the test gives, it is the `guild`'s cached member for that user, or a new member with its id, user and guild, which is then cached. A message and an interaction from one user in one server share the member, whichever is made first.
+  - A message built without `author` is unchanged.
+
+### Patch Changes
+
+- [#277](https://github.com/meocord/meocord/pull/277) [`55fe3e4`](https://github.com/meocord/meocord/commit/55fe3e4db223b342b7951e8e36e83cd479cfd21d) Thanks [@l7aromeo](https://github.com/l7aromeo)! - More mistakes MeoCord refuses as the bot loads are reported as one line, naming what to change first, instead of `Error during startup:` and a stack. This covers two classes of one name when either uses `@Cooldown` or `@Once`, `@Validate`, `@UsePipe` or `@Cooldown` on a handler they don't apply to, two component `customId` patterns that match the same ids, `sharding` settings in `meocord.config.ts` that `clientOptions` contradicts, and a self-contained build started on a platform its native addons weren't built for.
+
+  - Two component patterns that match the same ids, and the warning about two that can, are now reported by `MeoCordFactory.create()`, before `start()` attaches anything, and `meocord register` reports them too.
+  - The messages lead with the class, handler or file they are about, such as `Shop: two classes have this name; …`. A test that matches the old wording needs updating.
+
+- [#278](https://github.com/meocord/meocord/pull/278) [`2d26fc3`](https://github.com/meocord/meocord/commit/2d26fc397808f6888405690a5dbeedf3e912f93e) Thanks [@l7aromeo](https://github.com/l7aromeo)! - A production build keeps every class's own name. Before, when two modules declared a class of the same name, even a helper that never reaches MeoCord, `meocord build --prod` renamed one of them, such as `Shop` to `shop_controller_Shop`. A development build and your tests kept `Shop`. So in production, cooldowns were counted under the renamed class, errors and logs named it, and `ExecutionContext.getClass().name` returned it.
+
+  - If a controller was renamed this way, its cooldowns start over once, when you deploy this version. Nothing to do: the counts under the old name expire on their own.
+  - Two classes of one name are now refused in production as they already were in development and tests. That covers two controllers where either uses `@Cooldown` or `@Once`, and any two controllers or services under process sharding. If your bot stops at startup with this refusal, rename one of the two classes; the message names it.
+
 ## 4.1.0-beta.8
 
 ### Minor Changes

package/dist/cjs/_shared/{providers-CSLRsJOZ.cjs → providers-CyQkBU8g.cjs}

@@ -1,7 +1,7 @@
 'use strict';
 
 require('reflect-metadata');
-var themeResolvers = require('./theme-resolvers-CVm6n5-Q.cjs');
+var themeResolvers = require('./theme-resolvers-DLgZL-uE.cjs');
 var discord_js = require('discord.js');
 var enum_index = require('../enum/index.cjs');
 var node_async_hooks = require('node:async_hooks');

package/dist/cjs/_shared/{shard-context-DV-BymqL.cjs → shard-context-BWzzLmaJ.cjs}

@@ -1,9 +1,9 @@
 'use strict';
 
-var themeResolvers = require('./theme-resolvers-CVm6n5-Q.cjs');
-var translateError = require('./translate-error-Bts5vGXt.cjs');
+var themeResolvers = require('./theme-resolvers-DLgZL-uE.cjs');
+var translateError = require('./translate-error-yLJa8q4Q.cjs');
 var discord_js = require('discord.js');
-var providers = require('./providers-CSLRsJOZ.cjs');
+var providers = require('./providers-CyQkBU8g.cjs');
 require('reflect-metadata');
 var enum_index = require('../enum/index.cjs');
 var crypto = require('crypto');
@@ -11,6 +11,8 @@ var fs = require('fs');
 var path = require('path');
 var Table = require('cli-table3');
 require('node:crypto');
+require('node:fs');
+require('node:url');
 
 const NO_GLOBAL_STAGES = {
     guards: [],
@@ -21,7 +23,7 @@ const NO_GLOBAL_STAGES = {
 /** The global stages declared by `@MeoCord` on an application class. */ function appStages(app) {
     const options = Reflect.getMetadata(enum_index.MetadataKey.AppOptions, app);
     if (!options) {
-        throw new Error(`${app.name || 'The app'} is not decorated with @MeoCord().`);
+        throw themeResolvers.refuse(new Error(`${app.name || 'The app class'}: not decorated with @MeoCord().`));
     }
     return {
         guards: [
@@ -133,7 +135,7 @@ const stagesByGlobals = new WeakMap();
     for (const cls of classes){
         const other = byName.get(cls.name);
         if (other && other !== cls && (keyedByName(cls) || keyedByName(other))) {
-            throw new Error(`Two classes are named ${cls.name}, and @Cooldown and @Once tell classes apart by name, so they would share counts. ` + `Rename one of them.`);
+            throw themeResolvers.refuse(new Error(`${cls.name}: two classes have this name; @Cooldown and @Once tell classes apart by name, so they would share ` + `their counts. Rename one of them.`));
         }
         byName.set(cls.name, cls);
     }
@@ -159,13 +161,13 @@ const stagesByGlobals = new WeakMap();
     ];
     for (const [method, kind] of others){
         const { schema, pipes } = providers.handlerInputStages(prototype, method);
+        const handler = kind === 'message' ? 'a message handler without a pattern' : `${kind === 'autocomplete' || kind === 'event' ? 'an' : 'a'} ${kind} handler`;
         if (schema || pipes.length > 0) {
-            const handler = kind === 'message' ? 'a message handler without a pattern' : `${kind === 'autocomplete' || kind === 'event' ? 'an' : 'a'} ${kind} handler`;
-            throw new Error(`${controller.name}.${method} is ${handler}; @Validate and @UsePipe apply only to interaction and patterned ` + `message handlers, whose options, customId params, modal fields and pattern params they check.`);
+            throw themeResolvers.refuse(new Error(`${controller.name}.${method}: @Validate and @UsePipe are for interaction and patterned message handlers, ` + `whose options, customId params, modal fields and pattern params they check, and this is ${handler}.`));
         }
         // A controller's own @Cooldown skips these handlers; one on the method itself is a mistake.
         if (kind !== 'message' && providers.methodCooldowns(prototype, method).length > 0) {
-            throw new Error(`${controller.name}.${method} is ${kind === 'autocomplete' || kind === 'event' ? 'an' : 'a'} ${kind} handler; @Cooldown applies only to ` + `interaction and message handlers.`);
+            throw themeResolvers.refuse(new Error(`${controller.name}.${method}: @Cooldown is for interaction and message handlers, and this is ${handler}.`));
         }
     }
 }
@@ -1129,7 +1131,7 @@ function logRegistered(target, logger) {
                 }
                 // One handler under two spellings, such as 'card/{id}' and 'card/{cardId}', is one route
                 if (earlier.controllerClass === controllerClass && earlier.meta.methodName === meta.methodName) continue;
-                throw new Error(`"${earlier.pattern}" in ${earlier.controllerClass.name}.${earlier.meta.methodName} and "${pattern}" in ` + `${controllerClass.name}.${meta.methodName} match the same ${typeLabel(meta.type)} customIds, so only one ` + `of them could ever run. Change one pattern.`);
+                throw themeResolvers.refuse(new Error(`${earlier.controllerClass.name}.${earlier.meta.methodName}: "${earlier.pattern}" and "${pattern}" in ` + `${controllerClass.name}.${meta.methodName} match the same ${typeLabel(meta.type)} customIds, so only one ` + `of them could ever run. Change one pattern.`));
             }
         }
     }
@@ -2386,7 +2388,7 @@ const sortedEntries = (entries)=>entries.sort((a, b)=>a.usage < b.usage ? -1 : a
         }
         return this.controllerInstancesCache.get(controllerClass);
     }
-    /** The component routes, built and checked for overlaps on first use; the app calls it at startup. */ getComponentRoutes() {
+    /** The component routes, built and checked for overlaps on first use; the app calls it as it is created. */ getComponentRoutes() {
         if (this.componentRoutes) return this.componentRoutes;
         const routes = buildComponentRoutes([
             ...this.controllerClasses

package/dist/cjs/_shared/{sharded-cooldown-store-CBKsDtvc.cjs → sharded-cooldown-store-ComEj8So.cjs}

@@ -1,7 +1,7 @@
 'use strict';
 
 var node_crypto = require('node:crypto');
-var themeResolvers = require('./theme-resolvers-CVm6n5-Q.cjs');
+var themeResolvers = require('./theme-resolvers-DLgZL-uE.cjs');
 
 /** Whether an IPC message is one of MeoCord's own, rather than one of discord.js's. */ function isShardMessage(message) {
     return typeof message === 'object' && message !== null && 'meocord' in message;
@@ -34,7 +34,7 @@ var themeResolvers = require('./theme-resolvers-CVm6n5-Q.cjs');
     const own = clientOptions.shards ?? clientOptions.shardCount;
     if (sharding.mode === 'process') {
         if (own !== undefined) {
-            throw new Error("sharding.mode 'process' starts one shard per process, so clientOptions.shards and shardCount must be unset.");
+            throw themeResolvers.refuse(new Error("meocord.config.ts: sharding.mode 'process' starts one shard per process, so clientOptions.shards and shardCount " + 'must be unset.'));
         }
         return clientOptions;
     }
@@ -54,7 +54,7 @@ var themeResolvers = require('./theme-resolvers-CVm6n5-Q.cjs');
     }
     const agrees = shards === 'auto' ? clientOptions.shards === 'auto' : clientOptions.shardCount === shards || Array.isArray(clientOptions.shards) && clientOptions.shards.length === shards;
     if (!agrees) {
-        throw new Error(`sharding.shards (${String(shards)}) and clientOptions.shards/shardCount disagree; set the shards in one place.`);
+        throw themeResolvers.refuse(new Error(`meocord.config.ts: sharding.shards (${String(shards)}) and clientOptions.shards/shardCount disagree; set the ` + 'shards in one place.'));
     }
     return clientOptions;
 }

package/dist/cjs/_shared/{theme-resolvers-CVm6n5-Q.cjs → theme-resolvers-DLgZL-uE.cjs}

@@ -38,7 +38,7 @@ let configLoaded = false;
     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/theme-resolvers-CVm6n5-Q.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/theme-resolvers-DLgZL-uE.cjs', document.baseURI).href)));
         const loaded = load(compiledPath);
         return loaded.default ?? loaded;
     } catch (error) {
@@ -708,7 +708,8 @@ const PLAIN = "give a plain object, such as { ...value }, or a row's toObject(),
  * A locale is resolved to the catalog that serves it: the exact locale, then another of the same language (`es-419` to
  * `es-ES`, `en-GB` to `en-US`), then the default. A message missing from that catalog is looked up the same way.
  *
- * @typeParam C - The default catalog, whose keys and params every locale is checked against.
+ * @typeParam C - The default catalog. Every locale's keys are checked against it, and so are the `{params}` of
+ *   each message whose text the compiler can read; see {@link createTranslator}.
  *
  * @example
  * ```ts
@@ -754,8 +755,14 @@ const plurals = new Map();
     if (!rules) plurals.set(locale, rules = new Intl.PluralRules(locale));
     return message[rules.select(Number(count))] ?? message.other;
 }
+/** A `{param}`: word characters between braces, the reading `Placeholders` follows at compile time. */ const PLACEHOLDER = /\{(\w+)}/g;
+/** The `{param}` names of a message, in order, as translating fills them. */ function placeholderNames(message) {
+    return [
+        ...message.matchAll(PLACEHOLDER)
+    ].map(([, name])=>name);
+}
 function interpolate(message, params) {
-    return message.replace(/\{(\w+)}/g, (whole, name)=>Object.hasOwn(params, name) ? String(params[name]) : whole);
+    return message.replace(PLACEHOLDER, (whole, name)=>Object.hasOwn(params, name) ? String(params[name]) : whole);
 }
 class CatalogTranslator extends Translator {
     /** The catalogs that serve a locale, most specific first, ending with the default. */ [LOCALE_CHAIN](requested) {
@@ -823,6 +830,19 @@ class CatalogTranslator extends Translator {
  * Create it at module scope: command builders run when their class is decorated, before any container exists, and use
  * it for names and descriptions. Pass the same instance to `@MeoCord({ i18n })` to inject it as `Translator`.
  *
+ * Each locale is checked against the default catalog when the code compiles: every key must be one the default has,
+ * and a message may use only the `{params}` the default's message takes, and `{count}` in a plural's forms, in any
+ * order, all or some. The error names each mistake, such as:
+ *
+ * ```text
+ * id: ban.done takes no {usr}; the default is "Banned {user}."
+ * ```
+ *
+ * @remarks
+ * The compiler reads a message's params only from its text, which a catalog made with `defineCatalog`, written with
+ * `as const` or written inline keeps. A catalog from a plain variable or a JSON file types each message as `string`, so
+ * its params are checked by `expectCompleteCatalog` from `meocord/testing` instead, when a test runs.
+ *
  * @param options.default - The locale whose catalog is the reference and the last fallback.
  * @param options.locales - A catalog per discord.js `Locale`, including the default's.
  * @throws When the default locale has no catalog, or a key is not a Discord locale.
@@ -3811,6 +3831,7 @@ exports.ownHandlerList = ownHandlerList;
 exports.ownsAmbientTheme = ownsAmbientTheme;
 exports.parseSegment = parseSegment;
 exports.patternShape = patternShape;
+exports.placeholderNames = placeholderNames;
 exports.refuse = refuse;
 exports.registerClientTranslator = registerClientTranslator;
 exports.releaseAmbientTheme = releaseAmbientTheme;

package/dist/cjs/_shared/{translate-error-Bts5vGXt.cjs → translate-error-yLJa8q4Q.cjs}

@@ -1,6 +1,6 @@
 'use strict';
 
-var themeResolvers = require('./theme-resolvers-CVm6n5-Q.cjs');
+var themeResolvers = require('./theme-resolvers-DLgZL-uE.cjs');
 var enum_index = require('../enum/index.cjs');
 var discord_js = require('discord.js');
 

package/dist/cjs/common/index.cjs

@@ -1,10 +1,10 @@
 'use strict';
 
-var themeResolvers = require('../_shared/theme-resolvers-CVm6n5-Q.cjs');
-var translateError = require('../_shared/translate-error-Bts5vGXt.cjs');
+var themeResolvers = require('../_shared/theme-resolvers-DLgZL-uE.cjs');
+var translateError = require('../_shared/translate-error-yLJa8q4Q.cjs');
 require('reflect-metadata');
 var node_crypto = require('node:crypto');
-var shardedCooldownStore = require('../_shared/sharded-cooldown-store-CBKsDtvc.cjs');
+var shardedCooldownStore = require('../_shared/sharded-cooldown-store-ComEj8So.cjs');
 require('discord.js');
 require('path');
 require('../enum/index.cjs');

package/dist/cjs/core/index.cjs

@@ -1,16 +1,16 @@
 'use strict';
 
 require('reflect-metadata');
-var providers = require('../_shared/providers-CSLRsJOZ.cjs');
+var providers = require('../_shared/providers-CyQkBU8g.cjs');
 var inversify = require('inversify');
 var discord_js = require('discord.js');
-var themeResolvers = require('../_shared/theme-resolvers-CVm6n5-Q.cjs');
-require('../_shared/translate-error-Bts5vGXt.cjs');
+var themeResolvers = require('../_shared/theme-resolvers-DLgZL-uE.cjs');
+require('../_shared/translate-error-yLJa8q4Q.cjs');
 require('node:crypto');
-var shardedCooldownStore = require('../_shared/sharded-cooldown-store-CBKsDtvc.cjs');
+var shardedCooldownStore = require('../_shared/sharded-cooldown-store-ComEj8So.cjs');
 var lodashEs = require('lodash-es');
 var enum_index = require('../enum/index.cjs');
-var shardContext = require('../_shared/shard-context-DV-BymqL.cjs');
+var shardContext = require('../_shared/shard-context-BWzzLmaJ.cjs');
 var fs = require('fs');
 var path = require('path');
 require('node:async_hooks');
@@ -601,8 +601,6 @@ class MeoCordApp {
         this.attachEventHandlers();
         this.warnAboutMissingRequirements();
         this.noteGlobalStagesOnEvents();
-        // Built now rather than on the first click, so overlapping patterns are reported at startup
-        this.dispatcher.getComponentRoutes();
         const login = this.bot.login(this.discordToken);
         const stopped = new Promise((_, reject)=>{
             this.abortLogin = ()=>{
@@ -893,7 +891,8 @@ class MeoCordApp {
         this.close = ()=>this.closeClient();
         this.lifecycleUnits = lifecycleUnits ?? providers.classUnits(container, lifecycleClasses);
         this.bot = this.discordClient;
-        // Built now, so a pattern that cannot be read or two that match the same messages stop the bot before login
+        // Built now, so a pattern that cannot be read or two that match the same messages or customIds stop the bot as it
+        // is created, before start() attaches anything
         this.dispatcher = new shardContext.Dispatcher({
             container,
             controllerClasses,
@@ -903,6 +902,7 @@ class MeoCordApp {
             botUserId: ()=>this.bot.user?.id,
             warnUnanswered
         });
+        this.dispatcher.getComponentRoutes();
         this.shutdownTimeout = typeof shutdownTimeout === 'number' && shutdownTimeout >= 0 ? shutdownTimeout : DEFAULT_SHUTDOWN_TIMEOUT_MS;
     }
 }
@@ -960,7 +960,7 @@ class MeoCordApp {
     }
     const running = currentPlatform();
     if (isSamePlatform(built, running)) return;
-    throw new Error(`This build carries native addons compiled for ${describePlatform(built)}, but is running on ` + `${describePlatform(running)}. Compiled binaries only load on the platform they were built for. ` + 'Build on the same platform you deploy to -- for a container, run `meocord build` inside the image.');
+    throw themeResolvers.refuse(new Error(`${path.basename(distDir)}: this build carries native addons compiled for ${describePlatform(built)}, but is ` + `running on ${describePlatform(running)}. Compiled binaries only load on the platform they were built for. ` + 'Build on the same platform you deploy to -- for a container, run `meocord build` inside the image.'));
 }
 
 /** How long the manager waits for a shard to become ready before spawning the next. */ const SHARD_READY_TIMEOUT_MS = 30000;
@@ -1233,8 +1233,9 @@ const sleep = (ms)=>new Promise((resolve)=>setTimeout(resolve, ms));
    * @param target - The class `@MeoCord` decorates.
    * @returns The application, which `start()` logs in.
    * @throws Error when the class has no `@MeoCord`, when the built config is missing, when a provider cannot
-   *   be bound, such as one for a token MeoCord binds itself, or when two handlers take one command, or two builder
-   *   classes build one, naming both. In a built application it is logged first, as one line, so
+   *   be bound, such as one for a token MeoCord binds itself, when two handlers take one command, or two builder
+   *   classes build one, naming both, and for any other mistake it refuses as the app loads, such as two component
+   *   patterns that match the same customIds. In a built application it is logged first, as one line, so
    *   `isExplainedError()` tells a caller not to log it again.
    */ static create(target) {
         try {

package/dist/cjs/decorator/index.cjs

@@ -1,8 +1,8 @@
 'use strict';
 
 require('reflect-metadata');
-var themeResolvers = require('../_shared/theme-resolvers-CVm6n5-Q.cjs');
-var providers = require('../_shared/providers-CSLRsJOZ.cjs');
+var themeResolvers = require('../_shared/theme-resolvers-DLgZL-uE.cjs');
+var providers = require('../_shared/providers-CyQkBU8g.cjs');
 var enum_index = require('../enum/index.cjs');
 var inversify = require('inversify');
 require('discord.js');

package/dist/cjs/testing/index.cjs

@@ -2,11 +2,11 @@
 
 require('reflect-metadata');
 var inversify = require('inversify');
-var providers = require('../_shared/providers-CSLRsJOZ.cjs');
+var providers = require('../_shared/providers-CyQkBU8g.cjs');
 var discord_js = require('discord.js');
 var enum_index = require('../enum/index.cjs');
-var themeResolvers = require('../_shared/theme-resolvers-CVm6n5-Q.cjs');
-var shardContext = require('../_shared/shard-context-DV-BymqL.cjs');
+var themeResolvers = require('../_shared/theme-resolvers-DLgZL-uE.cjs');
+var shardContext = require('../_shared/shard-context-BWzzLmaJ.cjs');
 var node_crypto = require('node:crypto');
 require('node:async_hooks');
 require('path');
@@ -19,7 +19,7 @@ require('module');
 require('node:fs');
 require('chalk');
 require('node:url');
-require('../_shared/translate-error-Bts5vGXt.cjs');
+require('../_shared/translate-error-yLJa8q4Q.cjs');
 require('crypto');
 require('cli-table3');
 
@@ -957,10 +957,12 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
  * @remarks
  * Type guards such as `isButton()` run the real discord.js logic. An interaction gets an `id`, a `channelId` and a
  * `user` of its own unless given, and its `locale` is `'en-US'`; without a `guildId` it is a DM, and with one its
- * `member` is its user. Other data Discord always sends reads as Discord sends it, such as `false` for a flag and
- * `null` for what may be absent; what picks the handler, `commandName` or `customId`, is the test's to give. Replies
- * follow Discord's order, so a second `reply()` rejects, and {@link getResponse} reports what `respond()` sent. Every
- * method is a mock function, and one that returns a promise in discord.js resolves.
+ * `member` is its user: for a `user` given, when the test gives the `guild`, the guild's cached member for that user,
+ * so a message and an interaction from one user in that server share it. Other data Discord always sends reads as
+ * Discord sends it, such as `false` for a flag and `null` for what may be absent; what picks the handler, `commandName`
+ * or `customId`, is the test's to give. Replies follow Discord's order, so a second `reply()` rejects, and
+ * {@link getResponse} reports what `respond()` sent. Every method is a mock function, and one that returns a promise in
+ * discord.js resolves.
  *
  * @param Class - The discord.js class to mock.
  * @param props - Values for properties the class declares `readonly`; see {@link MockProps}.
@@ -1119,7 +1121,8 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
         const unset = (key)=>!Object.prototype.hasOwnProperty.call(instance, key);
         const generatedId = unset('id') ? instance.id = nextSnowflake() : undefined;
         defineCreatedTime(instance, generatedId);
-        if (unset('user')) instance.user = mockUser();
+        const userGiven = !unset('user');
+        if (!userGiven) instance.user = mockUser();
         if (unset('channelId')) instance.channelId = nextSnowflake();
         if (unset('guildId')) {
             instance.guildId = null;
@@ -1133,7 +1136,7 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
         let member;
         if (unset('member')) {
             Object.defineProperty(instance, 'member', {
-                get: ()=>own('guildId') ? member ?? (member = memberOf(own('user'))) : null,
+                get: ()=>own('guildId') ? member ?? (member = memberFor(own('guild'), own('user'), userGiven)) : null,
                 set: (value)=>Object.defineProperty(instance, 'member', {
                         value,
                         writable: true,
@@ -1414,10 +1417,28 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
     Object.assign(instance, props);
     return stubDeep(instance);
 }
-/** A member of a server, as the given user. */ function memberOf(user) {
-    return stubDeep(Object.assign(Object.create(discord_js.GuildMember.prototype), {
-        user
-    }));
+/** A member of `guild` as the user with `id`: its id, user and guild own values, as the gateway delivers a member. */ function memberIn(guild, id, user) {
+    const own = {
+        id: {
+            value: id
+        },
+        user: {
+            value: user
+        },
+        ...guild ? {
+            guild: {
+                value: guild
+            }
+        } : {}
+    };
+    return stubDeep(Object.defineProperties(Object.create(discord_js.GuildMember.prototype), own));
+}
+/**
+ * The member a server has for `user`. A user the test gave is looked up in the guild's member cache, and cached
+ * there when missing, as the gateway resolves a member, so every mock from that user in that server shares it.
+ */ function memberFor(guild, user, given) {
+    const make = ()=>memberIn(guild, user.id, user);
+    return given ? cached(cacheOf(guild?.members), user.id, make) : make();
 }
 /** The guild a mock message carries: a guild with the same stubbed managers as createMockGuild. */ function createMockGuildForMessage() {
     const guild = Object.create(discord_js.Guild.prototype);
@@ -1469,20 +1490,7 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
             ])),
         members: guildCaches ? new discord_js.Collection(users.map((id)=>[
                 id,
-                cached(guildCaches.members, id, ()=>{
-                    const user = cached(userCache, id, ()=>mockUser(id));
-                    return stubDeep(Object.defineProperties(Object.create(discord_js.GuildMember.prototype), {
-                        id: {
-                            value: id
-                        },
-                        user: {
-                            value: user
-                        },
-                        guild: {
-                            value: guild
-                        }
-                    }));
-                })
+                cached(guildCaches.members, id, ()=>memberIn(guild, id, cached(userCache, id, ()=>mockUser(id))))
             ])) : null,
         roles: new discord_js.Collection(guildCaches ? roles.map((id)=>[
                 id,
@@ -1583,9 +1591,10 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
  *
  * @remarks
  * `delete()`, `edit()`, `reply()`, `react()`, `pin()` and `unpin()` throw once the message is deleted, and `edit()`
- * and `reply()` resolve to a new mock message. Without overrides the message is empty, in a server, with its author
- * as its `member`; with `guild: null` it is a direct message. The users, roles and channels the content mentions are
- * cached as the gateway delivers them.
+ * and `reply()` resolve to a new mock message. Without overrides the message is empty, in a server, from a new
+ * person who is its `member`; with `guild: null` it is a direct message. Give `author` to send several messages as
+ * one user, such as to reach a per-user cooldown. An author given, and the users, roles and channels the content
+ * mentions, are cached as the gateway delivers them.
  *
  * @param overrides - The message's content, components and the rest; see {@link MockMessageOverrides}.
  *
@@ -1612,7 +1621,7 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
     const stubs = new Map();
     instance.deleted = false;
     // Constructor-assigned — set as prototype-based stubs; a user rather than a bot, as dispatch handles only those
-    instance.author = mockUser();
+    instance.author = overrides.author ?? mockUser();
     // Getters on the prototype — the proxy sees them as functions and returns
     // a mock fn, which is wrong. Pre-initialize as own properties to shadow
     // the prototype getters.
@@ -1628,9 +1637,12 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
         value: guild,
         writable: true
     });
-    // The author as a member of the message's server; a direct message has none
+    // The author as a member of the message's server; a direct message has none. A given author's member is the one
+    // the server caches, as the gateway resolves it, so every message from that author has the same member
+    const author = overrides.author;
+    const member = guild ? memberFor(guild, instance.author, author !== undefined) : null;
     Object.defineProperty(instance, 'member', {
-        value: guild ? memberOf(instance.author) : null,
+        value: member,
         writable: true
     });
     // In a server's text channel, the ids matching the objects; with no guild, a DM
@@ -1647,7 +1659,12 @@ const isSnowflake = (id)=>typeof id === 'string' && /^\d{1,20}$/.test(id);
         writable: true
     });
     const userCache = cacheOf(client.users);
-    for (const user of overrides.users ?? [])userCache?.set(user.id, user);
+    for (const user of [
+        ...overrides.users ?? [],
+        ...author ? [
+            author
+        ] : []
+    ])userCache?.set(user.id, user);
     // MessageMentions — constructor-assigned; what the content mentions, cached as the gateway delivers it
     instance.mentions = mentionsOf(overrides.content, client, guild);
     // Data a message always has, real rather than stubbed, so code reading it sees an empty message
@@ -2855,16 +2872,36 @@ function at(catalog, key) {
 }
 const MEOCORD_KEYS = new Set(leaves(themeResolvers.MEOCORD_MESSAGES).map(({ key })=>key));
 const isMeoCordKey = (key)=>key.startsWith('meocord.');
+/** The `{param}` names of a message, every form of a plural's, as translating fills them. */ function placeholders(message) {
+    const texts = typeof message === 'string' ? [
+        message
+    ] : Object.values(message);
+    return new Set(texts.flatMap((text)=>typeof text === 'string' ? themeResolvers.placeholderNames(text) : []));
+}
+/** A message as a report shows it: the text, or a plural's `other` form. */ const shown = (message)=>typeof message === 'string' ? message : String(message.other);
+/** The `{params}` a translation uses that the message it translates doesn't take; a plural's `{count}` is taken. */ function strayParams(translation, original) {
+    const taken = placeholders(original);
+    if (isPlural(original)) taken.add('count');
+    return [
+        ...placeholders(translation)
+    ].filter((name)=>!taken.has(name));
+}
 /**
  * Checks that every locale translates every message, and throws listing each gap, locale by locale.
  *
  * Use it in a test for a team that wants no fallback in production. It reports the messages a locale lacks, the
- * messages the default catalog doesn't have, and the plural forms a language needs but a plural lacks, such as
- * `few` for Russian.
+ * messages the default catalog doesn't have, the plural forms a language needs but a plural lacks, such as `few` for
+ * Russian, and each `{param}` a translation uses that the message it translates doesn't take. It reads the catalogs'
+ * own strings, so it checks the params of a catalog from a plain variable or a JSON file too, which the compiler
+ * can't.
  *
  * @remarks
- * MeoCord's own texts, a catalog's `meocord` group, fall back to English by design, so only a key MeoCord lacks is
- * reported. With `meocord: true`, every locale other than an English one must translate each of them as well.
+ * MeoCord's own texts, a catalog's `meocord` group, fall back to English by design, so only a key MeoCord lacks, and
+ * a `{param}` MeoCord's English doesn't take, is reported. With `meocord: true`, every locale other than an English
+ * one must translate each of them as well.
+ *
+ * A translation may leave a param out, as a language can say it without the name, so a missing param isn't
+ * reported.
  *
  * @param translator - A translator made by `createTranslator`.
  * @param options.meocord - Whether every locale that is not English must translate each of MeoCord's own texts.
@@ -2893,7 +2930,8 @@ const isMeoCordKey = (key)=>key.startsWith('meocord.');
  */ function expectCompleteCatalog(translator, options = {}) {
     const catalogs = translator[themeResolvers.CATALOGS];
     if (!catalogs) throw new Error('expectCompleteCatalog takes a translator made by createTranslator.');
-    const reference = leaves(catalogs[translator.defaultLocale] ?? {}).filter(({ key })=>!isMeoCordKey(key));
+    const defaultCatalog = catalogs[translator.defaultLocale] ?? {};
+    const reference = leaves(defaultCatalog).filter(({ key })=>!isMeoCordKey(key));
     const referenceKeys = new Set(reference.map(({ key })=>key));
     const gaps = [];
     for (const locale of translator.locales){
@@ -2904,10 +2942,15 @@ const isMeoCordKey = (key)=>key.startsWith('meocord.');
             const message = at(catalog, key);
             if (message === undefined) {
                 problems.push(`missing ${key}`);
-            } else if (plural && isPlural(message)) {
+                continue;
+            }
+            if (plural && isPlural(message)) {
                 const missing = categories.filter((category)=>typeof message[category] !== 'string');
                 if (missing.length > 0) problems.push(`${key} lacks ${missing.join(', ')}`);
             }
+            const original = at(defaultCatalog, key);
+            const stray = locale === translator.defaultLocale ? [] : strayParams(message, original);
+            for (const name of stray)problems.push(`${key} takes no {${name}}; the default is "${shown(original)}"`);
         }
         if (options.meocord && !locale.startsWith('en-')) {
             for (const key of MEOCORD_KEYS)if (at(catalog, key) === undefined) problems.push(`missing ${key}`);
@@ -2915,6 +2958,11 @@ const isMeoCordKey = (key)=>key.startsWith('meocord.');
         for (const { key } of leaves(catalog)){
             if (isMeoCordKey(key)) {
                 if (!MEOCORD_KEYS.has(key)) problems.push(`${key} is not one of MeoCord's texts`);
+                else {
+                    const english = at(themeResolvers.MEOCORD_MESSAGES, key);
+                    const stray = strayParams(at(catalog, key), english);
+                    for (const name of stray)problems.push(`${key} takes no {${name}}: MeoCord's English is "${shown(english)}"`);
+                }
             } else if (!referenceKeys.has(key)) problems.push(`${key} is not in the default catalog`);
         }
         if (problems.length > 0) gaps.push(`  ${locale}: ${problems.join('; ')}`);

package/dist/cli.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": 1,
-  "meocordVersion": "4.1.0-beta.8",
+  "meocordVersion": "4.1.0-beta.9",
   "name": "meocord",
   "description": "CLI for managing the MeoCord application",
   "usage": "[options] [command]",

package/dist/esm/bin/generator.js

@@ -13,6 +13,8 @@ import '../core/theme-scope.js';
 import '../common/translator.js';
 import 'node:crypto';
 import '../decorator/controller.decorator.js';
+import 'node:fs';
+import 'node:url';
 import '../core/theme-resolvers.js';
 import { ServiceGeneratorHelper } from './helper/service-generator.helper.js';
 import { GuardGeneratorHelper } from './helper/guard-generator.helper.js';

package/dist/esm/bin/helper/filter-generator.helper.js

@@ -9,6 +9,8 @@ import '../../core/theme-scope.js';
 import '../../common/translator.js';
 import 'node:crypto';
 import '../../decorator/controller.decorator.js';
+import 'node:fs';
+import 'node:url';
 import '../../core/theme-resolvers.js';
 import { validateAndFormatName, assertFilesAbsent, buildTemplate, createDirectoryIfNotExists, generateFile } from '../../util/generator-cli.util.js';
 

package/dist/esm/bin/helper/guard-generator.helper.js

@@ -9,6 +9,8 @@ import '../../core/theme-scope.js';
 import '../../common/translator.js';
 import 'node:crypto';
 import '../../decorator/controller.decorator.js';
+import 'node:fs';
+import 'node:url';
 import '../../core/theme-resolvers.js';
 import { validateAndFormatName, assertFilesAbsent, buildTemplate, createDirectoryIfNotExists, generateFile } from '../../util/generator-cli.util.js';
 

package/dist/esm/bin/helper/interceptor-generator.helper.js

@@ -9,6 +9,8 @@ import '../../core/theme-scope.js';
 import '../../common/translator.js';
 import 'node:crypto';
 import '../../decorator/controller.decorator.js';
+import 'node:fs';
+import 'node:url';
 import '../../core/theme-resolvers.js';
 import { validateAndFormatName, assertFilesAbsent, buildTemplate, createDirectoryIfNotExists, generateFile } from '../../util/generator-cli.util.js';