Skip to content

Show current thinking level in Telegram picker#78278

Merged
obviyus merged 1 commit into
openclaw:mainfrom
obviyus:fix/telegram-think-current-level
May 6, 2026
Merged

Show current thinking level in Telegram picker#78278
obviyus merged 1 commit into
openclaw:mainfrom
obviyus:fix/telegram-think-current-level

Conversation

@obviyus

@obviyus obviyus commented May 6, 2026

Copy link
Copy Markdown
Contributor

Summary:

  • Show the current thinking level above Telegram's native /think picker.
  • Resolve the displayed level from the target session override first, then agent/default model thinking config.
  • Cover explicit session and configured-default cases in Telegram native command tests.

Real behavior proof:

Verification:

  • pnpm test extensions/telegram/src/bot-native-commands.session-meta.test.ts
  • pnpm exec oxfmt --check --threads=1 CHANGELOG.md extensions/telegram/src/bot-native-commands.ts extensions/telegram/src/bot-native-commands.session-meta.test.ts
  • pnpm check:changed

@openclaw-barnacle openclaw-barnacle Bot added channel: telegram Channel integration: telegram size: S maintainer Maintainer-authored PR labels May 6, 2026
@obviyus obviyus self-assigned this May 6, 2026
@clawsweeper

clawsweeper Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

Codex review: passed.

Summary
The PR updates Telegram native /think argument menu rendering to prefix the current thinking level, adds precedence tests, and adds a changelog entry.

Reproducibility: yes. at source level: current main renders the Telegram native /think picker with only the generic argument-menu title while the docs say no-argument /think shows the current level. The contributor screenshot also proves the corrected visible behavior after the patch.

Real behavior proof
Sufficient (screenshot): The PR comment includes a real Telegram screenshot showing /think displaying the current thinking level above the native picker after the fix.

Next step before merge
No repair lane is needed; this automerge-opted PR has no actionable code findings and exact-head checks can gate merge.

Security
Cleared: The diff only changes Telegram command display logic, focused tests, and the changelog, with no dependency, workflow, secret, package, or code-execution surface change.

Review details

Best possible solution:

Land the PR once exact-head merge gates accept the supplied proof and CI, keeping shared thinking default resolution as the source of truth.

Do we have a high-confidence way to reproduce the issue?

Yes, at source level: current main renders the Telegram native /think picker with only the generic argument-menu title while the docs say no-argument /think shows the current level. The contributor screenshot also proves the corrected visible behavior after the patch.

Is this the best way to solve the issue?

Yes. The patch keeps the change inside Telegram native command title formatting, reuses the shared session/default thinking resolver path, and adds focused precedence coverage.

What I checked:

Likely related people:

  • steipete: Introduced the native /think menu session-model alignment path that this PR extends and has recent substantial Telegram native-command maintenance. (role: introduced behavior / recent maintainer; confidence: high; commits: c6770d369468, 05eda57b3c72; files: extensions/telegram/src/bot-native-commands.ts, extensions/telegram/src/bot-native-commands.session-meta.test.ts, src/auto-reply/commands-registry.ts)
  • Takhoffman: Recently changed the shared thinking default resolver contract that the Telegram title helper now delegates to. (role: adjacent owner; confidence: medium; commits: 87eee6e640ab; files: src/agents/model-thinking-default.ts, src/agents/model-selection.test.ts, src/auto-reply/reply/model-selection.ts)
  • obviyus: Has prior merged maintenance in the Telegram native-command session metadata path, independent of authoring this PR. (role: recent maintainer; confidence: medium; commits: c57960b8d1ea; files: extensions/telegram/src/bot-native-commands.ts)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 6febffb6feb0.

@obviyus

obviyus commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Tested manually, shows current thinking level now.

image

@obviyus

obviyus commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper automerge

@clawsweeper clawsweeper Bot added the clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge label May 6, 2026
@clawsweeper

clawsweeper Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper 🐠 automerge status

No new branch changes from this pass. ClawSweeper left the branch untouched instead of making a noisy edit.

Executor outcome: source PR branch changed while the repair worker was preparing its push; requeue against the latest head.
Worker summary: this PR is the canonical automerge repair target. It is open, small, branch-writable, security-cleared, and has a passing ClawSweeper code review, but automerge is blocked by the failing Real behavior proof check, so the next deterministic action is to repair the contributor branch and rerun exact-head gates rather than merge or close.

Worker actions:

  • fix_needed on this PR: planned - Allowed actions include fix and raise_pr, but merge and close are blocked by job policy. Because the canonical contributor PR is useful and branch-writable, repair the contributor branch instead of replacing it.
  • build_fix_artifact on cluster:automerge-openclaw-openclaw-78278: planned - A cluster-scoped fix artifact is required for a maintainer-calibrated open canonical PR that is not merge-ready yet.

No branch push, rebase, replacement PR, merge, or ClawSweeper re-review was started on this pass.

fish notes: model gpt-5.5, reasoning high.

Automerge progress:

  • 2026-05-06 05:31:19 UTC review queued b8b9c46b5816 (after repair)
  • 2026-05-06 05:39:50 UTC review passed b8b9c46b5816 (structured ClawSweeper verdict: pass (sha=b8b9c46b5816704789725d89fcb6f606aa341...)

@clawsweeper clawsweeper Bot force-pushed the fix/telegram-think-current-level branch from 171a9ad to 5f83a20 Compare May 6, 2026 05:07
@clawsweeper clawsweeper Bot added the proof: sufficient ClawSweeper judged the real behavior proof convincing. label May 6, 2026
@clawsweeper clawsweeper Bot force-pushed the fix/telegram-think-current-level branch from 5f83a20 to b8b9c46 Compare May 6, 2026 05:31
@obviyus

obviyus commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper stop

@clawsweeper clawsweeper Bot added clawsweeper:human-review Needs maintainer review before ClawSweeper can continue and removed clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge labels May 6, 2026
@clawsweeper

clawsweeper Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

🦞✅
Got it. ClawSweeper will leave this item for human review.

I added clawsweeper:human-review, removed clawsweeper:automerge, and paused the automation trail until a maintainer asks again.

@obviyus obviyus force-pushed the fix/telegram-think-current-level branch from b8b9c46 to 6a550f1 Compare May 6, 2026 05:52
@obviyus obviyus merged commit 98cbf7f into openclaw:main May 6, 2026
105 of 107 checks passed
@obviyus

obviyus commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Landed via rebase as 98cbf7f.

Verification:

  • pnpm test extensions/telegram/src/bot-native-commands.session-meta.test.ts
  • pnpm exec oxfmt --check --threads=1 CHANGELOG.md extensions/telegram/src/bot-native-commands.ts extensions/telegram/src/bot-native-commands.session-meta.test.ts
  • pnpm check:changed

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 9, 2026
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: telegram Channel integration: telegram clawsweeper:human-review Needs maintainer review before ClawSweeper can continue maintainer Maintainer-authored PR proof: sufficient ClawSweeper judged the real behavior proof convincing. size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant