saving-token 1.1.0 → 1.2.0

package/README.md

@@ -45,18 +45,20 @@ 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 refresh only the suggested models in an existing Codex installation, run:
+To choose a new writer model in an existing Codex installation, run:
 
 ```bash
 npx saving-token@latest update-models --agent codex
 ```
 
-It changes no writer configuration and shows no setup menus. Use the Bash
-script with `--update-models --target claude` or `cursor` for those platforms.
+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
 

package/package.json

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

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

@@ -98,7 +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            Update only model suggestions in an installed Skill.
+  --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
@@ -170,6 +170,22 @@ case "$target" in
   *) fail "Unsupported target '$target'. Expected codex, claude, or cursor." ;;
 esac
 
+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 "$prompt" "${model_options[@]}"; then
+    writer_model=$MENU_RESULT
+    return 0
+  fi
+  return 1
+}
+
 update_installed_model_suggestions() {
   local installed_setup temporary_setup model_line
 
@@ -205,21 +221,66 @@ update_installed_model_suggestions() {
   fi
 
   mv -f "$temporary_setup" "$installed_setup"
-  printf 'Updated model suggestions for %s in %s. Writer configuration was not changed.\n' "$target" "$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.'
-  model_options=()
-  for model in "${models[@]}"; do model_options[${#model_options[@]}]="$model|$model"; done
-  if choose_menu 'Choose a writer model:' "${model_options[@]}"; then
-    writer_model=$MENU_RESULT
-  else
+  if ! choose_writer_model 'Choose a writer model:'; then
     cancel_install
   fi
 fi
@@ -261,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"