fix(cli): rollback section state on strategy failure

Author: owanesh

cli/README.md

@@ -139,14 +139,19 @@ Without `--seed`, every run uses a fresh unseeded RNG (non-deterministic).
 ln -s function_body_tmp100.pkl strategies/data/function_body.pkl
 ```
 
-Strategies could work correctly when applied individually.
-
-Known issues:
-
-- `function_body_cloning` + `function_insertion` — cloned proxy calls break when new functions shift indices
-- `data_encryption` — injected decryption function may have mismatched signatures on certain modules
-
-Use `--validate` to catch failures, and `-e` to exclude problematic strategies:
+> [!TIP]
+> The CLI takes a snapshot of the section before each strategy. If a strategy crashes mid-execution, the section is automatically rolled back to its pre-strategy state, so partial modifications never corrupt the output.
+
+> [!WARNING]
+> **There are known issue on specific combos**
+>
+> - `function_insertion* + function_body_cloning`
+>   - `function_insertion` creates functions with `header=''` (empty string) instead of the expected `[[line, tag], ...]` list (`structural_perturbation.py:113`). When `function_body_cloning` runs next on the same section, it tries to access `header[0][0]` on one of those functions (`structural_perturbation.py:184-185`), which crashes because you can't index into an empty string. The CLI rolls back the section on failure, so the output stays valid — but `function_body_cloning` is effectively skipped.
+> - `data_encryption` — injected decryption function may have mismatched signatures on certain modules
+
+> ? NOTE: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nulla et euismod
+> nulla. Curabitur feugiat, tortor non consequat finibus, justo purus auctor
+> massa, nec semper lorem quam in massa.
 
 ```bash
 swamped obfuscate input.wasm -o out.wasm -s structural -e data_encryption function_body_cloning --validate

cli/swamped_cli.py

@@ -345,6 +345,7 @@ def cmd_obfuscate(args):
     for name in strategy_names:
         entry = STRATEGIES[name]
         logger.info("    -> %s", name)
+        snapshot = copy.deepcopy(section)
         t0 = time.monotonic()
         try:
             if name == "stack_op_insertion":
@@ -358,11 +359,12 @@ def cmd_obfuscate(args):
         except Exception as e:
             elapsed = time.monotonic() - t0
             timings.append((name, elapsed, False))
+            section = snapshot  # rollback to avoid corrupted state
             if args.strict:
                 logger.error("    [error] %s failed: %s", name, e)
                 sys.exit(1)
             else:
-                logger.warning("    [warning] %s failed: %s", name, e)
+                logger.warning("    [warning] %s failed (rolled back): %s", name, e)
 
     if args.timing and len(timings) > 1:
         total = sum(t for _, t, _ in timings)