@promptbook/cli 0.114.0-25 → 0.114.0-26

package/apps/agents-server/src/utils/agentProjects/agentProjectRuntimePm2.ts

@@ -1,6 +1,8 @@
 import { execFile } from 'child_process';
 import { createHash } from 'crypto';
 import { promisify } from 'util';
+import { UnexpectedError } from '../../../../../src/errors/UnexpectedError';
+import { spaceTrim } from '../../../../../src/utils/organization/spaceTrim';
 import { AGENT_PROJECT_RUNTIME_HOST } from './agentProjectRuntimeConstants';
 import type { AgentProjectRuntimeMode } from './AgentProjectRuntimeInfo';
 import { resolveAgentProjectRuntimeEnvironmentFlag } from './resolveAgentProjectRuntimeEnvironmentFlag';
@@ -44,8 +46,18 @@ const AGENT_PROJECT_PM2_PROCESS_LABEL_LENGTH = 36;
 
 /**
  * Max output retained from pm2 commands.
+ *
+ * Note: `pm2 jlist` embeds a full environment snapshot into every listed process, so the list grows by tens of
+ *       kilobytes with each started project. A budget of one megabyte was exhausted by roughly a dozen projects,
+ *       after which `execFile` killed every `pm2 jlist` call with `ERR_CHILD_PROCESS_STDIO_MAXBUFFER` and pm2
+ *       appeared to know no process at all.
+ */
+const PM2_COMMAND_MAX_BUFFER_BYTES = 64 * 1024 * 1024;
+
+/**
+ * pm2 error reported when no process of the requested name is registered.
  */
-const PM2_COMMAND_MAX_BUFFER_BYTES = 1024 * 1024;
+const PM2_MISSING_PROCESS_ERROR_PATTERN = /Process or Namespace .* not found/u;
 
 /**
  * Status snapshot for one pm2 process.
@@ -189,25 +201,46 @@ export async function startAgentProjectRuntimePm2Process(
 }
 
 /**
- * Stops one pm2 project process when it exists.
+ * Deletes every pm2 process registered under one name.
+ *
+ * Note: The delete is never skipped because pm2 could not be asked for its current process list. `pm2 start`
+ *       appends another process instead of replacing the existing one when a process of the same name is already
+ *       registered, so a delete which silently does nothing turns every later start of the same project into one
+ *       more duplicate. Only pm2 itself reporting an unknown name counts as "already gone", every other failure
+ *       is reported to the caller, which then must not start the project again.
  *
  * @param processName - pm2 process name.
  */
 export async function stopAgentProjectRuntimePm2Process(processName: string | null | undefined): Promise<void> {
-    if (!processName) {
+    if (!processName || !isAgentProjectRuntimePm2Enabled()) {
         return;
     }
 
-    const status = await resolveAgentProjectRuntimePm2Status(processName);
+    try {
+        await execFileAsync(PM2_COMMAND, ['delete', processName], {
+            maxBuffer: PM2_COMMAND_MAX_BUFFER_BYTES,
+            windowsHide: true,
+        });
+    } catch (error) {
+        if (isPm2MissingProcessError(error)) {
+            return;
+        }
 
-    if (!status.isKnown) {
-        return;
+        throw new UnexpectedError(
+            spaceTrim(
+                (block) => `
+                    Failed to delete the pm2 process \`${processName}\` of an agent project.
+
+                    Starting the project again would register one more pm2 process of the same name instead of
+                    replacing the existing one, so the project was left alone.
+
+                    **Cause:**
+                    ${block(error instanceof Error ? error.message : String(error))}
+                `,
+            ),
+        );
     }
 
-    await execFileAsync(PM2_COMMAND, ['delete', processName], {
-        maxBuffer: PM2_COMMAND_MAX_BUFFER_BYTES,
-        windowsHide: true,
-    });
     await savePm2ProcessList();
 }
 
@@ -366,6 +399,19 @@ async function savePm2ProcessList(): Promise<void> {
     });
 }
 
