Remove quarantine row cap; confirm large bulk operations instead

The 50-row cap was a foot-gun guard, but it cut against the goal:
quarantine exists specifically so users can be brave about deleting
options and recover later. Capping at 50 means a bulk delete of 80 rows
partially succeeds with no UI surface for the failures — worse UX than
just letting it through.

Replaces the hard cap with a browser confirmation prompt when bulk
deleting more than 25 selected options. The prompt names the count and
reminds the user that quarantine is recoverable.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: ilicfilip

js/admin-script.js

@@ -779,6 +779,19 @@ jQuery( document ).ready( function () {
 				return;
 			}
 
+			// Warn before quarantining a large batch — quarantine is recoverable
+			// but sifting through hundreds of rows to find a culprit is painful.
+			if ( bulkAction === 'delete' && requestData.option_names.length > 25 ) {
+				const msg = aaaOptionOptimizer.i18n.confirmBulkQuarantine.replace(
+					'%d',
+					requestData.option_names.length
+				);
+				// eslint-disable-next-line no-alert
+				if ( ! window.confirm( msg ) ) {
+					return;
+				}
+			}
+
 			const endpoint =
 				'delete' === bulkAction
 					? 'delete-options'

src/class-admin-page.php

@@ -240,6 +240,8 @@ public function enqueue_scripts( $hook ) {
 					'restore'                => \esc_html__( 'Restore', 'aaa-option-optimizer' ),
 					'permanentlyDelete'      => \esc_html__( 'Permanently delete', 'aaa-option-optimizer' ),
 					'confirmPermanentDelete' => \esc_html__( 'Permanently delete this option? This cannot be undone.', 'aaa-option-optimizer' ),
+					/* translators: %d: number of selected options */
+					'confirmBulkQuarantine'  => \esc_html__( 'You are about to quarantine %d options. They can be restored from the Quarantine tab. Continue?', 'aaa-option-optimizer' ),
 					'importSelectFile'       => \esc_html__( 'Select JSON file', 'aaa-option-optimizer' ),
 					'importOverwriteLabel'   => \esc_html__( 'Overwrite existing options', 'aaa-option-optimizer' ),
 					'importButton'           => \esc_html__( 'Import', 'aaa-option-optimizer' ),

src/class-quarantine.php

@@ -17,11 +17,6 @@ class Quarantine {
 	 */
 	const CRON_HOOK = 'aaa_option_optimizer_quarantine_cleanup';
 
-	/**
-	 * Hard cap on simultaneously quarantined options.
-	 */
-	const MAX_ROWS = 50;
-
 	/**
 	 * Default retention in days.
 	 */
@@ -68,18 +63,6 @@ public function quarantine( $option_name ) {
 			);
 		}
 
-		if ( Database::count_quarantine_rows() >= self::MAX_ROWS ) {
-			return new \WP_Error(
-				'quarantine_full',
-				\sprintf(
-					/* translators: %d: max quarantine rows */
-					\__( 'Quarantine is full (max %d options). Restore or permanently delete some entries first.', 'aaa-option-optimizer' ),
-					self::MAX_ROWS
-				),
-				[ 'status' => 409 ]
-			);
-		}
-
 		global $wpdb;
 
 		// Read the live row directly so we capture the raw stored value and autoload flag.