Viewer: More tools from game picker (rename, move)

Author: klieret

.cursor/rules/viewer.mdc

@@ -271,7 +271,10 @@ The application has two main pages:
 * **Batch Operations**:
   - "Select All" checkbox to toggle all games
   - Styled action dropdown with custom arrow and hover effects for batch operations on selected games
-  - Copy pathnames copies selected game paths as space-separated string to clipboard
+  - **Copy paths**: Copies selected game paths as space-separated string to clipboard
+  - **Add suffix**: Prompts user for suffix text and adds it to folder names with dot separator (e.g., `experiment1` → `experiment1.backup`)
+  - **Copy foldernames**: Copies only folder names (not full paths) as space-separated string to clipboard
+  - **Move to subfolder**: Prompts user for subfolder name and moves all selected folders into it (creates subfolder if needed)
 
 ### Layout
 
@@ -347,6 +350,36 @@ The trajectory template displays:
 
 **Content Preview**: Long message content (>5 lines) shows a preview with expand/collapse functionality to keep the interface manageable.
 
+## Multi-Select Actions Specification
+
+The game picker includes four batch operations for selected game folders, accessible via the action dropdown:
+
+### 1. Copy Paths (`copy-paths`)
+- Copies full relative paths of selected games as space-separated string
+- Example: `fuchur/experiment1 fuchur/experiment2`
+- Uses modern clipboard API with execCommand fallback for non-HTTPS environments
+
+### 2. Add Suffix (`add-suffix`)
+- Prompts user for suffix text using browser's native `prompt()`
+- Adds suffix with dot separator: `experiment1` → `experiment1.backup`
+- Server endpoint: `POST /rename-folders` with `action: "add-suffix"`
+- Validates folder existence and prevents name conflicts
+- Shows success message and refreshes page after operation
+
+### 3. Copy Foldernames (`copy-foldernames`)
+- Copies only folder names (not full paths) as space-separated string
+- Example: From `fuchur/experiment1` copies just `experiment1`
+- Uses modern clipboard API with execCommand fallback for non-HTTPS environments
+
+### 4. Move to Subfolder (`move-to-subfolder`)
+- Prompts user for subfolder name using browser's native `prompt()`
+- Moves selected folders to new/existing subfolder in their parent directory
+- Server endpoint: `POST /move-to-subfolder`
+- Creates subfolder if it doesn't exist
+- Validates subfolder name (no path separators)
+- Shows success message and refreshes page after operation
+
+
 ### Important
 
 * Always use the player name from the metadata.json file, do not assume players are called p1, p2, etc.

codeclash/viewer/app.py

@@ -647,4 +647,144 @@ def load_readme():
         return jsonify({"success": False, "error": str(e)})
 
 
+@app.route("/rename-folders", methods=["POST"])
+def rename_folders():
+    """Add suffix to selected folders"""
+    try:
+        data = request.get_json()
+        action = data.get("action")
+        paths = data.get("paths", [])
+        suffix = data.get("suffix", "")
+
+        if not paths:
+            return jsonify({"success": False, "error": "No paths provided"})
+
+        if action != "add-suffix":
+            return jsonify({"success": False, "error": "Invalid action"})
+
+        if not suffix:
+            return jsonify({"success": False, "error": "No suffix provided"})
+
+        successful_renames = []
+        failed_renames = []
+
+        for relative_path in paths:
+            try:
+                # Get the full path
+                old_path = LOG_BASE_DIR / relative_path
+
+                if not old_path.exists():
+                    failed_renames.append(f"{relative_path}: does not exist")
+                    continue
+
+                # Create new path with suffix
+                parent_dir = old_path.parent
+                old_name = old_path.name
+                new_name = f"{old_name}.{suffix}"
+                new_path = parent_dir / new_name
+
+                # Check if target already exists
+                if new_path.exists():
+                    failed_renames.append(f"{relative_path}: target already exists")
+                    continue
+
+                # Perform the rename
+                old_path.rename(new_path)
+                successful_renames.append(f"{relative_path} → {old_name}.{suffix}")
+
+            except Exception as e:
+                failed_renames.append(f"{relative_path}: {str(e)}")
+
+        if failed_renames:
+            error_msg = "Some renames failed:\n" + "\n".join(failed_renames)
+            if successful_renames:
+                error_msg += "\n\nSuccessful renames:\n" + "\n".join(successful_renames)
+            return jsonify({"success": False, "error": error_msg})
+
+        return jsonify(
+            {
+                "success": True,
+                "message": f"Successfully renamed {len(successful_renames)} folder(s)",
+                "renamed": successful_renames,
+            }
+        )
+
+    except Exception as e:
+        return jsonify({"success": False, "error": str(e)})
+
+
+@app.route("/move-to-subfolder", methods=["POST"])
+def move_to_subfolder():
+    """Move selected folders to a new subfolder"""
+    try:
+        data = request.get_json()
+        paths = data.get("paths", [])
+        subfolder_name = data.get("subfolder", "")
+
+        if not paths:
+            return jsonify({"success": False, "error": "No paths provided"})
+
+        if not subfolder_name:
+            return jsonify({"success": False, "error": "No subfolder name provided"})
+
+        # Validate subfolder name (no path separators, etc.)
+        if "/" in subfolder_name or "\\" in subfolder_name or ".." in subfolder_name:
+            return jsonify({"success": False, "error": "Invalid subfolder name"})
+
+        successful_moves = []
+        failed_moves = []
+
+        for relative_path in paths:
+            try:
+                # Get the full path
+                old_path = LOG_BASE_DIR / relative_path
+
+                if not old_path.exists():
+                    failed_moves.append(f"{relative_path}: does not exist")
+                    continue
+
+                # Determine where to create the subfolder
+                parent_dir = old_path.parent
+                subfolder_path = parent_dir / subfolder_name
+
+                # Create subfolder if it doesn't exist
+                subfolder_path.mkdir(exist_ok=True)
+
+                # Create new path inside subfolder
+                folder_name = old_path.name
+                new_path = subfolder_path / folder_name
+
+                # Check if target already exists
+                if new_path.exists():
+                    failed_moves.append(f"{relative_path}: target already exists in subfolder")
+                    continue
+
+                # Perform the move
+                old_path.rename(new_path)
+
+                # Calculate the new relative path for display
+                new_relative_path = str(new_path.relative_to(LOG_BASE_DIR))
+                successful_moves.append(f"{relative_path} → {new_relative_path}")
+
+            except Exception as e:
+                failed_moves.append(f"{relative_path}: {str(e)}")
+
+        if failed_moves:
+            error_msg = "Some moves failed:\n" + "\n".join(failed_moves)
+            if successful_moves:
+                error_msg += "\n\nSuccessful moves:\n" + "\n".join(successful_moves)
+            return jsonify({"success": False, "error": error_msg})
+
+        return jsonify(
+            {
+                "success": True,
+                "message": f"Successfully moved {len(successful_moves)} folder(s) to subfolder '{subfolder_name}'",
+                "moved": successful_moves,
+            }
+        )
+
+    except Exception as e:
+        return jsonify({"success": False, "error": str(e)})
+
+
 # Use run_viewer.py to launch the application