refactor(cli): hoist gen name validators to module constants

The name-validation regexes in the gen arg parsers were inline literals,
recreating a RegExp on every parser call and duplicating two identical
patterns across five sites. Hoist them to two module-level constants
(NAME_RE for model/migration, NAME_WITH_HYPHENS_RE for middleware/cron/
queue). Behavior is unchanged (anchored, non-global validators).

Author: hyfdev

packages/void/src/cli/gen-args.ts

@@ -57,12 +57,16 @@ const COLUMN_TYPES = new Set(['string', 'text', 'datetime', 'integer', 'boolean'
 
 const VALID_METHODS = new Set(['get', 'post', 'put', 'patch', 'delete']);
 
+// Generated-file name validators (hoisted once); *_HYPHENS variant also allows hyphens (cron/queue/middleware).
+const NAME_RE = /^[a-z_][a-z0-9_]*$/;
+const NAME_WITH_HYPHENS_RE = /^[a-z_][a-z0-9_-]*$/;
+
 function parseModelArgs(args: Array<string>): GenArgs {
   const [name, ...rest] = args;
   if (!name) {
     throw new Error('gen: Usage: void gen model <name> [columns...].');
   }
-  if (!/^[a-z_][a-z0-9_]*$/.test(name)) {
+  if (!NAME_RE.test(name)) {
     throw new Error(
       'gen: Model name must be lowercase alphanumeric with underscores and no leading digits.',
     );
@@ -94,7 +98,7 @@ function parseMigrationArgs(args: Array<string>): GenArgs {
   if (extra) {
     throw new Error(`gen: Unexpected argument '${extra}'.`);
   }
-  if (!/^[a-z_][a-z0-9_]*$/.test(name)) {
+  if (!NAME_RE.test(name)) {
     throw new Error(
       'gen: Migration name must be lowercase alphanumeric with underscores and no leading digits.',
     );
@@ -145,7 +149,7 @@ function parseMiddlewareArgs(args: Array<string>): GenArgs {
   if (extra) {
     throw new Error(`gen: Unexpected argument '${extra}'.`);
   }
-  if (!/^[a-z_][a-z0-9_-]*$/.test(name)) {
+  if (!NAME_WITH_HYPHENS_RE.test(name)) {
     throw new Error(
       'gen: Middleware name must be lowercase alphanumeric with underscores or hyphens and no leading digits.',
     );
@@ -186,7 +190,7 @@ function parseCronArgs(args: Array<string>): GenArgs {
   if (extra) {
     throw new Error(`gen: Unexpected argument '${extra}'.`);
   }
-  if (!/^[a-z_][a-z0-9_-]*$/.test(name)) {
+  if (!NAME_WITH_HYPHENS_RE.test(name)) {
     throw new Error(
       'gen: Cron name must be lowercase alphanumeric with underscores or hyphens and no leading digits.',
     );
@@ -202,7 +206,7 @@ function parseQueueArgs(args: Array<string>): GenArgs {
   if (extra) {
     throw new Error(`gen: Unexpected argument '${extra}'.`);
   }
-  if (!/^[a-z_][a-z0-9_-]*$/.test(name)) {
+  if (!NAME_WITH_HYPHENS_RE.test(name)) {
     throw new Error(
       'gen: Queue name must be lowercase alphanumeric with underscores or hyphens and no leading digits.',
     );

packages/void/test/unit/project-args.test.ts

@@ -103,8 +103,7 @@ describe('parseProjectArgs', () => {
   });
 
   describe('unknown-flag rejection', () => {
-    // `delete`/`logs`/`purge-cache` already reject unknown flags; status/link/
-    // rollback/cancel previously captured a leading flag as the positional id.
+    // status/link/rollback/cancel previously captured a leading flag as the positional id.
     it('rejects an unknown flag for status instead of capturing it as the name', () => {
       expect(() => parseProjectArgs(['status', '--foo'])).toThrow("project: Unknown flag '--foo'.");
     });