Skip to content

Safari web clipper (macOS, Mac App Store)#643

Draft
h4yfans wants to merge 5 commits into
mainfrom
safari-clipper
Draft

Safari web clipper (macOS, Mac App Store)#643
h4yfans wants to merge 5 commits into
mainfrom
safari-clipper

Conversation

@h4yfans

@h4yfans h4yfans commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Safari Web Clipper (macOS → Mac App Store)

Adds a Safari Web Extension target for the existing web clipper. Reuses the WXT bundle unchanged; the only product code change is allowing Safari's extension-origin scheme on the desktop capture server.

Spec: docs/superpowers/specs/2026-07-01-safari-web-clipper-design.md
Plan: docs/superpowers/plans/2026-07-01-safari-web-clipper.md

Done (code)

  • Task 1safari-web-extension:// added to the capture-server origin allowlist (EXTENSION_ORIGIN_PREFIXES, single source of truth). New isExtensionOrigin tests (Safari/Chrome/Firefox accept, web-origin/undefined reject). 26 main tests pass.
  • Task 2 — WXT Safari MV3 build scripts (dev:safari/build:safari/zip:safari). build:safari verified: .output/safari-mv3/manifest.json is MV3, no WXT warnings, all five risky manifest keys pass through. README build section added.

Remaining (macOS-only, draft until done)

  • Task 3safari-web-extension-packager → committed Xcode container app (com.memrynote.web-clipper) + com.apple.security.network.client entitlement (required for loopback under App Sandbox).
  • Task 4 — Acceptance gate: confirm the background fetch attaches Origin: safari-web-extension://<UUID> and reaches loopback under the sandbox. Go/no-go.
  • Task 5 — Code-sign, notarize, App Store Connect listing ("Data Not Collected"), submit; README install flow + release runbook.

Notes

  • Branched off origin/main, which does not include the uncommitted :firefox/:edge script edits in the working tree — reconcile script placement at merge.
  • Docs gate skipped (MEMRY_DOCS_IMPACT_SKIP=1): the allowlist line is an inert precursor; user-facing Safari docs land with Task 5 when the extension is actually installable.

@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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

Labels

dependencies documentation Improvements or additions to documentation test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant