fix(opencode): never save review as a level, support /ponytail default (#1103)

`/ponytail review` was saved to the OpenCode flag, so every later chat got a
one-line pointer to the review skill instead of the ruleset (the latch fixed
for Claude Code in #736). The plugin now accepts only off/lite/full/ultra,
and an old saved `review` falls back to the default.

`/ponytail default <level>` was read as one unknown level and silently did
nothing. It now saves the default via writeDefaultMode, as the Claude hook
and pi do.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Dietrich Gebert <dietrichgebert@gmail.com>

Author: 3 people

.opencode/plugins/ponytail.mjs

@@ -23,7 +23,7 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url));
 // The shared instruction builder is CommonJS; bridge to it from this ES module.
 const require = createRequire(import.meta.url);
 const { getPonytailInstructions } = require('../../hooks/ponytail-instructions');
-const { getDefaultMode, normalizePersistedMode } = require('../../hooks/ponytail-config');
+const { getDefaultMode, normalizeMode, writeDefaultMode } = require('../../hooks/ponytail-config');
 const { parseCommandFile, parseSkillFile } = require('./ponytail-frontmatter.cjs');
 
 // OpenCode has no flag-file convention of its own; keep mode beside its config.
@@ -36,7 +36,8 @@ const statePath = path.join(
 
 function readMode() {
   try {
-    return normalizePersistedMode(fs.readFileSync(statePath, 'utf8').trim()) || getDefaultMode();
+    // review is a one-shot command, never a level (#736); an old saved review falls back.
+    return normalizeMode(fs.readFileSync(statePath, 'utf8')) || getDefaultMode();
   } catch (e) {
     return getDefaultMode();
   }
@@ -53,9 +54,16 @@ function writeMode(mode) {
 // synchronous store if same-turn switching ever matters.
 function persistMode(args) {
   const wanted = String(args == null ? '' : args).trim();
+  // `/ponytail default <level>` saves the default for new sessions, like the Claude hook.
+  const [first, second] = wanted.toLowerCase().split(/\s+/);
+  if (first === 'default') {
+    const saved = writeDefaultMode(second);
+    if (saved) console.log('ponytail default ' + saved);
+    return;
+  }
   // Bare /ponytail switches ponytail on, or keeps the level when it already is (#639).
   if (!wanted && readMode() !== 'off') return;
-  const mode = wanted ? normalizePersistedMode(wanted) : (getDefaultMode() === 'off' ? 'full' : getDefaultMode());
+  const mode = wanted ? normalizeMode(wanted) : (getDefaultMode() === 'off' ? 'full' : getDefaultMode());
   if (!mode) return;
   writeMode(mode);
   console.log('ponytail ' + mode);

tests/opencode-plugin.test.js

@@ -265,6 +265,25 @@ test('unsupported /ponytail arguments do not reset the current mode', async () =
   assert.equal(fs.readFileSync(statePath, 'utf8'), 'ultra');
 });
 
+test('/ponytail review is not a level: the flag keeps the current mode (#736)', async () => {
+  const hooks = await loadPlugin.server({});
+  fs.writeFileSync(statePath, 'ultra');
+  await hooks['command.execute.before']({ command: 'ponytail', arguments: 'review', sessionID: 's' });
+  assert.equal(fs.readFileSync(statePath, 'utf8'), 'ultra');
+  // A review saved by an older version falls back to the default instead of latching.
+  fs.writeFileSync(statePath, 'review');
+  assert.match((await transform(hooks))[0], /PONYTAIL MODE ACTIVE — level: full/);
+});
+
+test('/ponytail default lite saves the default and leaves the session level alone', async () => {
+  const hooks = await loadPlugin.server({});
+  fs.writeFileSync(statePath, 'ultra');
+  await hooks['command.execute.before']({ command: 'ponytail', arguments: 'default lite', sessionID: 's' });
+  assert.equal(JSON.parse(fs.readFileSync(path.join(tmp, 'ponytail', 'config.json'), 'utf8')).defaultMode, 'lite');
+  assert.equal(fs.readFileSync(statePath, 'utf8'), 'ultra');
+  fs.rmSync(path.join(tmp, 'ponytail'), { recursive: true, force: true });
+});
+
 test('unrelated commands do not touch the flag', async () => {
   try { fs.unlinkSync(statePath); } catch (e) {}
   const hooks = await loadPlugin.server({});