Skip to content

DOJ-6143 + DOJ-6529: retire the Hormozi attribution and rename slide-design to slides-generate - #36

Open
lapc506 wants to merge 2 commits into
mainfrom
chore/drop-hormozi-slide-attribution
Open

DOJ-6143 + DOJ-6529: retire the Hormozi attribution and rename slide-design to slides-generate#36
lapc506 wants to merge 2 commits into
mainfrom
chore/drop-hormozi-slide-attribution

Conversation

@lapc506

@lapc506 lapc506 commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

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.md records that the decks it described are retired and superseded by branded-content-base.html. The name outlived the thing it named.

The sweep covers the English Hormozi-style and the Spanish estilo Hormozi in README.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 standard became 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-designslides-generate

commands/slides-generate.md was backed by a skill called slide-design. Its sibling slides-preview is backed by skills/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 as instructional-design-toolkit:slide-design while its command listed as instructional-design-toolkit:slides-generate, so the two halves of one feature appeared under different names.

git mv skills/slide-design skills/slides-generate
git grep -c 'slide-design'   # 12 refs across 4 files, all updated
git grep 'slide-design'      # after: zero

Why these two are one PR and not two

They edit the same three files, and in SKILL.md the same frontmatter block — the name: key sits four lines from the description: 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.md carried a "Cross-PR dependencies" section saying slide design "may delegate to a slide-designer agent" that "migrates from dojo-academy in DOJ-3709". Both halves resolve to nothing on this ref: DOJ-3709 merged as PRs #18 and #19, and agents/ contains no slide-designer — the agent that exists is slides-renderer. Rewritten as a ## Delegation section naming what actually ships.

Verification

grep -rn 'slide-design' . --exclude-dir=.git   # ZERO
grep -rni 'hormozi'    . --exclude-dir=.git    # ZERO (only .git gitdir path matches)

python3 scripts/ci/check_json_schemas.py       # 4 file(s) valid
python3 scripts/ci/check_frontmatter.py        # 73 file(s) valid, 2 skipped
python3 scripts/ci/check_agent_references.py   # 20 agent(s) indexed, all resolve

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/main returns 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

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.
@lapc506

lapc506 commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author

@dojo-code-reviewer review

@dojo-code-reviewer dojo-code-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ 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.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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 /

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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]

@dojo-code-reviewer dojo-code-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Approved

Approved — 0 blockers, 2 P3. Confidence: 4.60/5.00.

Walkthrough

⚠️ Governance Warning: This PR targets the 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-generate command.
  • 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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>

@dojo-code-reviewer dojo-code-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Approved

Approved — 0 blockers, 2 P3. Confidence: 4.60/5.00.

Walkthrough

⚠️ Governance Warning: This pull request targets the 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-generate command description and the skill reference from slide-design to slides-generate.
  • commands/slides-generate.md: Dropped "Hormozi" references and updated references to the slide skill.
  • skills/slides-generate/SKILL.md (renamed from skills/slide-design/SKILL.md): Cleaned up attributions and updated name/heuristics to match slides-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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 P3 (minor) — A duplicate word ("the") was introduced during the text update.

[pass 1]

@lapc506 lapc506 changed the title DOJ-6143: drop the Hormozi attribution from the slide design system DOJ-6143 + DOJ-6529: retire the Hormozi attribution and rename slide-design to slides-generate Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant