Miscs improvements (Interro02)

copienator/annotation_actions.py

@@ -6,6 +6,7 @@ from pathlib import Path
 from typing import Any
 
 from .json_io import read_json
+from .feedback_boxes import valid_feedback_box
 
 Log = Callable[[str], None]
 
@@ -25,8 +26,10 @@ def apply_checkbox_actions(
             continue
         result = labels_data[label]["result"]
         feedbacks = result.get("feedback", [])
-        global_feedbacks = [item for item in feedbacks if not item.get("box_2d")]
+        # Match the renderer's fallback for invalid boxes so checkbox indices
-        local_feedbacks = [item for item in feedbacks if item.get("box_2d")]
+        # still address the right comment when returned annotations are read.
+        global_feedbacks = [item for item in feedbacks if not valid_feedback_box(item.get("box_2d"))]
+        local_feedbacks = [item for item in feedbacks if valid_feedback_box(item.get("box_2d"))]
         local_feedbacks.sort(key=lambda item: item["box_2d"][0])
 
         for action in label_actions:

copienator/annotation_data.py

@@ -8,6 +8,7 @@ from typing import Any
 from PIL import Image
 
 from .json_io import read_json
+from .feedback_boxes import valid_feedback_box
 from .workspace import EvaluationWorkspace
 
 AnnotationData = dict[str, dict[str, dict[str, Any]]]
@@ -69,7 +70,7 @@ def _scaled_result(result: dict[str, Any], coordinates: GroupCoordinates | None)
         return scaled
     for feedback in scaled.get("feedback", []):
         box = feedback.get("box_2d")
-        if not box or len(box) != 4:
+        if not box or not valid_feedback_box(box):
             continue
         box[0] = int(box[0] * coordinates.height) // 1000
         box[2] = int(box[2] * coordinates.height) // 1000

copienator/commands/add_final_score.py

@@ -7,11 +7,17 @@ from pathlib import Path
 import pandas as pd
 from PIL import Image, ImageDraw, ImageFont
 
-from copienator.configuration import FINAL_SCORE_FONT_PATH, FINAL_SCORE_ODS_PATH, FINAL_SCORE_OUTPUT_DIR
+from copienator.configuration import (
+    FINAL_SCORE_FONT_PATH,
+    FINAL_SCORE_HISTOGRAM_PATH,
+    FINAL_SCORE_ODS_PATH,
+    FINAL_SCORE_OUTPUT_DIR,
+)
 
 # Configuration constants
 ODS_PATH = Path(FINAL_SCORE_ODS_PATH).expanduser()
 OUTPUT_DIR = Path(FINAL_SCORE_OUTPUT_DIR).expanduser()
+HISTOGRAM_PATH = Path(FINAL_SCORE_HISTOGRAM_PATH).expanduser()
 
 
 def score_font(size):
@@ -33,6 +39,35 @@ def get_rounded_score(score):
     except (ValueError, TypeError):
         return None
 
+
+def copy_return_artifacts(source_dir: Path, destination_dir: Path) -> None:
+    """Copy the metadata and optional individual answers for one student."""
+    for filename in ("score.json", "info.json"):
+        source = source_dir / filename
+        if source.is_file():
+            shutil.copy2(source, destination_dir / filename)
+        else:
+            print(f"Warning: Missing '{source}'.")
+
+    answers_source = source_dir / "answers"
+    answers_destination = destination_dir / "answers"
+    if answers_destination.is_symlink():
+        answers_destination.unlink()
+    elif answers_destination.is_dir():
+        shutil.rmtree(answers_destination)
+    if answers_source.is_dir():
+        shutil.copytree(answers_source, answers_destination)
+
+
+def copy_histogram(output_dir: Path) -> None:
+    """Copy the score histogram beside the per-student output folders."""
+    if not HISTOGRAM_PATH.is_file():
+        print(f"Warning: Missing histogram '{HISTOGRAM_PATH}'.")
+        return
+    destination = output_dir / "histogramme.pdf"
+    shutil.copy2(HISTOGRAM_PATH, destination)
+    print(f"Copied histogram: {destination}")
+
 def process_images(base_dir, output_dir):
     # 1. Load Data
     try:
@@ -56,60 +91,64 @@ def process_images(base_dir, output_dir):
         print(f"Error: Directory '{search_path}' not found.")
         sys.exit(1)
 
-    for img_path in sorted(search_path.glob("*/*.jpg")):
+    for student_source in sorted(path for path in search_path.iterdir() if path.is_dir()):
-        student_name = img_path.stem # Filename without extension
+        image_paths = sorted(student_source.glob("*.jpg"))
+        pdf_paths = sorted(student_source.glob("*.pdf"))
+        media_paths = image_paths or pdf_paths
+        if not media_paths:
+            print(f"Error: No JPG or PDF found in '{student_source}'.")
+            continue
+
+        student_name = media_paths[0].stem
+        student_output = output_dir / student_name
+        student_output.mkdir(parents=True, exist_ok=True)
+        # Remove files produced by the former flat output layout when migrating
+        # an existing export directory.
+        for suffix in (".jpg", ".pdf"):
+            legacy_output = output_dir / f"{student_name}{suffix}"
+            if legacy_output.is_file() or legacy_output.is_symlink():
+                legacy_output.unlink()
+        copy_return_artifacts(student_source, student_output)
 
         # 4. Find Score
         if student_name not in score_db:
             print(f"Error: Student '{student_name}' not found in ODS file.")
-            continue
+        else:
+            raw_score = score_db[student_name]
+            score = get_rounded_score(raw_score)
 
-        raw_score = score_db[student_name]
+            if score is None:
-        score = get_rounded_score(raw_score)
+                print(f"Error: Invalid score '{raw_score}' for '{student_name}'.")
+            else:
+                # 5. Process Images
+                for img_path in image_paths:
+                    try:
+                        with Image.open(img_path) as img:
+                            img = img.convert("RGB")
+                            draw = ImageDraw.Draw(img)
+                            width, _height = img.size
 
-        if score is None:
+                            font_size = int(width * 0.08)
-            print(f"Error: Invalid score '{raw_score}' for '{student_name}'.")
+                            font = score_font(font_size)
-            continue
+                            text = str(score)
 
-        # 5. Process Image
+                            bbox = draw.textbbox((0, 0), text, font=font)
-        try:
+                            text_w = bbox[2] - bbox[0]
-            with Image.open(img_path) as img:
-                img = img.convert("RGB")
-                draw = ImageDraw.Draw(img)
-                width, height = img.size
 
-                # Dynamic font size (15% of image height)
+                            # 30px padding, top right.
-                font_size = int(width * 0.08)
+                            x = width - text_w - 30
+                            y = 30
+                            draw.text((x, y), text, fill=(255, 0, 0), font=font)
 
-                font = score_font(font_size)
+                            img.save(student_output / img_path.name)
+                            print(f"Processed: {student_name} -> {score}")
+                    except Exception as e:
+                        print(f"Error processing image for '{student_name}': {e}")
 
-                text = str(score)
+        for pdf_path in pdf_paths:
+            shutil.copy2(pdf_path, student_output / pdf_path.name)
 
-                # Calculate text size and position (Top Right)
+    copy_histogram(output_dir)
-                bbox = draw.textbbox((0, 0), text, font=font)
-                text_w = bbox[2] - bbox[0]
-                text_h = bbox[3] - bbox[1]
-
-                # 30px padding
-                x = width - text_w - 30
-                y = 30
-
-                # Draw Text (Red)
-                draw.text((x, y), text, fill=(255, 0, 0), font=font)
-
-                # Save
-                save_path = output_dir / f"{student_name}.jpg"
-                img.save(save_path)
-                print(f"Processed: {student_name} -> {score}")
-
-        except Exception as e:
-            print(f"Error processing image for '{student_name}': {e}")
-
-    for pdf_path in sorted(search_path.glob("*/*.pdf")):
-        student_name = pdf_path.stem # Filename without extension
-        save_path = output_dir / f"{student_name}.pdf"
-
-        shutil.copy(str(pdf_path), str(save_path))
 
 
 def main(argv=None):

copienator/commands/annotating.py

@@ -32,6 +32,7 @@ from copienator import (
 from copienator.annotation_data import load_annotation_data
 from copienator.answer_info import build_answer_info
 from copienator.filesystem import staged_directory
+from copienator.feedback_boxes import valid_feedback_box
 from copienator.utils import natural_key
 
 MARGIN_LEFT = 300
@@ -326,6 +327,16 @@ def compose_label_image(base_img, label, result, hmin,
 
     # Filter deleted items (used by reading_annotations.py)
     feedbacks = [f for f in feedbacks if "to_delete" not in f]
+    # Never guess where an invalid rectangle belongs, or lose its comment.
+    # Use a copy so rendering cannot mutate saved correction data.
+    normalized = []
+    for feedback in feedbacks:
+        box = feedback.get("box_2d")
+        if box is not None and not valid_feedback_box(box):
+            print(f"Warning: Copie{with_id or ''} {label}: invalid feedback box {box!r}; displaying the comment without a rectangle.")
+            feedback = {**feedback, "box_2d": None}
+        normalized.append(feedback)
+    feedbacks = normalized
 
     global_fb = [f for f in feedbacks if not f.get('box_2d')]
     local_fb = [f for f in feedbacks if f.get('box_2d')]

copienator/commands/batch_status.py

@@ -9,10 +9,13 @@ from google import genai
 from copienator import configuration as config
 from copienator import (
     CliError,
+    EvaluationWorkspace,
     ExitCode,
     atomic_write_bytes,
     execute,
+    read_json,
     standard_parser,
+    workspace_from_args,
 )
 
 
@@ -43,6 +46,41 @@ def list_jobs(*, client=None) -> ExitCode:
     return ExitCode.SUCCESS
 
 
+def check_evaluation_jobs(workspace: EvaluationWorkspace, *, client=None) -> ExitCode:
+    """Only report readiness after checking every job recorded for this evaluation."""
+    if not workspace.batch_jobs_file.is_file():
+        print("Impossible de vérifier les batchs : batch_jobs.json est absent.")
+        return ExitCode.PARTIAL
+    manifest = read_json(workspace.batch_jobs_file)
+    jobs = manifest.get("jobs") if isinstance(manifest, dict) else None
+    if not isinstance(jobs, dict):
+        raise CliError(f"Invalid batch manifest: {workspace.batch_jobs_file}")
+    if not jobs:
+        print("Aucun job enregistré pour cette évaluation.")
+        return ExitCode.PARTIAL
+    if any(not isinstance(entry, dict) or not isinstance(entry.get("name"), str)
+           or not entry["name"].strip() for entry in jobs.values()):
+        raise CliError(f"Invalid batch job in {workspace.batch_jobs_file}")
+    client = client or _client()
+    ready = True
+    for tier, entry in jobs.items():
+        job = client.batches.get(name=entry["name"])
+        state = job.state.name if hasattr(job.state, "name") else job.state
+        print(f"{tier} — {entry['name']}: {state}")
+        if state != "JOB_STATE_SUCCEEDED":
+            ready = False
+            if getattr(job, "error", None):
+                print(f"  Erreur : {job.error}")
+        elif not getattr(getattr(job, "dest", None), "file_name", None):
+            ready = False
+            print("  Le fichier de résultats n’est pas encore disponible.")
+    if ready:
+        print("Tous les batchs de l’évaluation ont réussi. Les résultats sont prêts à récupérer.")
+        return ExitCode.SUCCESS
+    print("Les résultats ne sont pas tous prêts. Consultez à nouveau cette étape plus tard.")
+    return ExitCode.PARTIAL
+
+
 def download_job(
     job_name: str,
     *,
@@ -73,6 +111,8 @@ def build_parser() -> argparse.ArgumentParser:
     parser = standard_parser("List or download Gemini correction batch jobs")
     parser.add_argument("--download", metavar="JOB_NAME")
     parser.add_argument("--output", type=Path, help="Downloaded JSONL destination")
+    parser.add_argument("--evaluation", type=Path,
+                        help="Check readiness of jobs recorded in this evaluation's batch_jobs.json")
     return parser
 
 
@@ -80,6 +120,11 @@ def main(argv: Sequence[str] | None = None) -> int:
     parser = build_parser()
 
     def handle(args: argparse.Namespace) -> ExitCode:
+        if args.evaluation is not None:
+            if args.download or args.output is not None:
+                raise CliError("--evaluation cannot be combined with --download or --output",
+                               ExitCode.INVALID_ARGUMENTS)
+            return check_evaluation_jobs(workspace_from_args(args))
         if args.output is not None and not args.download:
             raise CliError("--output requires --download", ExitCode.INVALID_ARGUMENTS)
         if args.download:

copienator/commands/correction.py

@@ -18,6 +18,7 @@ from google import genai
 from copienator import configuration as config
 from copienator.commands import grouping
 from copienator import prompting
+from copienator.feedback_boxes import valid_feedback_box
 from copienator import (
     CliError,
     EvaluationWorkspace,
@@ -84,6 +85,8 @@ def flush_thread_log(tid=None):
 # --- Lock for thread-safe file writing ---
 io_lock = threading.Lock()
 pro_lock = threading.Lock()
+group_index_lock = threading.Lock()
+reserved_group_indices: dict[str, int] = {}
 pro_count = 0
 flash_count = 0
 pro_quota_exhausted = False
@@ -154,6 +157,7 @@ def configure_runtime(
     completed_tasks = []
     results = {label: [] for _file, label in tasks}
     thread_logs.clear()
+    reserved_group_indices.clear()
     pro_count = 0
     flash_count = 0
     pro_quota_exhausted = False
@@ -267,6 +271,9 @@ def correct_boxes_with_gemini(pid, label, pdf_path, original_feedbacks,
     for f in corrected_feedbacks:
         b = f.get("box_2d")
         if b:
+            if not valid_feedback_box(b) or any(value < 0 or value > 1000 for value in b):
+                f["box_2d"] = None
+                continue
             ymin_s, xmin_s, ymax_s, xmax_s = b
 
             # Y mapping: Add the group Y-offset (yming), then normalize to total_height
@@ -290,6 +297,16 @@ def get_next_group_idx(label):
     if not existing: return 0
     return max([int(f.stem.split("_")[1]) for f in existing])
 
+
+def reserve_next_group_idx(label: str) -> int:
+    """Reserve a unique zero-based group index for this correction run."""
+    with group_index_lock:
+        if label not in reserved_group_indices:
+            reserved_group_indices[label] = get_next_group_idx(label)
+        idx = reserved_group_indices[label]
+        reserved_group_indices[label] = idx + 1
+        return idx
+
 def handle_label_errors(pid, label, res, pdf_path):
     """Handles Gemini labeling errors, moves/copies files, and returns new tasks."""
     new_tasks = []
@@ -330,7 +347,7 @@ def handle_label_errors(pid, label, res, pdf_path):
             if pdf_path != old_pdf_path:
                 shutil.move(str(pdf_path), str(old_pdf_path))
 
-            idx = get_next_group_idx(new_label)
+            idx = reserve_next_group_idx(new_label)
             height = grouping.get_pdf_height(str(new_pdf_path))
             grouping.create_jpg(new_label, idx, [(pid, str(new_pdf_path), height)], GROUPS_DIR)
             tprint(f"\t\tMaking {new_label} group {idx+1}")
@@ -362,7 +379,7 @@ def handle_label_errors(pid, label, res, pdf_path):
             if not base_add_pdf_path.exists() and not add_pdf_path.exists():
                 shutil.copy(str(pdf_path), str(add_pdf_path))
                 tprint(f"\t\tCopying Copie{pid} : {label} -> {add_label}")
-                idx = get_next_group_idx(add_label)
+                idx = reserve_next_group_idx(add_label)
                 tprint(f"\t\tMaking {add_label} group {idx+1}")
                 height = grouping.get_pdf_height(str(add_pdf_path))
                 grouping.create_jpg(add_label, idx, [(pid, str(add_pdf_path), height)], GROUPS_DIR)
@@ -475,6 +492,9 @@ def process_single_task(task_tuple, precomputed_response=None):
                 for (i,f) in enumerate(res["feedback"]):
                     b = f.get("box_2d")
                     if b:
+                        if not valid_feedback_box(b):
+                            needs_correction.append(i)
+                            continue
                         ymin, _xmin, ymax, xmax = b
                         ymin = ymin * total_height // 1000
                         ymax = ymax * total_height // 1000
@@ -484,9 +504,11 @@ def process_single_task(task_tuple, precomputed_response=None):
                                    pid, label, group_name)
                             continue
 
-                        if (ymin < yming - 50 or ymax > ymaxg + 50 or xmax / 1000 > width_r):
+                        if (ymin < yming - 50 or ymax > ymaxg + 50
+                                or ymin > ymaxg + 50 or ymax < yming - 50
+                                or _xmin < 0 or xmax / 1000 > width_r):
                             needs_correction.append(i)
-                            break
+                            continue
                         if ymin < yming - 5:
                             ymin = yming - 5
                             b[0] = ymin * 1000 // total_height
@@ -575,7 +597,7 @@ def resolve_delayed_moves():
                                 if pdf_path != old_pdf_path:
                                     shutil.move(str(pdf_path), str(old_pdf_path))
 
-                                idx = get_next_group_idx(target_label)
+                                idx = reserve_next_group_idx(target_label)
                                 height = grouping.get_pdf_height(str(new_pdf_path))
                                 grouping.create_jpg(target_label, idx, [(pid, str(new_pdf_path), height)], GROUPS_DIR)
                                 new_tasks.append((str(GROUPS_DIR / target_label / f"Group_{idx+1}.jpg"), target_label, False))
@@ -593,7 +615,7 @@ def resolve_delayed_moves():
                                 resolved_any = True
 
                                 shutil.copy(str(pdf_path), str(add_pdf_path))
-                                idx = get_next_group_idx(target_label)
+                                idx = reserve_next_group_idx(target_label)
                                 height = grouping.get_pdf_height(str(add_pdf_path))
                                 grouping.create_jpg(target_label, idx, [(pid, str(add_pdf_path), height)], GROUPS_DIR)
                                 new_tasks.append((str(GROUPS_DIR / target_label / f"Group_{idx+1}.jpg"), target_label, False))
@@ -679,7 +701,7 @@ def run_configured(args: argparse.Namespace) -> ExitCode:
                             # pdf_path = copie_dir / f"{label}_old.pdf"
 
                     if pdf_path.exists():
-                        idx = get_next_group_idx(label)
+                        idx = reserve_next_group_idx(label)
                         height = grouping.get_pdf_height(str(pdf_path))
                         grouping.create_jpg(label, idx, [(pid, str(pdf_path), height)], GROUPS_DIR)
                         new_group_path = str(GROUPS_DIR / label / f"Group_{idx+1}.jpg")
@@ -852,7 +874,7 @@ def run_configured(args: argparse.Namespace) -> ExitCode:
     # Check for remaining unresolved delayed tasks
     unresolved_delayed = []
     with io_lock:
-        for label, batches in results.items():
+        for label, batches in sorted(results.items()):
             for batch in batches:
                 for p in batch:
                     res = p.get("result", {})
@@ -869,7 +891,7 @@ def run_configured(args: argparse.Namespace) -> ExitCode:
         manual_path = INPUT_DIR / "manual_resolutions.txt"
         atomic_write_text(
             manual_path,
-            "###  Use -> x>, -x, ss, sx, xx, xs\n"
+            "###  Use -> x>, -x, ss, sx, xx, xs, c{43}1>, c{43}2x\n"
             + "\n".join(unresolved_delayed)
             + "\n",
         )

copienator/commands/cutleft.py

@@ -65,7 +65,9 @@ def stitch_images(image_list: list[Image.Image]) -> Image.Image | None:
 
 @lru_cache(maxsize=3)
 def _get_pdf_pages_cached(pdf_path: Path) -> list[Image.Image]:
-    return convert_from_path(pdf_path)
+    # Label coordinates use the full, displayed MediaBox, including on PDFs
+    # previously cropped by crop-margins. Keep this in sync with split-answers.
+    return convert_from_path(pdf_path, use_cropbox=False)
 
 
 def get_pdf_pages(pdf_path: Path) -> list[Image.Image]:

copienator/commands/gemini_for_enonce.py

@@ -86,6 +86,11 @@ même si certains textes fournis sont dans une autre langue.
 Conserve les formules mathématiques, les labels exacts des questions et les
 clés JSON `rubrics`, `label` et `rubric_content` sans les traduire.
 Chaque question DOIT être notée sur exactement 4 points. Propose une répartition logique de ces points.
+Il est inutile d'indiquer dans `rubric_content` que le barème totalise 4 points :
+ce total est toujours implicite.
+N'utilise pas de caractères mathématiques Unicode dans `rubric_content`.
+Écris les expressions mathématiques en LaTeX, par exemple
+`$\\lfloor \\sqrt{k} \\rfloor$` plutôt qu'avec des symboles Unicode.
 Par exemple :
  - Au moins 2 points si le résultat est correct.
  - Mettre la moitié des points si le raisonnement est correct mais pas le résultat.

copienator/commands/giving_names.py

@@ -29,9 +29,20 @@ def build_parser() -> argparse.ArgumentParser:
         choices=ANNOTATION_CHOICES,
         help="Annotation directory to use",
     )
+    parser.add_argument(
+        "--update",
+        action="store_true",
+        help=(
+            "Update only the individual images in existing A Rendre/answers "
+            "directories, matching folders by their trailing copy ID"
+        ),
+    )
     return parser
 
 
+RETURN_COPY_ID = re.compile(r"\((\d+)\)$")
+
+
 def _read_expected_names(workspace: EvaluationWorkspace) -> set[str]:
     names_path = workspace.names_file()
     if not names_path.exists():
@@ -47,6 +58,70 @@ def _read_expected_names(workspace: EvaluationWorkspace) -> set[str]:
     }
 
 
+def _annotation_source(
+    workspace: EvaluationWorkspace,
+    annotation_dir_name: str,
+    copy_id: str,
+) -> Path | None:
+    selected = workspace.root / annotation_dir_name / f"Copie{copy_id}"
+    fallback = workspace.annotation_dir("simple") / f"Copie{copy_id}"
+    for candidate in (selected, fallback):
+        if (candidate / "score.json").is_file() and (
+            (candidate / "Concat.jpg").is_file()
+            or (candidate / "info.json").is_file()
+        ):
+            return candidate
+    return None
+
+
+def update_named_return_answers(
+    workspace: EvaluationWorkspace,
+    annotation_dir_name: str,
+) -> ExitCode:
+    """Refresh only answers/ in existing returns, preserving manual names."""
+    workspace.require_directories(annotation_dir_name, "A Rendre")
+    had_errors = False
+    found = False
+    for destination in sorted(workspace.return_dir.iterdir()):
+        if not destination.is_dir():
+            continue
+        match = RETURN_COPY_ID.search(destination.name)
+        if match is None:
+            print(
+                f"Warning: cannot identify a copy ID in {destination.name!r}; skipped",
+                file=sys.stderr,
+            )
+            had_errors = True
+            continue
+        found = True
+        copy_id = match.group(1)
+        source_folder = _annotation_source(
+            workspace, annotation_dir_name, copy_id
+        )
+        if source_folder is None:
+            print(
+                f"Warning: no annotation source found for Copie{copy_id}; skipped",
+                file=sys.stderr,
+            )
+            had_errors = True
+            continue
+        try:
+            publish_answer_returns(
+                workspace.root,
+                source_folder,
+                destination,
+                answers_only=True,
+            )
+            print(f"Updated answers for {destination.name} from Copie{copy_id}")
+        except (OSError, TypeError, ValueError) as exc:
+            print(f"Error updating answers for {destination.name}: {exc}", file=sys.stderr)
+            had_errors = True
+    if not found:
+        print("Warning: no identifiable student folders found in A Rendre", file=sys.stderr)
+        had_errors = True
+    return ExitCode.PARTIAL if had_errors else ExitCode.SUCCESS
+
+
 def prepare_named_returns(
     workspace: EvaluationWorkspace,
     annotation_dir_name: str,
@@ -74,9 +149,6 @@ def prepare_named_returns(
             had_errors = True
 
     assigned_names: set[str] = set()
-    selected_annotations = workspace.root / annotation_dir_name
-    fallback_annotations = workspace.annotation_dir("simple")
-
     for name, copy_ids in copies_map.items():
         if name == "Unknown":
             print(
@@ -92,16 +164,9 @@ def prepare_named_returns(
 
         safe_name = safe_filename(name)
         for copy_id in copy_ids:
-            selected = selected_annotations / f"Copie{copy_id}"
+            source_folder = _annotation_source(
-            fallback = fallback_annotations / f"Copie{copy_id}"
+                workspace, annotation_dir_name, copy_id
-            source_folder = None
+            )
-            for candidate in (selected, fallback):
-                if (candidate / "score.json").is_file() and (
-                    (candidate / "Concat.jpg").is_file()
-                    or (candidate / "info.json").is_file()
-                ):
-                    source_folder = candidate
-                    break
             if source_folder is None:
                 continue
 
@@ -160,7 +225,10 @@ def run(
     workspace: EvaluationWorkspace,
     *,
     annotation_dir: str,
+    update: bool = False,
 ) -> ExitCode:
+    if update:
+        return update_named_return_answers(workspace, annotation_dir)
     return prepare_named_returns(workspace, annotation_dir)
 
 
@@ -172,6 +240,7 @@ def main(argv: Sequence[str] | None = None) -> int:
         lambda args: run(
             workspace_from_args(args, repository=Path.cwd()),
             annotation_dir=args.annotation_dir,
+            update=args.update,
         ),
     )
 

copienator/commands/post_correction.py

@@ -9,6 +9,7 @@ from typing import Any
 from copienator import (
     EvaluationWorkspace,
     ExitCode,
+    atomic_write_bytes,
     atomic_write_json,
     evaluation_parser,
     execute,
@@ -18,6 +19,10 @@ from copienator import (
 
 WORD_LIST_FILE = Path(__file__).resolve().parents[1] / "data" / "liste_francais.txt"
 ACCENT_PATTERN = re.compile(r"[éèêëàâäîïôöùûüçœÉÈÊËÀÂÄÎÏÔÖÙÛÜÇŒ]")
+MATH_PATTERN = re.compile(
+    r"(\$\$.*?\$\$|\$.*?\$|\\\(.*?\\\)|\\\[.*?\\\])",
+    re.DOTALL,
+)
 
 
 def build_parser() -> argparse.ArgumentParser:
@@ -25,22 +30,37 @@ def build_parser() -> argparse.ArgumentParser:
 
 
 def escape_latex_underscores(text: str) -> str:
-    r"""Escape underscores outside LaTeX math environments."""
+    r"""Escape underscores outside math without double-escaping existing ones."""
-    math_pattern = re.compile(
+
-        r"(\$\$.*?\$\$|\$.*?\$|\\\(.*?\\\)|\\\[.*?\\\])",
+    def escape_plain(value: str) -> str:
-        re.DOTALL,
+        # Collapse any existing escape run as well, making cleanup idempotent.
-    )
+        return re.sub(r"\\*_", lambda _match: r"\_", value)
+
     parts: list[str] = []
     last_end = 0
-    for match in math_pattern.finditer(text):
+    for match in MATH_PATTERN.finditer(text):
         start, end = match.span()
-        parts.append(text[last_end:start].replace("_", r"\_"))
+        parts.append(escape_plain(text[last_end:start]))
         parts.append(match.group(0))
         last_end = end
-    parts.append(text[last_end:].replace("_", r"\_"))
+    parts.append(escape_plain(text[last_end:]))
     return "".join(parts)
 
 
+def normalize_overescaped_latex_commands(text: str) -> str:
+    r"""Collapse doubled command escapes inside LaTeX math environments.
+
+    Model responses occasionally contain ``\\mathbb`` after JSON decoding where
+    LaTeX requires ``\mathbb``.  A doubled backslash followed by whitespace is a
+    legitimate row break (for example in ``cases``), so it must be preserved.
+    """
+
+    def normalize_math(match: re.Match[str]) -> str:
+        return re.sub(r"\\\\(?=[A-Za-z{}])", r"\\", match.group(0))
+
+    return MATH_PATTERN.sub(normalize_math, text)
+
+
 def build_lookup_map(word_list_path: Path = WORD_LIST_FILE) -> dict[str, str]:
     words = word_list_path.read_text(encoding="utf-8").splitlines()
     lookup: dict[str, str] = {}
@@ -67,8 +87,23 @@ def fix_hex_corruption_safe(text: str) -> str:
     )
 
 
-def some_other_replacements(text: str) -> str:
+def repair_json_escape_corruption(text: str) -> str:
-    return text.replace("\neq", "\\neq").replace("\not", "\\not")
+    r"""Restore observed LaTeX commands consumed as JSON control escapes."""
+    replacements = (
+        ("\x0crac", r"\frac"),
+        ("\x0ceuille", r"\equiv"),
+        ("\theta", r"\theta"),
+        ("\times", r"\times"),
+        ("\textbackslash ", "\\"),
+        ("\negthinspace", r"\negthinspace"),
+        ("\neq", r"\neq"),
+        ("\not", r"\not"),
+        ("∈", r"\ensuremath{\in}"),
+        ("⊂", r"\ensuremath{\subset}"),
+    )
+    for broken, repaired in replacements:
+        text = text.replace(broken, repaired)
+    return text
 
 
 def clean_string(text: str, lookup: dict[str, str]) -> str:
@@ -79,7 +114,9 @@ def clean_string(text: str, lookup: dict[str, str]) -> str:
     text = re.sub(r" \x00{1,2} ", " à ", text)
     if "\x00" in text:
         text = fast_fix(text, lookup).replace("\x00", "")
-    return escape_latex_underscores(some_other_replacements(text))
+    text = repair_json_escape_corruption(text)
+    text = normalize_overescaped_latex_commands(text)
+    return escape_latex_underscores(text)
 
 
 def clean_obj(value: Any, lookup: dict[str, str]) -> Any:
@@ -104,6 +141,9 @@ def run(
     lookup = build_lookup_map(word_list_path)
     data = read_json(workspace.correction_file)
     cleaned = clean_obj(data, lookup)
+    backup = workspace.root / "correction_precleanup.json"
+    atomic_write_bytes(backup, workspace.correction_file.read_bytes())
+    print(f"Original JSON backed up to {backup}")
     atomic_write_json(workspace.correction_file, cleaned)
     print(f"Fixed JSON saved to {workspace.correction_file}")
     return ExitCode.SUCCESS

copienator/commands/reading_annotations.py

@@ -10,7 +10,7 @@ from pdf2image import convert_from_path
 from PIL import Image, ImageChops, ImageDraw, ImageFilter
 
 from copienator.commands import annotating
-from copienator import utils
+from copienator import configuration, utils
 from copienator import (
     EvaluationWorkspace,
     ExitCode,
@@ -24,6 +24,7 @@ from copienator.annotation_actions import apply_checkbox_actions, apply_score_ov
 from copienator.answer_info import build_answer_info
 from copienator.annotation_data import AnnotationData, load_annotation_data
 from copienator.filesystem import staged_files
+from copienator.return_answers import save_return_answer_options
 
 Image.MAX_IMAGE_PIXELS = None
 
@@ -69,10 +70,11 @@ def detect_checks_and_notes(
         print(f"  Resizing annotated PDF from {user_image.size} to {reference.size}")
         user_image = user_image.resize(reference.size, Image.Resampling.LANCZOS)
 
-    difference = np.abs(
+    # Keep the full-size difference in uint8.  Converting both tall group images
-        np.array(reference).astype(int) - np.array(user_image).astype(int)
+    # to the platform ``int`` dtype used several gigabytes per scan worker.
-    ).astype(np.uint8)
+    difference = np.asarray(
-    difference_gray = np.mean(difference, axis=2)
+        ImageChops.difference(reference, user_image), dtype=np.uint8
+    )
     keep_mask = Image.new("L", reference.size, 255)
     mask_draw = ImageDraw.Draw(keep_mask)
     actions: list[dict[str, Any]] = []
@@ -83,10 +85,14 @@ def detect_checks_and_notes(
         x1, y1, x2, y2 = map(int, raw_box["global_box"])
         x1, y1 = max(0, x1), max(0, y1)
         x2, y2 = min(reference.width, x2), min(reference.height, y2)
-        region = difference_gray[y1 + 5 : y2 - 5, x1 + 5 : x2 - 5]
+        region = difference[y1 + 5 : y2 - 5, x1 + 5 : x2 - 5]
         if region.size == 0:
             continue
-        density = np.sum(region > 30) / region.size
+        # Preserve the previous mean-across-RGB threshold, but allocate its
+        # temporary float array only for the small checkbox region.
+        density = np.count_nonzero(np.mean(region, axis=2) > 30) / (
+            region.shape[0] * region.shape[1]
+        )
         if density > 0.05:
             actions.append(raw_box)
             mask_draw.rectangle([x1 - 15, y1 - 15, x2 + 15, y2 + 15], fill=0)
@@ -95,13 +101,17 @@ def detect_checks_and_notes(
         if raw_box.get("type") == "score" and raw_box.get("value") == 0.0:
             mask_draw.rectangle([0, y1 - 10, reference.width, y2 + 10], fill=0)
 
+    del difference
     reference_blur = reference.filter(ImageFilter.GaussianBlur(2))
     user_blur = user_image.filter(ImageFilter.GaussianBlur(2))
     diff_image = ImageChops.difference(reference_blur, user_blur).convert("L")
-    alpha = np.where(np.array(diff_image) > 50, 255, 0).astype(np.uint8)
+    del reference_blur, user_blur
-    final_alpha = np.minimum(alpha, np.array(keep_mask))
+    alpha = np.asarray(diff_image, dtype=np.uint8).copy()
+    np.greater(alpha, 50, out=alpha)
+    alpha *= np.uint8(255)
+    np.minimum(alpha, np.asarray(keep_mask, dtype=np.uint8), out=alpha)
     notes = user_image.convert("RGBA")
-    notes.putalpha(Image.fromarray(final_alpha))
+    notes.putalpha(Image.fromarray(alpha))
     return actions, notes
 
 
@@ -150,8 +160,10 @@ def apply_actions_and_regenerate(
 
     labels_data = data[student_id]
     apply_checkbox_actions(labels_data, actions, print)
+    score_path = output_dir / "score.json"
+    preserve_score_file = update_score and score_path.is_file()
     if update_score:
-        apply_score_overrides(labels_data, output_dir / "score.json", print)
+        apply_score_overrides(labels_data, score_path, print)
 
     scores = dict.fromkeys(all_labels, "")
     answer_labels: list[str] = []
@@ -220,7 +232,8 @@ def apply_actions_and_regenerate(
                                          "touched.json", "answer_labels.json")) as staging:
         for label, image in dirty_images.items():
             image.save(staging / f"{label}.jpg")
-        atomic_write_json(staging / "score.json", scores)
+        if not preserve_score_file:
+            atomic_write_json(staging / "score.json", scores)
         atomic_write_json(staging / "info.json", build_answer_info(
             scores, labels_data, answer_labels
         ))
@@ -229,13 +242,41 @@ def apply_actions_and_regenerate(
         if filtered_image is not None:
             filtered_image.save(staging / "Concat_F.jpg")
 
+    if preserve_score_file:
+        print(f"  Preserved existing score.json in {output_dir}")
     print(f"  Saved regenerated files in {output_dir}")
     return ExitCode.PARTIAL if incomplete else ExitCode.SUCCESS
 
 
-def run(workspace: EvaluationWorkspace, *, update_score: bool = False) -> ExitCode:
+def run(
+    workspace: EvaluationWorkspace,
+    *,
+    update_score: bool = False,
+    return_answers_context: bool | None = None,
+    return_answers_question: bool | None = None,
+    return_answers_solution: bool | None = None,
+) -> ExitCode:
     workspace.require_files("labels", "correction.json")
     workspace.require_directories("Copies", "Par label", "Bnot")
+    if configuration.RETURN_ANSWERS_ENABLED:
+        save_return_answer_options(
+            workspace.root,
+            context=(
+                configuration.RETURN_ANSWERS_CONTEXT
+                if return_answers_context is None
+                else return_answers_context
+            ),
+            question=(
+                configuration.RETURN_ANSWERS_QUESTION
+                if return_answers_question is None
+                else return_answers_question
+            ),
+            solution=(
+                configuration.RETURN_ANSWERS_SOLUTION
+                if return_answers_solution is None
+                else return_answers_solution
+            ),
+        )
     all_labels = utils.read_all_labels(workspace.root)
     loaded = load_annotation_data(workspace)
     for warning in loaded.warnings:
@@ -276,7 +317,28 @@ def build_parser() -> argparse.ArgumentParser:
     parser.add_argument(
         "--update-score",
         action="store_true",
-        help="Override generated scores with values from existing score.json files",
+        help=(
+            "Regenerate images with current statement/solution PDFs while "
+            "preserving and applying existing score.json values"
+        ),
+    )
+    parser.add_argument(
+        "--return-answers-context",
+        action=argparse.BooleanOptionalAction,
+        default=configuration.RETURN_ANSWERS_CONTEXT,
+        help="Include applicable context pages in individual answer exports",
+    )
+    parser.add_argument(
+        "--return-answers-question",
+        action=argparse.BooleanOptionalAction,
+        default=configuration.RETURN_ANSWERS_QUESTION,
+        help="Include the current question PDF in individual answer exports",
+    )
+    parser.add_argument(
+        "--return-answers-solution",
+        action=argparse.BooleanOptionalAction,
+        default=configuration.RETURN_ANSWERS_SOLUTION,
+        help="Include the current solution PDF in individual answer exports",
     )
     return parser
 
@@ -285,7 +347,13 @@ def main(argv: Sequence[str] | None = None) -> int:
     parser = build_parser()
 
     def handle(args: argparse.Namespace) -> ExitCode:
-        return run(workspace_from_args(args), update_score=args.update_score)
+        return run(
+            workspace_from_args(args),
+            update_score=args.update_score,
+            return_answers_context=args.return_answers_context,
+            return_answers_question=args.return_answers_question,
+            return_answers_solution=args.return_answers_solution,
+        )
 
     return execute(parser, argv, handle)
 

copienator/commands/reading_grouped_annotations.py

@@ -9,6 +9,7 @@ from typing import Any
 
 from PIL import Image, ImageDraw
 
+from copienator import configuration
 from copienator import (
     EvaluationWorkspace,
     ExitCode,
@@ -29,9 +30,11 @@ from copienator.commands.reading_annotations import (
     has_significant_notes,
 )
 from copienator.filesystem import staged_files
+from copienator.return_answers import save_return_answer_options
 
 LabelNotes = dict[str, dict[str, Any]]
 ScanResult = tuple[dict[str, list[dict[str, Any]]], dict[str, LabelNotes]]
+SCAN_WORKERS = 2
 
 
 def get_extra_pdfs_as_images(
@@ -191,9 +194,11 @@ def apply_actions_and_regenerate_grouped(
     output_dir = workspace.root / annotation_dir / f"Copie{student_id}"
     labels_data = data.get(student_id, {})
     apply_checkbox_actions(labels_data, actions, logs.append)
+    score_path = output_dir / "score.json"
+    preserve_score_file = update_score and score_path.is_file()
     if update_score:
         apply_score_overrides(
-            labels_data, output_dir / "score.json", logs.append
+            labels_data, score_path, logs.append
         )
 
     selected_labels = selected_labels if selected_labels is not None else set()
@@ -350,7 +355,8 @@ def apply_actions_and_regenerate_grouped(
             atomic_write_json(staging / "refaire_simple_layout.json", simple_layout)
         for label, image in dirty_images.items():
             image.save(staging / f"{label}.jpg")
-        atomic_write_json(staging / "score.json", scores)
+        if not preserve_score_file:
+            atomic_write_json(staging / "score.json", scores)
         atomic_write_json(staging / "info.json", build_answer_info(
             scores, labels_data, answer_labels, touched
         ))
@@ -364,6 +370,8 @@ def apply_actions_and_regenerate_grouped(
                     [image for group in filtered_groups for image in group]
                 )
                 filtered_image.save(staging / "Concat_F.jpg")
+    if preserve_score_file:
+        logs.append(f"  Preserved existing score.json in {output_dir}")
     logs.append(f"  Saved regenerated files in {output_dir}")
     status = ExitCode.PARTIAL if incomplete else ExitCode.SUCCESS
     return status, "\n".join(logs)
@@ -461,6 +469,9 @@ def run(
     refaire: bool = False,
     update_score: bool = False,
     annotation_dir: str = "BGnot",
+    return_answers_context: bool | None = None,
+    return_answers_question: bool | None = None,
+    return_answers_solution: bool | None = None,
 ) -> ExitCode:
     workspace.require_files("labels", "correction.json")
     workspace.require_directories("Copies", "Par label", annotation_dir)
@@ -470,6 +481,25 @@ def run(
         workspace.require_files("refaire.json")
         workspace.require_directories("BRnot")
         refaire_list, refaire_by_student = _read_refaire(workspace)
+    if configuration.RETURN_ANSWERS_ENABLED:
+        save_return_answer_options(
+            workspace.root,
+            context=(
+                configuration.RETURN_ANSWERS_CONTEXT
+                if return_answers_context is None
+                else return_answers_context
+            ),
+            question=(
+                configuration.RETURN_ANSWERS_QUESTION
+                if return_answers_question is None
+                else return_answers_question
+            ),
+            solution=(
+                configuration.RETURN_ANSWERS_SOLUTION
+                if return_answers_solution is None
+                else return_answers_solution
+            ),
+        )
 
     all_labels = utils.read_all_labels(workspace.root)
     loaded = load_annotation_data(workspace)
@@ -494,7 +524,10 @@ def run(
         and path.is_dir()
         and not path.name.startswith("Copie")
     ]
-    with concurrent.futures.ThreadPoolExecutor(max_workers=6) as executor:
+    # Each worker decodes a full-height returned group and its reference image.
+    # Keep this stage deliberately narrow; answer regeneration below has its own
+    # parallel executor and a much smaller per-task memory footprint.
+    with concurrent.futures.ThreadPoolExecutor(max_workers=SCAN_WORKERS) as executor:
         futures = [
             executor.submit(_scan_annotation_directory, path, only_ids)
             for path in group_dirs
@@ -599,7 +632,28 @@ def build_parser() -> argparse.ArgumentParser:
     parser.add_argument(
         "--update-score",
         action="store_true",
-        help="Override generated scores with values from existing score.json files",
+        help=(
+            "Regenerate images with current statement/solution PDFs while "
+            "preserving and applying existing score.json values"
+        ),
+    )
+    parser.add_argument(
+        "--return-answers-context",
+        action=argparse.BooleanOptionalAction,
+        default=configuration.RETURN_ANSWERS_CONTEXT,
+        help="Include applicable context pages in individual answer exports",
+    )
+    parser.add_argument(
+        "--return-answers-question",
+        action=argparse.BooleanOptionalAction,
+        default=configuration.RETURN_ANSWERS_QUESTION,
+        help="Include the current question PDF in individual answer exports",
+    )
+    parser.add_argument(
+        "--return-answers-solution",
+        action=argparse.BooleanOptionalAction,
+        default=configuration.RETURN_ANSWERS_SOLUTION,
+        help="Include the current solution PDF in individual answer exports",
     )
     return parser
 
@@ -615,6 +669,9 @@ def main(argv: Sequence[str] | None = None) -> int:
             refaire=args.refaire,
             update_score=args.update_score,
             annotation_dir=args.annotation_dir,
+            return_answers_context=args.return_answers_context,
+            return_answers_question=args.return_answers_question,
+            return_answers_solution=args.return_answers_solution,
         )
 
     return execute(parser, argv, handle)

copienator/commands/resolve_manual.py

@@ -9,6 +9,7 @@ from pathlib import Path
 from typing import Any
 
 from pypdf import PdfWriter
+from copienator.pdf_cut import split_pdf
 
 from copienator import (
     CliError,
@@ -22,7 +23,8 @@ from copienator import (
 )
 
 OPERATORS = ("-x", "->", "x>", "ss", "sx", "xx", "xs")
-OPERATOR_PATTERN = re.compile(r"\s+(-x|->|x>|ss|sx|xx|xs)\s+")
+CUT_PATTERN = re.compile(r"c\{(\d+(?:\.\d+)?)\}([12])([x>])")
+OPERATOR_PATTERN = re.compile(r"\s+(-x|->|x>|ss|sx|xx|xs|c\{\d+(?:\.\d+)?\}[12][x>])\s+")
 COPY_PATTERN = re.compile(r"Copie(\d+)\s+(.+)")
 
 
@@ -34,6 +36,11 @@ class ManualInstruction:
     new_label: str
     pipe_first: bool
 
+    @property
+    def cut(self) -> tuple[float, int] | None:
+        match = CUT_PATTERN.fullmatch(self.operator)
+        return (float(match[1]), int(match[2])) if match else None
+
     @property
     def should_merge(self) -> bool:
         return self.operator.endswith(">")
@@ -48,10 +55,14 @@ def build_parser() -> argparse.ArgumentParser:
 
 
 def parse_instructions(path: Path) -> list[ManualInstruction]:
+    return parse_instruction_text(path.read_text(encoding="utf-8"))
+
+
+def parse_instruction_text(text: str) -> list[ManualInstruction]:
     instructions: list[ManualInstruction] = []
     malformed: list[int] = []
     for line_number, raw_line in enumerate(
-        path.read_text(encoding="utf-8").splitlines(), start=1
+        text.splitlines(), start=1
     ):
         line = raw_line.strip()
         if not line or line.startswith("###"):
@@ -64,7 +75,10 @@ def parse_instructions(path: Path) -> list[ManualInstruction]:
         right = line[operator_match.end() :].strip()
         copy_match = COPY_PATTERN.fullmatch(left)
         new_label = right.strip("|").strip()
-        if copy_match is None or not new_label:
+        cut_match = CUT_PATTERN.fullmatch(operator_match.group(1))
+        if (copy_match is None or not new_label
+                or (cut_match and (not 0 < float(cut_match[1]) < 100
+                                   or copy_match.group(2).strip() == new_label))):
             malformed.append(line_number)
             continue
         instructions.append(
@@ -97,15 +111,25 @@ def set_suffix_and_clean_error(
                 item["result"]["suffix"] = suffix
             error = item["result"].get("error", "")
             if new_label_target:
-                if f"wrg-lbl:{new_label_target}?delayed" in error:
+                # This instruction acknowledges this source/target conflict,
-                    item["result"]["error"] = (
+                # including decisions to keep/discard PDFs without merging.
-                        f"wrg-lbl-moved-to:{new_label_target}"
+                # Leave other pending targets (and other copies) untouched.
-                    )
+                result = item["result"]
-                if f"(delayed){new_label_target}" in error:
+                if "delayed" in result:
-                    item["result"]["error"] = error.replace(
+                    pending = [entry for entry in result["delayed"]
-                        f"(delayed){new_label_target}",
+                               if entry not in (["wrong-label", new_label_target],
-                        f"(->){new_label_target}",
+                                                ["add-label", new_label_target])]
-                    )
+                    if pending:
+                        result["delayed"] = pending
+                    else:
+                        result.pop("delayed")
+                if error in {f"wrg-lbl:{new_label_target}?",
+                             f"wrg-lbl:{new_label_target}?delayed",
+                             f"wrg-lbl:{new_label_target}?exists"}:
+                    error = f"wrg-lbl-moved-to:{new_label_target}"
+                error = error.replace(f"(delayed){new_label_target}", f"(->){new_label_target}")
+                error = error.replace(f"(->){new_label_target}?", f"(->){new_label_target}")
+                result["error"] = error
 
 
 def get_actual_pdf(copies_dir: Path, copy_id: str, label: str) -> Path:
@@ -154,6 +178,9 @@ def resolve_manual(workspace: EvaluationWorkspace) -> ExitCode:
         raise CliError("correction.json must contain a JSON object")
     results: dict[str, Any] = loaded
     instructions = parse_instructions(workspace.manual_resolutions_file)
+    cut_sources = [(item.copy_id, item.old_label) for item in instructions if item.cut]
+    if len(set(cut_sources)) != len(cut_sources):
+        raise CliError("Une seule coupe par label source est autorisée dans une résolution.")
 
     initial_paths: dict[tuple[str, str], Path] = {}
     current_paths: dict[tuple[str, str], Path] = {}
@@ -174,6 +201,15 @@ def resolve_manual(workspace: EvaluationWorkspace) -> ExitCode:
             key_new = (instruction.copy_id, instruction.new_label)
             source = initial_paths[key_old]
             destination = current_paths[key_new]
+            if instruction.cut:
+                percent, keep = instruction.cut
+                first = source.parent / f"temp_{len(temp_files)}.pdf"
+                second = source.parent / f"temp_{len(temp_files) + 1}.pdf"
+                temp_files.extend((first, second))
+                split_pdf(source, percent, first, second)
+                retained, source = (first, second) if keep == 1 else (second, first)
+                current_paths[key_old] = retained
+                files_to_old.add(initial_paths[key_old])
             temp_output = (
                 workspace.copies_dir
                 / f"Copie{instruction.copy_id}"

copienator/commands/splitting_int.py

@@ -80,15 +80,9 @@ def _save_cropped_page(
     y1: float,
     output_path: Path,
 ) -> None:
-    page = document[page_number]
+    # The source has been normalized by _prepare_split_pages: no rotation or
-    rotated_rectangle = page.rect * page.transformation_matrix
+    # CropBox translation remains in the coordinates passed to show_pdf_page.
-    visual_crop = pymupdf.Rect(
+    visual_crop = pymupdf.Rect(x0, y0, x1, y1)
-        rotated_rectangle.x0 + x0,
-        y0,
-        rotated_rectangle.x0 + x1,
-        y1,
-    )
-    unrotated_clip = visual_crop * page.derotation_matrix
     cropped = pymupdf.open()
     try:
         target_page = cropped.new_page(width=visual_crop.width, height=visual_crop.height)
@@ -96,14 +90,32 @@ def _save_cropped_page(
             target_page.rect,
             document,
             page_number,
-            rotate=-page.rotation,
+            clip=visual_crop,
-            clip=unrotated_clip,
         )
         cropped.save(output_path)
     finally:
         cropped.close()
 
 
+def _prepare_split_pages(document: pymupdf.Document) -> list[pymupdf.Rect]:
+    """Use the label preview's full-page coordinates; retain visible bounds.
+
+    Work only on the in-memory document. Baking rotation into the content after
+    restoring the MediaBox avoids show_pdf_page's rotated CropBox offsets.
+    Intersecting with the saved bounds later preserves prior margin cropping.
+    """
+    visible_bounds = []
+    for page in document:
+        crop = page.cropbox
+        media = page.mediabox
+        full_crop = pymupdf.Rect(media.x0, 0, media.x1, media.height)
+        page.set_cropbox(full_crop)
+        crop -= (full_crop.x0, full_crop.y0, full_crop.x0, full_crop.y0)
+        visible_bounds.append(crop * page.rotation_matrix)
+        page.remove_rotation()
+    return visible_bounds
+
+
 def _render_split_outputs(
     input_pdf: Path,
     coords_list: list[Coordinate],
@@ -112,6 +124,7 @@ def _render_split_outputs(
     """Render every current answer into an otherwise empty staging directory."""
     document = pymupdf.open(input_pdf)
     try:
+        visible_bounds = _prepare_split_pages(document)
         parsed = _parse_coordinates(coords_list)
         parts_by_label: defaultdict[str, list[Path]] = defaultdict(list)
         with tempfile.TemporaryDirectory(prefix="copienator-split-") as temp_directory:
@@ -160,16 +173,20 @@ def _render_split_outputs(
                     page = document[page_number]
                     y0 = (y_start / 1000) * page.rect.height if page_number == start_page else 0
                     y1 = (end_y / 1000) * page.rect.height if page_number == end_page else page.rect.height
-                    if y1 <= y0 + 1:
+                    clip = pymupdf.Rect(
+                        fraction_x0 * page.rect.width, y0,
+                        fraction_x1 * page.rect.width, y1,
+                    ) & visible_bounds[page_number] & page.rect
+                    if clip.is_empty or clip.height <= 1 or clip.width <= 1:
                         continue
                     part_path = temporary / f"part-{index}-{page_number}.pdf"
                     _save_cropped_page(
                         document,
                         page_number,
-                        fraction_x0 * page.rect.width,
+                        clip.x0,
-                        y0,
+                        clip.y0,
-                        fraction_x1 * page.rect.width,
+                        clip.x1,
-                        y1,
+                        clip.y1,
                         part_path,
                     )
                     parts_by_label[clean_label].append(part_path)
@@ -264,4 +281,3 @@ def main(argv: Sequence[str] | None = None) -> int:
 
 if __name__ == "__main__":
     raise SystemExit(main())
-

copienator/configuration.py

@@ -39,6 +39,7 @@ RETURN_ANSWERS_ENABLED = False
 RETURN_ANSWERS_CONTEXT = False
 RETURN_ANSWERS_QUESTION = True
 RETURN_ANSWERS_SOLUTION = False
+FINAL_SCORE_HISTOGRAM_PATH = Path("histogramme.pdf")
 for _name in dir(_configuration):
     if not _name.startswith("_"):
         globals()[_name] = getattr(_configuration, _name)

copienator/feedback_boxes.py

@@ -0,0 +1,13 @@
+"""Geometry checks shared by correction and annotation rendering."""
+
+import math
+from numbers import Real
+
+
+def valid_feedback_box(box) -> bool:
+    return (
+        isinstance(box, (list, tuple)) and len(box) == 4
+        and all(isinstance(value, Real) and not isinstance(value, bool)
+                and math.isfinite(value) for value in box)
+        and box[0] < box[2] and box[1] < box[3]
+    )

copienator/pdf_cut.py

@@ -0,0 +1,57 @@
+"""Split a PDF along its cumulative visible page height, in reading order."""
+
+import math
+from pathlib import Path
+
+import pymupdf
+
+from copienator.commands.splitting_int import _prepare_split_pages
+
+# Percentages emitted by the GUI have six decimal places. Recover exact page
+# boundaries despite rounding, without turning nearby in-page cuts into breaks.
+BOUNDARY_TOLERANCE_PERCENT = 0.000001
+
+
+def cut_position(heights: list[float], percent: float) -> tuple[int, float]:
+    """Return (page index, offset); offset zero denotes an exact page break."""
+    if not heights or any(height <= 0 for height in heights):
+        raise ValueError("Le PDF doit contenir des pages non vides.")
+    if not math.isfinite(percent) or not 0 < percent < 100:
+        raise ValueError("Le pourcentage de coupe doit être strictement entre 0 et 100.")
+    total = sum(heights)
+    position = total * percent / 100
+    start = 0.0
+    for index, height in enumerate(heights):
+        if index and abs(percent - start / total * 100) <= BOUNDARY_TOLERANCE_PERCENT:
+            return index, 0.0
+        end = start + height
+        if index + 1 < len(heights) and abs(percent - end / total * 100) <= BOUNDARY_TOLERANCE_PERCENT:
+            return index + 1, 0.0
+        if position < end:
+            return index, position - start
+        start = end
+    raise ValueError("Coupe hors du document.")
+
+
+def split_pdf(source: Path, percent: float, first_path: Path, second_path: Path) -> None:
+    """Preserve whole pages; clip only the page actually crossed by the cut."""
+    with pymupdf.open(source) as document, pymupdf.open() as first, pymupdf.open() as second:
+        index, offset = cut_position([page.rect.height for page in document], percent)
+        if index:
+            first.insert_pdf(document, from_page=0, to_page=index - 1)
+        if offset == 0:
+            second.insert_pdf(document, from_page=index)
+        else:
+            with pymupdf.open() as page_document:
+                page_document.insert_pdf(document, from_page=index, to_page=index)
+                visible = _prepare_split_pages(page_document)[0]
+                for target, clip in (
+                    (first, pymupdf.Rect(visible.x0, visible.y0, visible.x1, visible.y0 + offset)),
+                    (second, pymupdf.Rect(visible.x0, visible.y0 + offset, visible.x1, visible.y1)),
+                ):
+                    page = target.new_page(width=clip.width, height=clip.height)
+                    page.show_pdf_page(page.rect, page_document, 0, clip=clip)
+            if index + 1 < len(document):
+                second.insert_pdf(document, from_page=index + 1)
+        first.save(first_path)
+        second.save(second_path)

copienator/prompting.py

@@ -2,6 +2,11 @@ from pathlib import Path
 import io
 from . import utils
 
+PERSPECTIVE_GUIDANCE = (
+    "Ce barème est indicatif, si une réponse est entièrement correcte mais utilise "
+    "une méthode différente, elle mérite quand même tous les points."
+)
+
 main_prompt = """Je te fournis une image contenant plusieurs réponses manuscrites à un examen.
 
 Chaque réponse est séparée de la précédente par une ligne horizontale noire.
@@ -103,7 +108,13 @@ def make_prompt(input_dir,full_label):
     # print("Debug : l/t/c/p", full_label, text, corr, persp)
 
     if persp:
-        persp = "\n\nVoici des consignes de notation complémentaires : \n\n```\n" + persp +"\n```\n"
+        persp = (
+            "\n\nVoici des consignes de notation complémentaires :\n\n"
+            + PERSPECTIVE_GUIDANCE
+            + "\n\n```\n"
+            + persp
+            + "\n```\n"
+        )
     return main_prompt.replace("<<text>>", text).replace("<<corr>>", corr).replace("<<persp>>", persp).replace("<<label>>", full_label)
 
 

copienator/return_answers.py

@@ -8,9 +8,56 @@ from copienator import atomic_write_json, configuration, read_json, utils
 from copienator.filesystem import staged_directory
 from copienator.platform import safe_filename
 
+RETURN_ANSWER_OPTIONS_FILE = Path(".copienator") / "return_answers.json"
 
-def publish_answer_returns(root: Path, source: Path, destination: Path) -> None:
+
-    """Publish individual reviewed answers and per-question information."""
+def configured_return_answer_options() -> dict[str, bool]:
+    return {
+        "context": bool(configuration.RETURN_ANSWERS_CONTEXT),
+        "question": bool(configuration.RETURN_ANSWERS_QUESTION),
+        "solution": bool(configuration.RETURN_ANSWERS_SOLUTION),
+    }
+
+
+def save_return_answer_options(
+    root: Path,
+    *,
+    context: bool,
+    question: bool,
+    solution: bool,
+) -> None:
+    path = Path(root) / RETURN_ANSWER_OPTIONS_FILE
+    path.parent.mkdir(parents=True, exist_ok=True)
+    atomic_write_json(
+        path,
+        {"context": context, "question": question, "solution": solution},
+    )
+
+
+def load_return_answer_options(root: Path) -> dict[str, bool]:
+    options = configured_return_answer_options()
+    path = Path(root) / RETURN_ANSWER_OPTIONS_FILE
+    if not path.is_file():
+        return options
+    loaded = read_json(path)
+    if not isinstance(loaded, dict):
+        raise ValueError(f"Expected a return-answer options object in {path}")
+    for name in options:
+        if name in loaded:
+            if type(loaded[name]) is not bool:
+                raise ValueError(f"Expected a boolean for {name!r} in {path}")
+            options[name] = loaded[name]
+    return options
+
+
+def publish_answer_returns(
+    root: Path,
+    source: Path,
+    destination: Path,
+    *,
+    answers_only: bool = False,
+) -> None:
+    """Publish reviewed answers, optionally without touching return metadata."""
     scores = read_json(source / "score.json")
     if not isinstance(scores, dict):
         raise ValueError(f"Expected a score object in {source}")
@@ -36,6 +83,7 @@ def publish_answer_returns(root: Path, source: Path, destination: Path) -> None:
     if answers_dir.is_symlink():
         raise ValueError(f"Expected a real answer directory: {answers_dir}")
     if configuration.RETURN_ANSWERS_ENABLED:
+        options = load_return_answer_options(root)
         labels = [label for label, entry in info.items() if entry["present"] and entry["not_empty"]]
 
         # Import the rendering backend only when individual images are requested.
@@ -44,13 +92,13 @@ def publish_answer_returns(root: Path, source: Path, destination: Path) -> None:
 
         all_labels = utils.read_all_labels(root)
         with staged_directory(answers_dir) as staging:
-            for index, label in enumerate(sorted(labels, key=utils.natural_key), 1):
+            for label in sorted(labels, key=utils.natural_key):
                 paths = []
-                if configuration.RETURN_ANSWERS_CONTEXT:
+                if options["context"]:
                     paths.extend(utils.pdf_images_of_contexts(root, label, all_labels))
-                if configuration.RETURN_ANSWERS_QUESTION:
+                if options["question"]:
                     paths.append(utils.pdf_image_of_enonce(root, label))
-                if configuration.RETURN_ANSWERS_SOLUTION:
+                if options["solution"]:
                     paths.append(utils.pdf_image_of_solution(root, label))
                 images = []
                 for path in paths:
@@ -62,11 +110,16 @@ def publish_answer_returns(root: Path, source: Path, destination: Path) -> None:
                 with Image.open(source / f"{label}.jpg") as answer:
                     images.append(answer.convert("RGB"))
                 image = concatenate(images)
-                # Numbering avoids collisions between sanitized label filenames.
+                output = staging / f"{safe_filename(label)}.jpg"
-                image.save(staging / f"{index:03d} - {safe_filename(label)}.jpg")
+                if output.exists():
+                    raise ValueError(
+                        f"Answer labels produce the same filename in {answers_dir}: {label}"
+                    )
+                image.save(output)
     elif answers_dir.exists():
         # Replace the managed directory with an empty one to remove stale exports.
         with staged_directory(answers_dir):
             pass
-    atomic_write_json(destination / "info.json", info)
+    if not answers_only:
-    (destination / "touched.json").unlink(missing_ok=True)
+        atomic_write_json(destination / "info.json", info)
+        (destination / "touched.json").unlink(missing_ok=True)

copienator/utils.py

@@ -160,23 +160,38 @@ def compile_to_pdf(text, output_pdf_path):
         # env['TEXINPUTS'] = f".:{current_dir}:"
 
         try:
-            subprocess.run(
+            result = subprocess.run(
                 ['pdflatex', '-interaction=nonstopmode', tex_filename],
                 cwd=temp_dir,
-                stdout=subprocess.DEVNULL,
+                stdout=subprocess.PIPE,
-                stderr=subprocess.DEVNULL,
+                stderr=subprocess.STDOUT,
+                text=True,
                 check=False
             )
             if "minted" in text:
-                subprocess.run(
+                result = subprocess.run(
                     ['pdflatex', '-interaction=nonstopmode', tex_filename],
                     cwd=temp_dir,
-                    stdout=subprocess.DEVNULL,
+                    stdout=subprocess.PIPE,
-                    stderr=subprocess.DEVNULL,
+                    stderr=subprocess.STDOUT,
+                    text=True,
                     check=False)
 
+            if result.returncode != 0:
+                error_lines = [
+                    line.strip() for line in result.stdout.splitlines()
+                    if line.lstrip().startswith("!")
+                ]
+                detail = f": {error_lines[0]}" if error_lines else ""
+                print(
+                    f"Warning: LaTeX compilation failed for {output_pdf_path} "
+                    f"(exit code {result.returncode}){detail}"
+                )
+
             generated_pdf = os.path.join(temp_dir, pdf_filename)
             if os.path.exists(generated_pdf):
                 shutil.move(generated_pdf, output_pdf_path)
+            else:
+                print(f"Warning: LaTeX compilation produced no PDF for {output_pdf_path}")
         except Exception as e:
             print(f"Compilation error for {output_pdf_path}: {e}")

copienator_gui/app.py

@@ -2,6 +2,8 @@ from __future__ import annotations
 
 import os
 import queue
+import re
+import shutil
 import tkinter as tk
 from dataclasses import replace
 from pathlib import Path
@@ -10,6 +12,7 @@ from typing import Any
 
 from copienator import CliError, EvaluationWorkspace, ExitCode, atomic_write_json
 from copienator.copy_errors import copy_errors
+from copienator.filesystem import staged_files
 from copienator.platform import (
     WindowsLabelError,
     add_platform_executable_paths,
@@ -18,6 +21,9 @@ from copienator.platform import (
 )
 
 from .diagnostics import collect_diagnostics
+from .batch_monitor import BatchMonitor
+from .notifications import notify_desktop
+from .manual_resolution import ManualResolutionPanel
 from .refaire import SECTION as REFAIRE_SECTION
 from .refaire import (
     RefaireSelection,
@@ -52,6 +58,7 @@ STATUS_LABELS = {
 }
 
 DEFAULT_HTTPS_PROXY = "http://10.0.0.1:3128"
+PERSONAL_INTERRO_SOURCE = Path("/home/sebastien/Prépa/Staging/Interro")
 ANNOTATION_DIRECTORIES = ("BGnot", "Bnot", "Anot")
 ANNOTATION_VARIANT_DIRECTORIES = {
     "simple": "Anot",
@@ -60,6 +67,33 @@ ANNOTATION_VARIANT_DIRECTORIES = {
 }
 
 
+def get_personal_interro_files(
+    evaluation: Path, source_directory: Path = PERSONAL_INTERRO_SOURCE
+) -> tuple[Path, Path, Path]:
+    """Copy an Interro project's statement sources into the evaluation."""
+    match = re.fullmatch(r"Interro(\d+)", evaluation.name)
+    if match is None:
+        raise ValueError("Le dossier d’évaluation doit s’appeler Interro{id}, avec un identifiant numérique.")
+
+    project_name = evaluation.name
+    source_to_destination = (
+        (source_directory / f"{project_name}.pdf", "enonce.pdf"),
+        (source_directory / f"{project_name}.tex", "enonce.tex"),
+        (source_directory / f"{project_name}c.tex", "correction.tex"),
+    )
+    missing = [source.name for source, _destination in source_to_destination if not source.is_file()]
+    if missing:
+        raise FileNotFoundError(
+            "Fichier(s) source introuvable(s) dans "
+            f"{source_directory} : {', '.join(missing)}"
+        )
+
+    with staged_files(evaluation) as staging:
+        for source, destination_name in source_to_destination:
+            shutil.copy2(source, staging / destination_name)
+    return tuple(evaluation / name for _source, name in source_to_destination)
+
+
 class Tooltip:
     """Small delayed tooltip for Tk and ttk widgets."""
 
@@ -243,6 +277,10 @@ class CopienatorApp(tk.Tk):
         self.extra_var = tk.StringVar()
         self.command_var = tk.StringVar(value="Sélectionnez une étape.")
         self.info_var = tk.StringVar(value="Choisissez un dossier d’évaluation.")
+        self.batch_watch_status = tk.StringVar(value="Vérification automatique arrêtée.")
+        self.batch_monitor = BatchMonitor(
+            self, self._batch_results_ready, self._batch_watch_status_changed, self._append_console
+        )
 
         self._build_ui(show_personal_steps)
         self.extra_var.trace_add("write", lambda *_args: self._update_command_preview())
@@ -338,13 +376,23 @@ class CopienatorApp(tk.Tk):
 
         self.detail = ttk.Frame(main_pane, padding=(12, 6, 4, 4))
         self.detail.columnconfigure(0, weight=1)
-        self.detail.rowconfigure(2, weight=1)
+        self.detail.rowconfigure(3, weight=1)
         self.title_label = ttk.Label(self.detail, text="Sélectionnez une étape", font=("TkDefaultFont", 15, "bold"))
         self.title_label.grid(row=0, column=0, sticky="w")
         self.description_label = ttk.Label(self.detail, text="", wraplength=680, justify="left")
         self.description_label.grid(row=1, column=0, sticky="ew", pady=(5, 8))
+        ttk.Style(self).configure("MissingPrerequisite.TLabel", foreground="#c62828")
+        self.missing_requirements_label = ttk.Label(
+            self.detail,
+            text="",
+            style="MissingPrerequisite.TLabel",
+            wraplength=680,
+            justify="left",
+        )
+        self.missing_requirements_label.grid(row=2, column=0, sticky="ew", pady=(0, 8))
+        self.missing_requirements_label.grid_remove()
         form_container = ttk.Frame(self.detail)
-        form_container.grid(row=2, column=0, sticky="nsew")
+        form_container.grid(row=3, column=0, sticky="nsew")
         form_container.columnconfigure(0, weight=1)
         form_container.rowconfigure(0, weight=1)
         self.form_canvas = tk.Canvas(form_container, highlightthickness=0, width=1, height=1)
@@ -359,7 +407,7 @@ class CopienatorApp(tk.Tk):
         self.form.columnconfigure(1, weight=1)
 
         command_box = ttk.LabelFrame(self.detail, text="Commande", padding=6)
-        command_box.grid(row=3, column=0, sticky="ew", pady=(8, 5))
+        command_box.grid(row=4, column=0, sticky="ew", pady=(8, 5))
         command_box.columnconfigure(0, weight=1)
         ttk.Label(command_box, textvariable=self.command_var, wraplength=690, justify="left").grid(
             row=0, column=0, sticky="ew"
@@ -368,7 +416,7 @@ class CopienatorApp(tk.Tk):
         self.copy_command_button.grid(row=1, column=0, sticky="w", pady=(5, 0))
 
         buttons = ttk.Frame(self.detail)
-        buttons.grid(row=4, column=0, sticky="ew", pady=(5, 0))
+        buttons.grid(row=5, column=0, sticky="ew", pady=(5, 0))
         self.previous_button = ttk.Button(buttons, text="← Précédente", command=lambda: self._move_selection(-1))
         self.previous_button.pack(side="left")
         self.next_button = ttk.Button(buttons, text="Suivante →", command=lambda: self._move_selection(1))
@@ -502,6 +550,7 @@ class CopienatorApp(tk.Tk):
         if not evaluation or not evaluation.is_dir():
             messagebox.showerror("Dossier invalide", "Choisissez un dossier d’évaluation existant.")
             return
+        self.batch_monitor.stop()
         self._save_current_form()
         self.state_store.load(evaluation)
         for step in self.steps:
@@ -603,6 +652,40 @@ class CopienatorApp(tk.Tk):
         self._render_step()
         self.info_var.set("Fichiers revérifiés — " + ("manquants : " + ", ".join(missing) if missing else "tous les fichiers d’entrée sont présents."))
 
+    def _get_input_files(self) -> None:
+        evaluation = self.evaluation
+        if (
+            not self.show_personal_steps
+            or not evaluation
+            or not self.state_store.evaluation
+            or self.active_step_id
+            or self.runner.running
+        ):
+            return
+        try:
+            copied = get_personal_interro_files(evaluation)
+        except (OSError, ValueError) as exc:
+            messagebox.showerror("Récupération impossible", str(exc))
+            return
+
+        missing = self._missing_requirements(self.step_by_id["inputs"])
+        if missing:
+            self.state_store.update_step("inputs", status="ready")
+            self._populate_tree()
+            self._render_step()
+            messagebox.showwarning(
+                "Prérequis encore manquant",
+                "Les trois fichiers ont été copiés, mais l’étape ne peut pas encore "
+                "être terminée :\n\n" + "\n".join(missing),
+            )
+            return
+
+        self._mark_step("success", automatic=True, reason="Fichiers d’entrée récupérés")
+        self.info_var.set(
+            "Fichiers récupérés : " + ", ".join(path.name for path in copied)
+        )
+        self._move_selection_from("inputs", 1)
+
     def _on_tree_select(self, _event: tk.Event[Any] | None = None) -> None:
         selection = self.tree.selection()
         if not selection or selection[0].startswith("section:"):
@@ -669,7 +752,7 @@ class CopienatorApp(tk.Tk):
             return
         self._rendering = True
         redo = step.section == REFAIRE_SECTION
-        expanded_form = redo or step.id in {"page_splitter", "crop_blank_margins", "cutleft", "labels"}
+        expanded_form = redo or step.id in {"page_splitter", "crop_blank_margins", "cutleft", "labels", "manual_resolution"}
         self.console.configure(height=8 if expanded_form else 14)
         self.rowconfigure(1, weight=4 if expanded_form else 3)
         self.rowconfigure(2, weight=1 if expanded_form else 2)
@@ -692,10 +775,15 @@ class CopienatorApp(tk.Tk):
         if step.id == "statement" and saved_variant == "personal":
             description = ("Lit les blocs SHEETINFO de enonce.tex et récupère les énoncés, solutions "
                            "et barèmes du service personnel (localhost:8080). Crée un groupe par exercice. "
-                           "Les actions Gemini ci-dessous restent facultatives.")
+                           "Les deux étapes Gemini suivantes restent facultatives.")
-        if missing:
-            description += "\nPrérequis non détectés : " + ", ".join(missing)
         self.description_label.configure(text=description)
+        if missing:
+            self.missing_requirements_label.configure(
+                text="Prérequis non détectés : " + ", ".join(missing)
+            )
+            self.missing_requirements_label.grid()
+        else:
+            self.missing_requirements_label.grid_remove()
 
         row = self._render_context_controls(step, 0)
         if len(step.variants) > 1:
@@ -750,6 +838,18 @@ class CopienatorApp(tk.Tk):
             attach_tooltip(widget, spec.help)
             row += 1
 
+        if step.id == "manual_resolution":
+            def manual_evaluation():
+                target = self.arg_vars.get("target")
+                value = target.get().strip() if target else ""
+                return (self.repository / value).resolve() if value else self.evaluation
+
+            self.manual_panel = ManualResolutionPanel(self.form, manual_evaluation)
+            self.manual_panel.grid(row=row, column=0, columnspan=2, sticky="nsew", pady=8)
+            if "target" in self.arg_vars:
+                self.arg_vars["target"].trace_add("write", lambda *_args: self.manual_panel.reload())
+            row += 1
+
         show_extra = bool(step.extra_arguments_help) and (
             not step.extra_arguments_variants
             or variant.id in step.extra_arguments_variants
@@ -765,14 +865,6 @@ class CopienatorApp(tk.Tk):
             attach_tooltip(extra_entry, step.extra_arguments_help)
             row += 1
 
-        if self.show_personal_steps and step.id == "statement":
-            actions = ttk.LabelFrame(self.form, text="Après génération — facultatif", padding=7)
-            actions.grid(row=row, column=0, columnspan=2, sticky="ew", pady=(10, 8))
-            for ident, title in (("statement_groups", "Regrouper avec Gemini…"),
-                                 ("statement_persp", "Remplacer Persp avec Gemini…")):
-                ttk.Button(actions, text=title, command=lambda target=ident: self._select_statement_action(target)).pack(
-                    side="left", padx=(0, 6))
-
         self.run_button.configure(text="Enregistrer la sélection" if step.id == "refaire_selection" else ("Marquer terminée" if step.is_manual else "Exécuter"))
         self.skip_button.configure(state="normal" if step.optional else "disabled")
         self._rendering = False
@@ -780,6 +872,23 @@ class CopienatorApp(tk.Tk):
         self._update_controls()
 
     def _render_context_controls(self, step: StepDefinition, row: int) -> int:
+        if step.id == "batch_status":
+            controls = ttk.Frame(self.form)
+            controls.grid(row=row, column=0, columnspan=2, sticky="w", pady=8)
+            self.watch_batches_button = ttk.Button(
+                controls, text="Vérifier toutes les 5 minutes", command=self._start_batch_watch
+            )
+            self.watch_batches_button.pack(side="left")
+            self.stop_watch_batches_button = ttk.Button(
+                controls, text="Arrêter les vérifications", command=self.batch_monitor.stop
+            )
+            self.stop_watch_batches_button.pack(side="left", padx=8)
+            ttk.Label(self.form, textvariable=self.batch_watch_status).grid(
+                row=row + 1, column=0, columnspan=2, sticky="w"
+            )
+            ttk.Label(self.form, text="Gardez Copienator ouvert. Une notification de bureau sera envoyée lorsque tous les résultats seront prêts.",
+                      wraplength=650).grid(row=row + 2, column=0, columnspan=2, sticky="w", pady=8)
+            return row + 3
         if step.id == "rename":
             self.validate_rename_button = ttk.Button(
                 self.form, text="Valider sans renommer", command=self._validate_rename_step
@@ -790,7 +899,16 @@ class CopienatorApp(tk.Tk):
             ttk.Button(self.form, text="Recharger — vérifier à nouveau", command=self._reload_inputs).grid(
                 row=row, column=0, columnspan=2, sticky="w", pady=(0, 8)
             )
-            return row + 1
+            row += 1
+            if self.show_personal_steps:
+                self.get_input_files_button = ttk.Button(
+                    self.form, text="Get the files", command=self._get_input_files
+                )
+                self.get_input_files_button.grid(
+                    row=row, column=0, columnspan=2, sticky="w", pady=(0, 8)
+                )
+                row += 1
+            return row
         if step.section == REFAIRE_SECTION:
             evaluation = self.evaluation
             if step.id in {"refaire_selection", "refaire_correct"}:
@@ -891,14 +1009,6 @@ class CopienatorApp(tk.Tk):
             row += 1
         return row
 
-    def _select_statement_action(self, step_id: str) -> None:
-        self._save_current_form()
-        target = self.arg_vars.get("target")
-        if target:
-            self.state_store.update_step(step_id, values={"target": target.get()})
-        self.tree.selection_set(step_id)
-        self.tree.see(step_id)
-
     def _refaire_restart_controls(self, row: int) -> int:
         controls = ttk.Frame(self.form)
         controls.grid(row=row, column=0, columnspan=2, sticky="w", pady=(0, 5))
@@ -1226,6 +1336,8 @@ class CopienatorApp(tk.Tk):
         if not step or not evaluation or not self.state_store.evaluation:
             messagebox.showerror("Évaluation absente", "Chargez d’abord un dossier d’évaluation.")
             return
+        if step.id == "batch_status" and self.batch_monitor.active:
+            return
         if self.active_step_id or self.runner.running:
             messagebox.showwarning("Traitement en cours", "Interrompez le traitement actuel avant d’en lancer un autre.")
             return
@@ -1375,6 +1487,39 @@ class CopienatorApp(tk.Tk):
                 self._finish_process(int(return_code), bool(interrupted))
         self.after(60, self._poll_runner)
 
+    def _batch_watch_status_changed(self, message: str) -> None:
+        self.batch_watch_status.set(message)
+        self._update_controls()
+
+    def _start_batch_watch(self) -> None:
+        if self.active_step_id or self.runner.running or not self.state_store.evaluation:
+            return
+        step = self.step_by_id["batch_status"]
+        command = build_command(self.repository, step, step.variants[0], {},
+                                str(self.state_store.evaluation))
+        environment = build_runner_environment(
+            os.environ, self.api_key_var.get(), self.proxy_var.get(), self.use_proxy_var.get()
+        )
+        self.batch_monitor.start(command, self.repository, environment,
+                                 self.state_store.workspace.log_path("batch_status_watch"))
+
+    def _batch_results_ready(self) -> None:
+        evaluation = self.state_store.evaluation
+        if evaluation is None:
+            return
+        self.state_store.update_step("batch_status", status="success", return_code=0)
+        self._populate_tree()
+        message = f"{evaluation.name} : tous les résultats batch sont prêts à récupérer."
+        self.info_var.set(message)
+        try:
+            notify_desktop("Copienator — batchs terminés", message)
+        except (OSError, RuntimeError) as exc:
+            self.bell()
+            self._append_console(f"Notification de bureau indisponible : {exc}\n{message}\n")
+        if (self.current_step and self.current_step.id == "batch_status"
+                and not self.active_step_id and not self.runner.running):
+            self._move_selection_from("batch_status", 1)
+
     def _finish_process(self, return_code: int, interrupted: bool) -> None:
         step_id = self.active_step_id
         if not step_id:
@@ -1436,6 +1581,10 @@ class CopienatorApp(tk.Tk):
         self._populate_tree()
         self._update_controls()
         if status == "success":
+            if step_id == "batch_status" and self.batch_monitor.active:
+                self.batch_monitor.stop()
+                self._batch_results_ready()
+                return
             self.after_idle(lambda completed_id=step_id: self._move_selection_from(completed_id, 1))
 
     def _send_input(self) -> None:
@@ -1479,13 +1628,18 @@ class CopienatorApp(tk.Tk):
 
     def _update_controls(self) -> None:
         running = bool(self.active_step_id) or self.runner.running
+        if self.current_step and self.current_step.id == "batch_status":
+            self.watch_batches_button.configure(state="disabled" if running or self.batch_monitor.active or not self.state_store.evaluation else "normal")
+            self.stop_watch_batches_button.configure(state="normal" if self.batch_monitor.active else "disabled")
         if running and self.current_step and self.current_step.id in {"page_splitter", "cutleft"}:
             self.marked_copies_button.configure(state="disabled")
         if self.current_step and self.current_step.id == "rename":
             self.validate_rename_button.configure(
                 state="disabled" if running or not self.state_store.evaluation else "normal"
             )
-        self.run_button.configure(state="disabled" if running or not self.current_step else "normal")
+        watching_current = bool(self.current_step and self.current_step.id == "batch_status"
+                                and self.batch_monitor.active)
+        self.run_button.configure(state="disabled" if running or not self.current_step or watching_current else "normal")
         self.skip_button.configure(state="normal" if not running and self.current_step and self.current_step.optional else "disabled")
         self.interrupt_button.configure(state="normal" if running else "disabled")
         self.force_button.configure(state="normal" if running else "disabled")
@@ -1520,4 +1674,5 @@ class CopienatorApp(tk.Tk):
             except OSError:
                 pass
         self._save_current_form()
+        self.batch_monitor.stop()
         self.destroy()

copienator_gui/batch_monitor.py

@@ -0,0 +1,81 @@
+"""Non-blocking, cancellable batch checks while the desktop GUI is open."""
+
+import queue
+
+from .runner import ProcessRunner
+
+CHECK_INTERVAL_MS = 5 * 60 * 1000
+
+
+class BatchMonitor:
+    def __init__(self, scheduler, on_ready, on_status, on_output):
+        self.scheduler = scheduler
+        self.on_ready = on_ready
+        self.on_status = on_status
+        self.on_output = on_output
+        self.active = False
+        self.timer = None
+        self.runner = None
+
+    def start(self, command, cwd, environment, log_path):
+        if self.active:
+            return
+        self.arguments = (command, cwd, environment, log_path)
+        self.active = True
+        self._check()
+
+    def stop(self):
+        self.active = False
+        if self.timer is not None:
+            self.scheduler.after_cancel(self.timer)
+            self.timer = None
+        if self.runner is not None:
+            try:
+                self.runner.force_stop()
+            except OSError as exc:
+                self.on_output(f"Arrêt de la vérification : {exc}\n")
+            self.runner = None
+        self.on_status("Vérification automatique arrêtée.")
+
+    def _later(self):
+        self.timer = self.scheduler.after(CHECK_INTERVAL_MS, self._check)
+
+    def _check(self):
+        self.timer = None
+        if not self.active:
+            return
+        self.runner = ProcessRunner()
+        self.on_status("Vérification des batchs en cours…")
+        try:
+            self.runner.start(*self.arguments)
+        except (OSError, RuntimeError) as exc:
+            self.on_output(f"Vérification impossible : {exc}\n")
+            self.runner = None
+            self.on_status("Échec de la vérification. Nouvel essai dans 5 minutes.")
+            self._later()
+            return
+        self.timer = self.scheduler.after(100, self._poll)
+
+    def _poll(self):
+        self.timer = None
+        if not self.active or self.runner is None:
+            return
+        while True:
+            try:
+                event, payload = self.runner.events.get_nowait()
+            except queue.Empty:
+                break
+            if event in {"output", "runner_error"}:
+                self.on_output(str(payload))
+            elif event == "finished":
+                code, interrupted = payload
+                self.runner = None
+                if code == 0 and not interrupted:
+                    self.active = False
+                    self.on_status("Tous les résultats batch sont prêts.")
+                    self.on_ready()
+                else:
+                    self.on_status("Résultats pas encore prêts. Nouvelle vérification dans 5 minutes.")
+                    self._later()
+                return
+        self.timer = self.scheduler.after(100, self._poll)

copienator_gui/cut_helper.py

@@ -0,0 +1,131 @@
+from __future__ import annotations
+
+import tkinter as tk
+from pathlib import Path
+from tkinter import messagebox, ttk
+
+import pymupdf
+from PIL import Image, ImageTk
+
+from copienator.pdf_cut import cut_position
+
+PAGE_GAP = 28
+SNAP_PIXELS = 12
+
+
+def percentage_at_y(y: float, heights: list[float], scale: float) -> float:
+    """Convert canvas y to document height, snapping across inter-page gaps."""
+    top = 0.0
+    cumulative = 0.0
+    total = sum(heights)
+    for index, height in enumerate(heights):
+        bottom = top + height * scale
+        if index + 1 < len(heights) and bottom - SNAP_PIXELS <= y <= bottom + PAGE_GAP + SNAP_PIXELS:
+            return (cumulative + height) / total * 100
+        if y <= bottom:
+            return max(0.0, min(100.0, (cumulative + (y - top) / scale) / total * 100))
+        cumulative += height
+        top = bottom + PAGE_GAP
+    return 100.0
+
+
+def y_at_percentage(percent: float, heights: list[float], scale: float) -> float:
+    if percent <= 0:
+        return 0.0
+    if percent >= 100:
+        return sum(heights) * scale + (len(heights) - 1) * PAGE_GAP
+    index, offset = cut_position(heights, percent)
+    top = sum(heights[:index]) * scale + index * PAGE_GAP
+    return top - PAGE_GAP / 2 if index and offset == 0 else top + offset * scale
+
+
+def cut_operator(percent: float, keep: int, mode: str) -> str:
+    value = f"{percent:.6f}".rstrip("0").rstrip(".")
+    return f"c{{{value}}}{keep}{mode}"
+
+
+class CutHelper(tk.Toplevel):
+    def __init__(self, parent, path: Path, on_accept, initial=None):
+        # Load the source before creating a window so invalid PDFs leave no dialog.
+        with pymupdf.open(path) as document:
+            if not len(document):
+                raise ValueError("Le PDF est vide.")
+            self.heights = [page.rect.height for page in document]
+            width = min(850, parent.winfo_screenwidth() - 100)
+            self.scale = min(1.5, width / max(page.rect.width for page in document))
+            rendered = []
+            for page in document:
+                pix = page.get_pixmap(matrix=pymupdf.Matrix(self.scale, self.scale), alpha=False,
+                                      colorspace=pymupdf.csRGB)
+                rendered.append(Image.frombytes("RGB", (pix.width, pix.height), pix.samples))
+        super().__init__(parent)
+        self.title(f"Cut — {path.name}")
+        self.geometry(f"{width + 45}x{min(850, parent.winfo_screenheight() - 100)}")
+        self.transient(parent.winfo_toplevel())
+        self.on_accept = on_accept
+        self.percent = initial[0] if initial else 50.0
+        self.keep = tk.IntVar(value=initial[1] if initial else 1)
+        self.mode = tk.StringVar(value=initial[2] if initial else ">")
+        self.caption = tk.StringVar()
+        ttk.Label(self, text="Déplacez la barre rouge. Entrée : afficher la commande ; Échap : annuler.",
+                  wraplength=width).pack(anchor="w", padx=8, pady=5)
+        controls = ttk.Frame(self)
+        controls.pack(fill="x", padx=8)
+        ttk.Label(controls, text="Conserver à la source :").pack(side="left")
+        ttk.Radiobutton(controls, text="1 — début", variable=self.keep, value=1).pack(side="left")
+        ttk.Radiobutton(controls, text="2 — fin", variable=self.keep, value=2).pack(side="left")
+        ttk.Radiobutton(controls, text="Ajouter à la cible", variable=self.mode, value=">").pack(side="left")
+        ttk.Radiobutton(controls, text="Remplacer", variable=self.mode, value="x").pack(side="left")
+        ttk.Label(self, textvariable=self.caption).pack(anchor="w", padx=8, pady=5)
+        viewport = ttk.Frame(self)
+        viewport.pack(fill="both", expand=True)
+        self.canvas = tk.Canvas(viewport, background="#555555", highlightthickness=0)
+        scrollbar = ttk.Scrollbar(viewport, command=self.canvas.yview)
+        self.canvas.configure(yscrollcommand=scrollbar.set)
+        scrollbar.pack(side="right", fill="y")
+        self.canvas.pack(fill="both", expand=True)
+        self.images = [ImageTk.PhotoImage(image, master=self) for image in rendered]
+        top = 0.0
+        self.width = width
+        for index, image in enumerate(self.images):
+            self.canvas.create_image(0, top, anchor="nw", image=image)
+            top += self.heights[index] * self.scale
+            if index + 1 < len(self.images):
+                top += PAGE_GAP
+        self.canvas.configure(scrollregion=(0, 0, width, top))
+        self.bar = self.canvas.create_line(0, 0, width, 0, fill="#ff3030", width=4)
+        self.canvas.bind("<Button-1>", self.move_bar)
+        self.canvas.bind("<B1-Motion>", self.move_bar)
+        self.canvas.bind("<Button-4>", lambda event: self.canvas.yview_scroll(-3, "units"))
+        self.canvas.bind("<Button-5>", lambda event: self.canvas.yview_scroll(3, "units"))
+        self.canvas.bind("<MouseWheel>", lambda event: self.canvas.yview_scroll(-1 if event.delta > 0 else 1, "units"))
+        self.bind("<Return>", self.accept)
+        self.bind("<Escape>", lambda event: self.destroy())
+        self.keep.trace_add("write", lambda *_: self.draw_bar())
+        self.mode.trace_add("write", lambda *_: self.draw_bar())
+        self.draw_bar()
+        self.canvas.yview_moveto(max(0, (y_at_percentage(self.percent, self.heights, self.scale) - 200) / top))
+        self.focus_set()
+        self.grab_set()
+
+    def move_bar(self, event):
+        self.percent = percentage_at_y(self.canvas.canvasy(event.y), self.heights, self.scale)
+        self.draw_bar()
+
+    def draw_bar(self):
+        y = y_at_percentage(self.percent, self.heights, self.scale)
+        self.canvas.coords(self.bar, 0, y, self.width, y)
+        text = cut_operator(self.percent, self.keep.get(), self.mode.get())
+        if 0 < self.percent < 100:
+            index, offset = cut_position(self.heights, self.percent)
+            text += f" — entre les pages {index} et {index + 1}" if offset == 0 else f" — page {index + 1}"
+        self.caption.set(text)
+
+    def accept(self, event=None):
+        operator = cut_operator(self.percent, self.keep.get(), self.mode.get())
+        rounded = float(operator.split("{")[1].split("}")[0])
+        if not 0 < rounded < 100:
+            messagebox.showerror("Coupe invalide", "Chaque partie doit contenir une portion du PDF.", parent=self)
+            return
+        self.destroy()
+        self.on_accept(operator)

copienator_gui/manual_resolution.py

@@ -0,0 +1,154 @@
+from __future__ import annotations
+
+import tkinter as tk
+from pathlib import Path
+from tkinter import messagebox, ttk
+
+from copienator import CliError
+from copienator.commands.resolve_manual import get_actual_pdf, parse_instruction_text
+from copienator.platform import open_path
+from .cut_helper import CutHelper
+
+
+HELP = """La correction propose x> pour un mauvais label et -> pour une réponse supplémentaire lorsque le PDF cible existe déjà. Vérifiez les deux PDF avant de choisir.
+
+Format : Copie01 Label source OP Label cible|
+Conservez Copie suivi du numéro et les labels exacts (sans .pdf). Les espaces dans les labels sont acceptés ; entourez l’opérateur d’espaces.
+
+-> : fusionner dans la cible, conserver la source.
+x> : fusionner dans la cible, archiver la source.
+-x : remplacer la cible par une copie de la source, conserver la source.
+xx : remplacer la cible par une copie de la source, archiver la source.
+ss : conserver les deux PDF sans fusion.
+sx : conserver la source, archiver la cible, sans fusion.
+xs : archiver la source, conserver la cible, sans fusion.
+
+c{43}1> : couper la source à 43 %, conserver le début et ajouter la fin à la cible.
+c{43}2x : couper la source à 43 %, conserver la fin et remplacer la cible par le début.
+1 conserve le début, 2 conserve la fin ; > fusionne l’autre partie avec la cible, x la remplace. Le pourcentage porte sur la hauteur cumulée des pages visibles. Les décimales sont acceptées. Une seule coupe par label source est autorisée.
+Cut permet de placer la coupe, avec accrochage entre les pages. Entrée ferme l’aperçu et affiche la commande à recopier ci-dessus : aucun fichier n’est modifié. Une séparation entre pages conserve les pages entières. La source complète est archivée en _old ; les deux labels modifiés sont générés en _new et ajoutés à refaire.json.
+
+Pour les fusions : « Source x> Cible| » place la cible avant la source ; « Source x> |Cible » place la source avant la cible. Même règle avec -> et c{…}1>/c{…}2> (pour la partie transférée) ; sans |, la cible vient en premier.
+Vous pouvez changer l’opérateur, déplacer |, corriger les labels ou retirer une instruction. Les lignes vides et celles commençant par ### sont ignorées. Retirer/commenter une ligne ne résout pas son conflit.
+
+Enregistrez dans l’éditeur, puis cliquez sur Recharger et enfin sur Exécuter. Seul le fichier enregistré est appliqué. L’archivage utilise le suffixe _old ; les PDF créés utilisent _new. Après succès, manual_resolutions.txt est supprimé et correction.json est mis à jour. Si des PDF sont créés, refaire.json est généré : relancez la correction avec --refaire."""
+
+
+class ManualResolutionPanel(ttk.Frame):
+    def __init__(self, parent, get_evaluation):
+        super().__init__(parent)
+        self.get_evaluation = get_evaluation
+        self.pdf_buttons = []
+        self.cut_buttons = []
+        actions = ttk.Frame(self)
+        actions.pack(fill="x")
+        self.editor_button = ttk.Button(
+            actions, text="Ouvrir dans un éditeur de texte", command=self.open_editor
+        )
+        self.editor_button.pack(side="left")
+        ttk.Button(actions, text="Recharger", command=self.reload).pack(side="left", padx=6)
+        self.cut_result = tk.StringVar()
+        result = ttk.Frame(self)
+        result.pack(fill="x", pady=4)
+        ttk.Entry(result, textvariable=self.cut_result, state="readonly").pack(side="left", fill="x", expand=True)
+        ttk.Button(result, text="Copier la commande Cut", command=self.copy_cut_command).pack(side="left", padx=6)
+        self.status = ttk.Label(self, wraplength=650)
+        self.status.pack(fill="x", pady=4)
+        preview = ttk.Frame(self)
+        preview.pack(fill="both", expand=True)
+        self.text = tk.Text(preview, height=10, width=50, wrap="word", state="disabled")
+        scroll = ttk.Scrollbar(preview, command=self.text.yview)
+        self.text.configure(yscrollcommand=scroll.set)
+        scroll.pack(side="right", fill="y")
+        self.text.pack(fill="both", expand=True)
+        ttk.Label(self, text=HELP, wraplength=650, justify="left").pack(fill="x", pady=8)
+        self.reload()
+
+    def manual_path(self) -> Path | None:
+        evaluation = self.get_evaluation()
+        return evaluation / "manual_resolutions.txt" if evaluation else None
+
+    def open_editor(self):
+        path = self.manual_path()
+        if path is None or not path.is_file():
+            self.reload()
+            return
+        try:
+            # .txt is opened in the desktop's associated text editor.
+            open_path(path)
+        except (OSError, RuntimeError) as exc:
+            messagebox.showerror("Ouverture impossible", str(exc))
+
+    def open_pdf(self, evaluation, copy_id, label):
+        path = get_actual_pdf(evaluation / "Copies", copy_id, label)
+        try:
+            if not path.is_file():
+                raise FileNotFoundError(f"PDF introuvable : {path}")
+            open_path(path)
+        except (OSError, RuntimeError) as exc:
+            messagebox.showerror("Ouverture du PDF", str(exc))
+
+    def reload(self):
+        for button in self.pdf_buttons + self.cut_buttons:
+            button.destroy()
+        self.pdf_buttons.clear()
+        self.cut_buttons.clear()
+        self.cut_result.set("")
+        self.text.configure(state="normal")
+        self.text.delete("1.0", "end")
+        path = self.manual_path()
+        self.editor_button.configure(state="disabled")
+        try:
+            if path is None:
+                raise FileNotFoundError("Chargez une évaluation.")
+            content = path.read_text(encoding="utf-8")
+        except (OSError, UnicodeError) as exc:
+            self.status.configure(text=f"manual_resolutions.txt indisponible : {exc}")
+        else:
+            self.editor_button.configure(state="normal")
+            malformed = []
+            for number, raw in enumerate(content.splitlines(keepends=True), 1):
+                self.text.insert("end", raw.rstrip("\r\n"))
+                try:
+                    instructions = parse_instruction_text(raw)
+                except CliError:
+                    malformed.append(str(number))
+                else:
+                    if instructions:
+                        instruction = instructions[0]
+                        for title, label in (("PDF source", instruction.old_label),
+                                             ("PDF cible", instruction.new_label)):
+                            button = ttk.Button(
+                                self.text, text=title,
+                                command=lambda label=label, copy_id=instruction.copy_id, root=path.parent: self.open_pdf(root, copy_id, label),
+                            )
+                            self.pdf_buttons.append(button)
+                            self.text.window_create("end", window=button, padx=8)
+                        button = ttk.Button(self.text, text="Cut",
+                                            command=lambda item=instruction, root=path.parent: self.open_cut(root, item))
+                        self.cut_buttons.append(button)
+                        self.text.window_create("end", window=button, padx=8)
+                if raw.endswith("\n"):
+                    self.text.insert("end", "\n")
+            detail = (" — lignes invalides : " + ", ".join(malformed)) if malformed else (
+                f" — {len(self.pdf_buttons) // 2} instruction(s)"
+            )
+            self.status.configure(text=str(path) + detail)
+        finally:
+            self.text.configure(state="disabled")
+
+    def copy_cut_command(self):
+        if self.cut_result.get():
+            self.clipboard_clear()
+            self.clipboard_append(self.cut_result.get())
+
+    def open_cut(self, evaluation, instruction):
+        source = get_actual_pdf(evaluation / "Copies", instruction.copy_id, instruction.old_label)
+        def accepted(operator):
+            target = ("|" + instruction.new_label) if instruction.pipe_first else (instruction.new_label + "|")
+            self.cut_result.set(f"Copie{instruction.copy_id} {instruction.old_label} {operator} {target}")
+        try:
+            initial = (*instruction.cut, instruction.operator[-1]) if instruction.cut else None
+            CutHelper(self, source, accepted, initial)
+        except (OSError, RuntimeError, ValueError) as exc:
+            messagebox.showerror("Ouverture du PDF", str(exc))

copienator_gui/notifications.py

@@ -0,0 +1,37 @@
+"""Launch native desktop notifications without blocking Tk."""
+
+import base64
+import json
+import os
+import subprocess
+import sys
+
+from copienator.platform import find_executable
+
+
+def notify_desktop(title: str, message: str) -> None:
+    if sys.platform.startswith("linux"):
+        executable = find_executable("notify-send")
+        if not executable:
+            raise RuntimeError("Installez notify-send (libnotify) pour les notifications de bureau.")
+        command = [executable, "--app-name=Copienator", "--", title, message]
+    elif sys.platform == "darwin":
+        command = ["/usr/bin/osascript", "-e",
+                   f"display notification {json.dumps(message, ensure_ascii=False)} with title {json.dumps(title, ensure_ascii=False)}"]
+    elif os.name == "nt":
+        # Encode the script and quote strings as literals; no shell interpolation.
+        quote = lambda value: "'" + value.replace("'", "''") + "'"
+        script = (
+            "Add-Type -AssemblyName System.Windows.Forms;"
+            "$notice = New-Object System.Windows.Forms.NotifyIcon;"
+            "$notice.Icon = [System.Drawing.SystemIcons]::Information;"
+            "$notice.Visible = $true;"
+            f"$notice.ShowBalloonTip(10000, {quote(title)}, {quote(message)}, "
+            "[System.Windows.Forms.ToolTipIcon]::Info);"
+            "Start-Sleep -Seconds 12; $notice.Dispose()"
+        )
+        command = ["powershell.exe", "-NoProfile", "-NonInteractive", "-WindowStyle", "Hidden",
+                   "-EncodedCommand", base64.b64encode(script.encode("utf-16-le")).decode("ascii")]
+    else:
+        raise RuntimeError("Notifications de bureau indisponibles sur ce système.")
+    subprocess.Popen(command, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)

copienator_gui/workflow.py

@@ -7,6 +7,7 @@ from collections.abc import Iterable
 from dataclasses import dataclass
 from pathlib import Path
 
+from copienator import configuration
 from copienator.configuration import ALWAYS_CROP
 
 EVALUATION = "${evaluation}"
@@ -18,6 +19,7 @@ class ArgumentSpec:
     label: str
     kind: str = "text"  # text, int, bool, choice, path
     flag: str | None = None
+    false_flag: str | None = None
     default: object = ""
     choices: tuple[str, ...] = ()
     help: str = ""
@@ -87,6 +89,38 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
         ident, label, None, "manual"
     )
 
+    return_answer_arguments = ()
+    if configuration.RETURN_ANSWERS_ENABLED:
+        return_answer_arguments = (
+            ArgumentSpec(
+                "return_answers_context",
+                "Inclure le contexte dans answers",
+                "bool",
+                "--return-answers-context",
+                "--no-return-answers-context",
+                default=configuration.RETURN_ANSWERS_CONTEXT,
+                help="Inclut les pages de contexte applicables avant chaque réponse individuelle publiée dans answers.",
+            ),
+            ArgumentSpec(
+                "return_answers_question",
+                "Inclure l’énoncé dans answers",
+                "bool",
+                "--return-answers-question",
+                "--no-return-answers-question",
+                default=configuration.RETURN_ANSWERS_QUESTION,
+                help="Inclut l’énoncé actuel avant chaque réponse individuelle publiée dans answers.",
+            ),
+            ArgumentSpec(
+                "return_answers_solution",
+                "Inclure la correction dans answers",
+                "bool",
+                "--return-answers-solution",
+                "--no-return-answers-solution",
+                default=configuration.RETURN_ANSWERS_SOLUTION,
+                help="Inclut la correction actuelle avant chaque réponse individuelle publiée dans answers.",
+            ),
+        )
+
     steps = [
         StepDefinition(
             "inputs",
@@ -102,10 +136,9 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
             "Prétraitement de l’énoncé",
             "Analyser l’énoncé",
             "Détecte les questions, leurs groupes, le corrigé et les indications de barème.",
-            (
+            ((python("personal", "Énoncés et solutions personnels (SHEETINFO)", "statement-personal"),)
-                python("gemini", "Analyse avec Gemini", "statement"),
+             if show_personal_steps else ())
-            ) + ((python("personal", "Énoncés et solutions personnels (SHEETINFO)", "statement-personal"),)
+            + (python("gemini", "Analyse avec Gemini", "statement"),),
-                 if show_personal_steps else ()),
             arguments=(
                 arg_target("le dossier de l’évaluation"),
                 ArgumentSpec(
@@ -353,7 +386,7 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
                     "bool",
                     "--overwrite",
                     help="Relance les corrections demandées même lorsqu’un résultat existe déjà.",
-                    variants=("live", "batch", "hybrid", "refaire"),
+                    variants=("live", "batch", "hybrid", "refaire", "integrate"),
                 ),
                 ArgumentSpec(
                     "limit",
@@ -395,24 +428,8 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
             "batch_status",
             "Correction",
             "Consulter l’état des batchs",
-            "Affiche les jobs Gemini en cours. L’identifiant de téléchargement est facultatif.",
+            "Vérifie les jobs enregistrés pour l’évaluation. Passe à la récupération uniquement lorsque tous ont réussi et que leurs résultats sont disponibles ; sinon, reste sur cette étape.",
             (python("default", "État des batchs", "batch-status"),),
-            arguments=(
-                ArgumentSpec(
-                    "download",
-                    "Télécharger le job",
-                    "text",
-                    "--download",
-                    help="Saisissez l’identifiant complet d’un job Gemini terminé pour télécharger son fichier de résultats.",
-                ),
-                ArgumentSpec(
-                    "output",
-                    "Fichier JSONL de destination",
-                    "path",
-                    "--output",
-                    help="Choisissez le fichier JSONL dans lequel enregistrer le job téléchargé. Ce champ n’est utilisé que si un identifiant de job est fourni.",
-                ),
-            ),
             optional=True,
             skip_for_live_correction=True,
         ),
@@ -430,7 +447,7 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
             "post_correction",
             "Correction",
             "Nettoyer la correction",
-            "Corrige certains problèmes d’encodage et prépare le texte pour LaTeX.",
+            "Sauvegarde correction.json dans correction_precleanup.json, puis corrige certains problèmes d’encodage et prépare le texte pour LaTeX. La sauvegarde est remplacée à chaque nettoyage.",
             (python("default", "Post-correction", "post-correction"),),
             arguments=(arg_target("le dossier de l’évaluation"),),
             requires=("correction.json",),
@@ -550,10 +567,10 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
                 arg_target("le dossier de l’évaluation"),
                 ArgumentSpec(
                     "update_score",
-                    "Réappliquer les score.json",
+                    "Régénérer avec les nouveaux énoncés/corrigés et les score.json",
                     "bool",
                     "--update-score",
-                    help="Réutilise les valeurs présentes dans les fichiers score.json pour remplacer les scores lus dans les annotations.",
+                    help="Régénère les images avec les PDF actuels d’énoncé et de correction ; pour chaque label, le score.json existant prévaut sur le score relu dans l’annotation manuscrite.",
                 ),
                 ArgumentSpec(
                     "refaire",
@@ -573,7 +590,7 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
                     help="Indique le dossier d’annotations du passage principal dans lequel intégrer les questions refaites.",
                     variants=("grouped",),
                 ),
-            ),
+            ) + return_answer_arguments,
         ),
         StepDefinition(
             "giving_names",
@@ -592,6 +609,13 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
                     help="Choisissez le dossier d’annotations utilisé pour construire les fichiers nommés dans A Rendre.",
                     positional=True,
                 ),
+                ArgumentSpec(
+                    "update",
+                    "Mettre à jour uniquement les images de answers",
+                    "bool",
+                    "--update",
+                    help="Met à jour uniquement les images du dossier answers de chaque élève existant, en identifiant la copie par le numéro final entre parenthèses et sans modifier le nom du dossier ni les autres fichiers.",
+                ),
             ),
             artifacts=("A Rendre",),
         ),
@@ -649,7 +673,9 @@ def build_workflow(show_personal_steps: bool) -> list[StepDefinition]:
             "final_score",
             "Étapes personnelles",
             "Ajouter le score final",
-            "Génère les fichiers de diffusion avec le score final.",
+            "Prérequis immédiat : gestion_classe wse doit avoir été exécuté juste avant. "
+            "Génère ensuite les dossiers de diffusion avec le score final, puis copie "
+            "gestion_classe/Staging/histogramme.pdf dans le dossier Server/copies de l’évaluation.",
             (python("default", "Score final", "add-final-score", supports_verbose=False),),
             arguments=(arg_target("le dossier de l’évaluation"),),
             personal=True,
@@ -789,6 +815,9 @@ def build_command(
 
     positionals: list[str] = []
     options: list[str] = []
+    if step.id == "batch_status":
+        # Use the loaded evaluation without introducing another input field.
+        options.extend(("--evaluation", evaluation_arg))
     for spec in step.arguments:
         if spec.variants and variant.id not in spec.variants:
             continue
@@ -796,6 +825,8 @@ def build_command(
         if spec.kind == "bool":
             if bool(value) and spec.flag:
                 options.append(spec.flag)
+            elif not bool(value) and spec.false_flag:
+                options.append(spec.false_flag)
             continue
         if value is None or str(value).strip() == "":
             continue

default_config.py

@@ -24,6 +24,7 @@ ALWAYS_CROP = False
 CURRENT_SCORE_ODS_PATH = Path("current_eval.ods")
 FINAL_SCORE_ODS_PATH = Path("simple_eval.ods")
 FINAL_SCORE_OUTPUT_DIR = Path("Server") / "copies"
+FINAL_SCORE_HISTOGRAM_PATH = Path("histogramme.pdf")
 FINAL_SCORE_FONT_PATH = None
 
 # Modèle pour des choses très légères
@@ -82,6 +83,7 @@ LATEX_BEFORE = r"""\documentclass[varwidth=24.8cm,margin=0.4cm]{standalone}
 \usepackage{minted}
 \usepackage{graphicx}
 \usepackage{enumitem}
+\usepackage{multicol}
 \begin{document}
     \begin{minipage}{24.8cm}
 """

tests/test_add_final_score.py

@@ -0,0 +1,99 @@
+import contextlib
+import io
+import tempfile
+import unittest
+from pathlib import Path
+from unittest.mock import patch
+
+import pandas as pd
+from PIL import Image
+
+from copienator.commands import add_final_score
+
+
+class AddFinalScoreTests(unittest.TestCase):
+    def test_creates_complete_student_directory(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            root = Path(temporary)
+            source = root / "A Rendre" / "Student (01)"
+            answers = source / "answers"
+            answers.mkdir(parents=True)
+            Image.new("RGB", (300, 200), "white").save(source / "Student.jpg")
+            (source / "Student.pdf").write_bytes(b"pdf")
+            (source / "score.json").write_bytes(b'{"Ex 1": "4"}')
+            (source / "info.json").write_bytes(b'{"Ex 1": {}}')
+            (answers / "Ex 1.jpg").write_bytes(b"answer")
+
+            output = root / "output"
+            output.mkdir()
+            (output / "Student.jpg").write_bytes(b"legacy jpeg")
+            (output / "Student.pdf").write_bytes(b"legacy pdf")
+            stale_answers = output / "Student" / "answers"
+            stale_answers.mkdir(parents=True)
+            (stale_answers / "001 - Ex 1.jpg").write_bytes(b"stale")
+            scores = pd.DataFrame({0: ["Student"], 1: [12.39]})
+            histogram = root / "histogramme.pdf"
+            histogram.write_bytes(b"histogram")
+            with patch.object(add_final_score.pd, "read_excel", return_value=scores), \
+                    patch.object(add_final_score, "HISTOGRAM_PATH", histogram), \
+                    contextlib.redirect_stdout(io.StringIO()):
+                add_final_score.process_images(root, output)
+
+            student = output / "Student"
+            self.assertEqual(
+                {path.name for path in student.iterdir()},
+                {"Student.jpg", "Student.pdf", "score.json", "info.json", "answers"},
+            )
+            self.assertEqual((student / "Student.pdf").read_bytes(), b"pdf")
+            self.assertEqual((student / "score.json").read_bytes(), b'{"Ex 1": "4"}')
+            self.assertEqual((student / "info.json").read_bytes(), b'{"Ex 1": {}}')
+            self.assertEqual(
+                (student / "answers" / "Ex 1.jpg").read_bytes(), b"answer"
+            )
+            self.assertFalse((output / "Student.jpg").exists())
+            self.assertFalse((output / "Student.pdf").exists())
+            self.assertFalse((student / "answers" / "001 - Ex 1.jpg").exists())
+            self.assertEqual((output / "histogramme.pdf").read_bytes(), b"histogram")
+
+    def test_omits_answers_directory_when_it_was_not_generated(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            root = Path(temporary)
+            source = root / "A Rendre" / "Student (01)"
+            source.mkdir(parents=True)
+            Image.new("RGB", (300, 200), "white").save(source / "Student.jpg")
+            (source / "Student.pdf").write_bytes(b"pdf")
+            (source / "score.json").write_text("{}")
+            (source / "info.json").write_text("{}")
+
+            output = root / "output"
+            scores = pd.DataFrame({0: ["Student"], 1: [10]})
+            with patch.object(add_final_score.pd, "read_excel", return_value=scores), \
+                    contextlib.redirect_stdout(io.StringIO()):
+                add_final_score.process_images(root, output)
+
+            self.assertFalse((output / "Student" / "answers").exists())
+
+    def test_missing_histogram_warns_without_discarding_student_outputs(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            root = Path(temporary)
+            source = root / "A Rendre" / "Student (01)"
+            source.mkdir(parents=True)
+            Image.new("RGB", (300, 200), "white").save(source / "Student.jpg")
+            scores = pd.DataFrame({0: ["Student"], 1: [10]})
+            output = root / "output"
+            messages = io.StringIO()
+
+            with patch.object(
+                add_final_score.pd, "read_excel", return_value=scores
+            ), patch.object(
+                add_final_score, "HISTOGRAM_PATH", root / "missing.pdf"
+            ), contextlib.redirect_stdout(messages):
+                add_final_score.process_images(root, output)
+
+            self.assertTrue((output / "Student" / "Student.jpg").is_file())
+            self.assertFalse((output / "histogramme.pdf").exists())
+            self.assertIn("Missing histogram", messages.getvalue())
+
+
+if __name__ == "__main__":
+    unittest.main()

tests/test_answer_returns.py

@@ -11,7 +11,10 @@ from PIL import Image
 from copienator import EvaluationWorkspace, atomic_write_json, configuration, read_json
 from copienator.commands import annotating, giving_names
 from copienator.commands import reading_grouped_annotations as reader
-from copienator.return_answers import publish_answer_returns
+from copienator.return_answers import (
+    publish_answer_returns,
+    save_return_answer_options,
+)
 
 
 class AnswerReturnTests(unittest.TestCase):
@@ -54,7 +57,7 @@ class AnswerReturnTests(unittest.TestCase):
             ), patch.object(annotating, "make_base_image", side_effect=self.supplement):
                 publish_answer_returns(self.root, self.source, self.destination)
                 files = sorted((self.destination / "answers").glob("*.jpg"))
-                self.assertEqual([p.name for p in files], ["001 - Ex 1.jpg", "002 - Ex 2.jpg"])
+                self.assertEqual([p.name for p in files], ["Ex 1.jpg", "Ex 2.jpg"])
                 with Image.open(files[0]) as image:
                     self.assertEqual(image.size, (100, 30 + 20 * sum((context, question, solution))))
                     colors = []
@@ -70,6 +73,38 @@ class AnswerReturnTests(unittest.TestCase):
                         self.assertTrue(all(abs(a - b) < 10 for a, b in zip(pixel, color)))
                 self.assertEqual(read_json(self.destination / "info.json"), read_json(self.source / "info.json"))
 
+    def test_saved_reading_options_override_configuration_for_answers(self):
+        save_return_answer_options(
+            self.root,
+            context=True,
+            question=False,
+            solution=True,
+        )
+        with patch.multiple(
+            configuration,
+            RETURN_ANSWERS_ENABLED=True,
+            RETURN_ANSWERS_CONTEXT=False,
+            RETURN_ANSWERS_QUESTION=True,
+            RETURN_ANSWERS_SOLUTION=False,
+        ), patch.object(
+            annotating, "make_base_image", side_effect=self.supplement
+        ):
+            publish_answer_returns(self.root, self.source, self.destination)
+
+        with Image.open(self.destination / "answers" / "Ex 1.jpg") as image:
+            self.assertEqual(image.size, (100, 70))
+            for y, color in (
+                (10, (0, 0, 255)),
+                (30, (255, 255, 0)),
+                (50, (255, 0, 0)),
+            ):
+                self.assertTrue(
+                    all(
+                        abs(actual - expected) < 10
+                        for actual, expected in zip(image.getpixel((50, y)), color)
+                    )
+                )
+
     def test_missing_supplement_is_optional_and_failure_preserves_old_answers(self):
         with patch.multiple(configuration, RETURN_ANSWERS_ENABLED=True,
                             RETURN_ANSWERS_CONTEXT=False, RETURN_ANSWERS_QUESTION=True,
@@ -78,7 +113,7 @@ class AnswerReturnTests(unittest.TestCase):
         ):
             (self.root / "Text2" / "Ex 1.pdf").unlink()
             publish_answer_returns(self.root, self.source, self.destination)
-            path = self.destination / "answers" / "001 - Ex 1.jpg"
+            path = self.destination / "answers" / "Ex 1.jpg"
             with Image.open(path) as image:
                 self.assertEqual(image.size, (100, 30))
             original = path.read_bytes()
@@ -86,7 +121,7 @@ class AnswerReturnTests(unittest.TestCase):
             with self.assertRaises(FileNotFoundError):
                 publish_answer_returns(self.root, self.source, self.destination)
             self.assertEqual(path.read_bytes(), original)
-            self.assertTrue((self.destination / "answers" / "002 - Ex 2.jpg").exists())
+            self.assertTrue((self.destination / "answers" / "Ex 2.jpg").exists())
 
     def test_disabling_clears_individual_images_but_retains_info(self):
         with patch.multiple(configuration, RETURN_ANSWERS_ENABLED=True,
@@ -135,8 +170,99 @@ class AnswerReturnTests(unittest.TestCase):
             self.assertEqual(giving_names.run(workspace, annotation_dir="BGnot"), 0)
         self.assertEqual({p.name for p in self.destination.iterdir()}, {"answers", "score.json", "info.json"})
 
+    def test_update_uses_copy_id_from_renamed_folder_and_touches_only_answers(self):
+        workspace = EvaluationWorkspace(self.root)
+        workspace.copies_dir.mkdir()
+        atomic_write_json(workspace.copies_dir / "Copie01.json", {"name": "Student"})
+        renamed = self.destination.with_name("Nom modifié manuellement (01)")
+        self.destination.rename(renamed)
+        (renamed / "Nom personnalisé.jpg").write_bytes(b"keep-jpeg")
+        (renamed / "Nom personnalisé.pdf").write_bytes(b"keep-pdf")
+        (renamed / "score.json").write_bytes(b"keep-score")
+        (renamed / "info.json").write_bytes(b"keep-info")
+        answers = renamed / "answers"
+        answers.mkdir()
+        (answers / "obsolete.jpg").write_bytes(b"obsolete")
+        preserved = {
+            path.name: path.read_bytes()
+            for path in renamed.iterdir()
+            if path.is_file()
+        }
+
+        with patch.multiple(
+            configuration,
+            RETURN_ANSWERS_ENABLED=True,
+            RETURN_ANSWERS_CONTEXT=False,
+            RETURN_ANSWERS_QUESTION=False,
+            RETURN_ANSWERS_SOLUTION=False,
+        ), contextlib.redirect_stdout(io.StringIO()):
+            self.assertEqual(
+                giving_names.run(workspace, annotation_dir="BGnot", update=True),
+                0,
+            )
+
+        self.assertFalse((workspace.return_dir / "Student (01)").exists())
+        self.assertEqual(
+            {
+                path.name: path.read_bytes()
+                for path in renamed.iterdir()
+                if path.is_file()
+            },
+            preserved,
+        )
+        self.assertEqual(
+            sorted(path.name for path in answers.iterdir()),
+            ["Ex 1.jpg", "Ex 2.jpg"],
+        )
+
 
 class CompiledMembershipTests(unittest.TestCase):
+    def test_update_score_preserves_file_and_manual_value_wins(self):
+        with tempfile.TemporaryDirectory() as directory:
+            workspace = EvaluationWorkspace(Path(directory))
+            output = workspace.root / "BGnot" / "Copie01"
+            output.mkdir(parents=True)
+            original_score = b'{\n  "Ex 1": "3.5"\n}\n'
+            (output / "score.json").write_bytes(original_score)
+            answer = workspace.root / "answer.pdf"
+            answer.touch()
+            data = {
+                "01": {
+                    "Ex 1": {
+                        "result": {"score": 1, "feedback": []},
+                        "pdf_path": answer,
+                        "coordinates": (0, 0),
+                    }
+                }
+            }
+            rendered_scores = []
+
+            def compose(base, label, result, *args, **kwargs):
+                rendered_scores.append(result["score"])
+                return Image.new("RGB", (100, 50), "white"), 0
+
+            with patch.object(
+                annotating, "make_base_image", return_value=(None, 0, 0)
+            ), patch.object(
+                annotating, "compose_label_image", side_effect=compose
+            ), patch.object(
+                reader, "get_extra_pdfs_as_images", return_value=[]
+            ), patch.object(reader, "save_paginated_pdf"):
+                status, _ = reader.apply_actions_and_regenerate_grouped(
+                    workspace,
+                    data,
+                    "01",
+                    [{"label": "Ex 1", "type": "score", "value": "2"}],
+                    {},
+                    ["Ex 1"],
+                    update_score=True,
+                )
+
+            self.assertEqual(status, 0)
+            self.assertEqual(rendered_scores, ["3.5"])
+            self.assertEqual((output / "score.json").read_bytes(), original_score)
+            self.assertEqual(read_json(output / "info.json")["Ex 1"]["score"], "3.5")
+
     def test_membership_matches_actual_groups_and_saves_every_nonempty_answer(self):
         with tempfile.TemporaryDirectory() as directory:
             root = Path(directory)

tests/test_batch_monitor.py

@@ -0,0 +1,69 @@
+import queue
+import unittest
+from unittest.mock import Mock, patch
+
+from copienator_gui.batch_monitor import BatchMonitor, CHECK_INTERVAL_MS
+from copienator_gui.notifications import notify_desktop
+
+
+class BatchMonitorTests(unittest.TestCase):
+    def setUp(self):
+        self.scheduler = Mock()
+        self.scheduler.after.side_effect = lambda delay, callback: (delay, callback)
+        self.ready = Mock()
+        self.status = Mock()
+        self.monitor = BatchMonitor(self.scheduler, self.ready, self.status, Mock())
+        self.runner = Mock()
+        self.runner.events = queue.Queue()
+        self.factory = patch("copienator_gui.batch_monitor.ProcessRunner", return_value=self.runner)
+        self.factory.start()
+        self.addCleanup(self.factory.stop)
+
+    def start(self):
+        self.monitor.start(["check"], "/tmp", {}, None)
+
+    def test_checks_immediately_retries_in_five_minutes_and_notifies_once(self):
+        self.start()
+        self.runner.start.assert_called_once()
+        self.start()
+        self.runner.start.assert_called_once()
+        self.runner.events.put(("finished", (4, False)))
+        self.monitor._poll()
+        self.assertEqual(self.monitor.timer[0], CHECK_INTERVAL_MS)
+        self.assertEqual(CHECK_INTERVAL_MS, 300000)
+        self.ready.assert_not_called()
+        self.monitor.timer[1]()
+        self.assertEqual(self.runner.start.call_count, 2)
+        self.runner.events.put(("finished", (0, False)))
+        self.monitor._poll()
+        self.assertFalse(self.monitor.active)
+        self.assertIsNone(self.monitor.timer)
+        self.ready.assert_called_once()
+        self.monitor._poll()
+        self.ready.assert_called_once()
+
+    def test_stop_cancels_timer_and_running_check_without_notification(self):
+        self.start()
+        timer = self.monitor.timer
+        self.monitor.stop()
+        self.scheduler.after_cancel.assert_called_once_with(timer)
+        self.runner.force_stop.assert_called_once()
+        self.runner.events.put(("finished", (0, False)))
+        self.monitor._poll()
+        self.monitor._check()
+        self.ready.assert_not_called()
+        self.runner.start.assert_called_once()
+
+    def test_start_failure_retries_without_reporting_readiness(self):
+        self.runner.start.side_effect = OSError("unavailable")
+        self.start()
+        self.assertEqual(self.monitor.timer[0], CHECK_INTERVAL_MS)
+        self.ready.assert_not_called()
+
+    def test_linux_notification_passes_text_as_arguments(self):
+        with patch("copienator_gui.notifications.sys.platform", "linux"), patch(
+            "copienator_gui.notifications.find_executable", return_value="/usr/bin/notify-send"
+        ), patch("copienator_gui.notifications.subprocess.Popen") as launch:
+            notify_desktop("Copienator", "Interro02 : résultats prêts")
+        self.assertEqual(launch.call_args.args[0], ["/usr/bin/notify-send", "--app-name=Copienator",
+                                                   "--", "Copienator", "Interro02 : résultats prêts"])

tests/test_batch_status_readiness.py

@@ -0,0 +1,54 @@
+import tempfile
+import unittest
+from pathlib import Path
+from types import SimpleNamespace
+from unittest.mock import Mock, call, patch
+
+from copienator import CliError, EvaluationWorkspace, ExitCode, atomic_write_json
+from copienator.commands.batch_status import check_evaluation_jobs, main
+
+
+class BatchReadinessTests(unittest.TestCase):
+    def setUp(self):
+        self.temp = tempfile.TemporaryDirectory()
+        self.addCleanup(self.temp.cleanup)
+        self.workspace = EvaluationWorkspace(Path(self.temp.name))
+        self.client = Mock()
+
+    def manifest(self, jobs):
+        atomic_write_json(self.workspace.batch_jobs_file, {"jobs": jobs})
+
+    def test_only_recorded_jobs_are_checked_and_all_must_have_results(self):
+        self.manifest({"flash": {"name": "batches/flash"}, "pro": {"name": "batches/pro"}})
+        succeeded = SimpleNamespace(state="JOB_STATE_SUCCEEDED",
+                                    dest=SimpleNamespace(file_name="files/result"))
+        for state in ("JOB_STATE_PENDING", "JOB_STATE_RUNNING", "JOB_STATE_FAILED",
+                      "JOB_STATE_CANCELLED", "JOB_STATE_EXPIRED", "UNKNOWN", "JOB_STATE_SUCCEEDED"):
+            with self.subTest(state=state):
+                self.client.reset_mock()
+                self.client.batches.get.side_effect = [succeeded, SimpleNamespace(
+                    state=SimpleNamespace(name=state), dest=SimpleNamespace(file_name="files/pro"))]
+                result = check_evaluation_jobs(self.workspace, client=self.client)
+                self.assertEqual(result, ExitCode.SUCCESS if state == "JOB_STATE_SUCCEEDED" else ExitCode.PARTIAL)
+                self.assertEqual(self.client.batches.get.call_args_list,
+                                 [call(name="batches/flash"), call(name="batches/pro")])
+                self.client.batches.list.assert_not_called()
+                self.client.files.download.assert_not_called()
+        self.client.batches.get.side_effect = [succeeded, SimpleNamespace(
+            state="JOB_STATE_SUCCEEDED", dest=None)]
+        self.assertEqual(check_evaluation_jobs(self.workspace, client=self.client), ExitCode.PARTIAL)
+
+    def test_missing_empty_and_invalid_manifest_cannot_report_success(self):
+        self.assertEqual(check_evaluation_jobs(self.workspace, client=self.client), ExitCode.PARTIAL)
+        self.manifest({})
+        self.assertEqual(check_evaluation_jobs(self.workspace, client=self.client), ExitCode.PARTIAL)
+        for jobs in ([], {"flash": {}}, {"flash": {"name": ""}}, {"flash": None}):
+            self.manifest(jobs)
+            with self.assertRaises(CliError):
+                check_evaluation_jobs(self.workspace, client=self.client)
+        self.client.batches.get.assert_not_called()
+
+    def test_cli_returns_readiness_code_for_selected_evaluation(self):
+        with patch("copienator.commands.batch_status.check_evaluation_jobs", return_value=ExitCode.PARTIAL) as check:
+            self.assertEqual(main(["--evaluation", str(self.workspace.root)]), ExitCode.PARTIAL)
+        self.assertEqual(check.call_args.args[0].root, self.workspace.root)

tests/test_feedback_boxes.py

@@ -0,0 +1,82 @@
+import copy
+import json
+import tempfile
+import unittest
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+from PIL import Image
+
+from copienator import EvaluationWorkspace, atomic_write_json
+from copienator.annotation_data import GroupCoordinates, _scaled_result
+from copienator.annotation_actions import apply_checkbox_actions
+from copienator.commands import annotating, correction
+from copienator.feedback_boxes import valid_feedback_box
+
+
+class FeedbackBoxTests(unittest.TestCase):
+    def test_checkbox_for_promoted_feedback_deletes_the_correct_comment(self):
+        feedback = [{"text": "Invalid local", "box_2d": [753, 680, 287, 946]},
+                    {"text": "Existing global", "box_2d": None}]
+        apply_checkbox_actions({"Ex 5": {"result": {"feedback": feedback}}},
+                               [{"label": "Ex 5", "type": "del_global", "index": 0}], lambda _: None)
+        self.assertTrue(feedback[0]["to_delete"])
+        self.assertNotIn("to_delete", feedback[1])
+
+    def test_bad_boxes_keep_their_comment_as_global_feedback_without_mutating_data(self):
+        for box in ([753, 680, 287, 946], [10, 50, 20, 30], [10, 20, 10, 30],
+                    [1, 2, 3], [None, 0, 10, 20], [float("nan"), 0, 10, 20]):
+            with self.subTest(box=box):
+                result = {"score": 2, "feedback": [{"text": "Important comment", "box_2d": box}]}
+                original_box = result["feedback"][0]["box_2d"]
+                scaled = _scaled_result(result, GroupCoordinates(2415, 3297, 1655, 3297))
+                callback = Mock()
+                render = Mock(return_value=Image.new("RGBA", (200, 40), "white"))
+                with patch.object(annotating, "render_score_text", return_value=Image.new("RGBA", (200, 40))):
+                    image, _ = annotating.compose_label_image(Image.new("RGBA", (800, 100)), "Ex 5", scaled,
+                                                               2415, render_fn=render, draw_callback=callback)
+                self.assertIsNotNone(image)
+                render.assert_called_once_with("Important comment", unittest.mock.ANY)
+                self.assertFalse(any(call.args[0] == "local_rect" for call in callback.call_args_list))
+                self.assertIs(result["feedback"][0]["box_2d"], original_box)
+
+    def test_valid_boxes_remain_local(self):
+        result = {"feedback": [{"text": "Local", "box_2d": [10, 20, 30, 40]}]}
+        before = copy.deepcopy(result)
+        callback = Mock()
+        with patch.object(annotating, "render_score_text", return_value=Image.new("RGBA", (200, 40))):
+            annotating.compose_label_image(Image.new("RGBA", (800, 100)), "Ex 5", result, 0,
+                                            render_fn=Mock(return_value=Image.new("RGBA", (200, 40))),
+                                            draw_callback=callback)
+        self.assertTrue(any(call.args[0] == "local_rect" for call in callback.call_args_list))
+        self.assertEqual(result, before)
+        self.assertTrue(valid_feedback_box([10, 20, 30, 40]))
+
+    def test_invalid_auxiliary_response_loses_only_its_rectangle(self):
+        returned = [{"text": "Keep this", "box_2d": [753, 680, 287, 946]}]
+        with patch.object(correction.prompting, "request_for_box_correction", return_value=([], {})), patch.object(
+            correction, "call_gemini_with_retries", return_value=json.dumps(returned)
+        ):
+            feedback = correction.correct_boxes_with_gemini("26", "Ex 5", Path("unused.pdf"), [], 0, 1000, 1, 1000)
+        self.assertEqual(feedback, [{"text": "Keep this", "box_2d": None}])
+
+    def test_correction_requests_repair_for_inverted_boxes_and_falls_back_without_losing_comments(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            root = Path(temporary)
+            group = root / "Par label" / "Ex 5" / "Group_1.jpg"
+            group.parent.mkdir(parents=True)
+            group.touch()
+            atomic_write_json(group.with_suffix(".json"), [["26", 0, 1000, 1, "Ex 5"]])
+            args = correction.build_parser().parse_args([str(root)])
+            correction.configure_runtime(EvaluationWorkspace(root), [(str(group), "Ex 5")], args, api_client=Mock())
+            response = [{"id": "26", "result": {"score": 2, "error": "", "feedback": [
+                {"text": "First", "box_2d": [753, 680, 287, 946]},
+                {"text": "Second", "box_2d": [100, 900, 200, 100]}]}}]
+            with patch.object(correction.prompting, "generate_request", return_value=([], {})), patch.object(
+                correction, "correct_boxes_with_gemini", side_effect=RuntimeError("repair failed")
+            ) as repair:
+                correction.process_single_task((str(group), "Ex 5"), json.dumps(response))
+            repair.assert_called_once()
+            feedback = correction.results["Ex 5"][0][0]["result"]["feedback"]
+            self.assertEqual([f["text"] for f in feedback], ["First", "Second"])
+            self.assertTrue(all(f["box_2d"] is None for f in feedback))

tests/test_gui_convenience.py

@@ -52,7 +52,11 @@ class GuiConvenienceTests(unittest.TestCase):
         (self.evaluation / "enonce.pdf").unlink()
         self.app._reload_inputs()
         self.assertEqual(self.app.state_store.step("inputs")["status"], "ready")
-        self.assertIn("enonce.pdf", self.app.description_label.cget("text"))
+        self.assertIn("enonce.pdf", self.app.missing_requirements_label.cget("text"))
+        style = ttk.Style(self.app)
+        self.assertEqual(
+            style.lookup("MissingPrerequisite.TLabel", "foreground"), "#c62828"
+        )
 
     def test_compact_evaluation_input_and_open_folder_button(self):
         self.assertEqual(int(self.app.evaluation_entry.cget("width")), 42)
@@ -64,6 +68,66 @@ class GuiConvenienceTests(unittest.TestCase):
             self.app.open_evaluation_button.invoke()
         opened.assert_called_once_with(self.evaluation)
 
+    def test_manual_resolution_panel_follows_command_target(self):
+        manual = self.evaluation / "manual_resolutions.txt"
+        manual.write_text("Copie01 A -> B|\n", encoding="utf-8")
+        self.app.tree.selection_set("manual_resolution")
+        self.app.update()
+        self.assertEqual(len(self.app.manual_panel.pdf_buttons), 2)
+        other = self.repository / "Other"
+        other.mkdir()
+        other_manual = other / "manual_resolutions.txt"
+        other_manual.write_text("Copie02 C xs D\n", encoding="utf-8")
+        self.app.arg_vars["target"].set(str(other))
+        self.app.update()
+        self.assertIn("Copie02", self.app.manual_panel.text.get("1.0", "end"))
+        with patch("copienator_gui.manual_resolution.open_path") as opened:
+            self.app.manual_panel.editor_button.invoke()
+        opened.assert_called_once_with(other_manual)
+
+    def test_batch_status_waits_until_all_jobs_are_ready(self):
+        self.app.state_store.update_step("correction", variant="batch")
+        self.app.state_store.update_step("batch_status", visited=True)
+        self.app.tree.selection_set("batch_status")
+        self.app.update()
+        command = self.app._make_command()
+        self.assertIn("--evaluation", command)
+        self.assertFalse(self.app.arg_vars)
+        self.app.active_step_id = "batch_status"
+        self.app._finish_process(4, False)
+        self.app.update()
+        self.assertEqual(self.app.current_step.id, "batch_status")
+        self.app.active_step_id = "batch_status"
+        self.app._finish_process(0, False)
+        self.app.update()
+        self.assertEqual(self.app.current_step.id, "fetch_batches")
+
+    def test_batch_watch_buttons_use_loaded_evaluation_and_stop_on_reload(self):
+        self.app.state_store.update_step("correction", variant="batch")
+        self.app.tree.selection_set("batch_status")
+        self.app.update()
+        with patch.object(self.app.batch_monitor, "start") as start:
+            self.app.watch_batches_button.invoke()
+        self.assertEqual(start.call_args.args[0][-2:], ["--evaluation", str(self.evaluation)])
+        self.app.batch_monitor.active = True
+        self.app._update_controls()
+        self.assertEqual(str(self.app.stop_watch_batches_button.cget("state")), "normal")
+        self.app.stop_watch_batches_button.invoke()
+        self.assertFalse(self.app.batch_monitor.active)
+        self.app.batch_monitor.active = True
+        self.app._load_evaluation()
+        self.assertFalse(self.app.batch_monitor.active)
+
+    def test_background_batch_readiness_notifies_without_changing_other_step(self):
+        self.app.tree.selection_set("inputs")
+        self.app.update()
+        with patch("copienator_gui.app.notify_desktop") as notify:
+            self.app._batch_results_ready()
+        notify.assert_called_once()
+        self.assertIn(self.evaluation.name, notify.call_args.args[1])
+        self.assertEqual(self.app.current_step.id, "inputs")
+        self.assertEqual(self.app.state_store.step("batch_status")["status"], "success")
+
     def test_optional_blank_crop_can_target_a_copy_or_be_skipped(self):
         copies = self.evaluation/"Copies"
         copies.mkdir()

tests/test_gui_core.py

@@ -38,6 +38,7 @@ from copienator_gui.app import (
     build_runner_environment,
     copy_pdf_paths,
     detected_annotation_directories,
+    get_personal_interro_files,
     has_manual_conflicts,
     plotting_shortcut_lines,
     process_status,
@@ -45,6 +46,7 @@ from copienator_gui.app import (
 from copienator_gui.diagnostics import collect_diagnostics
 from copienator_gui.runner import ProcessRunner
 from copienator_gui.state import StateStore
+from copienator_gui import workflow as workflow_module
 from copienator_gui.workflow import build_command, build_workflow, evaluation_argument
 from copienator.commands.copies_tools import rename_all, rotate_all
 from copienator.platform import (
@@ -489,7 +491,7 @@ class StandardCliTests(unittest.TestCase):
             "giving_names": (
                 "giving_names",
                 "default",
-                {"target": evaluation, "annotation_dir": "BGnot"},
+                {"target": evaluation, "annotation_dir": "BGnot", "update": True},
             ),
             "grouping": ("grouping", "default", {"target": evaluation}),
             "post_correction": (
@@ -1224,6 +1226,27 @@ class StandardCliTests(unittest.TestCase):
             self.assertEqual(module.results, {"Ex 1": []})
             self.assertEqual(module.tasks_to_process, [("new.jpg", "Ex 1")])
 
+    def test_correction_reserves_unique_group_indices_concurrently(self) -> None:
+        module = self.modules["correction"]
+        with tempfile.TemporaryDirectory() as directory:
+            groups = Path(directory) / "Par label"
+            label_dir = groups / "Ex 8 : 2)"
+            label_dir.mkdir(parents=True)
+            (label_dir / "Group_3.jpg").write_bytes(b"existing")
+
+            with patch.object(module, "GROUPS_DIR", groups):
+                module.reserved_group_indices.clear()
+                with ThreadPoolExecutor(max_workers=8) as executor:
+                    indices = list(
+                        executor.map(
+                            module.reserve_next_group_idx,
+                            ["Ex 8 : 2)"] * 8,
+                        )
+                    )
+
+            self.assertEqual(sorted(indices), list(range(3, 11)))
+            self.assertEqual(len(indices), len(set(indices)))
+
     def test_correction_reset_restores_old_and_deletes_new_files(self) -> None:
         module = self.modules["correction"]
         with tempfile.TemporaryDirectory() as directory:
@@ -1332,6 +1355,43 @@ class StandardCliTests(unittest.TestCase):
 
     def test_post_correction_main_cleans_json_atomically(self) -> None:
         module = self.modules["post_correction"]
+        cleaned_text = module.clean_string(
+            r"outside_name already\_escaped; "
+            r"bad $\\mathbb{U}_n = \\{z \\in \\mathbb{C}\\}$; "
+            r"valid $\begin{cases} A=0 \\ B=1 \end{cases}$",
+            {},
+        )
+        self.assertEqual(
+            cleaned_text,
+            r"outside\_name already\_escaped; "
+            r"bad $\mathbb{U}_n = \{z \in \mathbb{C}\}$; "
+            r"valid $\begin{cases} A=0 \\ B=1 \end{cases}$",
+        )
+        self.assertEqual(module.clean_string(cleaned_text, {}), cleaned_text)
+        corrupted_json_escapes = (
+            "Formulas $"
+            + "\x0c"
+            + "rac{1}{2}$, $"
+            + "\t"
+            + "heta "
+            + "\x0c"
+            + "euille 0 [2"
+            + "\t"
+            + "extbackslash pi]$, $e "
+            + "\t"
+            + "imes x$, and ["
+            + "\n"
+            + "egthinspace[0,n]"
+        )
+        self.assertEqual(
+            module.clean_string(corrupted_json_escapes, {}),
+            r"Formulas $\frac{1}{2}$, $\theta \equiv 0 [2\pi]$, "
+            r"$e \times x$, and [\negthinspace[0,n]",
+        )
+        self.assertEqual(
+            module.clean_string("x ∈ A and B ⊂ C", {}),
+            r"x \ensuremath{\in} A and B \ensuremath{\subset} C",
+        )
         with tempfile.TemporaryDirectory() as directory:
             evaluation = Path(directory) / "Exam"
             evaluation.mkdir()
@@ -1339,11 +1399,20 @@ class StandardCliTests(unittest.TestCase):
                 evaluation / "correction.json",
                 {"text": "outside_name and $math_name$", "suffix": "_new"},
             )
+            original = (evaluation / "correction.json").read_bytes()
+            backup = evaluation / "correction_precleanup.json"
+            backup.write_bytes(b"previous backup")
             self.assertEqual(module.main([str(evaluation)]), 0)
+            self.assertEqual(backup.read_bytes(), original)
             self.assertEqual(
                 read_json(evaluation / "correction.json"),
                 {"text": r"outside\_name and $math_name$", "suffix": "_new"},
             )
+            cleaned = (evaluation / "correction.json").read_bytes()
+            with patch.object(module, "atomic_write_bytes", side_effect=OSError("backup failed")):
+                self.assertNotEqual(module.main([str(evaluation)]), 0)
+            self.assertEqual((evaluation / "correction.json").read_bytes(), cleaned)
+            self.assertEqual(backup.read_bytes(), original)
 
     def test_manual_resolution_with_no_actions_is_safe(self) -> None:
         module = self.modules["resolve_manual"]
@@ -1660,6 +1729,53 @@ class WorkflowTests(unittest.TestCase):
         self.assertNotIn("update_ods", standard_ids)
         self.assertIn("update_ods", personal_ids)
 
+    def test_get_personal_interro_files_copies_the_three_expected_sources(self) -> None:
+        with tempfile.TemporaryDirectory() as directory:
+            root = Path(directory)
+            source = root / "source"
+            evaluation = root / "Interro07"
+            source.mkdir()
+            evaluation.mkdir()
+            contents = {
+                "Interro07.pdf": b"pdf",
+                "Interro07.tex": b"statement",
+                "Interro07c.tex": b"correction",
+            }
+            for name, content in contents.items():
+                (source / name).write_bytes(content)
+
+            copied = get_personal_interro_files(evaluation, source)
+
+            self.assertEqual(
+                copied,
+                (
+                    evaluation / "enonce.pdf",
+                    evaluation / "enonce.tex",
+                    evaluation / "correction.tex",
+                ),
+            )
+            self.assertEqual((evaluation / "enonce.pdf").read_bytes(), b"pdf")
+            self.assertEqual((evaluation / "enonce.tex").read_bytes(), b"statement")
+            self.assertEqual((evaluation / "correction.tex").read_bytes(), b"correction")
+
+    def test_get_personal_interro_files_validates_name_and_all_sources_first(self) -> None:
+        with tempfile.TemporaryDirectory() as directory:
+            root = Path(directory)
+            source = root / "source"
+            source.mkdir()
+            invalid = root / "DS07"
+            invalid.mkdir()
+            with self.assertRaisesRegex(ValueError, "Interro"):
+                get_personal_interro_files(invalid, source)
+
+            evaluation = root / "Interro07"
+            evaluation.mkdir()
+            (evaluation / "enonce.pdf").write_bytes(b"existing")
+            (source / "Interro07.pdf").write_bytes(b"new")
+            with self.assertRaisesRegex(FileNotFoundError, "Interro07.tex"):
+                get_personal_interro_files(evaluation, source)
+            self.assertEqual((evaluation / "enonce.pdf").read_bytes(), b"existing")
+
     def test_first_visit_automation_is_declared_on_expected_steps(self) -> None:
         auto_start = {
             step.id for step in self.steps.values() if step.auto_start_first_visit
@@ -1835,6 +1951,12 @@ class WorkflowTests(unittest.TestCase):
                     self.assertTrue(spec.help.strip())
                     self.assertTrue(spec.help.rstrip().endswith("."))
 
+    def test_final_score_documents_immediate_gestion_classe_prerequisite(self) -> None:
+        description = self.steps["final_score"].description
+        self.assertIn("gestion_classe wse", description)
+        self.assertIn("juste avant", description)
+        self.assertIn("histogramme.pdf", description)
+
     def test_options_previously_requiring_free_form_arguments_have_controls(self) -> None:
         status = self.command(
             "batch_status",
@@ -1843,7 +1965,7 @@ class WorkflowTests(unittest.TestCase):
         )
         self.assertEqual(
             command_arguments(status),
-            ["--download", "files/job-1", "--output", "/tmp/result.jsonl"],
+            ["--evaluation", self.evaluation],
         )
 
         grouped = self.command(
@@ -1857,7 +1979,46 @@ class WorkflowTests(unittest.TestCase):
         )
         self.assertEqual(
             command_arguments(grouped),
-            [self.evaluation, "--refaire", "--annotation-dir", "Bnot"],
+            [
+                self.evaluation,
+                "--refaire",
+                "--annotation-dir",
+                "Bnot",
+                "--no-return-answers-context",
+                "--return-answers-question",
+                "--return-answers-solution",
+            ],
+        )
+
+    def test_return_answer_controls_follow_configuration_and_can_be_hidden(self) -> None:
+        with patch.multiple(
+            workflow_module.configuration,
+            RETURN_ANSWERS_ENABLED=True,
+            RETURN_ANSWERS_CONTEXT=True,
+            RETURN_ANSWERS_QUESTION=False,
+            RETURN_ANSWERS_SOLUTION=True,
+        ):
+            step = next(
+                item for item in build_workflow(True) if item.id == "read_annotations"
+            )
+        specs = {spec.name: spec for spec in step.arguments}
+        self.assertIs(specs["return_answers_context"].default, True)
+        self.assertIs(specs["return_answers_question"].default, False)
+        self.assertIs(specs["return_answers_solution"].default, True)
+
+        with patch.object(
+            workflow_module.configuration, "RETURN_ANSWERS_ENABLED", False
+        ):
+            hidden_step = next(
+                item for item in build_workflow(True) if item.id == "read_annotations"
+            )
+        self.assertFalse(
+            {
+                "return_answers_context",
+                "return_answers_question",
+                "return_answers_solution",
+            }
+            & {spec.name for spec in hidden_step.arguments}
         )
 
     def test_verbose_is_added_only_to_commands_that_support_it(self) -> None:

tests/test_gui_cut_helper.py

@@ -0,0 +1,56 @@
+import os
+import tempfile
+import tkinter as tk
+import unittest
+from pathlib import Path
+from types import SimpleNamespace
+
+import pymupdf
+
+from copienator_gui.cut_helper import CutHelper, PAGE_GAP
+from copienator_gui.manual_resolution import ManualResolutionPanel
+
+
+@unittest.skipUnless(os.environ.get("DISPLAY"), "requires a display")
+class CutHelperTests(unittest.TestCase):
+    def test_cut_button_drag_to_page_gap_and_enter_only_produces_command(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            directory = Path(temporary)
+            source = directory / "Copies" / "Copie16" / "A.pdf"
+            source.parent.mkdir(parents=True)
+            with pymupdf.open() as document:
+                for height in (200, 400):
+                    page = document.new_page(width=300, height=height)
+                    page.insert_text((20, 40), "Student answer")
+                document.save(source)
+            manual = directory / "manual_resolutions.txt"
+            manual.write_text("Copie16 A -> B|\n")
+            original = source.read_bytes()
+            root = tk.Tk()
+            try:
+                panel = ManualResolutionPanel(root, lambda: directory)
+                panel.pack()
+                root.update()
+                self.assertEqual(len(panel.cut_buttons), 1)
+                panel.cut_buttons[0].invoke()
+                helper = next(child for child in panel.winfo_children() if isinstance(child, CutHelper))
+                helper.canvas.yview_moveto(0)
+                root.update()
+                helper.move_bar(SimpleNamespace(y=200 * helper.scale + PAGE_GAP / 2))
+                self.assertIn("entre les pages 1 et 2", helper.caption.get())
+                helper.keep.set(2)
+                helper.mode.set("x")
+                helper.focus_force()
+                root.update()
+                helper.event_generate("<Return>")
+                root.update()
+                self.assertFalse(helper.winfo_exists())
+                self.assertEqual(panel.cut_result.get(), "Copie16 A c{33.333333}2x B|")
+                panel.copy_cut_command()
+                self.assertEqual(root.clipboard_get(), panel.cut_result.get())
+                self.assertEqual(source.read_bytes(), original)
+                self.assertEqual(manual.read_text(), "Copie16 A -> B|\n")
+                panel.reload()
+                self.assertEqual(len(panel.cut_buttons), 1)
+            finally:
+                root.destroy()

tests/test_gui_manual_resolution.py

@@ -0,0 +1,74 @@
+import os
+import tempfile
+import tkinter as tk
+import unittest
+from pathlib import Path
+from unittest.mock import call, patch
+
+from copienator_gui.manual_resolution import ManualResolutionPanel
+
+
+@unittest.skipUnless(os.environ.get("DISPLAY"), "requires a display")
+class ManualResolutionPanelTests(unittest.TestCase):
+    def setUp(self):
+        self.temp = tempfile.TemporaryDirectory()
+        self.addCleanup(self.temp.cleanup)
+        self.evaluation = Path(self.temp.name)
+        self.root = tk.Tk()
+        self.addCleanup(self.root.destroy)
+        self.path = self.evaluation / "manual_resolutions.txt"
+
+    def panel(self):
+        panel = ManualResolutionPanel(self.root, lambda: self.evaluation)
+        panel.pack()
+        self.root.update()
+        return panel
+
+    def test_preview_editor_and_each_pdf_pair_use_shared_resolution_rules(self):
+        content = "### Instructions\nCopie01 Ex 1 x> |Ex 2\n\nCopie02 Ex 3 ss Ex 4|\n"
+        self.path.write_text(content, encoding="utf-8")
+        paths = []
+        for copy, label in (("01", "Ex 1_new"), ("01", "Ex 2_old"),
+                            ("02", "Ex 3"), ("02", "Ex 4")):
+            path = self.evaluation / "Copies" / f"Copie{copy}" / f"{label}.pdf"
+            path.parent.mkdir(parents=True, exist_ok=True)
+            path.touch()
+            paths.append(path)
+        panel = self.panel()
+        self.assertEqual(panel.text.get("1.0", "end-1c"), content)
+        self.assertEqual(len(panel.pdf_buttons), 4)
+        self.assertEqual([button.cget("text") for button in panel.pdf_buttons],
+                         ["PDF source", "PDF cible"] * 2)
+        with patch("copienator_gui.manual_resolution.open_path") as opened:
+            panel.editor_button.invoke()
+            for button in panel.pdf_buttons:
+                button.invoke()
+        self.assertEqual(opened.call_args_list, [call(self.path), *map(call, paths)])
+
+    def test_reload_keeps_invalid_lines_visible_and_removes_stale_actions(self):
+        self.path.write_text("Copie01 A -> B|\n", encoding="utf-8")
+        panel = self.panel()
+        self.path.write_text("bad instruction\nCopie02 C sx D\n", encoding="utf-8")
+        panel.reload()
+        self.assertIn("bad instruction", panel.text.get("1.0", "end"))
+        self.assertIn("lignes invalides : 1", panel.status.cget("text"))
+        self.assertEqual(len(panel.pdf_buttons), 2)
+        self.path.unlink()
+        panel.reload()
+        self.assertFalse(panel.pdf_buttons)
+        self.assertEqual(str(panel.editor_button.cget("state")), "disabled")
+
+    def test_missing_source_does_not_prevent_opening_destination(self):
+        self.path.write_text("Copie01 A -> B|\n", encoding="utf-8")
+        destination = self.evaluation / "Copies" / "Copie01" / "B.pdf"
+        destination.parent.mkdir(parents=True)
+        destination.touch()
+        panel = self.panel()
+        with patch("copienator_gui.manual_resolution.open_path") as opened, patch(
+            "copienator_gui.manual_resolution.messagebox.showerror"
+        ) as error:
+            panel.pdf_buttons[0].invoke()
+            opened.assert_not_called()
+            panel.pdf_buttons[1].invoke()
+        opened.assert_called_once_with(destination)
+        self.assertIn("A.pdf", error.call_args.args[1])

tests/test_manual_cut.py

@@ -0,0 +1,125 @@
+import tempfile
+import unittest
+from pathlib import Path
+from unittest.mock import patch
+
+import numpy as np
+import pymupdf
+
+from copienator import CliError, EvaluationWorkspace, atomic_write_json, read_json
+from copienator.commands.resolve_manual import parse_instruction_text, resolve_manual
+from copienator.pdf_cut import cut_position, split_pdf
+from copienator_gui.cut_helper import PAGE_GAP, cut_operator, percentage_at_y, y_at_percentage
+
+
+def make_pdf(path, heights=(300, 700), rotation=0, cropped=False):
+    with pymupdf.open() as document:
+        for index, height in enumerate(heights):
+            page = document.new_page(width=200, height=height)
+            page.draw_rect(pymupdf.Rect(0, 0, 200, height / 2), color=None, fill=(1, 0, 0))
+            page.draw_rect(pymupdf.Rect(0, height / 2, 200, height), color=None, fill=(0, 0, 1))
+            page.insert_text((30, 30), f"Page {index + 1}")
+            if cropped:
+                page.set_cropbox(pymupdf.Rect(10, 20, 180, height - 30))
+            page.set_rotation(rotation)
+        document.save(path)
+
+
+class ManualCutTests(unittest.TestCase):
+    def setUp(self):
+        self.temp = tempfile.TemporaryDirectory()
+        self.addCleanup(self.temp.cleanup)
+        self.root = Path(self.temp.name)
+
+    def test_parser_accepts_new_operators_and_rejects_invalid_cuts(self):
+        for keep in (1, 2):
+            for action in (">", "x"):
+                item = parse_instruction_text(f"Copie16 Ex 4 : 1) c{{43.125}}{keep}{action} |Ex 4 : 2)")[0]
+                self.assertEqual(item.cut, (43.125, keep))
+                self.assertEqual(item.should_merge, action == ">")
+                self.assertTrue(item.pipe_first)
+        for operator in ("c{0}1>", "c{100}1>", "c{-1}1>", "c{101}1>", "c{43}3>",
+                         "c{43}1s", "c{NaN}1>"):
+            with self.assertRaises(CliError):
+                parse_instruction_text(f"Copie16 A {operator} B")
+        with self.assertRaises(CliError):
+            parse_instruction_text("Copie16 A c{43}1> A")
+
+    def test_page_boundary_preserves_whole_pages_without_clipping(self):
+        source, first, second = [self.root / name for name in ("source.pdf", "first.pdf", "second.pdf")]
+        make_pdf(source, heights=(100, 200), rotation=180)
+        before = source.read_bytes()
+        with patch("pymupdf.Page.show_pdf_page", side_effect=AssertionError("must not clip whole pages")):
+            split_pdf(source, 33.333333, first, second)
+        for path, height in ((first, 100), (second, 200)):
+            with pymupdf.open(path) as pdf:
+                self.assertEqual(len(pdf), 1)
+                self.assertEqual(pdf[0].rect.height, height)
+                self.assertEqual(pdf[0].rotation, 180)
+        self.assertEqual(source.read_bytes(), before)
+
+    def test_in_page_cut_preserves_visible_pixels_for_cropped_rotated_pages(self):
+        source, first, second = [self.root / name for name in ("source.pdf", "first.pdf", "second.pdf")]
+        for rotation in (0, 90, 180, 270):
+            with self.subTest(rotation=rotation):
+                make_pdf(source, heights=(400,), rotation=rotation, cropped=True)
+                with pymupdf.open(source) as pdf:
+                    pix = pdf[0].get_pixmap()
+                    expected = np.frombuffer(pix.samples, np.uint8).reshape(pix.height, pix.width, 3)
+                split_pdf(source, 50, first, second)
+                for path, pixels in ((first, expected[:len(expected)//2]), (second, expected[len(expected)//2:])):
+                    with pymupdf.open(path) as pdf:
+                        pix = pdf[0].get_pixmap()
+                        actual = np.frombuffer(pix.samples, np.uint8).reshape(pix.height, pix.width, 3)
+                    self.assertEqual(actual.shape, pixels.shape)
+                    self.assertLess(np.abs(actual.astype(float) - pixels).mean(), 0.1)
+
+    def test_cut_within_page_preserves_subsequent_pages(self):
+        source, first, second = [self.root / name for name in ("source.pdf", "first.pdf", "second.pdf")]
+        make_pdf(source)
+        split_pdf(source, 15, first, second)
+        with pymupdf.open(first) as pdf:
+            self.assertEqual([page.rect.height for page in pdf], [150])
+        with pymupdf.open(second) as pdf:
+            self.assertEqual([page.rect.height for page in pdf], [150, 700])
+
+    def test_resolver_archives_source_and_recorrrects_both_labels(self):
+        for keep in (1, 2):
+            for mode in (">", "x"):
+                for pipe_first in (False, True):
+                    with self.subTest(keep=keep, mode=mode, pipe_first=pipe_first):
+                        root = self.root / f"{keep}{mode == '>'}{pipe_first}"
+                        copies = root / "Copies" / "Copie16"
+                        copies.mkdir(parents=True)
+                        source, target = copies / "A.pdf", copies / "B.pdf"
+                        make_pdf(source)
+                        make_pdf(target, heights=(80,))
+                        original, destination = source.read_bytes(), target.read_bytes()
+                        atomic_write_json(root / "correction.json", {
+                            label: [[{"id": "16", "result": {}}]] for label in ("A", "B")
+                        })
+                        new_label = "|B" if pipe_first else "B|"
+                        (root / "manual_resolutions.txt").write_text(f"Copie16 A c{{30}}{keep}{mode} {new_label}\n")
+                        self.assertEqual(resolve_manual(EvaluationWorkspace(root)), 0)
+                        self.assertEqual((copies / "A_old.pdf").read_bytes(), original)
+                        self.assertEqual((copies / "B_old.pdf").read_bytes(), destination)
+                        retained, moved = (300, 700) if keep == 1 else (700, 300)
+                        with pymupdf.open(copies / "A_new.pdf") as pdf:
+                            self.assertEqual([page.rect.height for page in pdf], [retained])
+                        with pymupdf.open(copies / "B_new.pdf") as pdf:
+                            expected = ([moved, 80] if pipe_first else [80, moved]) if mode == ">" else [moved]
+                            self.assertEqual([page.rect.height for page in pdf], expected)
+                        self.assertEqual(read_json(root / "refaire.json"), [["Copie16", ["A", "B"]]])
+                        self.assertTrue(all(read_json(root / "correction.json")[label][0][0]["result"]["suffix"] == "_new" for label in ("A", "B")))
+                        self.assertFalse(list(copies.glob("temp_*.pdf")))
+
+    def test_helper_snaps_to_gap_and_round_trips_percentage(self):
+        heights = [100, 200]
+        for y in (95, 100, 100 + PAGE_GAP / 2, 100 + PAGE_GAP + 5):
+            percent = percentage_at_y(y, heights, 1)
+            self.assertEqual(cut_position(heights, percent), (1, 0))
+            self.assertEqual(y_at_percentage(percent, heights, 1), 100 + PAGE_GAP / 2)
+        self.assertEqual(cut_operator(43, 1, ">"), "c{43}1>")
+        self.assertEqual(cut_operator(100/3, 2, "x"), "c{33.333333}2x")
+        self.assertEqual(cut_position(heights, 33.333333), (1, 0))
+        self.assertNotEqual(cut_position(heights, 33.3)[1], 0)

tests/test_manual_resolution_state.py

@@ -0,0 +1,65 @@
+import copy
+import tempfile
+import unittest
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+import pymupdf
+
+from copienator import EvaluationWorkspace, atomic_write_json, read_json
+from copienator.commands import correction, resolve_manual
+
+
+class ManualResolutionStateTests(unittest.TestCase):
+    def test_acknowledgement_clears_only_this_target_and_copy(self):
+        for error in ("wrg-lbl:B?", "wrg-lbl:B?delayed", "wrg-lbl:B?exists", "al:(->)B?(->)C?", "al:(delayed)B"):
+            with self.subTest(error=error):
+                source = {"id": "01", "result": {"error": error, "delayed": [
+                    ["wrong-label", "B"], ["add-label", "B"], ["add-label", "C"]]}}
+                other = {"id": "02", "result": {"error": error, "delayed": [["wrong-label", "B"]]}}
+                before_other = copy.deepcopy(other)
+                results = {"A": [[source, other]]}
+                resolve_manual.set_suffix_and_clean_error(results, "01", "A", None, "B")
+                self.assertEqual(source["result"]["delayed"], [["add-label", "C"]])
+                self.assertNotIn("B?", source["result"]["error"])
+                self.assertEqual(other, before_other)
+                resolve_manual.set_suffix_and_clean_error(results, "01", "A", None, "C")
+                self.assertNotIn("delayed", source["result"])
+
+    def fixture(self, root, operator):
+        copies = root / "Copies" / "Copie01"
+        copies.mkdir(parents=True)
+        for label in ("A", "B"):
+            with pymupdf.open() as doc:
+                page = doc.new_page(width=200, height=200)
+                page.insert_text((20, 40), label)
+                doc.save(copies / f"{label}.pdf")
+        atomic_write_json(root / "correction.json", {
+            "A": [[{"id": "01", "result": {"error": "wrg-lbl:B?", "delayed": [["wrong-label", "B"]]}}]],
+            "B": [[{"id": "01", "result": {"error": ""}}]],
+        })
+        (root / "manual_resolutions.txt").write_text(f"Copie01 A {operator} B|\n")
+        return EvaluationWorkspace(root)
+
+    def test_every_resolution_acknowledges_pending_conflict(self):
+        for operator in (*resolve_manual.OPERATORS, "c{50}1>", "c{50}2x"):
+            with self.subTest(operator=operator), tempfile.TemporaryDirectory() as temporary:
+                workspace = self.fixture(Path(temporary), operator)
+                resolve_manual.resolve_manual(workspace)
+                result = read_json(workspace.correction_file)["A"][0][0]["result"]
+                self.assertNotIn("delayed", result)
+                self.assertFalse(workspace.manual_resolutions_file.exists())
+
+    def test_refaire_does_not_recreate_a_resolved_source_conflict(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            workspace = self.fixture(Path(temporary), "x>")
+            resolve_manual.resolve_manual(workspace)
+            (workspace.groups_dir / "B").mkdir(parents=True)
+            args = correction.build_parser().parse_args([str(workspace.root), "--refaire"])
+            correction.configure_runtime(workspace, [], args, api_client=Mock())
+            with patch.object(correction.grouping, "get_pdf_height", return_value=400), patch.object(
+                correction.grouping, "create_jpg"
+            ), patch.object(correction, "process_single_task", return_value=[]):
+                self.assertEqual(correction.run_configured(args), 0)
+            self.assertFalse(workspace.manual_resolutions_file.exists())
+            self.assertNotIn("delayed", correction.results["A"][0][0]["result"])

tests/test_prompting.py

@@ -0,0 +1,33 @@
+import unittest
+from unittest.mock import patch
+
+from copienator import prompting
+
+
+class PromptingTests(unittest.TestCase):
+    def test_perspective_is_inserted_with_alternative_method_guidance(self) -> None:
+        with patch.object(
+            prompting, "get_label_text_content", return_value="Question"
+        ), patch.object(
+            prompting, "get_label_sol_content", return_value="Solution"
+        ), patch.object(
+            prompting, "get_label_persp_content", return_value="Barème détaillé"
+        ):
+            prompt = prompting.make_prompt("evaluation", "Ex 1")
+
+        self.assertIn(prompting.PERSPECTIVE_GUIDANCE, prompt)
+        self.assertIn("Barème détaillé", prompt)
+
+    def test_alternative_method_guidance_is_omitted_without_perspective(self) -> None:
+        with patch.object(
+            prompting, "get_label_text_content", return_value="Question"
+        ), patch.object(
+            prompting, "get_label_sol_content", return_value="Solution"
+        ), patch.object(prompting, "get_label_persp_content", return_value=None):
+            prompt = prompting.make_prompt("evaluation", "Ex 1")
+
+        self.assertNotIn(prompting.PERSPECTIVE_GUIDANCE, prompt)
+
+
+if __name__ == "__main__":
+    unittest.main()

tests/test_splitting_geometry.py

@@ -0,0 +1,92 @@
+"""Compare extracted answers against the same Poppler view used for labels."""
+
+import tempfile
+import unittest
+from pathlib import Path
+
+import numpy as np
+import pymupdf
+from pdf2image import convert_from_path
+
+from copienator.commands.splitting_int import _render_split_outputs
+
+
+class SplittingGeometryTests(unittest.TestCase):
+    def make_source(self, path, rotation, cropped=True):
+        with pymupdf.open() as document:
+            page = document.new_page(width=800, height=1000)
+            # Asymmetric colours exercise both translation and orientation.
+            for row in range(20):
+                for column in range(8):
+                    colour = (row / 20, column / 8, (row + column) % 7 / 7)
+                    page.draw_rect(
+                        pymupdf.Rect(column * 100, row * 50,
+                                     (column + 1) * 100, (row + 1) * 50),
+                        color=None, fill=colour,
+                    )
+            page.insert_text((200, 500), "Middle of the answer")
+            if cropped:
+                page.set_cropbox(pymupdf.Rect(20, 80, 760, 920))
+            page.set_rotation(rotation)
+            document.save(path)
+
+    def assert_matches_preview(self, answer, preview, bounds):
+        actual = np.asarray(convert_from_path(answer, dpi=72)[0]).astype(float)
+        expected = np.asarray(preview.crop(bounds)).astype(float)
+        self.assertEqual(actual.shape, expected.shape)
+        self.assertLess(np.abs(actual - expected).mean(), 0.5)
+
+    def test_cropped_and_uncropped_pages_at_every_rotation(self):
+        visible = {
+            0: (20, 80, 760, 920),
+            90: (80, 20, 920, 760),
+            180: (40, 80, 780, 920),
+            270: (80, 40, 920, 780),
+        }
+        for rotation in (0, 90, 180, 270):
+            for cropped in (False, True):
+                for full_answer in (False, True):
+                    with self.subTest(rotation=rotation, cropped=cropped, full=full_answer):
+                        with tempfile.TemporaryDirectory() as temporary:
+                            root = Path(temporary)
+                            source = root / "copy.pdf"
+                            self.make_source(source, rotation, cropped)
+                            before = source.read_bytes()
+                            preview = convert_from_path(source, dpi=72, use_cropbox=False)[0]
+                            width, height = preview.size
+                            bounds = visible[rotation] if cropped else (0, 0, width, height)
+                            if full_answer:
+                                coordinates = [("A", 0, 0, 10, 0, 100)]
+                            else:
+                                coordinates = [("A", 0, 250, 260, 0, 100),
+                                               ("_", 0, 711, 721, 0, 100)]
+                                bounds = (bounds[0], max(bounds[1], height // 4),
+                                          bounds[2], min(bounds[3], height * 3 // 4))
+                            _render_split_outputs(source, coordinates, root)
+                            self.assert_matches_preview(root / "A.pdf", preview, bounds)
+                            self.assertEqual(source.read_bytes(), before)
+
+    def test_answer_continues_to_bottom_then_next_page(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            root = Path(temporary)
+            source = root / "copy.pdf"
+            first = root / "first.pdf"
+            second = root / "second.pdf"
+            self.make_source(first, 180)
+            self.make_source(second, 0)
+            with pymupdf.open() as document:
+                for path in (first, second):
+                    with pymupdf.open(path) as part:
+                        document.insert_pdf(part)
+                document.save(source)
+            previews = convert_from_path(source, dpi=72, use_cropbox=False)
+            _render_split_outputs(source, [("A", 0, 700, 720, 0, 100),
+                                           ("_", 1, 211, 230, 500, 600)], root)
+            rendered = convert_from_path(root / "A.pdf", dpi=72)
+            self.assertEqual(len(rendered), 2)
+            for actual, preview, bounds in zip(rendered, previews,
+                                               [(40, 700, 780, 920), (20, 80, 760, 250)]):
+                expected = np.asarray(preview.crop(bounds)).astype(float)
+                actual = np.asarray(actual).astype(float)
+                self.assertEqual(actual.shape, expected.shape)
+                self.assertLess(np.abs(actual - expected).mean(), 0.5)

tests/test_statement_personal_flow.py

@@ -4,13 +4,14 @@ import os
 import tempfile
 import unittest
 from pathlib import Path
+from tkinter import ttk
 from types import SimpleNamespace
 from unittest.mock import Mock, patch
 
 from copienator import CliError, EvaluationWorkspace, ExitCode
 from copienator.commands import enonce_info as personal
 from copienator.commands import gemini_for_enonce as gemini
-from copienator_gui.app import CopienatorApp
+from copienator_gui.app import CopienatorApp, get_personal_interro_files
 from copienator_gui.workflow import build_workflow
 
 
@@ -53,7 +54,7 @@ class PersonalStatementTests(unittest.TestCase):
         enabled = {step.id: step for step in build_workflow(True)}
         self.assertEqual([variant.id for variant in standard["statement"].variants], ["gemini"])
         self.assertEqual([variant.program for variant in enabled["statement"].variants],
-                         ["statement", "statement-personal"])
+                         ["statement-personal", "statement"])
         for ident in ("statement_groups", "statement_persp"):
             self.assertNotIn(ident, standard)
             self.assertTrue(enabled[ident].optional)
@@ -120,6 +121,11 @@ class SelectiveGeminiTests(unittest.TestCase):
             self.assertEqual(call.kwargs["contents"][0].parts[0].text, gemini.PROMPT_4)
             self.assertTrue(call.kwargs["config"].automatic_function_calling.disable)
 
+    def test_rubric_prompt_omits_the_assumed_total_and_requires_latex(self):
+        self.assertIn("total est toujours implicite", gemini.PROMPT_4)
+        self.assertIn("caractères mathématiques Unicode", gemini.PROMPT_4)
+        self.assertIn(r"$\lfloor \sqrt{k} \rfloor$", gemini.PROMPT_4)
+
     def test_incomplete_or_failed_rubrics_preserve_entire_persp(self):
         before = self.snapshot()
         for last in (self.response({"rubrics": []}), RuntimeError("API unavailable")):
@@ -140,6 +146,48 @@ class SelectiveGeminiTests(unittest.TestCase):
 
 @unittest.skipUnless(os.environ.get("DISPLAY"), "Tk requires a display")
 class PersonalStatementGuiTests(unittest.TestCase):
+    def test_get_file_button_completes_inputs_and_advances(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            repository = Path(temporary)
+            evaluation = repository / "Interro12"
+            source = repository / "source"
+            evaluation.mkdir()
+            source.mkdir()
+            (repository / "names").touch()
+            for name, content in (
+                ("Interro12.pdf", b"pdf"),
+                ("Interro12.tex", b"statement"),
+                ("Interro12c.tex", b"correction"),
+            ):
+                (source / name).write_bytes(content)
+            app = CopienatorApp(repository, True, evaluation)
+            try:
+                app.update()
+                buttons = [
+                    child
+                    for child in app.form.winfo_children()
+                    if isinstance(child, ttk.Button)
+                ]
+                get_button = next(
+                    button for button in buttons if button.cget("text") == "Get the files"
+                )
+                with patch(
+                    "copienator_gui.app.get_personal_interro_files",
+                    side_effect=lambda target: get_personal_interro_files(target, source),
+                ):
+                    get_button.invoke()
+                    app.update()
+
+                self.assertEqual(app.state_store.step("inputs")["status"], "success")
+                self.assertEqual(app.current_step.id, "statement")
+                self.assertEqual((evaluation / "enonce.pdf").read_bytes(), b"pdf")
+                self.assertEqual((evaluation / "enonce.tex").read_bytes(), b"statement")
+                self.assertEqual((evaluation / "correction.tex").read_bytes(), b"correction")
+            finally:
+                for callback in app.tk.splitlist(app.tk.call("after", "info")):
+                    app.after_cancel(callback)
+                app.destroy()
+
     def test_personal_requirements_and_optional_button_command_previews(self):
         with tempfile.TemporaryDirectory() as temporary:
             evaluation = Path(temporary)
@@ -149,12 +197,19 @@ class PersonalStatementGuiTests(unittest.TestCase):
                 app.update()
                 app.tree.selection_set("statement")
                 app.update()
-                app._select_variant(1)
+                self.assertEqual(app.variant_var.get(), "personal")
                 self.assertIn("statement-personal", app.command_var.get())
                 self.assertEqual(app._missing_requirements(app.current_step), [])
                 self.assertIn("SHEETINFO", app.description_label.cget("text"))
+                self.assertFalse(
+                    any(
+                        isinstance(child, ttk.LabelFrame)
+                        and child.cget("text") == "Après génération — facultatif"
+                        for child in app.form.winfo_children()
+                    )
+                )
                 for ident, flag in (("statement_groups", "--groups-only"), ("statement_persp", "--persp-only")):
-                    app._select_statement_action(ident)
+                    app.tree.selection_set(ident)
                     app.update()
                     self.assertEqual(app.current_step.id, ident)
                     self.assertIn(flag, app.command_var.get())

tests/test_utils_compile.py

@@ -0,0 +1,48 @@
+import contextlib
+import io
+import subprocess
+import tempfile
+import unittest
+from pathlib import Path
+from unittest.mock import patch
+
+from copienator import utils
+
+
+class CompileToPdfTests(unittest.TestCase):
+    def test_warns_when_latex_fails_even_if_partial_pdf_exists(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            output = Path(temporary) / "result.pdf"
+
+            def failed_run(*_args, **kwargs):
+                (Path(kwargs["cwd"]) / "text.pdf").write_bytes(b"partial")
+                return subprocess.CompletedProcess(
+                    args=[], returncode=1,
+                    stdout="! LaTeX Error: Something's wrong--perhaps a missing \\item.\n",
+                )
+
+            console = io.StringIO()
+            with patch.object(utils.subprocess, "run", side_effect=failed_run), \
+                    contextlib.redirect_stdout(console):
+                utils.compile_to_pdf("broken", output)
+
+            self.assertEqual(output.read_bytes(), b"partial")
+            self.assertIn("Warning: LaTeX compilation failed", console.getvalue())
+            self.assertIn("missing \\item", console.getvalue())
+
+    def test_warns_when_no_pdf_is_produced(self):
+        with tempfile.TemporaryDirectory() as temporary:
+            output = Path(temporary) / "result.pdf"
+            result = subprocess.CompletedProcess(args=[], returncode=0, stdout="")
+
+            console = io.StringIO()
+            with patch.object(utils.subprocess, "run", return_value=result), \
+                    contextlib.redirect_stdout(console):
+                utils.compile_to_pdf("valid", output)
+
+            self.assertFalse(output.exists())
+            self.assertIn("produced no PDF", console.getvalue())
+
+
+if __name__ == "__main__":
+    unittest.main()