dsh-root-planner-preset 1.4.2 → 1.4.3

package/build-patch.mjs

@@ -71,7 +71,7 @@ const patch = `# Generated by build-patch.mjs from persona.txt. Do not hand-edit
           - id: persona
             name: '@deepseek-ai/dsh-persona'
             config:
-              suffix: Your working directory is {{cwd}}.
+              suffix: Your working directory is the session workspace; call the working_directory tool to read its absolute path when you need it.
               prefix: |-
 <PERSONA>
           - id: agent-instructions
@@ -259,7 +259,7 @@ const patch = `# Generated by build-patch.mjs from persona.txt. Do not hand-edit
           - id: persona
             name: '@deepseek-ai/dsh-persona'
             config:
-              suffix: Your working directory is {{cwd}}.
+              suffix: Your working directory is the session workspace; call the working_directory tool to read its absolute path when you need it.
               prefix: |-
 <PERSONA>
           - id: agent-instructions

package/cordis.patch.yml

@@ -17,7 +17,7 @@
           - id: persona
             name: '@deepseek-ai/dsh-persona'
             config:
-              suffix: Your working directory is {{cwd}}.
+              suffix: Your working directory is the session workspace; call the working_directory tool to read its absolute path when you need it.
               prefix: |-
                 A task may be handled by a single agent or by a tree of agents. You are one node in that tree, and the same rules govern every node, so behavior stays predictable no matter where a task lands.
 
@@ -276,7 +276,7 @@
           - id: persona
             name: '@deepseek-ai/dsh-persona'
             config:
-              suffix: Your working directory is {{cwd}}.
+              suffix: Your working directory is the session workspace; call the working_directory tool to read its absolute path when you need it.
               prefix: |-
                 A task may be handled by a single agent or by a tree of agents. You are one node in that tree, and the same rules govern every node, so behavior stays predictable no matter where a task lands.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-root-planner-preset",
-  "version": "1.4.2",
+  "version": "1.4.3",
   "type": "module",
   "description": "DSH agent preset: the root agent plans and decides, subagent workers read, write and run commands inside their own _agents/ workspace, and nothing outlives its task.",
   "license": "MIT",

package/verify-patch.mjs

@@ -51,7 +51,13 @@ check(presetRow?.config?.order === 5, 'preset order is 5 (after the shipped pres
 
 const plugins = presetRow?.config?.plugins ?? [];
 const personaRow = plugins.find((row) => row.id === 'persona');
-check(personaRow?.config?.suffix === 'Your working directory is {{cwd}}.', 'persona suffix matches the deployment text');
+// The suffix is intentionally not the deployment's text any more: that one interpolates
+// {{cwd}}, which DSH 0.2.1 rejects (only provider and model are registered variables). The
+// variable-free replacement is asserted below for both presets.
+check(
+  typeof personaRow?.config?.suffix === 'string' && personaRow.config.suffix.length > 0,
+  'persona suffix is set',
+);
 
 const expected = readFileSync(join(here, 'persona.txt'), 'utf8').replace(/\r\n/g, '\n').trimEnd();
 check(personaRow?.config?.prefix === expected, 'parsed persona is byte-identical to persona.txt');
@@ -248,6 +254,23 @@ for (const scoped of SCOPE_LOCAL_TOOLS) {
   );
 }
 
+// DSH 0.2.1 validates every `{{name}}` in a prompt section against the registered prompt
+// variables, which on this machine are `provider` and `model` only. The persona suffix used
+// to say "Your working directory is {{cwd}}." and that now fails the whole run with
+//   unknown prompt variable "{{cwd}}" in section "deployment:persona-suffix"
+// so the suffix is variable-free and points at the tool that reports the path.
+for (const [label, rows] of [
+  ['the root preset', plugins],
+  ['the worker preset', workerPlugins],
+]) {
+  const suffix = rows.find((entry) => entry.id === 'persona')?.config?.suffix ?? '';
+  check(typeof suffix === 'string' && suffix.length > 0, `${label} declares a persona suffix`);
+  check(
+    !/\{\{/.test(suffix),
+    `${label} persona suffix declares no prompt variables (only provider and model are registered)`,
+  );
+}
+
 // "No unbounded process": with the shell tools present, the property lives in each
 // preset's own rows rather than in a mask — the worker needs it as much as the root.
 for (const [label, rows] of [