Round 3: outline groups once, empty placeholders, table styles and flags

[uvrt]

Production feedback round 3, items D3, A3 and A4.

D3, the outline lists every grouped shape twice. slide.shapes lists a group and then its members; the outline walked the groups as well. It now walks the slide's own shapes and their groups once and marks each group around its blocks, <!-- 256.5 group "Agenda item 1" --> ... <!-- /256.5 --> (docx-agent's closing form), nested as the groups are. ppt_read_slides outline, outline_blocks, find_text and replace_text follow. The other readers of slide.shapes (fit, collisions, layout and design facts, comments) already deduplicated; a new test holds them to the same facts grouped or not. Repro d3_group_outline.py: BUG PRESENT -> fixed.

A3, empty placeholders. slide.empty_placeholders() and slide.delete_empty_placeholders(titles=False) (one undo step). Empty: a p:sp placeholder with no text body, no paragraphs, or only whitespace/empty paragraphs, and no field. A placeholder that took a picture, table, chart or SmartArt (a p:pic/p:graphicFrame) is not empty. Date, footer, slide number and header are left out, and so is a hidden shape. Titles and subtitles count (they show "Click to add title/subtitle"), but deleting keeps an empty title unless asked: a title names the slide in navigation and to a screen reader. The tools report empty_placeholders: [{shape, type, idx}] in every changing call's checks and in check (with fit, the default), and save_document lists the ones left under unresolved. ppt_manage_slides gains delete_empty_placeholders.

A4, table styles and Table Design flags. deck.table_styles -> [{id, name, builtin}] lists the deck's own styles (tableStyles.xml) and then PowerPoint's 74 built-in styles (GUIDs and names from pptx2svg's catalogue, measured in PowerPoint). The default is marked. table.style can be read, and set by name or id. table.flags and table.set_flags(...) cover the six flags. add_table(style=) also takes a name and refuses an id that neither the deck nor PowerPoint knows. Tools: ppt_add_table style, ppt_edit_table set_style/set_flags, style and flags in table results, and describe theme.table_styles.

Budget: core unchanged (3,398 estimated, about 5,029 counted, budget 5,500). All definitions 10,269 -> 10,464 (guard 10,500, stays). Strict tool set unchanged.

Golden p14: a chart slide on "Title and Content" with its empty body deleted, and a table in the template's own style ("Harbor Lane Table", added to the business review's tableStyles.xml by the input builder) with banding off. 10 calls, 18/18 checks.

PowerPoint verification (PowerPoint 16, Dutch UI; PDF exports and Normal-view screenshots):

• An empty body placeholder (no text, or one empty paragraph) shows "Klikken om tekst toe te voegen" ("Click to add text") with the content icons in edit view. The PDF draws no prompt.
• A whitespace-only body shows no prompt but draws a bullet, both in edit view and in the PDF. It stays reported: it is a stray bullet.
• Several empty paragraphs show nothing at all.
• An empty placeholder's own fill is drawn in the PDF. So delete_empty_placeholders now keeps a placeholder that draws a fill or outline (its own or its layout's) and reports it as kept_drawn (commit 8f3e17c).
• Built-in table styles: when PowerPoint saves a deck using Light Style 2 - Accent 3, Medium Style 2 - Accent 2 and Dark Style 1 - Accent 1, tableStyles.xml keeps only def and the deck's own style. Built-ins are referenced by GUID only, as implemented. PowerPoint writes the flag attributes in the same order as set_flags.
• The p14 output exports with no prompt or repair dialog. It shows the branded header and no banding. Its table-slide title wrapped to two lines in PowerPoint, so the title is shorter now and check_p14 checks one line as drawn.
• The renderer gaps this found are fixed separately in Empty placeholder boxes, Light Style 2 lines, a deck style's unstated edges (measured) pptx2svg#24.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WCxhY1fvWQPiWCwz7SYUVh