dsh-nextsteps 1.3.1 → 1.3.3

package/index.js

@@ -124,14 +124,13 @@ const PARAMETER_SCHEMA = {
   required: ['followups']
 };
 
-function toolDescription(settings) {
-  const language = LANGUAGE_NAMES[settings.lang] ?? LANGUAGE_NAMES.auto;
+function toolDescription() {
   return [
     'MANDATORY in every turn: once all real work and other tool calls are done, call this tool BEFORE writing your final answer — the answer must come after this tool returns, so the turn ends cleanly right after it. Never write your final answer before calling it.',
-    `Suggest exactly ${settings.maxFollowups} followup prompts the user is likely to want next.`,
+    'Suggest exactly the number of followup prompts stated in the nextsteps section of the system prompt.',
     'Good suggestions include alternatives to the latest implementation, related features, cleanup opportunities, tests to add, or verification steps you could run yourself; if plan steps remain, one suggestion may be continuing with the next step.',
     `Each prompt must be short (usually one sentence), goal-oriented — name the outcome, not the steps — stand alone without conversation context, and be phrased in the user's first-person voice.`,
-    `Write every prompt in ${language}.`,
+    'Write every prompt and label in the language the nextsteps section of the system prompt states.',
     'Never suggest actions the user would carry out themselves outside this session (e.g. "commit these changes"); judge by the tools you actually have.',
     'This never waits for an answer: its result only confirms the suggestions were recorded.',
     'Example input:',
@@ -139,12 +138,25 @@ function toolDescription(settings) {
   ].join('\n');
 }
 
+function nextstepsSectionText() {
+  const settings = settingsLoad();
+  const language = LANGUAGE_NAMES[settings.lang] ?? LANGUAGE_NAMES.auto;
+  const languageRule = settings.lang === 'auto'
+    ? `Write every prompt and label in the same language the conversation has been using.`
+    : `LANGUAGE OVERRIDE, applies to the suggest_followups tool only: write every prompt and label in ${language} — even if the conversation or the session history is in another language. Never copy the conversation's language here.`;
+  return [
+    'nextsteps (the followup suggestion cards):',
+    `- Suggest exactly ${settings.maxFollowups} followup prompts when you call suggest_followups.`,
+    `- ${languageRule}`
+  ].join('\n');
+}
+
 const PLUGIN_REV = 'sf-2';
 
-function toolSpec(settings) {
+function toolSpec() {
   return {
     name: TOOL_NAME,
-    description: toolDescription(settings),
+    description: toolDescription(),
     parameters: PARAMETER_SCHEMA,
     output: {
       schema: { type: 'string' },
@@ -205,11 +217,14 @@ function turnTranscript(events, afterSeq) {
 
 function generateSystem(settings) {
   const language = LANGUAGE_NAMES[settings.lang] ?? LANGUAGE_NAMES.auto;
+  const languageRule = settings.lang === 'auto'
+    ? 'Write every prompt and label in the same language the conversation has been using.'
+    : `LANGUAGE OVERRIDE, applies to this answer only: write every prompt and label in ${language} — even if the transcript is in another language. Never copy the transcript's language here.`;
   return [
     'You suggest followup prompts the user of a coding-agent chat may want next. You answer with JSON only — no prose, no code fence.',
     `Output shape: {"followups":[{"prompt":"...","label":"..."}]} with exactly ${settings.maxFollowups} items.`,
     'Each prompt is one short sentence in the user\'s first-person voice, goal-oriented, standing alone without the transcript; each label is 2-5 words.',
-    `Write every prompt and label in ${language}.`,
+    languageRule,
     'Good suggestions: alternatives to the latest change, related features, cleanup, tests, verification steps. If the transcript shows remaining plan steps, one suggestion may continue the next step.'
   ].join('\n');
 }
@@ -302,20 +317,35 @@ async function apply(ctx) {
   let settings = settingsLoad();
   const json = (value, status = 200) => new Response(JSON.stringify(value), { status, headers: { 'content-type': 'application/json; charset=utf-8', 'cache-control': 'no-store' } });
   const isExternal = (next) => typeof next.model === 'string' && next.model !== '';
-  try {
-    if (!isExternal(settings)) ctx.tools.register(toolSpec(settings));
-  } catch (error) {
-    console.error('dsh-nextsteps: tool registration failed:', error);
-  }
   const reapplyTool = (next) => {
     settings = next;
     try {
-      if (isExternal(next)) ctx.tools.unregister(TOOL_NAME);
-      else ctx.tools.register(toolSpec(settings));
+      try { ctx.tools.unregister(TOOL_NAME); } catch { /* not registered yet */ }
+      if (!isExternal(next)) ctx.tools.register(toolSpec());
     } catch (error) {
       console.error('dsh-nextsteps: tool re-registration failed:', error);
     }
   };
+  try {
+    if (!isExternal(settings)) ctx.tools.register(toolSpec());
+  } catch (error) {
+    console.error('dsh-nextsteps: tool registration failed:', error);
+  }
+  if (typeof ctx.inject === 'function') {
+    ctx.inject(['tools', 'systemPrompt'], (sub) => {
+      try {
+        try { sub.tools.unregister(TOOL_NAME); } catch { /* not registered yet */ }
+        if (!isExternal(settings)) sub.tools.register(toolSpec());
+      } catch (error) {
+        console.error('dsh-nextsteps: tool registration via inject failed:', error);
+      }
+      try {
+        sub.systemPrompt.section({ name: 'dsh-nextsteps', order: 640, interpolate: false, text: () => nextstepsSectionText() });
+      } catch (error) {
+        console.error('dsh-nextsteps: system prompt section rejected:', error);
+      }
+    });
+  }
   ctx.connection?.fetch?.register({
     path: '/api/dsh-nextsteps.settings',
     methods: ['GET', 'POST'],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-nextsteps",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "type": "module",
   "exports": {
     ".": "./index.js",