@heylemon/lemonade 0.0.4 → 0.0.6

package/skills/pdf/scripts/fill_pdf_form_with_annotations.py

@@ -1,25 +1,19 @@
 import json
 import sys
-import os
 
 from pypdf import PdfReader, PdfWriter
 from pypdf.annotations import FreeText
 
 
-# Fills a PDF by adding text annotations defined in `fields.json`. See FORMS.md.
 
 
 def transform_from_image_coords(bbox, image_width, image_height, pdf_width, pdf_height):
-    """Transform bounding box from image coordinates to PDF coordinates"""
-    # Image coordinates: origin at top-left, y increases downward
-    # PDF coordinates: origin at bottom-left, y increases upward
     x_scale = pdf_width / image_width
     y_scale = pdf_height / image_height
 
     left = bbox[0] * x_scale
     right = bbox[2] * x_scale
 
-    # Flip Y coordinates for PDF
     top = pdf_height - (bbox[1] * y_scale)
     bottom = pdf_height - (bbox[3] * y_scale)
 
@@ -27,67 +21,43 @@ def transform_from_image_coords(bbox, image_width, image_height, pdf_width, pdf_
 
 
 def transform_from_pdf_coords(bbox, pdf_height):
-    """Transform bounding box from pdfplumber coordinates to pypdf coordinates.
-
-    pdfplumber uses y=0 at top, y increases downward (like images).
-    pypdf FreeText expects y=0 at bottom, y increases upward.
-    Both use the same scale (PDF points), so only Y needs flipping.
-    """
     left = bbox[0]
     right = bbox[2]
 
-    # bbox is [left, top, right, bottom] where top < bottom (y=0 at top)
-    # pypdf wants [left, bottom, right, top] where bottom < top (y=0 at bottom)
-    pypdf_top = pdf_height - bbox[1]      # flip the "top" value
-    pypdf_bottom = pdf_height - bbox[3]   # flip the "bottom" value
+    pypdf_top = pdf_height - bbox[1]      
+    pypdf_bottom = pdf_height - bbox[3]   
 
     return left, pypdf_bottom, right, pypdf_top
 
 
 def fill_pdf_form(input_pdf_path, fields_json_path, output_pdf_path):
-    """Fill the PDF form with data from fields.json"""
-    input_abs = os.path.abspath(input_pdf_path)
-    output_abs = os.path.abspath(output_pdf_path)
-    if input_abs == output_abs:
-        print("ERROR: Refusing to overwrite input PDF. Use a different output path.")
-        sys.exit(1)
     
-    # `fields.json` format described in FORMS.md.
     with open(fields_json_path, "r") as f:
         fields_data = json.load(f)
     
-    # Open the PDF
     reader = PdfReader(input_pdf_path)
     writer = PdfWriter()
     
-    # Copy all pages to writer
     writer.append(reader)
     
-    # Get PDF dimensions for each page
     pdf_dimensions = {}
     for i, page in enumerate(reader.pages):
         mediabox = page.mediabox
         pdf_dimensions[i + 1] = [mediabox.width, mediabox.height]
     
-    # Process each form field
     annotations = []
     for field in fields_data["form_fields"]:
         page_num = field["page_number"]
 
-        # Get page dimensions and transform coordinates.
         page_info = next(p for p in fields_data["pages"] if p["page_number"] == page_num)
         pdf_width, pdf_height = pdf_dimensions[page_num]
 
-        # Detect coordinate system: pdf_width/pdf_height = PDF coords, image_width/image_height = image coords
         if "pdf_width" in page_info:
-            # PDF coordinates from structure extraction (pdfplumber style)
-            # Only need Y-flip, no scaling
             transformed_entry_box = transform_from_pdf_coords(
                 field["entry_bounding_box"],
                 float(pdf_height)
             )
         else:
-            # Image coordinates - need scaling and Y-flip
             image_width = page_info["image_width"]
             image_height = page_info["image_height"]
             transformed_entry_box = transform_from_image_coords(
@@ -96,7 +66,6 @@ def fill_pdf_form(input_pdf_path, fields_json_path, output_pdf_path):
                 float(pdf_width), float(pdf_height)
             )
         
-        # Skip empty fields
         if "entry_text" not in field or "text" not in field["entry_text"]:
             continue
         entry_text = field["entry_text"]
@@ -108,8 +77,6 @@ def fill_pdf_form(input_pdf_path, fields_json_path, output_pdf_path):
         font_size = str(entry_text.get("font_size", 14)) + "pt"
         font_color = entry_text.get("font_color", "000000")
 
-        # Font size/color seems to not work reliably across viewers:
-        # https://github.com/py-pdf/pypdf/issues/2084
         annotation = FreeText(
             text=text,
             rect=transformed_entry_box,
@@ -120,10 +87,8 @@ def fill_pdf_form(input_pdf_path, fields_json_path, output_pdf_path):
             background_color=None,
         )
         annotations.append(annotation)
-        # page_number is 0-based for pypdf
         writer.add_annotation(page_number=page_num - 1, annotation=annotation)
         
-    # Save the filled PDF
     with open(output_pdf_path, "wb") as output:
         writer.write(output)
     
@@ -139,4 +104,4 @@ if __name__ == "__main__":
     fields_json = sys.argv[2]
     output_pdf = sys.argv[3]
     
-    fill_pdf_form(input_pdf, fields_json, output_pdf)
+    fill_pdf_form(input_pdf, fields_json, output_pdf)

package/skills/pptx/SKILL.md

@@ -1,38 +1,11 @@
 ---
 name: pptx
-description: "Presentation creation, editing, and analysis. When Claude needs to work with presentations (.pptx files) for: (1) Creating new presentations, (2) Modifying or editing content, (3) Working with layouts, (4) Adding comments or speaker notes, or any other presentation tasks"
+description: "Use this skill any time a .pptx file is involved in any way — as input, output, or both. This includes: creating slide decks, pitch decks, or presentations; reading, parsing, or extracting text from any .pptx file (even if the extracted content will be used elsewhere, like in an email or summary); editing, modifying, or updating existing presentations; combining or splitting slide files; working with templates, layouts, speaker notes, or comments. Trigger whenever the user mentions \"deck,\" \"slides,\" \"presentation,\" or references a .pptx filename, regardless of what they plan to do with the content afterward. If a .pptx file needs to be opened, created, or touched, use this skill."
 license: Proprietary. LICENSE.txt has complete terms
 ---
 
 # PPTX Skill
 
-## IMPORTANT: Save to Desktop
-
-**Always save created `.pptx` files to `~/Desktop/`** (e.g. `~/Desktop/presentation.pptx`). Never save to the agent workspace or hidden directories — the user needs easy access to the file.
-
-## CRITICAL: Integration Priority
-
-### 1. `lemon-slides` CLI (For Google Slides)
-If user wants a Google Slides presentation (shareable, collaborative), use `lemon-slides`:
-- `lemon-slides create "Title"` - Create a new presentation
-- `lemon-slides read <id>` - Read a presentation
-
-### 2. Local PPTX (Recommended for Quality)
-For best results, create presentations locally using pptxgenjs:
-- Full control over every element
-- Professional, polished output
-- Files user can open in PowerPoint, Keynote, or upload to Google Slides
-
-### 3. Browser (LAST RESORT)
-Only if `lemon-slides` CLI fails AND user explicitly requests Google Slides in browser.
-
-## Workflow Summary (Local Creation)
-
-1. **Create slides** using pptxgenjs (JavaScript)
-2. **Generate `.pptx` file** locally
-3. **QA the output** using visual inspection
-4. **Deliver** the file to the user
-
 ## Quick Reference
 
 | Task | Guide |
@@ -161,7 +134,7 @@ Choose colors that match your topic — don't default to generic blue. Use these
 - **Don't create text-only slides** — add images, icons, charts, or visual elements; avoid plain title + bullets
 - **Don't forget text box padding** — when aligning lines or shapes with text edges, set `margin: 0` on the text box or offset the shape to account for padding
 - **Don't use low-contrast elements** — icons AND text need strong contrast against the background; avoid light text on light backgrounds or dark text on dark backgrounds
-- **NEVER use horizontal lines to seperate title and body** — use whitespace or background color instead
+- **NEVER use accent lines under titles** — these are a hallmark of AI-generated slides; use whitespace or background color instead
 
 ---
 

package/skills/pptx/editing.md

@@ -39,7 +39,7 @@ When using an existing presentation as a template:
 
 6. **Clean**: `python scripts/clean.py unpacked/`
 
-7. **Pack**: `python scripts/office/pack.py unpacked/ ~/Desktop/output.pptx --original template.pptx`
+7. **Pack**: `python scripts/office/pack.py unpacked/ output.pptx --original template.pptx`
 
 ---
 
@@ -81,7 +81,7 @@ Removes slides not in `<p:sldIdLst>`, unreferenced media, orphaned rels.
 ### pack.py
 
 ```bash
-python scripts/office/pack.py unpacked/ ~/Desktop/output.pptx --original input.pptx
+python scripts/office/pack.py unpacked/ output.pptx --original input.pptx
 ```
 
 Validates, repairs, condenses XML, re-encodes smart quotes.

package/skills/pptx/pptxgenjs.md

@@ -13,9 +13,7 @@ pres.title = 'Presentation Title';
 let slide = pres.addSlide();
 slide.addText("Hello World!", { x: 0.5, y: 0.5, fontSize: 36, color: "363636" });
 
-const os = require('os');
-const path = require('path');
-pres.writeFile({ fileName: path.join(os.homedir(), "Desktop", "Presentation.pptx") });
+pres.writeFile({ fileName: "Presentation.pptx" });
 ```
 
 ## Layout Dimensions
@@ -37,6 +35,9 @@ slide.addText("Simple Text", {
   color: "363636", bold: true, align: "center", valign: "middle"
 });
 
+// Character spacing (use charSpacing, not letterSpacing which is silently ignored)
+slide.addText("SPACED TEXT", { x: 1, y: 1, w: 8, h: 1, charSpacing: 6 });
+
 // Rich text arrays
 slide.addText([
   { text: "Bold ", options: { bold: true } },
@@ -100,8 +101,35 @@ slide.addShape(pres.shapes.RECTANGLE, {
   x: 1, y: 1, w: 3, h: 2,
   fill: { color: "0088CC", transparency: 50 }
 });
+
+// Rounded rectangle (rectRadius only works with ROUNDED_RECTANGLE, not RECTANGLE)
+// ⚠️ Don't pair with rectangular accent overlays — they won't cover rounded corners. Use RECTANGLE instead.
+slide.addShape(pres.shapes.ROUNDED_RECTANGLE, {
+  x: 1, y: 1, w: 3, h: 2,
+  fill: { color: "FFFFFF" }, rectRadius: 0.1
+});
+
+// With shadow
+slide.addShape(pres.shapes.RECTANGLE, {
+  x: 1, y: 1, w: 3, h: 2,
+  fill: { color: "FFFFFF" },
+  shadow: { type: "outer", color: "000000", blur: 6, offset: 2, angle: 135, opacity: 0.15 }
+});
 ```
 
+Shadow options:
+
+| Property | Type | Range | Notes |
+|----------|------|-------|-------|
+| `type` | string | `"outer"`, `"inner"` | |
+| `color` | string | 6-char hex (e.g. `"000000"`) | No `#` prefix, no 8-char hex — see Common Pitfalls |
+| `blur` | number | 0-100 pt | |
+| `offset` | number | 0-200 pt | **Must be non-negative** — negative values corrupt the file |
+| `angle` | number | 0-359 degrees | Direction the shadow falls (135 = bottom-right, 270 = upward) |
+| `opacity` | number | 0.0-1.0 | Use this for transparency, never encode in color string |
+
+To cast a shadow upward (e.g. on a footer bar), use `angle: 270` with a positive offset — do **not** use a negative offset.
+
 **Note**: Gradient fills are not natively supported. Use a gradient image as a background instead.
 
 ---
@@ -345,15 +373,32 @@ titleSlide.addText("My Title", { placeholder: "title" });
    color: "#FF0000"     // ❌ WRONG
    ```
 
-2. **Use `bullet: true`** - NEVER unicode symbols like "•" (creates double bullets)
+2. **NEVER encode opacity in hex color strings** - 8-char colors (e.g., `"00000020"`) corrupt the file. Use the `opacity` property instead.
+   ```javascript
+   shadow: { type: "outer", blur: 6, offset: 2, color: "00000020" }          // ❌ CORRUPTS FILE
+   shadow: { type: "outer", blur: 6, offset: 2, color: "000000", opacity: 0.12 }  // ✅ CORRECT
+   ```
+
+3. **Use `bullet: true`** - NEVER unicode symbols like "•" (creates double bullets)
 
-3. **Use `breakLine: true`** between array items or text runs together
+4. **Use `breakLine: true`** between array items or text runs together
 
-4. **Avoid `lineSpacing` with bullets** - causes excessive gaps; use `paraSpaceAfter` instead
+5. **Avoid `lineSpacing` with bullets** - causes excessive gaps; use `paraSpaceAfter` instead
 
-5. **Each presentation needs fresh instance** - don't reuse `pptxgen()` objects
+6. **Each presentation needs fresh instance** - don't reuse `pptxgen()` objects
+
+7. **NEVER reuse option objects across calls** - PptxGenJS mutates objects in-place (e.g. converting shadow values to EMU). Sharing one object between multiple calls corrupts the second shape.
+   ```javascript
+   const shadow = { type: "outer", blur: 6, offset: 2, color: "000000", opacity: 0.15 };
+   slide.addShape(pres.shapes.RECTANGLE, { shadow, ... });  // ❌ second call gets already-converted values
+   slide.addShape(pres.shapes.RECTANGLE, { shadow, ... });
+
+   const makeShadow = () => ({ type: "outer", blur: 6, offset: 2, color: "000000", opacity: 0.15 });
+   slide.addShape(pres.shapes.RECTANGLE, { shadow: makeShadow(), ... });  // ✅ fresh object each time
+   slide.addShape(pres.shapes.RECTANGLE, { shadow: makeShadow(), ... });
+   ```
 
-6. **Don't use `ROUNDED_RECTANGLE` with accent borders** - rectangular overlay bars won't cover rounded corners. Use `RECTANGLE` instead.
+8. **Don't use `ROUNDED_RECTANGLE` with accent borders** - rectangular overlay bars won't cover rounded corners. Use `RECTANGLE` instead.
    ```javascript
    // ❌ WRONG: Accent bar doesn't cover rounded corners
    slide.addShape(pres.shapes.ROUNDED_RECTANGLE, { x: 1, y: 1, w: 3, h: 1.5, fill: { color: "FFFFFF" } });

package/skills/pptx/scripts/add_slide.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 """Add a new slide to an unpacked PPTX directory.
 
 Usage: python add_slide.py <unpacked_dir> <source>
@@ -26,14 +25,12 @@ from pathlib import Path
 
 
 def get_next_slide_number(slides_dir: Path) -> int:
-    """Find the next available slide number."""
     existing = [int(m.group(1)) for f in slides_dir.glob("slide*.xml")
                 if (m := re.match(r"slide(\d+)\.xml", f.name))]
     return max(existing) + 1 if existing else 1
 
 
 def create_slide_from_layout(unpacked_dir: Path, layout_file: str) -> None:
-    """Create a new slide from a layout template."""
     slides_dir = unpacked_dir / "ppt" / "slides"
     rels_dir = slides_dir / "_rels"
     layouts_dir = unpacked_dir / "ppt" / "slideLayouts"
@@ -43,13 +40,11 @@ def create_slide_from_layout(unpacked_dir: Path, layout_file: str) -> None:
         print(f"Error: {layout_path} not found", file=sys.stderr)
         sys.exit(1)
 
-    # Auto-select destination name
     next_num = get_next_slide_number(slides_dir)
     dest = f"slide{next_num}.xml"
     dest_slide = slides_dir / dest
     dest_rels = rels_dir / f"{dest}.rels"
 
-    # 1. Create minimal slide XML that references the layout
     slide_xml = '''<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
 <p:sld xmlns:a="http://schemas.openxmlformats.org/drawingml/2006/main" xmlns:r="http://schemas.openxmlformats.org/officeDocument/2006/relationships" xmlns:p="http://schemas.openxmlformats.org/presentationml/2006/main">
   <p:cSld>
@@ -75,7 +70,6 @@ def create_slide_from_layout(unpacked_dir: Path, layout_file: str) -> None:
 </p:sld>'''
     dest_slide.write_text(slide_xml, encoding="utf-8")
 
-    # 2. Create relationships file pointing to the layout
     rels_dir.mkdir(exist_ok=True)
     rels_xml = f'''<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
@@ -83,33 +77,26 @@ def create_slide_from_layout(unpacked_dir: Path, layout_file: str) -> None:
 </Relationships>'''
     dest_rels.write_text(rels_xml, encoding="utf-8")
 
-    # 3. Add to [Content_Types].xml
     _add_to_content_types(unpacked_dir, dest)
 
-    # 4. Add relationship to presentation.xml.rels and get rId
     rid = _add_to_presentation_rels(unpacked_dir, dest)
 
-    # 5. Get next slide ID
     next_slide_id = _get_next_slide_id(unpacked_dir)
 
-    # Output
     print(f"Created {dest} from {layout_file}")
     print(f'Add to presentation.xml <p:sldIdLst>: <p:sldId id="{next_slide_id}" r:id="{rid}"/>')
 
 
 def duplicate_slide(unpacked_dir: Path, source: str) -> None:
-    """Duplicate a slide and update all references."""
     slides_dir = unpacked_dir / "ppt" / "slides"
     rels_dir = slides_dir / "_rels"
 
     source_slide = slides_dir / source
 
-    # Validate source exists
     if not source_slide.exists():
         print(f"Error: {source_slide} not found", file=sys.stderr)
         sys.exit(1)
 
-    # Auto-select destination name
     next_num = get_next_slide_number(slides_dir)
     dest = f"slide{next_num}.xml"
     dest_slide = slides_dir / dest
@@ -117,14 +104,11 @@ def duplicate_slide(unpacked_dir: Path, source: str) -> None:
     source_rels = rels_dir / f"{source}.rels"
     dest_rels = rels_dir / f"{dest}.rels"
 
-    # 1. Copy slide XML
     shutil.copy2(source_slide, dest_slide)
 
-    # 2. Copy relationships file (if exists)
     if source_rels.exists():
         shutil.copy2(source_rels, dest_rels)
 
-        # 3. Remove notes references from new rels file
         rels_content = dest_rels.read_text(encoding="utf-8")
         rels_content = re.sub(
             r'\s*<Relationship[^>]*Type="[^"]*notesSlide"[^>]*/>\s*',
@@ -133,22 +117,17 @@ def duplicate_slide(unpacked_dir: Path, source: str) -> None:
         )
         dest_rels.write_text(rels_content, encoding="utf-8")
 
-    # 4. Add to [Content_Types].xml
     _add_to_content_types(unpacked_dir, dest)
 
-    # 5. Add relationship to presentation.xml.rels
     rid = _add_to_presentation_rels(unpacked_dir, dest)
 
-    # 6. Get next slide ID
     next_slide_id = _get_next_slide_id(unpacked_dir)
 
-    # Output
     print(f"Created {dest} from {source}")
     print(f'Add to presentation.xml <p:sldIdLst>: <p:sldId id="{next_slide_id}" r:id="{rid}"/>')
 
 
 def _add_to_content_types(unpacked_dir: Path, dest: str) -> None:
-    """Add new slide to [Content_Types].xml."""
     content_types_path = unpacked_dir / "[Content_Types].xml"
     content_types = content_types_path.read_text(encoding="utf-8")
 
@@ -160,7 +139,6 @@ def _add_to_content_types(unpacked_dir: Path, dest: str) -> None:
 
 
 def _add_to_presentation_rels(unpacked_dir: Path, dest: str) -> str:
-    """Add relationship to presentation.xml.rels. Returns the new rId."""
     pres_rels_path = unpacked_dir / "ppt" / "_rels" / "presentation.xml.rels"
     pres_rels = pres_rels_path.read_text(encoding="utf-8")
 
@@ -178,7 +156,6 @@ def _add_to_presentation_rels(unpacked_dir: Path, dest: str) -> str:
 
 
 def _get_next_slide_id(unpacked_dir: Path) -> int:
-    """Get the next available slide ID for presentation.xml."""
     pres_path = unpacked_dir / "ppt" / "presentation.xml"
     pres_content = pres_path.read_text(encoding="utf-8")
     slide_ids = [int(m) for m in re.findall(r'<p:sldId[^>]*id="(\d+)"', pres_content)]
@@ -186,16 +163,9 @@ def _get_next_slide_id(unpacked_dir: Path) -> int:
 
 
 def parse_source(source: str) -> tuple[str, str | None]:
-    """Parse source argument to determine if it's a slide or layout.
-
-    Returns:
-        ("slide", None) if source is a slide file like "slide2.xml"
-        ("layout", filename) if source is a layout like "slideLayout2.xml"
-    """
     if source.startswith("slideLayout") and source.endswith(".xml"):
         return ("layout", source)
 
-    # Otherwise treat as a slide file
     return ("slide", None)
 
 

package/skills/pptx/scripts/clean.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 """Remove unreferenced files from an unpacked PPTX directory.
 
 Usage: python clean.py <unpacked_dir>
@@ -26,14 +25,12 @@ import re
 
 
 def get_slides_in_sldidlst(unpacked_dir: Path) -> set[str]:
-    """Get slide filenames referenced in presentation.xml sldIdLst."""
     pres_path = unpacked_dir / "ppt" / "presentation.xml"
     pres_rels_path = unpacked_dir / "ppt" / "_rels" / "presentation.xml.rels"
 
     if not pres_path.exists() or not pres_rels_path.exists():
         return set()
 
-    # Build rId -> slide filename mapping from presentation.xml.rels
     rels_dom = defusedxml.minidom.parse(str(pres_rels_path))
     rid_to_slide = {}
     for rel in rels_dom.getElementsByTagName("Relationship"):
@@ -43,16 +40,13 @@ def get_slides_in_sldidlst(unpacked_dir: Path) -> set[str]:
         if "slide" in rel_type and target.startswith("slides/"):
             rid_to_slide[rid] = target.replace("slides/", "")
 
-    # Get rIds from sldIdLst in presentation.xml
     pres_content = pres_path.read_text(encoding="utf-8")
     referenced_rids = set(re.findall(r'<p:sldId[^>]*r:id="([^"]+)"', pres_content))
 
-    # Return slide filenames that are in sldIdLst
     return {rid_to_slide[rid] for rid in referenced_rids if rid in rid_to_slide}
 
 
 def remove_orphaned_slides(unpacked_dir: Path) -> list[str]:
-    """Remove slides not referenced in sldIdLst."""
     slides_dir = unpacked_dir / "ppt" / "slides"
     slides_rels_dir = slides_dir / "_rels"
     pres_rels_path = unpacked_dir / "ppt" / "_rels" / "presentation.xml.rels"
@@ -63,21 +57,17 @@ def remove_orphaned_slides(unpacked_dir: Path) -> list[str]:
     referenced_slides = get_slides_in_sldidlst(unpacked_dir)
     removed = []
 
-    # Find and remove orphaned slide files
     for slide_file in slides_dir.glob("slide*.xml"):
         if slide_file.name not in referenced_slides:
-            # Remove slide file
             rel_path = slide_file.relative_to(unpacked_dir)
             slide_file.unlink()
             removed.append(str(rel_path))
 
-            # Remove slide rels file
             rels_file = slides_rels_dir / f"{slide_file.name}.rels"
             if rels_file.exists():
                 rels_file.unlink()
                 removed.append(str(rels_file.relative_to(unpacked_dir)))
 
-    # Remove relationships from presentation.xml.rels for deleted slides
     if removed and pres_rels_path.exists():
         rels_dom = defusedxml.minidom.parse(str(pres_rels_path))
         changed = False
@@ -99,7 +89,6 @@ def remove_orphaned_slides(unpacked_dir: Path) -> list[str]:
 
 
 def remove_trash_directory(unpacked_dir: Path) -> list[str]:
-    """Remove [trash] directory if it exists."""
     trash_dir = unpacked_dir / "[trash]"
     removed = []
 
@@ -115,7 +104,6 @@ def remove_trash_directory(unpacked_dir: Path) -> list[str]:
 
 
 def get_slide_referenced_files(unpacked_dir: Path) -> set:
-    """Get files referenced directly from slides."""
     referenced = set()
     slides_rels_dir = unpacked_dir / "ppt" / "slides" / "_rels"
 
@@ -138,7 +126,6 @@ def get_slide_referenced_files(unpacked_dir: Path) -> set:
 
 
 def remove_orphaned_rels_files(unpacked_dir: Path) -> list[str]:
-    """Remove .rels files for unreferenced resources."""
     resource_dirs = ["charts", "diagrams", "drawings"]
     removed = []
     slide_referenced = get_slide_referenced_files(unpacked_dir)
@@ -164,7 +151,6 @@ def remove_orphaned_rels_files(unpacked_dir: Path) -> list[str]:
 
 
 def get_referenced_files(unpacked_dir: Path) -> set:
-    """Get all files referenced in .rels files."""
     referenced = set()
 
     for rels_file in unpacked_dir.rglob("*.rels"):
@@ -183,7 +169,6 @@ def get_referenced_files(unpacked_dir: Path) -> set:
 
 
 def remove_orphaned_files(unpacked_dir: Path, referenced: set) -> list[str]:
-    """Remove files not in the referenced set."""
     resource_dirs = ["media", "embeddings", "charts", "diagrams", "tags", "drawings", "ink"]
     removed = []
 
@@ -200,7 +185,6 @@ def remove_orphaned_files(unpacked_dir: Path, referenced: set) -> list[str]:
                 file_path.unlink()
                 removed.append(str(rel_path))
 
-    # Clean up unreferenced theme files
     theme_dir = unpacked_dir / "ppt" / "theme"
     if theme_dir.exists():
         for file_path in theme_dir.glob("theme*.xml"):
@@ -208,13 +192,11 @@ def remove_orphaned_files(unpacked_dir: Path, referenced: set) -> list[str]:
             if rel_path not in referenced:
                 file_path.unlink()
                 removed.append(str(rel_path))
-                # Also remove corresponding .rels
                 theme_rels = theme_dir / "_rels" / f"{file_path.name}.rels"
                 if theme_rels.exists():
                     theme_rels.unlink()
                     removed.append(str(theme_rels.relative_to(unpacked_dir)))
 
-    # Remove orphaned notes slides
     notes_dir = unpacked_dir / "ppt" / "notesSlides"
     if notes_dir.exists():
         for file_path in notes_dir.glob("*.xml"):
@@ -237,7 +219,6 @@ def remove_orphaned_files(unpacked_dir: Path, referenced: set) -> list[str]:
 
 
 def update_content_types(unpacked_dir: Path, removed_files: list[str]) -> None:
-    """Remove Content-Type overrides for deleted files."""
     ct_path = unpacked_dir / "[Content_Types].xml"
     if not ct_path.exists():
         return
@@ -258,18 +239,14 @@ def update_content_types(unpacked_dir: Path, removed_files: list[str]) -> None:
 
 
 def clean_unused_files(unpacked_dir: Path) -> list[str]:
-    """Remove all unreferenced files from the unpacked directory."""
     all_removed = []
 
-    # Remove orphaned slides first (not in sldIdLst)
     slides_removed = remove_orphaned_slides(unpacked_dir)
     all_removed.extend(slides_removed)
 
-    # Remove [trash] directory
     trash_removed = remove_trash_directory(unpacked_dir)
     all_removed.extend(trash_removed)
 
-    # Keep cleaning until nothing more is removed
     while True:
         removed_rels = remove_orphaned_rels_files(unpacked_dir)
         referenced = get_referenced_files(unpacked_dir)