Skip to content

fix: preserve full X share copy and repo url#975

Merged
lefarcen merged 1 commit intomainfrom
fix/issue-967-x-share-truncation
Apr 9, 2026
Merged

fix: preserve full X share copy and repo url#975
lefarcen merged 1 commit intomainfrom
fix/issue-967-x-share-truncation

Conversation

@anthhub
Copy link
Copy Markdown

@anthhub anthhub commented Apr 9, 2026

What

Fix the Rewards X share template so the full share copy and GitHub repo URL are preserved when opening the X composer.

Why

Closes #967.

The previous X share URL put both the marketing copy and the repo link into the text query param. In practice this caused the compose content to truncate after Slack and the GitHub link to disappear.

How

  • keep the shared marketing copy as the X text payload
  • pass the GitHub repo as the dedicated X url query param
  • update the shared rewards template test to assert the split text / url behavior
  • leave WhatsApp and other share templates unchanged

Affected areas

  • Desktop app (Electron shell)
  • Controller (backend / API)
  • Web dashboard (React UI)
  • OpenClaw runtime
  • Skills
  • Shared schemas / packages
  • Build / CI / Tooling

Checklist

  • pnpm typecheck passes
  • pnpm lint passes
  • pnpm test passes
  • pnpm generate-types run (if API routes/schemas changed)
  • No credentials or tokens in code or logs
  • No any types introduced (use unknown with narrowing)

Notes for reviewers

Manual verification in the web rewards flow confirms the X composer now keeps the full share copy and includes the GitHub URL / preview card.

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@lefarcen lefarcen merged commit 883d3b7 into main Apr 9, 2026
12 checks passed
@lefarcen lefarcen mentioned this pull request Apr 10, 2026
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.

X (Twitter) sharing text is truncated and incomplete

2 participants