GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||
GeezerVison is showing github.com — feat(capturekit): add CaptureKit integration by josselinlbe · Pull Request #9834 · simstudioai/sim ConversationSummaryAdds a CaptureKit integration: tools, block, icon and generated docs page. I work on CaptureKit, so this PR comes from the CaptureKit team. Capture any web page as a PNG, JPEG, WebP image or a PDF, by viewport, full page or CSS selector on 17+ emulated devices; extract a page's metadata, links and Markdown; fetch raw HTML; and get an AI summary of what a website offers. Operations:
Credentials are a CaptureKit API key on the block ( Product update (if relevant)Marketing, compliance and research teams can archive pages, track competitor pages, and turn websites into clean input for agents. The block appears in the integrations catalog with 7 templates and grounded skills; the docs page is Type of Change
Testing
Checklist
Screenshots/Videos
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Add a CaptureKit block for web page capture and content: - Take a screenshot of a page as PNG, JPEG, WebP or PDF, full page or by CSS selector, on 17+ emulated devices, stored as a file - Extract a page's metadata, internal, external and social links, and its Markdown - Fetch a page's raw HTML without a browser - Get an AI summary of what a website offers, its audience and contacts Outputs follow the documented OpenAPI schemas and were checked against live responses. Credentials are a CaptureKit API key sent in the x-api-key header.
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
[Medium impact] The changes affect workflows that choose to use the new CaptureKit integration. No blocking behavior defect was established, but the two explicit repository requirements must be satisfied before merging. FindingsSummaryAdds CaptureKit with four API-key-backed operations:
Two repository requirements remain: keep single-consumer helpers beside their caller and document the new exported icon functions. Reviews (1) · Last reviewed commit: "feat(capturekit): add CaptureKit integra..." · Reviewed by Greptile |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| /** Reads a `CaptureKitPageMetadata` from an untyped CaptureKit API payload. */ | ||
| export function mapCaptureKitPageMetadata(value: unknown): CaptureKitPageMetadata | null { |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Helpers live apart from caller
Both exported helpers in utils.ts are used only by extract_content.ts. The repository requires helpers with one consumer to stay inline. Move both functions into extract_content.ts and remove this extra module before merging.
Context Used: CLAUDE.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 7c1ebb8. Both helpers moved into extract_content.ts, their only consumer, and tools/capturekit/utils.ts is removed.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| ) | ||
| } | ||
|
|
||
| export function CaptureKitIcon(props: SVGProps<SVGSVGElement>) { |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Public icons lack required docs
The new CaptureKitIcon exports here and in apps/docs/components/icons.tsx lack TSDoc. The repository requires TSDoc on exported APIs. Add a short declaration comment above both functions before merging.
Context Used: CLAUDE.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 7c1ebb8. Both CaptureKitIcon exports, in apps/sim/components/icons.tsx and apps/docs/components/icons.tsx, now have a TSDoc line.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
- Keep the two response mappers in extract_content.ts, their only caller - Document the CaptureKitIcon export - Read the operation from the mapper context so agent tools coerce the screenshot's numeric inputs too
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.