8000 feat(clipper): Edge build, Edge Add-ons package and Edge smoke test by adeelahmad · Pull Request #285 · adeelahmad/lense · GitHub
[go: up one dir, main page]

Skip to content

feat(clipper): Edge build, Edge Add-ons package and Edge smoke test - #285

Merged
adeelahmad merged 3 commits into
mainfrom
claude/project-thread-sr7fzu
Oct 10, 2026
Merged

adeelahmad merged 3 commits into
mainfrom
claude/project-thread-sr7fzu

Conversation

@adeelahmad
@adeelahmad adeelahmad commented Oct 10, 2026 •
Copy link
Copy Markdown
Owner

Requested by Adeel · project thread

Before: the web clipper (#277) built only for Chrome, and nothing checked it in Edge.

After: every run and release also produces lens-clipper-edge.zip, which you can load unpacked in Edge or upload to Edge Add-ons as it is. CI loads that build into real Microsoft Edge and clips a page into a stand-in Lens.

How: extensions/browser/targets/edge.json is the Edge manifest overlay, which the existing Browser extension workflow builds and zips with the other targets. Store publishing stays in that workflow (#294). extensions/browser/edge/smoke.mjs (Playwright core, msedge channel, under xvfb) checks the options page and popup render, then clips an article and checks the result: front matter title and author, list and code block, footer dropped, and the push carries the hook token and the Markdown. .github/workflows/clipper-edge.yml runs it and installs Edge if the runner lacks it. I ran it locally against Edge 155 and Chromium, and all checks pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JdVNcZL5BUSx3iwStCtq3U

A MV3 extension in extensions/browser: clip the page (Readability), the
selection or a picked element to Markdown with MarkDownload-style
templates, and send it to Lens through a namespace's import webhook.
Shared core for the Firefox, Edge and Safari targets; CI builds a zip
per target and attaches them to releases.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018uA6PhZXrmpgsbfDh2W23u
…t in CI

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JdVNcZL5BUSx3iwStCtq3U
@adeelahmad adeelahmad self-assigned this Oct 10, 2026
Base automatically changed from claude/project-thread-sin0jk to main October 10, 2026 20:18
@adeelahmad
adeelahmad merged commit ca81b32 into main Oct 10, 2026
11 checks passed
@adeelahmad
adeelahmad deleted the claude/project-thread-sr7fzu branch October 10, 2026 20:46
adeelahmad added a commit that referenced this pull request Oct 10, 2026
…308)

#296, #291, #285 and #294 merged after the README refresh, so they move from
"In progress" to the feature list, and the web clipper guide's status names
Edge as built.


Claude-Session: https://claude.ai/code/session_01HDx7PPUauyjMn1LZs8JAsS

Co-authored-by: Adeel Ahmad <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0