Skip to content

Add curlshot skill for web page screenshots, PDFs and HTML rendering 🤖🤖🤖 - #5125

Open
SergiuPogor wants to merge 1 commit into
github:mainfrom
SergiuPogor:add-curlshot-skill
Open

SergiuPogor wants to merge 1 commit into
github:mainfrom
SergiuPogor:add-curlshot-skill

Conversation

@SergiuPogor

Copy link
Copy Markdown

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 main branch for this pull request.

Description

Adds one skill:

  • skills/curlshot/SKILL.md: capturing a web page as an image (PNG, JPEG, WebP), a PDF, or a scrolling video (MP4, WebM, GIF) with one HTTP request to the CurlShot screenshot API, plus rendering local HTML or Markdown to an image or PDF via a POST body. It covers the curl -G --data-urlencode call pattern, a table of the common options (full page, device presets, viewport, dark mode, element selector, cookie-banner/ad blocking, wait conditions, cache), how errors come back (JSON error_code / error_message, unknown options rejected rather than ignored), and the hosted MCP server as an alternative for MCP-capable agents.

Requirements and limitations are stated in the skill: it is a hosted, paid service with a free tier (100 screenshots/month, no card; paid plans from $19/month), it needs an access key in CURLSHOT_API_KEY, the key must never be printed or committed, and localhost/private-network URLs are refused (send the page HTML instead). Links point only to the docs, the signup page needed to get a key, and the MIT-licensed source repo the skill comes from (https://github.com/SergiuPogor/curlshot-mcp).

Disclosure: I build CurlShot.


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

npm run skill:validate passes (all 436 skills valid, including curlshot), npx @microsoft/vally-cli lint skills/curlshot passes (2/2 checks), and npm run build regenerated docs/README.skills.md (one added row, included in this commit). Line endings normalized with bash eng/fix-line-endings.sh. The skill's API examples have not been run through GitHub Copilot itself, hence the unchecked box above. This PR was prepared with an AI coding agent.


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

🤖 Generated with Claude Code

Adds skills/curlshot: capture web pages (PNG/JPEG/WebP, PDF, scrolling
video) and render HTML/Markdown via the CurlShot HTTP API or its hosted
MCP server. README skills table regenerated with npm run build.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@SergiuPogor
SergiuPogor requested review from a team as code owners October 16, 2026 15:25
@github-actions github-actions Bot added new-submission PR adds at least one new contribution skills PR touches skills awaiting-automation merge-risk:medium labels Oct 16, 2026
@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:high — Privileged execution, automation, or review-policy change
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access, including a maintainer with admin or maintain permission.

Why this tier
  • Label needs-review:HIGH flags a high contributor-risk signal

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ✅ Passed Passed · logs
Skill validation ✅ Passed Passed · logs
Skill lint (vally) ✅ Passed Passed · logs
Risk scan ✅ Passed Passed · logs
Contributor reputation ✅ Passed Passed · logs
Duplicate resource scan ✅ Passed Passed · logs
PR quality signal ⏭️ Skipped Skipped by its workflow · logs

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s); an approval from a maintainer with admin or maintain permission
  • The core-maintainers pool is not staffed yet; an approver with admin or maintain permission is required instead.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for fa8ea2c · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 1 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

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

@github-actions

Copy link
Copy Markdown
Contributor

🔴 Contributor Reputation Check: HIGH risk

Check Risk
Profile HIGH
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

@github-actions github-actions Bot added the needs-review:HIGH Contributor reputation check flagged HIGH risk label Oct 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

✅ All checks passed

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

Summary

Level Finding
ℹ️ ✅ curlshot (2/2 checks passed)
ℹ️ ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️ ✓ spec-compliance: All spec checks passed.
ℹ️ ✓ [valid-refs] All file references across 1 skill(s) are valid.
ℹ️ ✓ valid-refs: All file references resolve to existing files within the skill directory.
ℹ️ 1 skill(s) linted, 1 passed
Full linter output
### Linting skills/curlshot
✅ curlshot (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

@github-actions github-actions Bot added merge-risk:high ready-for-review Submission passed intake validation and is ready for maintainer review and removed merge-risk:medium awaiting-automation labels Oct 16, 2026

This branch has not been deployed

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

Labels

merge-risk:high needs-review:HIGH Contributor reputation check flagged HIGH risk new-submission PR adds at least one new contribution ready-for-review Submission passed intake validation and is ready for maintainer review skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant