Skip to content

Fix Twitter URL facet replacement#82

Merged
rosethornbush merged 1 commit into
mainfrom
capy/fix-twitter-url-replace
Jun 7, 2026
Merged

Fix Twitter URL facet replacement#82
rosethornbush merged 1 commit into
mainfrom
capy/fix-twitter-url-replace

Conversation

@rosethornbush
Copy link
Copy Markdown
Contributor

@rosethornbush rosethornbush commented Jun 7, 2026

Summary\n- Replace Twitter URL facets exactly once, preferring the original URL text when present\n- Prevent expanded URL replacements from being processed again via display text and producing duplicated prefixes/slashes\n\n## Validation\n- Targeted Twitter transform reproduction for the Notion Status URL\n- pnpm exec tsc -p packages/platforms/tsconfig.json --noEmit\n- oxlint packages/platforms/src/platforms/twitter.ts\n- pnpm --filter @embedly/platforms exec oxfmt --check src/platforms/twitter.ts

Open in Capy

@rosethornbush rosethornbush added the capy Created by Capy label Jun 7, 2026 — with Capy AI
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 7, 2026

⚠️ No Changeset found

Latest commit: 8d35459

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jun 7, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
embedly-docs 8d35459 Commit Preview URL

Branch Preview URL
Jun 07 2026, 07:26 AM

@rosethornbush rosethornbush merged commit 677faee into main Jun 7, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

capy Created by Capy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant