Skip to content

Add pptx-denote skill to remove all slide notes from a .pptx file#2045

Open
justinyoo wants to merge 82 commits into
github:stagedfrom
justinyoo:skills/pptx-denote
Open

Add pptx-denote skill to remove all slide notes from a .pptx file#2045
justinyoo wants to merge 82 commits into
github:stagedfrom
justinyoo:skills/pptx-denote

Conversation

@justinyoo

@justinyoo justinyoo commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, workflow, or canvas extension file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, workflow, or canvas extension with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the staged branch for this pull request.

Note: The "tested with GitHub Copilot" box is left unchecked pending the author's confirmation that the skill has been exercised end-to-end with Copilot.


Description

Adds a new pptx-denote skill that removes all slide notes from a PowerPoint (.pptx) presentation using the open-source python-pptx library.

Given an input .pptx file, the skill clears the notes text from every slide and writes a new file. When no output path is supplied, it defaults to <input>-denoted.pptx. No paid services are involved.

Files added:

  • skills/pptx-denote/SKILL.md
  • skills/pptx-denote/scripts/remove_notes.py

Validation:

  • npm run skill:validate -> all skills valid (including pptx-denote).
  • npm run build -> docs/README.skills.md regenerated with the new entry.

Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • New canvas extension.
  • Update to existing instruction, prompt, agent, plugin, skill, workflow, or canvas extension.
  • Other (please specify):

Additional Notes

The skill requires python-pptx to be installed (pip install python-pptx), as documented in SKILL.md.


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings June 18, 2026 06:56
@github-actions github-actions Bot added branched-main PR appears to include plugin files materialized from main external-plugin PR updates plugins/external.json ready-for-review Submission passed intake validation and is ready for maintainer review labels Jun 18, 2026
@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

✅ External plugin PR checks passed

  • Changed entries detected: 0
  • Workflow state label: ready-for-review

Per-plugin quality summary

Plugin skill-validator install smoke test overall source tree
none not_run not_run not_run n/a

No changed external plugin entries were detected in this PR.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@github-actions github-actions Bot removed the external-plugin PR updates plugins/external.json label Jun 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 14 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 4
ℹ️ Info 1
Severity Rule File Line Match
🟠 package-exec-command docs/README.skills.md 31 | [acreadiness-assess](../skills/acreadiness-assess/SKILL.md)&lt;br /&gt;`gh skills install github/awesome-copilot acreadiness-assess` | Run the AgentRC readiness assessment on the curre
🟠 unpinned-version-indicator plugins/aws-cloud-development/skills/aws-cost-optimize/SKILL.md 110 High: Score &gt; 20 | Medium: Score 5-20 | Low: Score &lt; 5
🟠 unpinned-version-indicator plugins/aws-cloud-development/skills/aws-resource-health-diagnose/SKILL.md 97 --query-string 'filter @​​type = "REPORT" | filter @​​initDuration &gt; 0 | stats count() as coldStarts by bin(1h)'
🟠 unpinned-version-indicator plugins/aws-cloud-development/skills/aws-resource-health-diagnose/SKILL.md 113 - **High**: Performance degradation, error rates &gt;5%, intermittent failures
ℹ️ skill-script-touched skills/pptx-denote/scripts/remove_notes.py 1 skills/pptx-denote/scripts/remove_notes.py

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions github-actions Bot added the skill-check-warning Skill validator reported warnings label Jun 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Skill Validator Results

⚠️ Warnings or advisories found

Scope Checked
Skills 8
Agents 0
Total 8
Severity Count
❌ Errors 0
⚠️ Warnings 5
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ Found 8 skill(s)
ℹ️ [aws-cost-optimize] 📊 aws-cost-optimize: 2,004 BPE tokens [chars/4: 2,047] (detailed ✓), 19 sections, 6 code blocks
ℹ️ [aws-resource-health-diagnose] 📊 aws-resource-health-diagnose: 1,695 BPE tokens [chars/4: 1,743] (detailed ✓), 31 sections, 5 code blocks
ℹ️ [aws-resource-query] 📊 aws-resource-query: 5,900 BPE tokens [chars/4: 5,521] (comprehensive ✗), 154 sections, 23 code blocks
ℹ️ [aws-resource-query] ⚠ Skill is 5,900 BPE tokens (chars/4 estimate: 5,521) — "comprehensive" skills hurt performance by 2.9pp on average. Consider splitting into 2–3 focused skills.
ℹ️ [aws-well-architected-review] 📊 aws-well-architected-review: 1,860 BPE tokens [chars/4: 1,890] (detailed ✓), 24 sections, 4 code blocks
ℹ️ [from-the-other-side-anitta] 📊 from-the-other-side-anitta: 788 BPE tokens [chars/4: 999] (detailed ✓), 15 sections, 0 code blocks
ℹ️ [from-the-other-side-anitta] ⚠ No code blocks — agents perform better with concrete snippets and commands.
ℹ️ [from-the-other-side-quinn] 📊 from-the-other-side-quinn: 559 BPE tokens [chars/4: 695] (detailed ✓), 12 sections, 0 code blocks
ℹ️ [from-the-other-side-quinn] ⚠ No code blocks — agents perform better with concrete snippets and commands.
Full validator output
Found 8 skill(s)
[aws-cost-optimize] 📊 aws-cost-optimize: 2,004 BPE tokens [chars/4: 2,047] (detailed ✓), 19 sections, 6 code blocks
[aws-resource-health-diagnose] 📊 aws-resource-health-diagnose: 1,695 BPE tokens [chars/4: 1,743] (detailed ✓), 31 sections, 5 code blocks
[aws-resource-query] 📊 aws-resource-query: 5,900 BPE tokens [chars/4: 5,521] (comprehensive ✗), 154 sections, 23 code blocks
[aws-resource-query]    ⚠  Skill is 5,900 BPE tokens (chars/4 estimate: 5,521) — "comprehensive" skills hurt performance by 2.9pp on average. Consider splitting into 2–3 focused skills.
[aws-well-architected-review] 📊 aws-well-architected-review: 1,860 BPE tokens [chars/4: 1,890] (detailed ✓), 24 sections, 4 code blocks
[from-the-other-side-anitta] 📊 from-the-other-side-anitta: 788 BPE tokens [chars/4: 999] (detailed ✓), 15 sections, 0 code blocks
[from-the-other-side-anitta]    ⚠  No code blocks — agents perform better with concrete snippets and commands.
[from-the-other-side-quinn] 📊 from-the-other-side-quinn: 559 BPE tokens [chars/4: 695] (detailed ✓), 12 sections, 0 code blocks
[from-the-other-side-quinn]    ⚠  No code blocks — agents perform better with concrete snippets and commands.
[from-the-other-side-wiggins] 📊 from-the-other-side-wiggins: 776 BPE tokens [chars/4: 981] (detailed ✓), 16 sections, 0 code blocks
[from-the-other-side-wiggins]    ⚠  No code blocks — agents perform better with concrete snippets and commands.
[pptx-denote] 📊 pptx-denote: 235 BPE tokens [chars/4: 261] (compact ✓), 2 sections, 2 code blocks
[pptx-denote]    ⚠  No numbered workflow steps — agents follow sequenced procedures more reliably.
✅ All checks passed (8 skill(s))

@justinyoo

Copy link
Copy Markdown
Contributor Author

@aaronpowell I'm not sure why this PR has included the ones other than mine. Should I close this and recreate another one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branched-main PR appears to include plugin files materialized from main ready-for-review Submission passed intake validation and is ready for maintainer review skill-check-warning Skill validator reported warnings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants