beautiful_deck + tikz: cross-skill consolidation, script extractions, label-clearance hardening#18
Open
nsmiller2501 wants to merge 7 commits into
Open
Conversation
…tions Cross-skill (the paired-session win): - Lift Beamer parameterized-style rule into tikz_rules.md as Rule 9 - Collapse beautiful_deck Step 4.4 (Rules 1-6, ~42L) to a pointer - tikz_rules.md is now single source of truth for both generation + audit Modal extractions in beautiful_deck (loaded only when triggered): - theme_path_b.md: Step 1 sub-steps 1.1-1.6 (original-design path) - alternative_formats.md: Q5 non-Beamer formats - rhetoric_audit_prompt.md: verbatim Step 7 sub-agent prompt - graphics_audit_prompt.md: verbatim Step 8 sub-agent prompt tikz trim: - Pass 6 debug bounding-box pass extracted to debug_bounding_box.md - Known Limitations table moved to tikz_rules.md Line counts: beautiful_deck/SKILL.md 657 -> 546; tikz/SKILL.md 124 -> 101. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Reference section (Three Laws + Aristotelian triad, ~20L) collapsed to pointer at rhetoric_of_decks.md §I, §II — duplication, not load-bearing - Step 6 /tikz invocation trimmed to short pointer (the /tikz skill describes its own protocol; restating it here was duplication) - Delete .claude/skills/beautiful_deck/rhetoric_of_decks_full_essay.md. Identical copy already exists at deprecated/presentations/. Update both READMEs to reference the archived path. Line counts: beautiful_deck/SKILL.md 546 -> 507. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-on to #9. Continues the paired-skill work: extracts deterministic scripts out of both
/beautiful_deckand/tikz, hardens/tikzlabel clearance, and consolidates cross-skill duplication so the two skills share a single source of truth.What changed
Script extractions
.claude/skills/tikz/scripts/audit_passes.sh: the shell-based pre-check / picture-count / candidate-section indexer that previously lived inline in SKILL.md./tikznow invokes it at Step 2 (audit) and Step 5 (recompile verification)..claude/skills/beautiful_deck/scripts/compile_loop.sh: the deterministicpdflatexcompile-and-classify loop that previously lived inline in/beautiful_deck's compile step. Returns a structured exit signal so the orchestrator can decide whether to re-run or escalate.Both skills'
allowed-toolslists have been augmented with the new script paths.Cross-skill audit (commit
d088e8d-derived)/beautiful_deckStep 4.4; the canonical formulas / clearance tables now live only in/tikz/tikz_rules.mdand are referenced from both skills.theme_path_b.md,alternative_formats.md,graphics_audit_prompt.md, andrhetoric_audit_prompt.mdmaterial out ofSKILL.mdinto modal sibling files that load on demand.rhetoric_of_decks_full_essay.mdfrom.claude/skills/beautiful_deck/(the canonical lives atpresentations/rhetoric_of_decks_full_essay.md)./tikzlabel-clearance hardening (commit7d46e16-derived)tikz_rules.md: minimum edge-label-to-arrow gap raised, "label touches arrow shaft" promoted from advisory to hard failure./beautiful_decktrim (commit7834716-derived)/tikzafter the cross-skill consolidation.Why
/beautiful_deckwrites safe TikZ at generation time (Step 4.4),/tikzaudits residual issues at the end. Bundling them as one follow-on PR mirrors how Revise /beautiful_deck and /tikz as a shared deck-quality workflow #9 paired them.Testing
/beautiful_deckend-to-end against a small academic-seminar deck and confirmedcompile_loop.shproduces the same classify-and-loop behavior as the prior inline steps./tikzagainst a deck with known label/arrow collisions and confirmedaudit_passes.shsurfaces the same candidate sections as the prior inline shell snippets.beautiful_deckStep 4.4 still resolves thetikz_rules.mdreference correctly after the consolidation.