@promptbook/cli 0.113.0-5 → 0.113.0-8

package/src/cli/cli-commands/coder/run.ts

@@ -1,9 +1,9 @@
 import colors from 'colors';
 import { Command as Program /* <- Note: [🔸] Using Program because Command is misleading name */ } from 'commander';
 import { spaceTrim } from 'spacetrim';
-import { NotAllowed } from '../../../errors/NotAllowed';
 import { assertsError } from '../../../errors/assertsError';
 import type { $side_effect } from '../../../utils/organization/$side_effect';
+import { createPositiveIntegerOptionParser } from '../common/createPositiveIntegerOptionParser';
 import { handleActionErrors } from '../common/handleActionErrors';
 import type { PromptRunnerCliOptions } from '../common/promptRunnerCliOptions';
 import {
@@ -12,10 +12,7 @@ import {
     normalizePromptRunnerCliOptions,
     PROMPT_RUNNER_DESCRIPTION,
 } from '../common/promptRunnerCliOptions';
-import {
-    DEFAULT_WAIT_AFTER_ERROR_MS,
-    parseOptionalWaitDuration,
-} from './waitOptions';
+import { DEFAULT_WAIT_AFTER_ERROR_MS, parseOptionalWaitDuration } from './waitOptions';
 
 /**
  * Initializes `coder run` command for Promptbook CLI utilities
@@ -69,7 +66,7 @@ export function $initializeCoderRunCommand(program: Program): $side_effect {
     command.option(
         '--limit <run-count>',
         'Stop after processing this many prompt runs',
-        parsePositiveIntegerOption,
+        createPositiveIntegerOptionParser('--limit'),
     );
     command.option(
         '--wait-after-prompt <duration>',
@@ -213,27 +210,6 @@ function parseIntOption(value: string): number {
     return parsed;
 }
 
-/**
- * Parses a positive integer option value.
- *
- * @private internal utility of `coder run` command
- */
-function parsePositiveIntegerOption(value: string): number {
-    const parsed = Number(value);
-
-    if (!Number.isInteger(parsed) || parsed <= 0) {
-        throw new NotAllowed(
-            spaceTrim(`
-                Invalid value for \`--limit\`: \`${value}\`.
-
-                Use a positive integer.
-            `),
-        );
-    }
-
-    return parsed;
-}
-
 /**
  * Joins one Commander option that may be parsed either as a single string or a variadic token array.
  *

package/src/cli/cli-commands/common/createPositiveIntegerOptionParser.ts

@@ -0,0 +1,31 @@
+import { spaceTrim } from 'spacetrim';
+import { NotAllowed } from '../../../errors/NotAllowed';
+
+/**
+ * Creates a Commander argument parser that accepts only positive integers.
+ *
+ * The returned parser is meant to be passed as the coercion callback of `command.option(...)`.
+ * It throws a branded `NotAllowed` error with a clear message referencing the given option name
+ * when the provided value is not a positive integer.
+ *
+ * @private internal utility of `promptbookCli`
+ */
+export function createPositiveIntegerOptionParser(optionName: string): (value: string) => number {
+    return (value: string): number => {
+        const parsedValue = Number(value);
+
+        if (!Number.isInteger(parsedValue) || parsedValue <= 0) {
+            throw new NotAllowed(
+                spaceTrim(`
+                    Invalid value for \`${optionName}\`: \`${value}\`.
+
+                    Use a positive integer.
+                `),
+            );
+        }
+
+        return parsedValue;
+    };
+}
+
+// Note: [🟡] Code for CLI option parser [createPositiveIntegerOptionParser](src/cli/cli-commands/common/createPositiveIntegerOptionParser.ts) should never be published outside of `@promptbook/cli`

package/src/commitments/TEAM/TEAM.ts

@@ -113,10 +113,10 @@ const teamToolTitles: Record<string_javascript_name, string> = {};
  *
  * @private
  */
-const TEAM_SYSTEM_MESSAGE_GUIDANCE_LINES = [
-    '-   If a teammate is relevant to the request, consult that teammate using the matching tool.',
-    '-   Do not ask the user for information that a listed teammate can provide directly.',
-] as const;
+const TEAM_SYSTEM_MESSAGE_GUIDANCE = spaceTrim(`
+    -   If a teammate is relevant to the request, consult that teammate using the matching tool.
+    -   Do not ask the user for information that a listed teammate can provide directly.
+`);
 /**
  * Constant for remote agents by Url.
  */
@@ -352,7 +352,7 @@ function buildTeamSystemMessageBody(teamEntries: ReadonlyArray<TeamToolEntry>):
 
     return spaceTrim(
         (block) => `
-            ${block(TEAM_SYSTEM_MESSAGE_GUIDANCE_LINES.join('\n'))}
+            ${block(TEAM_SYSTEM_MESSAGE_GUIDANCE)}
 
             ${block(teammateSections.join('\n\n'))}
         `,