DOJ-6143 + DOJ-6529: retire the Hormozi attribution and rename slide-design to slides-generate - #36
DOJ-6143 + DOJ-6529: retire the Hormozi attribution and rename slide-design to slides-generate#36lapc506 wants to merge 2 commits into
Conversation
The slide system is no longer described by whose presentation style inspired it. `skills/slide-design/SKILL.md` already records that the decks it replaced are retired and superseded by `branded-content-base.html`, so the attribution had outlived the thing it attributed. Scope is the slide-style attribution ONLY. Both spellings were swept: the English `Hormozi-style` and the Spanish `estilo Hormozi` in README.md — the second is the one a grep for the English form silently misses. The principles the system actually runs on are kept and now stand on their own merit rather than on a name: extreme clarity, zero decoration, surgical use of colour, one idea per slide. The "industry standard" appeal to authority behind the 35-40% image-density rule is replaced by its actual reason (pure text decks read as corporate training). Deliberately NOT touched: `_research/` references naming Hormozi as a person to study, and any citation of "$100M Offers" as a source work. Those are attribution of a published book, not of a visual style.
|
@dojo-code-reviewer review |
There was a problem hiding this comment.
✅ Approved
Approved — 0 blockers, 2 P3. Confidence: 4.60/5.00.
Walkthrough
Summary of Review
This PR successfully removes obsolete references to 'Hormozi' / 'Hormozi-style' from the slide design system documentation, replacing authority-based justification with the core design principles themselves (e.g., extreme clarity, minimal decoration, and pacing rules).
Reviewed Areas & Files:
README.md: Verified command and skill descriptions.commands/slides-generate.md: Reviewed HTML generation prompts and mapping rules.skills/slide-design/SKILL.md: Checked slide design philosophy and image density guidelines.
Safety Rationale:
This is a purely documentation/markdown-based update that does not alter any code or logic, making it completely safe to merge.
Verdict:
Approved — 0 blockers, 2 P3.
🔵 P3 — Minor
commands/slides-generate.md:146— 🔵 P3 (minor) — Removing the word 'Hormozi-style' left a dangling 'standard.' word at the beginning of this line. Removing 'standard.' makes the sentence read more naturally as 'pure text decks read as corporate training'.
[pass 1]
commands/slides-generate.md:283— 🔵 P3 (minor) — Replacing 'Hormozi-style' with 'the' results in a duplicate 'the' ('enforce the the mapping rules'). Let's remove the extra 'the'.
[pass 1]
Total findings: 2 business context (2 total)
| @@ -145,7 +145,7 @@ Parse the video brief's script sections. For each section: | |||
| - **NEVER skip image generation** — if a slide describes a diagram, loop, | |||
| flowchart, process, or comparison, generate the image via Gemini API. | |||
There was a problem hiding this comment.
🔵 P3 (minor) — Removing the word 'Hormozi-style' left a dangling 'standard.' word at the beginning of this line. Removing 'standard.' makes the sentence read more naturally as 'pure text decks read as corporate training'.
[pass 1]
| For this command, expect (when a consumer like `dojo-academy` is installed): | ||
| - Structural overlays (priority ~50) — e.g. slide-design: enforce the | ||
| Hormozi-style mapping rules (35-40% image slides, alternating content / | ||
| the mapping rules (35-40% image slides, alternating content / |
There was a problem hiding this comment.
🔵 P3 (minor) — Replacing 'Hormozi-style' with 'the' results in a duplicate 'the' ('enforce the the mapping rules'). Let's remove the extra 'the'.
[pass 1]
There was a problem hiding this comment.
✅ Approved
Approved — 0 blockers, 2 P3. Confidence: 4.60/5.00.
Walkthrough
main branch directly. Under the repo's GitFlow process, branches should normally target develop before migrating to main. This warning is informational only.
Walkthrough
This PR removes obsolete references and attributions to Alex Hormozi's visual style from the slide design system across README.md, commands/slides-generate.md, and skills/slide-design/SKILL.md. The core design rules (such as the 35–40% image density rule) are kept intact but justified on their own pedagogical and stylistic merits.
Reviewed Files
README.md: Verified the complete deletion of "estilo Hormozi" in Spanish prose.commands/slides-generate.md: Audited the updated instructions for the/slides-generatecommand.skills/slide-design/SKILL.md: Audited the core slide-design principles and rules.
Safety Rationale
This is a safe, documentation-only and prompt-only change with zero impact on executable code or Layer 1 invariants.
Verdict
Approved — 0 blockers, 2 P3.
🔵 P3 — Minor
commands/slides-generate.md:148— 🔵 P3 (minor) — The word 'standard' on the next line is an orphan left over from the previous phrasing ('this is the Hormozi industry standard'). Leaving it in place results in an ungrammatical and confusing sentence: 'pure text decks read as corporate training standard.'
[pass 1]
commands/slides-generate.md:284— 🔵 P3 (minor) — A duplicate word 'the' has been introduced due to the line break ('enforce the' at the end of line 283 and 'the mapping rules' on line 284).
[pass 1]
Total findings: 2 business context (2 total)
| flowchart, process, or comparison, generate the image via Gemini API. | ||
| Text-only slides for visual concepts make the deck look unfinished. | ||
| - **35-40% of slides must be image slides** — this is the Hormozi industry | ||
| - **35-40% of slides must be image slides** — pure text decks read as corporate training |
There was a problem hiding this comment.
🔵 P3 (minor) — The word 'standard' on the next line is an orphan left over from the previous phrasing ('this is the Hormozi industry standard'). Leaving it in place results in an ungrammatical and confusing sentence: 'pure text decks read as corporate training standard.'
[pass 1]
| - Structural overlays (priority ~50) — e.g. slide-design: enforce the | ||
| Hormozi-style mapping rules (35-40% image slides, alternating content / | ||
| the mapping rules (35-40% image slides, alternating content / | ||
| statement, max 4 bullets per content slide), the `slides-` filename |
There was a problem hiding this comment.
🔵 P3 (minor) — A duplicate word 'the' has been introduced due to the line break ('enforce the' at the end of line 283 and 'the mapping rules' on line 284).
[pass 1]
The skill that backs `/slides-generate` was named `slide-design`, so the command and its skill did not share a name. Its sibling `slides-preview` already does — `commands/slides-preview.md` + `skills/slides-preview/`. This repo has ten such command/skill pairs; the slides pair was the exception. Also retires a stale paragraph in the command. "Cross-PR dependencies" said slide design may be delegated to a `slide-designer` agent that "migrates from dojo-academy in DOJ-3709". DOJ-3709 merged (PRs #18/#19), and no `slide-designer` agent exists in `agents/` — the renderer that does exist is `slides-renderer`. The paragraph named two things that resolve to nothing. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
✅ Approved
Approved — 0 blockers, 2 P3. Confidence: 4.60/5.00.
Walkthrough
main branch directly. Per GitFlow standards, features and chores should target develop before being merged into main. Since this is a minor chore and does not contain blocking issues, we can proceed, but please ensure future PRs follow the GitFlow target branch convention.
Walkthrough
This PR drops the "Hormozi" slide style attribution across the toolkit's design system documentation and commands, replacing it with voice-neutral, principles-first design guidelines (extreme clarity, zero decoration, surgical color use, and high image density). It also updates all references of the slide-design skill to its new name slides-generate to align with the core command.
Files Reviewed
README.md: Updated the/slides-generatecommand description and the skill reference fromslide-designtoslides-generate.commands/slides-generate.md: Dropped "Hormozi" references and updated references to the slide skill.skills/slides-generate/SKILL.md(renamed fromskills/slide-design/SKILL.md): Cleaned up attributions and updated name/heuristics to matchslides-generate.skills/workbook-generate/SKILL.md: Updated cross-references pointing to the renamed slide skill.
Safety Rationale
The changes are purely documentation and instruction-text cleanups that remove obsolete attributions without altering any runtime code or breaking xAPI standard invariants. Therefore, it is completely safe to merge.
Verdict
Approved — 0 blockers, 2 P3.
🔵 P3 — Minor
commands/slides-generate.md:148— 🔵 P3 (minor) — The sentence is grammatically mismatched due to the partial deletion of the original phrasing. It currently reads: "...pure text decks read as corporate training standard. For a 15-slide deck...", which is awkward. Please remove "standard." from the beginning of line 149 or adjust the phrasing to flow naturally.
[pass 1]
commands/slides-generate.md:284— 🔵 P3 (minor) — A duplicate word ("the") was introduced during the text update.
[pass 1]
Total findings: 2 business context (2 total)
| flowchart, process, or comparison, generate the image via Gemini API. | ||
| Text-only slides for visual concepts make the deck look unfinished. | ||
| - **35-40% of slides must be image slides** — this is the Hormozi industry | ||
| - **35-40% of slides must be image slides** — pure text decks read as corporate training |
There was a problem hiding this comment.
🔵 P3 (minor) — The sentence is grammatically mismatched due to the partial deletion of the original phrasing. It currently reads: "...pure text decks read as corporate training standard. For a 15-slide deck...", which is awkward. Please remove "standard." from the beginning of line 149 or adjust the phrasing to flow naturally.
[pass 1]
| Hormozi-style mapping rules (35-40% image slides, alternating content / | ||
| - Structural overlays (priority ~50) — e.g. `slides-generate`: enforce the | ||
| the mapping rules (35-40% image slides, alternating content / | ||
| statement, max 4 bullets per content slide), the `slides-` filename |
There was a problem hiding this comment.
🔵 P3 (minor) — A duplicate word ("the") was introduced during the text update.
[pass 1]
The slide system was named after whose presentation style inspired it, and its skill was not named after the command it backs. Both are the same artifact telling you the wrong thing about itself. 4 files.
1 — The attribution is retired
The repo already said so:
skills/slide-design/SKILL.mdrecords that the decks it described are retired and superseded bybranded-content-base.html. The name outlived the thing it named.The sweep covers the English
Hormozi-styleand the Spanishestilo HormoziinREADME.md. That second one is the interesting failure: a grep for the English form returns clean while the concept is still there in another language. Same idea, two spellings, only one of them findable by the obvious search.What replaces it is not a different name — it is the principles standing on their own merit: extreme clarity, zero decoration, surgical use of colour, one idea per slide. The 35-40% image-density rule kept its number and lost its justification-by-authority;
This is the Hormozi industry standardbecame the actual reason, that pure text decks read as corporate training. A rule that cites a person is only as durable as that person's relevance; a rule that cites its consequence survives.2 — The skill is renamed
slide-design→slides-generatecommands/slides-generate.mdwas backed by a skill calledslide-design. Its siblingslides-previewis backed byskills/slides-preview/, same name, no translation step.This repo has ten command/skill pairs that share a name —
course-audit,course-diff,course-retro,course-revise,course-visualize,new-course,new-path,session-plan-audit,slides-preview,workbook-generate. The slides pair was the only exception, and it showed at the runtime surface: the skill listed asinstructional-design-toolkit:slide-designwhile its command listed asinstructional-design-toolkit:slides-generate, so the two halves of one feature appeared under different names.Why these two are one PR and not two
They edit the same three files, and in
SKILL.mdthe same frontmatter block — thename:key sits four lines from thedescription:that carried the attribution. Split, they conflict with each other by construction, and the second one to land pays for the split. Together they are 4 files, which still reads in one sitting.Also fixed, because the rename walked past it
commands/slides-generate.mdcarried a "Cross-PR dependencies" section saying slide design "may delegate to aslide-designeragent" that "migrates fromdojo-academyin DOJ-3709". Both halves resolve to nothing on this ref: DOJ-3709 merged as PRs #18 and #19, andagents/contains noslide-designer— the agent that exists isslides-renderer. Rewritten as a## Delegationsection naming what actually ships.Verification
One correction to this PR's earlier body
It said
_research/material and citations of $100M Offers were deliberately not touched. That is true of the dojo-academy companion PR (DOJ-6144, 10 book citations) and vacuous here:git grep -c '100M' origin/mainreturns nothing in this repo, and there is no_research/. All nine Hormozi hits in IDT were visual-style attribution, so "delete all" is literal and admits no exception. Recorded rather than quietly deleted, because the original sentence would have let a reader believe a carve-out was applied that never existed.Closes DOJ-6143
Closes DOJ-6529
Part of DOJ-6525
Created by Claude Code on behalf of @lapc506