fix(qoder): keep the ruleset on the /ponytail default turn (#1114)

Author: htjworld

hooks/ponytail-mode-tracker.js

@@ -42,6 +42,7 @@ function finish() {
     // Match /ponytail commands
     let modeSwitched = false;
     let deactivated = false;
+    let defaultNotice = '';
     if (/^[/@$]ponytail/.test(prompt)) {
       const parts = prompt.split(/\s+/);
       const cmd = parts[0].replace(/^[@$]/, '/');
@@ -62,9 +63,12 @@ function finish() {
           const dmode = parts[2];
           if (dmode === 'off' || dmode === 'lite' || dmode === 'full' || dmode === 'ultra') {
             writeDefaultMode(dmode);
-            writeHookOutput('UserPromptSubmit', dmode, 'PONYTAIL DEFAULT SET — new sessions start in ' + dmode + '.');
+            defaultNotice = 'PONYTAIL DEFAULT SET — new sessions start in ' + dmode + '.';
+            if (!isQoder) writeHookOutput('UserPromptSubmit', dmode, defaultNotice);
           }
-          return; // don't fall through to the session-mode switch
+          // Qoder still needs this turn's ruleset: the block below folds the
+          // notice into it. mode stays null, so the session level is untouched.
+          if (!isQoder) return; // don't fall through to the session-mode switch
         }
         if (arg === 'lite') mode = 'lite';
         else if (arg === 'full') mode = 'full';
@@ -146,10 +150,12 @@ function finish() {
       if (currentMode && currentMode !== 'off') {
         // ponytail: one JSON per invocation — mode-switch confirmation is
         // folded into the ruleset header so Qoder gets both in one write.
-        const header = modeSwitched
+        const header = (defaultNotice ? defaultNotice + '\n\n' : '') + (modeSwitched
           ? 'PONYTAIL MODE CHANGED — level: ' + currentMode + '\n\n'
-          : '';
+          : '');
         writeHookOutput('UserPromptSubmit', currentMode, header + getPonytailInstructions(currentMode));
+      } else if (defaultNotice) {
+        writeHookOutput('UserPromptSubmit', currentMode, defaultNotice);
       }
     }
   } catch (e) {

tests/hooks.test.js

@@ -636,6 +636,16 @@ assert.match(
   /PONYTAIL MODE CHANGED — level: ultra/,
 );
 
+// `/ponytail default lite` on Qoder: the turn still gets the live ruleset, with
+// the confirmation folded into the same JSON, and the live level stays ultra.
+const qoderDefaultEnv = { ...qoderEnv, XDG_CONFIG_HOME: path.join(qoderHome, '.config') };
+result = run('ponytail-mode-tracker.js', qoderDefaultEnv, JSON.stringify({ prompt: '/ponytail default lite' }));
+assert.equal(result.status, 0, result.stderr);
+assert.equal(fs.readFileSync(qoderState, 'utf8'), 'ultra');
+output = JSON.parse(result.stdout);
+assert.match(output.hookSpecificOutput.additionalContext, /^PONYTAIL DEFAULT SET — new sessions start in lite\./);
+assert.match(output.hookSpecificOutput.additionalContext, /PONYTAIL MODE ACTIVE — level: ultra/);
+
 // "stop ponytail": deactivates and persists off so a later prompt does
 // not mistake the missing flag for first-run initialization.
 result = run(
@@ -657,6 +667,12 @@ assert.equal(result.status, 0, result.stderr);
 assert.equal(result.stdout, '', 'Qoder must stay off on later prompts');
 assert.equal(fs.readFileSync(qoderState, 'utf8'), 'off');
 
+// While off, `/ponytail default` only confirms; it must not switch ponytail on.
+result = run('ponytail-mode-tracker.js', qoderDefaultEnv, JSON.stringify({ prompt: '/ponytail default full' }));
+assert.equal(result.status, 0, result.stderr);
+assert.equal(JSON.parse(result.stdout).hookSpecificOutput.additionalContext, 'PONYTAIL DEFAULT SET — new sessions start in full.');
+assert.equal(fs.readFileSync(qoderState, 'utf8'), 'off');
+
 // Subagent injection via PreToolUse (task|Task matcher): when ponytail is
 // active, the subagent hook injects the ruleset. Qoder shares the same
 // ponytail-subagent.js script; the isQoder branch outputs hookSpecificOutput