Add Gemini CLI Agent Skill support (one-command install)#2
Add Gemini CLI Agent Skill support (one-command install)#2Andrei-WongE wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a Gemini CLI–compatible “academic-pptx” skill packaging to make the repository installable via gemini skills install, while preserving the legacy Claude Code layout.
Changes:
- Introduces
gemini/skill entrypoint and reference docs for Gemini CLI. - Preserves a parallel
claude/folder with the original flat layout. - Updates repository-level README and adds MIT
LICENSE.
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
gemini/SKILL.md |
Gemini skill entrypoint and academic presentation rules/QA checklist. |
gemini/references/content_guidelines.md |
Detailed content/argument guidelines for academic decks. |
gemini/references/slide_patterns.md |
Slide-by-slide implementation patterns and examples. |
claude/SKILL.md |
Claude variant entrypoint; updates metadata (incl. license). |
claude/content_guidelines.md |
Claude variant of content guidelines. |
claude/slide_patterns.md |
Claude variant of slide patterns/examples. |
README.md |
New Gemini-focused install/usage docs and dual-platform structure explanation. |
LICENSE |
Adds MIT license text. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```bash | ||
| gemini skills install https://github.com/Andrei-WongE/academic-pptx-skill | ||
| ``` |
There was a problem hiding this comment.
The Quick Install command uses a different GitHub URL than the one referenced in the PR description. Please ensure the install command points to the canonical repository URL for this project so gemini skills install ... works as documented.
| ### Avoid (Academic-Specific) | ||
|
|
||
| - **No decorative icons** — icons in colored circles, stock images, clip art are inappropriate for analytical academic presentations. | ||
| - **No accent lines under titles** — use whitespace instead. |
There was a problem hiding this comment.
This guidance says “No accent lines under titles,” but the referenced slide patterns include divider/underline shapes directly under titles. Please reconcile these docs (either allow subtle divider rules, or update the patterns to match the ‘whitespace instead’ rule) so the skill doesn’t give conflicting instructions.
| - **No accent lines under titles** — use whitespace instead. | |
| - **No decorative accent lines directly under titles** — rely on whitespace for hierarchy. Simple, thin divider rules that are part of a clean template and align with the grid are acceptable, but avoid heavy or ornamental underlines. |
| }); | ||
|
|
||
| // Author and affiliation | ||
| slide.addText("Jane Smith¹ · John Doe²\n¹ University of X ²aboratory for Y", { |
There was a problem hiding this comment.
Typo in the example affiliation text: "²aboratory" is missing the leading "L" (should be "² Laboratory").
| slide.addText("Jane Smith¹ · John Doe²\n¹ University of X ²aboratory for Y", { | |
| slide.addText("Jane Smith¹ · John Doe²\n¹ University of X ² Laboratory for Y", { |
| { text: "a significant earnings premium is detectable at age 35, 30 years after treatment.", options: { breakLine: true, breakLine: true } }, | ||
| { text: "2. Effects are largest for the most disadvantaged: ", options: { bold: true, breakLine: false } }, | ||
| { text: "the Q1 earnings premium (28%) is 3.5× the Q4 effect (8%).", options: { breakLine: true, breakLine: true } }, |
There was a problem hiding this comment.
In the JavaScript example, the options object repeats the same key (breakLine) twice. Duplicate keys are legal but confusing and can trigger lint/type-check issues when copied into real code; remove the duplicate so the intent is unambiguous.
| { text: "a significant earnings premium is detectable at age 35, 30 years after treatment.", options: { breakLine: true, breakLine: true } }, | |
| { text: "2. Effects are largest for the most disadvantaged: ", options: { bold: true, breakLine: false } }, | |
| { text: "the Q1 earnings premium (28%) is 3.5× the Q4 effect (8%).", options: { breakLine: true, breakLine: true } }, | |
| { text: "a significant earnings premium is detectable at age 35, 30 years after treatment.", options: { breakLine: true } }, | |
| { text: "2. Effects are largest for the most disadvantaged: ", options: { bold: true, breakLine: false } }, | |
| { text: "the Q1 earnings premium (28%) is 3.5× the Q4 effect (8%).", options: { breakLine: true } }, |
Makes this repo directly installable as a Gemini CLI skill:
Main modifications:
gemini skills install https://github.com/Gabberflast/academic-pptx-skill now works
Tested on Gemini CLI v 0.31.0