Skip to content

refactor: move PPT/share guidance into references/#98

Open
lukesw55 wants to merge 1 commit into
zarazhangrui:mainfrom
lukesw55:refactor/skill-references
Open

refactor: move PPT/share guidance into references/#98
lukesw55 wants to merge 1 commit into
zarazhangrui:mainfrom
lukesw55:refactor/skill-references

Conversation

@lukesw55

@lukesw55 lukesw55 commented Jul 2, 2026

Copy link
Copy Markdown

Proposal, third item from #95. This one changes how the skill is organized, so feel free to push back on structure or naming.

SKILL.md drops from 380 to 308 lines by moving two blocks of detail into on-demand files:

  • references/ppt-conversion.md — the detailed PPT conversion workflow and JSON interpretation guidance
  • references/share-export.md — deploy/PDF gotchas, troubleshooting and delivery messaging

SKILL.md keeps the core loop for each phase plus a one-line pointer to the reference file, following the same progressive-disclosure pattern the skill already uses for STYLE_PRESETS.md and the bold template pack: the always-loaded instructions stay lean, and the agent reads the detail only in the phase that needs it.

No guidance was dropped; the moved content is verbatim-or-tightened in the reference files. Two small additions while moving:

  • an explicit "do not deploy publicly without user approval" line in the share flow
  • guidance to split a too-dense source slide into multiple HTML slides instead of shrinking content

If this lands, I have a small follow-up: a stdlib-only validation script plus CI workflow that checks internal links, frontmatter, and drift between the root skill and the plugins/ copy. I'll open it separately since it depends on this structure.

Root and plugins/ copies updated together.

Keep the always-loaded SKILL.md lean (380 -> 308 lines) and load
conversion and share/export detail on demand, following the skill
progressive-disclosure pattern. Moved content is preserved in
references/ppt-conversion.md and references/share-export.md.
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.

2 participants