Fix timing of teardown

We just tear down the document build dir at the end, no matter what.
The way it was before it would tear down, _then_ show an error message
if there was an error. Now the output is more logical. (sure, same result)

Author: dannyadair

src/pdfbaker/baker.py

@@ -138,8 +138,8 @@ def bake(self) -> None:
                 if isinstance(pdf_files, Path):
                     pdf_files = [pdf_files]
                 pdfs_created.extend(pdf_files)
-                if not self.keep_build:
-                    doc.teardown()
+            if not self.keep_build:
+                doc.teardown()
 
         if pdfs_created:
             self.log_info("Successfully created PDFs:")

src/pdfbaker/document.py

@@ -157,30 +157,22 @@ def _process_with_custom_bake(self, bake_path: Path) -> Path | list[Path]:
 
     def process(self) -> Path | list[Path]:
         """Process document using standard processing."""
-        try:
-            if "variants" in self.config:
-                # Multiple PDF documents
-                pdf_files = []
-                for variant in self.config["variants"]:
-                    self.log_info_subsection(
-                        'Processing variant "%s"...', variant["name"]
-                    )
-                    variant_config = deep_merge(self.config, variant)
-                    variant_config["variant"] = variant
-                    variant_config = render_config(variant_config)
-                    page_pdfs = self._process_pages(variant_config)
-                    pdf_files.append(self._finalize(page_pdfs, variant_config))
-                return pdf_files
-
-            # Single PDF document
-            doc_config = render_config(self.config)
-            page_pdfs = self._process_pages(doc_config)
-            return self._finalize(page_pdfs, doc_config)
-        except Exception:
-            # Ensure build directory is cleaned up if processing fails
-            if not self.baker.keep_build:
-                self.teardown()
-            raise
+        if "variants" in self.config:
+            # Multiple PDF documents
+            pdf_files = []
+            for variant in self.config["variants"]:
+                self.log_info_subsection('Processing variant "%s"...', variant["name"])
+                variant_config = deep_merge(self.config, variant)
+                variant_config["variant"] = variant
+                variant_config = render_config(variant_config)
+                page_pdfs = self._process_pages(variant_config)
+                pdf_files.append(self._finalize(page_pdfs, variant_config))
+            return pdf_files
+
+        # Single PDF document
+        doc_config = render_config(self.config)
+        page_pdfs = self._process_pages(doc_config)
+        return self._finalize(page_pdfs, doc_config)
 
     def _process_pages(self, config: dict[str, Any]) -> list[Path]:
         """Process pages with given configuration."""