feat(telegram): add forum topic creation support#17035
feat(telegram): add forum topic creation support#17035steipete merged 287 commits intoopenclaw:mainfrom
Conversation
| if (action === "createForumTopic") { | ||
| const chatId = readStringOrNumberParam(params, "chatId", { | ||
| required: true, | ||
| }); | ||
| const name = readStringParam(params, "name", { required: true }); | ||
| const iconColor = readNumberParam(params, "iconColor", { integer: true }); | ||
| const iconCustomEmojiId = readStringParam(params, "iconCustomEmojiId"); |
There was a problem hiding this comment.
missing action gate check - other actions like sendMessage, deleteMessage, editMessage, and sendSticker include if (!isActionEnabled(...)) checks to respect the action gating configuration
| if (action === "createForumTopic") { | |
| const chatId = readStringOrNumberParam(params, "chatId", { | |
| required: true, | |
| }); | |
| const name = readStringParam(params, "name", { required: true }); | |
| const iconColor = readNumberParam(params, "iconColor", { integer: true }); | |
| const iconCustomEmojiId = readStringParam(params, "iconCustomEmojiId"); | |
| if (action === "createForumTopic") { | |
| if (!isActionEnabled("createForumTopic")) { | |
| throw new Error("Telegram createForumTopic is disabled."); | |
| } | |
| const chatId = readStringOrNumberParam(params, "chatId", { | |
| required: true, | |
| }); | |
| const name = readStringParam(params, "name", { required: true }); |
Prompt To Fix With AI
This is a comment left during a code review.
Path: src/agents/tools/telegram-actions.ts
Line: 331:337
Comment:
missing action gate check - other actions like `sendMessage`, `deleteMessage`, `editMessage`, and `sendSticker` include `if (!isActionEnabled(...))` checks to respect the action gating configuration
```suggestion
if (action === "createForumTopic") {
if (!isActionEnabled("createForumTopic")) {
throw new Error("Telegram createForumTopic is disabled.");
}
const chatId = readStringOrNumberParam(params, "chatId", {
required: true,
});
const name = readStringParam(params, "name", { required: true });
```
How can I resolve this? If you propose a fix, please make it concise.| actions.add("sticker"); | ||
| actions.add("sticker-search"); | ||
| } | ||
| actions.add("topic-create"); |
There was a problem hiding this comment.
topic-create added unconditionally - other actions like react, delete, edit, and sticker use gate() checks to respect configuration. consider adding if (gate("createForumTopic")) { actions.add("topic-create"); } for consistency
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Prompt To Fix With AI
This is a comment left during a code review.
Path: src/channels/plugins/actions/telegram.ts
Line: 63:63
Comment:
`topic-create` added unconditionally - other actions like `react`, `delete`, `edit`, and `sticker` use `gate()` checks to respect configuration. consider adding `if (gate("createForumTopic")) { actions.add("topic-create"); }` for consistency
<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>
How can I resolve this? If you propose a fix, please make it concise.
skills/frontend-design/SKILL.md
Outdated
| --- | ||
| name: frontend-design | ||
| description: Create distinctive, production-grade frontend interfaces with high design quality. Use this skill when the user asks to build web components, pages, or applications. Generates creative, polished code that avoids generic AI aesthetics. | ||
| --- | ||
|
|
||
| This skill guides creation of distinctive, production-grade frontend interfaces that avoid generic "AI slop" aesthetics. Implement real working code with exceptional attention to aesthetic details and creative choices. | ||
|
|
||
| The user provides frontend requirements: a component, page, application, or interface to build. They may include context about the purpose, audience, or technical constraints. | ||
|
|
||
| ## Design Thinking | ||
|
|
||
| Before coding, understand the context and commit to a BOLD aesthetic direction: | ||
|
|
||
| - **Purpose**: What problem does this interface solve? Who uses it? | ||
| - **Tone**: Pick an extreme: brutally minimal, maximalist chaos, retro-futuristic, organic/natural, luxury/refined, playful/toy-like, editorial/magazine, brutalist/raw, art deco/geometric, soft/pastel, industrial/utilitarian, etc. There are so many flavors to choose from. Use these for inspiration but design one that is true to the aesthetic direction. | ||
| - **Constraints**: Technical requirements (framework, performance, accessibility). | ||
| - **Differentiation**: What makes this UNFORGETTABLE? What's the one thing someone will remember? | ||
|
|
||
| **CRITICAL**: Choose a clear conceptual direction and execute it with precision. Bold maximalism and refined minimalism both work - the key is intentionality, not intensity. | ||
|
|
||
| Then implement working code (HTML/CSS/JS, React, Vue, etc.) that is: | ||
|
|
||
| - Production-grade and functional | ||
| - Visually striking and memorable | ||
| - Cohesive with a clear aesthetic point-of-view | ||
| - Meticulously refined in every detail | ||
|
|
||
| ## Frontend Aesthetics Guidelines | ||
|
|
||
| Focus on: | ||
|
|
||
| - **Typography**: Choose fonts that are beautiful, unique, and interesting. Avoid generic fonts like Arial and Inter; opt instead for distinctive choices that elevate the frontend's aesthetics; unexpected, characterful font choices. Pair a distinctive display font with a refined body font. | ||
| - **Color & Theme**: Commit to a cohesive aesthetic. Use CSS variables for consistency. Dominant colors with sharp accents outperform timid, evenly-distributed palettes. | ||
| - **Motion**: Use animations for effects and micro-interactions. Prioritize CSS-only solutions for HTML. Focus on high-impact moments: one well-orchestrated page load with staggered reveals (animation-delay) creates more delight than scattered micro-interactions. Use scroll-triggering and hover states that surprise. | ||
| - **Spatial Composition**: Unexpected layouts. Asymmetry. Overlap. Diagonal flow. Grid-breaking elements. Generous negative space OR controlled density. | ||
| - **Backgrounds & Visual Details**: Create atmosphere and depth rather than defaulting to solid colors. Add contextual effects and textures that match the overall aesthetic. Apply creative forms like gradient meshes, noise textures, geometric patterns, layered transparencies, dramatic shadows, decorative borders, custom cursors, and grain overlays. | ||
|
|
||
| NEVER use generic AI-generated aesthetics like overused font families (Inter, Roboto, Arial, system fonts, Space Grotesk), cliched color schemes (particularly purple gradients on white backgrounds), predictable layouts and component patterns, and cookie-cutter design that lacks context-specific character. | ||
|
|
There was a problem hiding this comment.
this file appears unrelated to the PR's stated purpose (telegram forum topic creation). the PR description makes no mention of adding a frontend-design skill. this may have been included accidentally from commit 1754daa
Prompt To Fix With AI
This is a comment left during a code review.
Path: skills/frontend-design/SKILL.md
Line: 1:39
Comment:
this file appears unrelated to the PR's stated purpose (telegram forum topic creation). the PR description makes no mention of adding a frontend-design skill. this may have been included accidentally from commit 1754daa0
How can I resolve this? If you propose a fix, please make it concise.|
RE: |
bfc1ccb to
f92900f
Compare
|
Friendly bump 👋 — happy to address any feedback or make changes if needed. All CI checks are green (aside from the fork-related |
This reverts commit c529e60.
… pipe" This reverts commit 8b14052.
This reverts commit 53af9f7.
This reverts commit b6d934c.
…e in build logs" This reverts commit 717caa9.
…mage" This reverts commit 2ab6313.
This reverts commit 32c66af.
This reverts commit 59e0e7e.
This reverts commit b91e437.
This reverts commit d4a142f.
This reverts commit 06b961b.
…ew/reset" This reverts commit d6acd71.
This reverts commit 34b18ea.
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
24 similar comments
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
|
Closing this PR because it looks dirty (too many unrelated commits). Please recreate the PR from a clean branch. |
* Revert "fix(gateway): set explicit chat timeouts for mesh gateway calls" This reverts commit c529e60. * Revert "fix: capture init script exit codes instead of swallowing via pipe" This reverts commit 8b14052. * Revert "feat(docker): add init script support via /openclaw-init.d/" This reverts commit 53af9f7. * Revert "Agents: improve Windows scaffold helpers for venture studio" This reverts commit b6d934c. * chore: Fix types in tests 1/N. * chore: Fix types in tests 2/N. * Revert "fix: remove stderr suppression so install failures are visible in build logs" This reverts commit 717caa9. * Revert "fix(docker): ensure memory-lancedb deps installed in Docker image" This reverts commit 2ab6313. * Revert "fix: add windowsHide: true to spawn in runCommandWithTimeout" This reverts commit 32c66af. * Revert "Onboarding: fix webchat URL loopback and canonical session" This reverts commit 59e0e7e. * Revert "feat(linq): add interactive onboarding adapter" This reverts commit b91e437. * Revert "feat: add Linq channel — real iMessage via API, no Mac required" This reverts commit d4a142f. * docs: clarify discord proxy scope for startup REST calls * Revert "fix: flatten remaining anyOf/oneOf in Gemini schema cleaning" This reverts commit 06b961b. * Revert "fix: session-memory hook finds previous session file after /new/reset" This reverts commit d6acd71. * Revert "fix: respect OPENCLAW_HOME for isolated gateway instances" This reverts commit 34b18ea. * fix(process): harden graceful kill-tree cancellation semantics * fix(slack): scope attachment extraction to forwarded shares * docs(changelog): note process kill-tree hotfix * docs(changelog): note slack forwarded attachment hotfix * fix(session-memory): harden reset transcript recovery * revert(telegram): undo accidental merge of PR openclaw#18601 * fix(ui): preserve locale bootstrap and trusted-proxy overview behavior * fix(scripts): harden Windows UI spawn behavior * fix(slack): validate interaction payloads and handle malformed actions * fix(mattermost): harden react remove flag parsing * docs(changelog): record PR 18608 fixups * fix(heartbeat): bound responsePrefix strip for ack detection * chore: Fix types in tests 3/N. * chore: chore: Fix types in tests 4/N. * chore: Fix types in tests 5/N. * chore: Fix types in tests 6/N. * chore: Format files. * chore: Fix types that were broken due to reverts. * chore: Cleanup unused vars that were leftover from the reverts. * fix(actions): layer per-account gate fallback * fix(subagents): pass group context in /subagents spawn * fix(failover): align abort timeout detection and regressions * fix(models): sync auth-profiles before availability checks * fix(ui): correct usage range totals and muted styles * Revert "feat: show transcript file size in session status" This reverts commit 15dd2cd. * revert(doctor): undo accidental merge of PR openclaw#18591 * fix(agents): align session lock hold budget with run timeouts * Revert "fix: resolve openclaw#12770 - update Antigravity default model and trim leading whitespace in BlueBubbles replies" This reverts commit e179d45. * revert(tools): undo accidental merge of PR openclaw#18584 * revert(tools): finish rollback of PR openclaw#18584 * chore: Fix Slack test. * revert: remove accidentally merged video-quote-finder skill (openclaw#18550) * revert: accidental merge of OC-09 sandbox env sanitization change * fix(doctor): move forced exit to top-level command * chore: Fix types in tests 7/N. * chore: Fix types in tests 8/N. * chore: Fix types in tests 9/N. * chore: Fix types in tests 10/N. * chore: Fix types in tests 11/N. * chore: chore: Fix types in tests 12/N. * chore: Fix type errors from reverts. * fix(gateway): remove watch-mode build/start race (openclaw#18782) * fix(doctor): repair googlechat open dm wildcard auto-fix * test(extensions): cast fetch mocks to satisfy tsgo * fix(gateway): harden channel health monitor recovery * fix(reply): track messaging media aliases for dedupe * refactor(plugins): split before-agent hooks by model and prompt phases * revert(telegram): undo accidental merge of PR openclaw#18564 * fix(agents): restore multi-image image tool schema contract * chore: Format files. * fix(ui): gate sessions refresh on successful delete * revert(docs): undo accidental merge of openclaw#18516 * revert(exec): undo accidental merge of PR openclaw#18521 * docs(cron): clarify webhook posting summary condition * fix(gateway): preserve chat.history context under hard caps * chore: Fix types in tests 13/N. * chore: Fix types in tests 14/N. * chore: Fix types in tests 15/N. * chore: Fix types in tests 16/N. * chore: Fix types in tests 17/N. * chore: Fix types in tests 18/N. * chore: Format files. * revert(sandbox): revert SHA-1 slug restoration * test(session): cover stale threadId fallback * test(status): cover token summary variants * test(telegram): cover getFile file-too-big errors * test(voice-call): cover stream disconnect auto-end * chore(format): fix test import order * test(agents): cover tool result media placeholders * chore: chore: Fix types in tests 19/N. * chore: Fix types in tests 20/N. * chore: Fix types in tests 21/N. * chore: Fix types in tests 22/N. * chore: Fix types in tests 23/N. * docs(voice-call): document stale call reaper config * fix(doctor): audit env-only gateway tokens * fix(sessions): purge deleted transcript archives * test(docker): cover browser install build arg * revert(gateway): restore loopback auth setup * revert(voice-call): undo cached greeting note * revert(voice-call): undo oxfmt formatting * revert(voice-call): undo oxfmt formatting pass * revert(voice-call): remove cached inbound greeting * test: stabilize infra tests * fix(subagents): harden announce retry guards * Revert "fix(whatsapp): allow per-message link preview override\n\nWhatsApp messages default to enabling link previews for URLs. This adds\nsupport for overriding this behavior per-message via the \nparameter (e.g. from tool options), consistent with Telegram.\n\nFix: Updated internal WhatsApp Web API layers to pass option\ndown to Baileys ." This reverts commit 1bef2fc. * fix(telegram): clear offsets on token change * test(agents): cover exec non-zero exits * CI: use self-hosted for labeler/automation * Revert "channels: migrate extension account listing to factory" This reverts commit d24340d. * chore(format) * chore: wtf. * chore: Fix types. * chore: Fix types in tests 24/N. * chore: Fix types in tests 25/N. * chore: Fix types in tests 26/N. * chore: Fix types in tests 27/N. * chore: Fix types in tests 28/N. * chore: Fix types in tests 29/N. * chore: Fix types in tests 30/N. * chore: Fix types in tests 31/N. * chore: Fix types in tests 32/N. * fix(telegram): add initial message debounce for better push notifications (openclaw#18147) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 5e2285b Co-authored-by: Marvae <[email protected]> Co-authored-by: obviyus <[email protected]> Reviewed-by: @obviyus * style(telegram): format dispatch files * chore: Fix types in tests 33/N. * chore: Fix types in tests 34/N. * chore: Fix types in tests 35/N. * chore: Fix types in tests 36/N. * chore: Fix types in tests 37/N. * chore: Fix types in tests 38/N. * chore: Fix types in tests 39/N. * chore: Fix types in tests 40/N. * chore: Fix types in tests 41/N. * chore: Fix types in tests 42/N. * chore: Fix types in tests 43/N. * chore: Fix types in tests 44/N. * chore: Fix types in tests 45/N. * chore: Typecheck tests. * chore: Fix broken test. * chore: Fix hanging test. * fix(telegram): avoid duplicate preview bubbles in partial stream mode (openclaw#18956) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: cf4eca7 Co-authored-by: obviyus <[email protected]> Co-authored-by: obviyus <[email protected]> Reviewed-by: @obviyus * fix: before_tool_call hook double-fires with abort signal (openclaw#16852) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 6269d61 Co-authored-by: sreuter <[email protected]> Co-authored-by: obviyus <[email protected]> Reviewed-by: @obviyus * Revert "Default Telegram polls to public" This reverts commit c43e95e. * Revert "Fix Telegram poll action wiring" This reverts commit 556b531. * Revert "Add Telegram polls action to config typing" This reverts commit 5cbfaf5. * Revert "fix(telegram): wire sendPollTelegram into channel action handler (openclaw#16977)" This reverts commit 7bb9a7d. * CI: remove formal models conformance workflow (openclaw#19007) * fix: preserve telegram dm topic thread ids * style: drop aidev-note prefix in telegram comments * test: pass extensionContext in abort dedupe e2e * fix: align tool execute arg parsing for hooks * test: type telegram action mock passthrough args * Configure: make model picker allowlist searchable * Configure: improve searchable model picker token matching * Docs: add screenshot showing model picker usability issue * fix: searchable model picker in configure (openclaw#19010) (thanks @bjesuiter) * fix(extensions): revert openai codex auth plugin (PR openclaw#18009) * feat(telegram): add channel_post support for bot-to-bot communication (openclaw#17857) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 27a343c Co-authored-by: theSamPadilla <[email protected]> Co-authored-by: obviyus <[email protected]> Reviewed-by: @obviyus * Revert "fix: handle forum/topics in Telegram DM thread routing (openclaw#17980)" This reverts commit e20b87f. * Revert: undo openclaw#17974 README change * voice-call: harden closed-loop turn loop and transcript routing (openclaw#19140) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 14a3edb Co-authored-by: mbelinky <[email protected]> Co-authored-by: mbelinky <[email protected]> Reviewed-by: @mbelinky * iOS onboarding: stop auth step-3 retry loop churn (openclaw#19153) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: a38ec42 Co-authored-by: mbelinky <[email protected]> Co-authored-by: mbelinky <[email protected]> Reviewed-by: @mbelinky * Revert: fully roll back openclaw#17974 zh-cn UI README * chore(subagents): add regression coverage and changelog * fix(daemon): scope token drift warnings * test(web): fix baileys mock typing * test(cron): cover webhook session rollover overrides * docs(changelog): note webhook session reuse fix * fix(discord): normalize command allowFrom prefixes * fix(cli): honor update restart overrides * fix(cron): add spin-loop regression coverage * test(gateway): cover trusted proxy trimming * test(discord): cover audioAsVoice replies * test(feishu): cover post mentions for other users * fix(discord): preserve DM lastRoute user target * Revert "fix(browser): track original port mapping for EADDRINUSE fallback" This reverts commit 8e55503. * Revert "fix(browser): handle EADDRINUSE with automatic port fallback" This reverts commit 0e6daa2. * test(discord): fix mock call arg typing * Revert: fully roll back openclaw#17986 templates * test: add fetch mock helper and reaction coverage * CLI: approve latest pending device request * docs(readme): remove Android install link * revert(agents): remove llms.txt discovery prompt (openclaw#19192) * fix(ui): revert PR openclaw#18093 directive tags (openclaw#19188) * test(discord): cover auto-thread skip types * test(update): cover restart gating * docs(zai): document tool_stream defaults * revert: per-model thinkingDefault override (openclaw#19195) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: fe2c59e Co-authored-by: sebslight <[email protected]> Co-authored-by: sebslight <[email protected]> Reviewed-by: @sebslight * fix(gateway): make stale token cleanup non-fatal * Agents: add before_message_write persistence regression tests * fix(mattermost): surface reactions support * Tests: fix fetch mock typings for type-aware checks * revert: fix models set catalog validation (openclaw#19194) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7e3b2ff Co-authored-by: sebslight <[email protected]> Co-authored-by: sebslight <[email protected]> Reviewed-by: @sebslight * test: cover cron telemetry and typed fetch mocks * revert(agents): revert base64 image validation (openclaw#19221) * docs(cli): add components send example * test(sessions): add delivery info regression coverage * fix(daemon): guard preferred node selection * test(auto-reply): cover sender_id metadata * revert: PR 18288 accidental merge (openclaw#19224) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 3cda315 Co-authored-by: sebslight <[email protected]> Co-authored-by: sebslight <[email protected]> Reviewed-by: @sebslight * test(telegram): cover autoSelectFamily env precedence * test(cron): add model fallback regression coverage * test(release): add appcast regression coverage * docs(changelog): remove revert entries * docs: add maintainer application section * docs: refine maintainer application guidance * docs: add vision doc and link from README * docs: add community plugins guide * Update auto-response message for third-party extensions * update my contributing list * iOS: use operator session for ChatSheet RPCs (openclaw#19320) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 0753b3a Co-authored-by: mbelinky <[email protected]> Co-authored-by: mbelinky <[email protected]> Reviewed-by: @mbelinky * fix: sanitize native command names for Telegram API (openclaw#19257) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: b608be3 Co-authored-by: akramcodez <[email protected]> Co-authored-by: obviyus <[email protected]> Reviewed-by: @obviyus * docs(slack): add assistant:write requirement for typing status * chore: document sessions_spawn response note and subagent context prefix * feat(ios): auto-select local signing team (openclaw#18421) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: bbb9c3a Co-authored-by: ngutman <[email protected]> Co-authored-by: ngutman <[email protected]> Reviewed-by: @ngutman * fix(bluebubbles): recover outbound message IDs and include sender metadata * fix cron announce routing and timeout handling * changelog: add @tyler6204 credit for today's entries * feat: share to openclaw ios app (openclaw#19424) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 0a7ab85 Co-authored-by: mbelinky <[email protected]> Co-authored-by: mbelinky <[email protected]> Reviewed-by: @mbelinky * Docs: expand multi-agent routing * docs(changelog): add missing 2026.2.16 entries and reorder by user impact * chore(release): bump version to 2026.2.17 * fix(signal): canonicalize message targets in tool and inbound flows * docs: tighten contribution guidance and vision links * docs: tighten PR scope and review-size policy in vision * fix(gateway): block cross-session fallback in node event delivery * fix(gateway): make health monitor checks single-flight * fix(ios): harden share relay routing and delivery guards * fix(telegram): normalize topic-create targets and add regression tests * feat(cron): add default stagger controls for scheduled jobs * fix(cron): retry next-second schedule compute on undefined * docs(security): harden gateway security guidance * feat(models): support anthropic sonnet 4.6 * fix: wire agents.defaults.imageModel into media understanding auto-discovery resolveAutoEntries only checked a hardcoded list of providers (openai, anthropic, google, minimax) when looking for an image model. agents.defaults.imageModel was never consulted by the media understanding pipeline — it was only wired into the explicit `image` tool. Add resolveImageModelFromAgentDefaults that reads the imageModel config (primary + fallbacks) and inserts it into the auto-discovery chain before the hardcoded provider list. runProviderEntry already falls back to describeImageWithModel (via pi-ai) for providers not in the media understanding registry, so no additional provider registration is needed. Co-Authored-By: Claude Sonnet 4.5 <[email protected]> (cherry picked from commit b381029) * docs: update AGENTS instructions * fix(subagent): harden read-tool overflow guards and sticky reply threading (openclaw#19508) * fix(gateway): avoid premature agent.wait completion on transient errors * fix(agent): preemptively guard tool results against context overflow * fix: harden tool-result context guard and add message_id metadata * fix: use importOriginal in session-key mock to include DEFAULT_ACCOUNT_ID The run.skill-filter test was mocking ../../routing/session-key.js with only buildAgentMainSessionKey and normalizeAgentId, but the module also exports DEFAULT_ACCOUNT_ID which is required transitively by src/web/auth-store.ts. Switch to importOriginal pattern so all real exports are preserved alongside the mocked functions. * pi-runner: guard accumulated tool-result overflow in transformContext * PI runner: compact overflowing tool-result context * Subagent: harden tool-result context recovery * Enhance tool-result context handling by adding support for legacy tool outputs and improving character estimation for message truncation. This includes a new function to create legacy tool results and updates to existing functions to better manage context overflow scenarios. * Enhance iMessage handling by adding reply tag support in send functions and tests. This includes modifications to prepend or rewrite reply tags based on provided replyToId, ensuring proper message formatting for replies. * Enhance message delivery across multiple channels by implementing sticky reply context for chunked messages. This includes preserving reply references in Discord, Telegram, and iMessage, ensuring that follow-up messages maintain their intended reply targets. Additionally, improve handling of reply tags in system prompts and tests to support consistent reply behavior. * Enhance read tool functionality by implementing auto-paging across chunks when no explicit limit is provided, scaling output budget based on model context window. Additionally, add tests for adaptive reading behavior and capped continuation guidance for large outputs. Update related functions to support these features. * Refine tool-result context management by stripping oversized read-tool details payloads during compaction, ensuring repeated read calls do not bypass context limits. Introduce new utility functions for handling truncation content and enhance character estimation for tool results. Add tests to validate the removal of excessive details in context overflow scenarios. * Refine message delivery logic in Matrix and Telegram by introducing a flag to track if a text chunk was sent. This ensures that replies are only marked as delivered when a text chunk has been successfully sent, improving the accuracy of reply handling in both channels. * fix: tighten reply threading coverage and prep fixes (openclaw#19508) (thanks @tyler6204) * fix(hooks): backport internal message hook bridge with safe delivery semantics * fix(subagent): update SUBAGENT_SPAWN_ACCEPTED_NOTE for clarity on auto-announcement behavior * fix: follow-up slack streaming routing/tests (openclaw#9972) (thanks @natedenh) * fix: reduce default image dimension from 2000px to 1200px Large images (2000px) consume excessive context tokens when sent to LLMs. 1200px provides sufficient detail for most use cases while significantly reducing token usage. The 5MB byte limit remains unchanged as JPEG compression at 1200px naturally produces smaller files. (cherry picked from commit 4018212) * fix(agents): make image sanitization dimension configurable * docs(tokens): document image dimension token tradeoffs * Whatsapp/add resolve outbound target tests (openclaw#19345) * test(whatsapp): add resolveWhatsAppOutboundTarget test suite * style: auto-format files * fix(test): correct mock order for invalid allowList entry test * feat(skills): Add 'Use when / Don't use when' routing blocks (openclaw#14521) * feat(skills): add 'Use when / Don't use when' blocks to skill descriptions Based on OpenAI's Shell + Skills + Compaction best practices article. Key changes: - Added clear routing logic to skill descriptions - Added negative examples to prevent misfires - Added templates/examples to github skill - Included Blake's specific setup notes for openhue Skills updated: - apple-reminders: Clarify vs Clawdbot cron - github: Clarify vs local git operations - imsg: Clarify vs other messaging channels - openhue: Add device inventory, room layout - tmux: Clarify vs exec tool - weather: Add location defaults, format codes Reference: https://developers.openai.com/blog/skills-shell-tips * fix(skills): restore metadata and generic CLI examples --------- Co-authored-by: Peter Steinberger <[email protected]> * feat(agents): add generic provider api key rotation (openclaw#19587) * feat(skills): improve descriptions with routing logic (openclaw#14577) * feat(skills): improve descriptions with routing logic Apply OpenAI's recommended pattern for skill descriptions: - Add 'Use when' conditions for clear triggering - Add 'NOT for' negative examples to reduce misfires - Make descriptions act as routing logic, not marketing copy Based on: https://developers.openai.com/blog/skills-shell-tips/ Skills updated: - coding-agent: clarify when to delegate vs direct edit - github: add boundaries vs browser/scripting - weather: add scope limitations Glean reported 20% drop in skill triggering without negative examples, recovering after adding them. This change brings Clawdbot skills in line with that pattern. * docs(skills): clarify routing boundaries (openclaw#14577) (thanks @DylanWoodAkers) * docs(changelog): add PR 14577 release note (openclaw#14577) (thanks @DylanWoodAkers) --------- Co-authored-by: ClawdBotWolf <[email protected]> Co-authored-by: Peter Steinberger <[email protected]> * Add frontend-design skill * feat(telegram): add forum topic creation support (openclaw#10427) Add `topic-create` action to the Telegram message adapter, enabling programmatic creation of forum topics in supergroups. Changes: - Add `createForumTopicTelegram()` to `src/telegram/send.ts` - Add `createForumTopic` handler in `telegram-actions.ts` - Wire `topic-create` action in Telegram adapter - Register `topic-create` in message action names and spec The bot requires `can_manage_topics` permission in the target group. Supports optional `iconColor` and `iconCustomEmojiId` parameters. Closes openclaw#10427 * chore: fix formatting in frontend-design SKILL.md * fix: add action gate check and config type for createForumTopic Address review feedback: - Add isActionEnabled() gate in telegram-actions.ts - Add gate() check in telegram adapter listActions - Add createForumTopic to TelegramActionConfig type * fix(telegram): normalize topic-create targets and add regression tests --------- Co-authored-by: Peter Steinberger <[email protected]> Co-authored-by: Gustavo Madeira Santana <[email protected]> Co-authored-by: cpojer <[email protected]> Co-authored-by: Sebastian <[email protected]> Co-authored-by: Josh Avant <[email protected]> Co-authored-by: Shadow <[email protected]> Co-authored-by: Hongwei Ma <[email protected]> Co-authored-by: Marvae <[email protected]> Co-authored-by: obviyus <[email protected]> Co-authored-by: Ayaan Zaidi <[email protected]> Co-authored-by: Ayaan Zaidi <[email protected]> Co-authored-by: Sascha Reuter <[email protected]> Co-authored-by: sreuter <[email protected]> Co-authored-by: Nimrod Gutman <[email protected]> Co-authored-by: Vignesh <[email protected]> Co-authored-by: Benjamin Jesuiter <[email protected]> Co-authored-by: Sam Padilla <[email protected]> Co-authored-by: Muhammed Mukhthar CM <[email protected]> Co-authored-by: Mariano <[email protected]> Co-authored-by: Shakker <[email protected]> Co-authored-by: Mariano Belinky <[email protected]> Co-authored-by: Shadow <[email protected]> Co-authored-by: Sk Akram <[email protected]> Co-authored-by: akramcodez <[email protected]> Co-authored-by: Onur <[email protected]> Co-authored-by: Tyler Yust <[email protected]> Co-authored-by: ngutman <[email protected]> Co-authored-by: Pablo Nunez <[email protected]> Co-authored-by: Claude Sonnet 4.5 <[email protected]> Co-authored-by: Tyler Yust <[email protected]> Co-authored-by: Han Xiao <[email protected]> Co-authored-by: Verite Igiraneza <[email protected]> Co-authored-by: Blakeshannon <[email protected]> Co-authored-by: Peter Steinberger <[email protected]> Co-authored-by: DylanWoodAkers <[email protected]> Co-authored-by: ClawdBotWolf <[email protected]> Co-authored-by: Claw <[email protected]>
Summary
Adds
topic-createas a new Telegram message action, enabling programmatic creation of forum topics in supergroups via themessagetool.Closes #10427
Changes
src/telegram/send.ts— AddcreateForumTopicTelegram()function that calls the Telegram Bot API'screateForumTopicmethodsrc/agents/tools/telegram-actions.ts— AddcreateForumTopicaction handlersrc/channels/plugins/actions/telegram.ts— Wiretopic-createinto the Telegram adapter'slistActionsandhandleActionsrc/channels/plugins/message-action-names.ts— Registertopic-createaction namesrc/infra/outbound/message-action-spec.ts— Add target mode fortopic-createUsage
Optional parameters:
iconColor— Topic icon color (one of the six Telegram-supported values)iconCustomEmojiId— Custom emoji ID for the topic iconRequirements
The bot needs
can_manage_topicspermission in the target supergroup (set via BotFather or group admin settings).Testing
pnpm buildpassespnpm lintpasses (0 warnings, 0 errors)Greptile Summary
Adds
topic-createas a new Telegram message action for programmatic forum topic creation in supergroups. The implementation spans the full action pipeline: acreateForumTopicTelegram()function in the send layer, action handler registration with proper gating in both the agent tools and channel adapter, config type extension, and action name/target mode registration.createForumTopicdefaults to enabled (consistent withreactions,sendMessage,deleteMessage,editMessage) and can be disabled viachannels.telegram.actions.createForumTopic: falseiconColorandiconCustomEmojiIdare optional pass-through parameters validated by the Telegram API server-sideskills/frontend-design/SKILL.mdfile from a prior commit in the branch historyConfidence Score: 4/5
skills/frontend-design/SKILL.mdfile is unrelated to the PR scope but harmless.Last reviewed commit: b5bfb73