Skip to content

ci: stop unconfigured-publisher noise (remove Telegram + fail-safe npm/ClawHub)#7

Merged
acidkill merged 2 commits into
mainfrom
ci/remove-telegram-notify
May 28, 2026
Merged

ci: stop unconfigured-publisher noise (remove Telegram + fail-safe npm/ClawHub)#7
acidkill merged 2 commits into
mainfrom
ci/remove-telegram-notify

Conversation

@acidkill
Copy link
Copy Markdown
Owner

@acidkill acidkill commented May 28, 2026

Summary

Stop CI going red on publish channels that were never configured. Today only PyPI + GitHub Release are live (where 2.0.0 and 2.1.0 actually shipped); npm (plugin + SDK), ClawHub and VS Code have no working tokens / the npm package names don't even exist yet.

  • Remove .github/workflows/notify-telegram.yml — not configured, fails on every tag/RC push.
  • Fail-safe publishers — a missing secret is now a clean skip (::notice), a failed publish is a ::warning, never a red build:
    • release.yml: ClawHub (skip if no CLAWHUB_TOKEN; its unguarded clawdhub login was what reddened the v2.1.0 release), npm + SDK (skip if no NPM_TOKEN, tolerant publish).
    • publish-npm.yml / publish-sdk.yml (rolling RC): skip if no NPM_TOKEN so they stop failing on every push to main.

PyPI + GitHub Release are untouched.

Context (diagnosed)

  • surreal-memory on PyPI: 2.0.0 and 2.1.0 — release works.
  • npm registry: surrealmemory, @acidkill/surreal-memory-client, @surrealmemory/openclaw-plugin → all 404 (don't exist); npm was never actually published for any version.
  • ClawHub: CLAWHUB_TOKEN empty → "Token required".

Test plan

  • all 3 workflow YAMLs parse; jobs intact
  • no telegram references left under .github/
  • next tag/release: green, with notice/skip on unconfigured channels

Toni Nowak added 2 commits May 28, 2026 16:15
The Telegram notifier isn't configured (no bot token/chat) and fails on every
tag push — including prerelease/RC pushes — producing useless red alerts.
Remove it. (The product-level Telegram backup feature, smem_telegram_backup,
is unaffected — this only deletes the CI notification workflow.)
The release publishes only to PyPI + GitHub today; npm (plugin + SDK), ClawHub,
and VS Code were never configured (no/invalid tokens, npm package names don't
exist) and were failing — ClawHub even reddened the whole release because its
`clawdhub login` wasn't guarded.

Guard every optional publisher so a missing secret is a clean skip (notice),
and a failed publish is a warning, never a red build:
- release.yml: ClawHub (skip if no CLAWHUB_TOKEN + guard login), npm + SDK
  (skip if no NPM_TOKEN, tolerant publish).
- publish-npm.yml / publish-sdk.yml (rolling RC): skip if no NPM_TOKEN so they
  stop failing on every push to main.

PyPI + GitHub Release are unaffected and remain the live channels (where 2.0.0
and 2.1.0 actually shipped).
@acidkill acidkill changed the title ci: remove unconfigured Telegram release notification ci: stop unconfigured-publisher noise (remove Telegram + fail-safe npm/ClawHub) May 28, 2026
@acidkill acidkill merged commit dfaae88 into main May 28, 2026
7 checks passed
@acidkill acidkill deleted the ci/remove-telegram-notify branch May 28, 2026 14:39
@acidkill acidkill mentioned this pull request May 28, 2026
3 tasks
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.

1 participant