saving-token 1.0.0 → 1.2.0

package/README.md

@@ -6,6 +6,9 @@ unchanged.
 
 Inspired by [karpathy-guidelines](https://github.com/multica-ai/andrej-karpathy-skills).
 
+The writer is used only when the host model is stronger; otherwise the host
+works directly.
+
 ## Install
 
 ### Interactive setup
@@ -42,10 +45,21 @@ After publication, run:
 npx saving-token@latest install --agent codex
 ```
 
-The wrapper supports `install --agent codex` and keeps the interactive menus.
+The wrapper supports `install --agent codex` and `update-models --agent codex`.
 Use the Bash script directly for `--target`, `--model`, `--reasoning-effort`,
 or `--yes`.
 
+To choose a new writer model in an existing Codex installation, run:
+
+```bash
+npx saving-token@latest update-models --agent codex
+```
+
+Use **Up/Down** and **Enter** to choose the model. It updates the model list
+and only the writer's model field; it preserves the reasoning effort and shows
+no platform, effort, or confirmation menus. Use the Bash script with
+`--update-models --target claude` or `cursor` for those platforms.
+
 ### Installed paths
 
 | Platform | Skill | Writer configuration |

package/bin/saving-token.cjs

@@ -8,11 +8,11 @@ const args = process.argv.slice(2);
 
 if (
   args.length !== 3 ||
-  args[0] !== 'install' ||
+  !['install', 'update-models'].includes(args[0]) ||
   args[1] !== '--agent' ||
   args[2] !== 'codex'
 ) {
-  console.error('Usage: saving-token install --agent codex');
+  console.error('Usage: saving-token <install|update-models> --agent codex');
   process.exit(1);
 }
 
@@ -24,7 +24,12 @@ const setupPath = path.join(
   'scripts',
   'setup.sh'
 );
-const result = spawnSync('bash', [setupPath, '--target', 'codex'], {
+const setupArgs = [setupPath, '--target', 'codex'];
+if (args[0] === 'update-models') {
+  setupArgs.push('--update-models');
+}
+
+const result = spawnSync('bash', setupArgs, {
   stdio: 'inherit'
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saving-token",
-  "version": "1.0.0",
+  "version": "1.2.0",
   "bin": {
     "saving-token": "bin/saving-token.cjs"
   },

package/skills/saving-token/SKILL.md

@@ -18,18 +18,25 @@ If your identity is `saving-token-writer` or `saving-token:saving-token-writer`,
 - If a writable native writer or independent model configuration is unavailable, report instruction-only compatibility and ask before implementing directly.
 - Configuration is not runtime proof. Check exposed model metadata and disclose mismatches; never silently substitute a model yourself. Claude Code and Cursor may apply host-level fallback.
 
+## Model gate
+
+- Delegate only when the current host model is stronger than the configured writer model.
+- If the comparison is unknown, do not delegate; the parent completes the task directly under normal authorization.
+
 ## Delegate
 
 - Delegate every file write, including edits, deletions, documents, temporary scripts, and formatting.
 - Give goals and acceptance criteria, not implementation text to paste. Prefer file paths and issue locations over copied source or conversation history.
 - Batch related changes into one scoped task and reuse the writer for revisions.
 - The writer follows project rules and authorization boundaries, preserves user edits, implements directly, and runs relevant checks.
+- Put exact focused commands in `Verify`; the writer runs them with the change.
+- For slow E2E or independent confirmation, use a verification-only native subagent. It must not modify files and returns commands, exit codes, result counts when available, and code failures or environmental blockers.
 
 ```text
 Goal: <expected behavior>
-Scope: <paths; prohibited changes>
-Verify: <acceptance criteria and checks>
-Return: changed paths; check results; blockers.
+Scope: <paths; prohibited changes; or verification-only, no file changes>
+Verify: <exact commands>
+Return: changed paths; commands, exit codes, result counts when available; code failures or environmental blockers.
 ```
 
 ## Accept

package/skills/saving-token/assets/claude/saving-token-writer.md.template

@@ -12,4 +12,5 @@ disallowedTools:
 Implement directly. Never delegate or invoke another subagent.
 Follow project rules and authorization boundaries, and preserve existing user edits.
 Make only the requested changes and run the requested checks.
-Return changed paths, check commands and results, and blockers only.
+For verification-only work, do not modify files.
+Return changed paths; commands, exit codes, result counts when available; and code failures or environmental blockers only.

package/skills/saving-token/assets/codex/saving-token-writer.toml.template

@@ -7,5 +7,6 @@ developer_instructions = """
 Implement directly. Do not delegate further.
 Follow project rules and preserve existing user edits.
 Run the requested checks.
-Return changed paths, check results, and blockers only.
+For verification-only work, do not modify files.
+Return changed paths; commands, exit codes, result counts when available; and code failures or environmental blockers only.
 """

package/skills/saving-token/assets/cursor/saving-token-writer.md.template

@@ -11,4 +11,5 @@ is_background: false
 Implement directly. Never delegate or invoke another subagent.
 Follow project rules and authorization boundaries, and preserve existing user edits.
 Make only the requested changes and run the requested checks.
-Return changed paths, check commands and results, and blockers only.
+For verification-only work, do not modify files.
+Return changed paths; commands, exit codes, result counts when available; and code failures or environmental blockers only.

package/skills/saving-token/scripts/setup.sh

@@ -90,6 +90,7 @@ usage() {
   cat <<'HELP'
 Usage: setup.sh [--target codex|claude|cursor] [--model ID]
                 [--reasoning-effort EFFORT] [--yes]
+       setup.sh --update-models --target codex|claude|cursor
 
 Use Up/Down and Enter to choose a platform, model, effort, and confirmation.
 Esc cancels before any files are changed.
@@ -97,6 +98,7 @@ Esc cancels before any files are changed.
   --model ID                 Skip the model menu; required without a terminal.
   --reasoning-effort EFFORT   Optional. Omit to use the host's default behavior.
   --yes                      Approve installation and displayed replacements.
+  --update-models            Choose and update the writer model in an installed Skill.
   -h, --help                 Show this help.
 
 Model suggestions are not an account availability check. Use only models and
@@ -112,6 +114,7 @@ reasoning_effort=''
 model_was_set=false
 effort_was_set=false
 assume_yes=false
+update_models=false
 
 while [[ $# -gt 0 ]]; do
   case "$1" in
@@ -124,12 +127,17 @@ while [[ $# -gt 0 ]]; do
       esac
       shift 2 ;;
     --yes) assume_yes=true; shift ;;
+    --update-models) update_models=true; shift ;;
     -h|--help) usage; exit 0 ;;
     *) fail "Unknown option: $1" ;;
   esac
 done
 
 [[ -n "${HOME:-}" ]] || fail 'HOME is not set.'
+if [[ "$update_models" == true ]]; then
+  [[ -n "$target" ]] || fail 'Pass --target with --update-models.'
+  [[ "$model_was_set" == false && "$effort_was_set" == false && "$assume_yes" == false ]] || fail '--update-models cannot be combined with --model, --reasoning-effort, or --yes.'
+fi
 if [[ -z "$target" ]]; then
   [[ -t 0 ]] || fail 'Pass --target in non-interactive use.'
   if choose_menu 'Choose a platform:' 'Codex|codex' 'Claude Code|claude' 'Cursor|cursor'; then
@@ -145,30 +153,134 @@ case "$target" in
     skill_target="$codex_home/skills/saving-token"
     agent_dir="$codex_home/agents"
     agent_file='saving-token-writer.toml'
-    models=(gpt-5.6-luna gpt-5.6-terra gpt-5.3-codex-spark)
+    models=(gpt-6-luna gpt-6-sol gpt-6-astra)
     ;;
   claude)
     skill_target="$HOME/.claude/skills/saving-token"
     agent_dir="$HOME/.claude/agents"
     agent_file='saving-token-writer.md'
-    models=(haiku sonnet opus)
+    models=(claude-haiku-4-5-20251001 claude-sonnet-5 claude-opus-5-5 claude-fable-5-1)
     ;;
   cursor)
     skill_target="$HOME/.cursor/skills/saving-token"
     agent_dir="$HOME/.cursor/agents"
     agent_file='saving-token-writer.md'
-    models=(composer-2.5 gpt-5.6-sol)
+    models=(composer-2.5 gpt-5.6-luna gpt-5.6-terra gpt-5.6-sol claude-sonnet-5 claude-opus-5-5 claude-fable-5-1)
     ;;
   *) fail "Unsupported target '$target'. Expected codex, claude, or cursor." ;;
 esac
 
-if [[ "$model_was_set" != true ]]; then
-  [[ -t 0 ]] || fail 'Pass --model in non-interactive use.'
+agent_target="$agent_dir/$agent_file"
+
+choose_writer_model() {
+  local prompt=$1
+  local -a model_options
+  local model
+
   model_options=()
   for model in "${models[@]}"; do model_options[${#model_options[@]}]="$model|$model"; done
-  if choose_menu 'Choose a writer model:' "${model_options[@]}"; then
+  if choose_menu "$prompt" "${model_options[@]}"; then
     writer_model=$MENU_RESULT
+    return 0
+  fi
+  return 1
+}
+
+update_installed_model_suggestions() {
+  local installed_setup temporary_setup model_line
+
+  installed_setup="$skill_target/scripts/setup.sh"
+  [[ -f "$installed_setup" ]] || fail "No installed Skill script found at $installed_setup. Run install first."
+
+  model_line="    models=(${models[*]})"
+
+  temporary_setup="$(mktemp "$(dirname -- "$installed_setup")/.saving-token-setup.XXXXXX")"
+  cp -p "$installed_setup" "$temporary_setup"
+  if ! awk -v target="$target" -v model_line="$model_line" '
+    $0 == "  " target ")" { in_target = 1; print; next }
+    in_target && $0 == "    ;;" { in_target = 0; print; next }
+    in_target && $0 ~ /^    models=\([^)]*\)$/ {
+      replacements++
+      print model_line
+      next
+    }
+    { print }
+    END { exit replacements == 1 ? 0 : 1 }
+  ' "$installed_setup" > "$temporary_setup"; then
+    rm -f "$temporary_setup"
+    fail "Could not find one model list for $target in $installed_setup. Run install to refresh the Skill."
+  fi
+  if ! bash -n "$temporary_setup"; then
+    rm -f "$temporary_setup"
+    fail 'Updated Skill script failed syntax validation.'
+  fi
+  if cmp -s "$installed_setup" "$temporary_setup"; then
+    rm -f "$temporary_setup"
+    printf 'Model suggestions for %s are already current.\n' "$target"
+    return
+  fi
+
+  mv -f "$temporary_setup" "$installed_setup"
+  printf 'Updated model suggestions for %s in %s.\n' "$target" "$installed_setup"
+}
+
+update_writer_model() {
+  local temporary_agent model_line cursor_effort line
+
+  [[ -f "$agent_target" ]] || fail "No writer configuration found at $agent_target. Run install first."
+  if [[ "$target" == codex ]]; then
+    model_line="model = \"$writer_model\""
   else
+    cursor_effort=''
+    if [[ "$target" == cursor ]]; then
+      while IFS= read -r line || [[ -n "$line" ]]; do
+        if [[ "$line" =~ ^model:\ \"[^\"]*(\[effort=[^]]+\])\"$ ]]; then
+          cursor_effort=${BASH_REMATCH[1]}
+          break
+        fi
+      done < "$agent_target"
+    fi
+    model_line="model: \"$writer_model$cursor_effort\""
+  fi
+
+  temporary_agent="$(mktemp "$(dirname -- "$agent_target")/.${agent_file}.XXXXXX")"
+  cp -p "$agent_target" "$temporary_agent"
+  if ! awk -v target="$target" -v model_line="$model_line" '
+    (target == "codex" && $0 ~ /^model = "[^"]*"$/) ||
+    (target != "codex" && $0 ~ /^model: "[^"]*"$/) {
+      replacements++
+      print model_line
+      next
+    }
+    { print }
+    END { exit replacements == 1 ? 0 : 1 }
+  ' "$agent_target" > "$temporary_agent"; then
+    rm -f "$temporary_agent"
+    fail "Could not find one model field in $agent_target. Run install to refresh the writer configuration."
+  fi
+  if cmp -s "$agent_target" "$temporary_agent"; then
+    rm -f "$temporary_agent"
+    printf 'Writer model for %s is already %s.\n' "$target" "$writer_model"
+    return
+  fi
+
+  mv -f "$temporary_agent" "$agent_target"
+  printf 'Updated writer model for %s to %s.\n' "$target" "$writer_model"
+}
+
+if [[ "$update_models" == true ]]; then
+  [[ -t 0 ]] || fail '--update-models requires a terminal to choose a writer model.'
+  if ! choose_writer_model 'Choose a writer model to update:'; then
+    cancel_install
+  fi
+  update_installed_model_suggestions
+  update_writer_model
+  exit 0
+fi
+
+if [[ "$model_was_set" != true ]]; then
+  [[ -t 0 ]] || fail 'Pass --model in non-interactive use.'
+  if ! choose_writer_model 'Choose a writer model:'; then
     cancel_install
   fi
 fi
@@ -210,7 +322,6 @@ if [[ "$target" == claude ]]; then
 fi
 
 template_path="$SKILL_SOURCE/assets/$target/$agent_file.template"
-agent_target="$agent_dir/$agent_file"
 distributed_files=(SKILL.md scripts/setup.sh assets/codex/saving-token-writer.toml.template assets/claude/saving-token-writer.md.template assets/cursor/saving-token-writer.md.template)
 for file in "${distributed_files[@]}"; do
   [[ -f "$SKILL_SOURCE/$file" ]] || fail "Missing required Skill file: $SKILL_SOURCE/$file"