+/**
+ * Returns whether one failed pm2 command failed only because pm2 knows no process of the requested name.
+ */
+function isPm2MissingProcessError(error: unknown): boolean {
+    if (!error || typeof error !== 'object') {
+        return false;
+    }
+
+    const { stderr } = error as { readonly stderr?: unknown };
+
+    return typeof stderr === 'string' && PM2_MISSING_PROCESS_ERROR_PATTERN.test(stderr);
+}
+
 /**
  * Creates the fallback status for missing or unavailable pm2 processes.
  */

package/apps/agents-server/src/utils/userChat/createImmediateUserChatAnswerModelRequirements.ts

@@ -19,6 +19,7 @@ const IMMEDIATE_USER_CHAT_ANSWER_INSTRUCTION_COMMITMENTS = new Set<string>([
     'STYLE',
     'STYLES',
     'WRITING RULES',
+    'WRITING RULE',
     'FORMAT',
     'FORMATS',
     'SCENARIO',

package/esm/index.es.js

@@ -48,7 +48,7 @@ const BOOK_LANGUAGE_VERSION = '2.0.0';
  * @generated
  * @see https://github.com/webgptorg/promptbook
  */
-const PROMPTBOOK_ENGINE_VERSION = '0.114.0-25';
+const PROMPTBOOK_ENGINE_VERSION = '0.114.0-26';
 /**
  * TODO: string_promptbook_version should be constrained to the all versions of Promptbook engine
  * Note: [💞] Ignore a discrepancy between file name and entity name
@@ -20063,29 +20063,36 @@ class UseUserLocationCommitmentDefinition extends BaseCommitmentDefinition {
 /**
  * `WRITING RULES` commitment definition.
  *
- * It adds constraints that apply strictly to writing style and presentation
- * rather than task-solving behavior.
+ * The `WRITING RULES`/`WRITING RULE` commitment adds constraints that apply strictly to writing style
+ * and presentation rather than task-solving behavior.
+ *
+ * Example usage in agent source:
+ *
+ * ```book
+ * WRITING RULES Keep every response under 120 words
+ * WRITING RULE Always end the message with an emoji
+ * ```
  *
  * @private [🪔] Maybe export the commitments through some package
  */
 class WritingRulesCommitmentDefinition extends BaseCommitmentDefinition {
-    constructor() {
-        super('WRITING RULES');
+    constructor(type = 'WRITING RULES') {
+        super(type);
     }
     /**
-     * Short one-line description of `WRITING RULES`.
+     * Short one-line description of `WRITING RULES`/`WRITING RULE`.
      */
     get description() {
         return 'Add writing-only constraints such as tone, length, formatting, or emoji usage.';
     }
     /**
-     * Icon for `WRITING RULES`.
+     * Icon for `WRITING RULES`/`WRITING RULE`.
      */
     get icon() {
         return '📝';
     }
     /**
-     * Markdown documentation for `WRITING RULES`.
+     * Markdown documentation for `WRITING RULES`/`WRITING RULE` commitment.
      */
     get documentation() {
         return spaceTrim$1(`
@@ -20095,6 +20102,7 @@ class WritingRulesCommitmentDefinition extends BaseCommitmentDefinition {
 
             ## Key aspects
 
+            - All writing rules are treated equally regardless of singular/plural form.
             - Use it for writing-only constraints such as tone, formatting, length, emoji usage, punctuation, or reading level.
             - Do **not** use it for problem-solving behavior, policy, tool usage, or decision-making logic. Use \`RULE\` for that.
             - Newer writing-rules blocks override conflicting earlier writing-rules blocks.
@@ -20293,7 +20301,8 @@ const COMMITMENT_REGISTRY = [
     new LanguageCommitmentDefinition('LANGUAGE'),
     new LanguageCommitmentDefinition('LANGUAGES'),
     new WritingSampleCommitmentDefinition(),
-    new WritingRulesCommitmentDefinition(),
+    new WritingRulesCommitmentDefinition('WRITING RULES'),
+    new WritingRulesCommitmentDefinition('WRITING RULE'),
     new SampleCommitmentDefinition('SAMPLE'),
     new SampleCommitmentDefinition('EXAMPLE'),
     new FormatCommitmentDefinition('FORMAT'),