Skip to content

Add Latchshot public-page capture skill 🤖🤖🤖#2374

Open
BaiqingL wants to merge 1 commit into
github:mainfrom
BaiqingL:add-latchshot-page-capture-skill
Open

Add Latchshot public-page capture skill 🤖🤖🤖#2374
BaiqingL wants to merge 1 commit into
github:mainfrom
BaiqingL:add-latchshot-page-capture-skill

Conversation

@BaiqingL

Copy link
Copy Markdown

Summary

  • Add the latchshot-page-capture Agent Skill and its dependency-free Node.js helper.
  • Latchshot is a hosted third-party service maintained by this contributor. The skill keeps it optional and preserves local-browser workflows for private pages or unsupported actions.
  • Provide a constrained no-key JPEG proof; authenticated capture and quota reads use an environment-only API key.
  • Limit the workflow to public HTTP(S) pages and PNG, JPEG, or PDF artifacts. Exclude authenticated/private pages, raw HTML, extraction, arbitrary browser actions, CAPTCHA handling, proxies, and bypass behavior.
  • Keep upgrade, checkout, payment, and other commercial actions outside the skill.
  • Complement the existing local-browser screenshot skills with a hosted public-page-to-artifact workflow.

The helper uses a fixed service origin, validates URLs before requests, bounds response reads, validates MIME type and file signatures, and writes artifacts atomically. Source: https://github.com/BaiqingL/latchshot-page-capture

Validation

  • npm run skill:validate: all 387 skills valid
  • Microsoft Vally: 2/2 checks passed
  • npm run build: reproducible; generated index is current
  • bash eng/fix-line-endings.sh
  • Skill Creator quick validator: passed
  • Agent Skills reference validator: passed
  • codespell: passed
  • Node.js syntax and help smoke test: passed
  • Helper is byte-identical to the published v1.1.0 helper, whose standalone suite passes 9/9 tests

The installed GitHub CLI here is 2.89.0, below the 2.90.0 requirement for the generated gh skills command, so I did not claim a local gh skills install test.

@BaiqingL
BaiqingL requested a review from aaronpowell as a code owner July 21, 2026 08:13
@github-actions github-actions Bot added new-submission PR adds at least one new contribution skills PR touches skills labels Jul 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 3 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 1
ℹ️ Info 1
Severity Rule File Line Match
🟠 package-exec-command docs/README.skills.md 31 | [acreadiness-assess](../skills/acreadiness-assess/SKILL.md)<br />`gh skills install github/awesome-copilot acreadiness-assess` | Run the AgentRC readiness assessment on the curre
ℹ️ skill-script-touched skills/latchshot-page-capture/scripts/latchshot.mjs 1 skills/latchshot-page-capture/scripts/latchshot.mjs

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

@github-actions

github-actions Bot commented Jul 21, 2026

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 Jul 21, 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
ℹ️ npm warn EBADENGINE Unsupported engine {
ℹ️ npm warn EBADENGINE package: 'commander@15.0.0',
ℹ️ npm warn EBADENGINE required: { node: '>=22.12.0' },
ℹ️ npm warn EBADENGINE current: { node: 'v20.20.2', npm: '10.8.2' }
ℹ️ npm warn EBADENGINE }
ℹ️ npm warn deprecated prebuild-install@7.1.3: No longer maintained. Please contact the author of the relevant native addon; alternatives are available.
ℹ️ ✅ latchshot-page-capture (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.
Full linter output
### Linting skills/latchshot-page-capture
npm warn EBADENGINE Unsupported engine {
npm warn EBADENGINE   package: 'commander@15.0.0',
npm warn EBADENGINE   required: { node: '>=22.12.0' },
npm warn EBADENGINE   current: { node: 'v20.20.2', npm: '10.8.2' }
npm warn EBADENGINE }
npm warn deprecated prebuild-install@7.1.3: No longer maintained. Please contact the author of the relevant native addon; alternatives are available.
✅ latchshot-page-capture (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

@BaiqingL
BaiqingL force-pushed the add-latchshot-page-capture-skill branch from cbade68 to aaa0f88 Compare July 21, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:HIGH Contributor reputation check flagged HIGH risk new-submission PR adds at least one new contribution skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant