Skip to content

fix(models): label picker auth via effective provider order#83726

Merged
clawsweeper[bot] merged 2 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-83697
May 18, 2026
Merged

fix(models): label picker auth via effective provider order#83726
clawsweeper[bot] merged 2 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-83697

Conversation

@clawsweeper

@clawsweeper clawsweeper Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

Makes #83697 merge-ready for the ClawSweeper automerge loop.
The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

  • Repair fallback: GitHub rejected the repair branch push because it updates workflow files and the ClawSweeper app token does not have workflows permission

Inherited issue-closing references from the source PR:
Fixes #83574

Co-author credit kept:

fish notes: model gpt-5.5, reasoning high; reviewed against 8ca2924.

@clawsweeper clawsweeper Bot requested a review from a team as a code owner May 18, 2026 18:40
@clawsweeper clawsweeper Bot added agents Agent runtime and tooling size: S clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. mantis: telegram-visible-proof Mantis should capture Telegram visible proof. P2 Normal backlog priority with limited blast radius. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. clawsweeper Tracked by ClawSweeper automation labels May 18, 2026
@clawsweeper clawsweeper Bot mentioned this pull request May 18, 2026
Closed
@openclaw-barnacle openclaw-barnacle Bot removed proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. labels May 18, 2026
@clawsweeper

clawsweeper Bot commented May 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Codex review: passed.

Workflow note: Future ClawSweeper reviews update this same comment in place.

How this review workflow works
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

Summary
The PR passes the effective OpenAI/Codex auth provider set into /models provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.

Reproducibility: yes. Current main lacks acceptedProviderIds in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

PR rating
Overall: 🐚 platinum hermit
Proof: 🦞 diamond lobster ✨ media proof bonus
Patch quality: 🐚 platinum hermit
Summary: Focused implementation with strong visual and terminal proof; remaining gating is ordinary exact-head CI/proof completion.

Rank-up moves:

  • Wait for required exact-head checks, including the in-progress native Telegram proof if it is required for this PR.
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

PR egg
✨ Hatched: 🥚 common Sunspot Test Hopper

       /\  .---.  /\         
      /  \/     \/  \        
     /   ( -   - )   \       
    |       ._.       |      
    |   /|  ===  |\   |      
     \  \|______/|/  /       
      '._  `--'  _.'         
         '-.__.-'            
       _/|_|  |_|\_          
      /__|      |__\         
       .-----------.         
      '-------------'

Rarity: 🥚 common.
Trait: purrs at green checks.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Sunspot Test Hopper in ClawSweeper.

What is this egg doing here?
  • Eggs appear after the PR passes real-behavior proof. It is here for vibes, not verdicts: it does not change labels, ratings, merge decisions, or automation.
  • The shell reacts to review momentum: open follow-up work warms it up, re-review makes it wobble, and a clean final review lets it hatch.
  • How to hatch it: reach status: 👀 ready for maintainer look or status: 🚀 automerge armed; that usually means sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness.
  • The hatch is seeded from this repository and PR number, so the same PR keeps the same creature; the reviewed head SHA can only change safe visual details.
  • Rarity is just collectible sparkle: 🥚 common, 🌱 uncommon, 💎 rare, ✨ glimmer, and 🌈 legendary.

Real behavior proof
Sufficient (screenshot): The source PR body includes terminal output from the real formatter path, and inspected Mantis screenshots show the Telegram picker header changing to the Codex OAuth label after the fix.

Risk before merge
Why this matters: - Exact-head native Telegram proof was still running when inspected; merge automation should wait for required checks to finish.

Maintainer options:

  1. Decide the mitigation before merge
    Land the replacement after required exact-head checks finish, keeping the fix limited to shared provider-header labeling and leaving model selection/routing behavior unchanged.
  2. Pause or close
    Do not merge this PR until maintainers decide whether the risk is worth taking.

Next step before merge
No repair lane is needed because the patch is focused, proof-backed, and has no actionable review findings; normal checks/automerge should gate the exact head.

Security
Cleared: The diff is limited to display-label plumbing, tests, and a changelog entry; it adds no dependency, workflow, secret-handling, or code-execution surface.

Review details

Best possible solution:

Land the replacement after required exact-head checks finish, keeping the fix limited to shared provider-header labeling and leaving model selection/routing behavior unchanged.

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

Yes. Current main lacks acceptedProviderIds in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Is this the best way to solve the issue?

Yes. Reusing the existing OpenAI/Codex accepted-provider helper in the shared header formatter is the narrow maintainable fix and avoids changing dispatch routing or adding configuration.

Label justifications:

  • P2: The PR fixes a concrete auth-provider display bug with limited blast radius but real user cost-control ambiguity.

What I checked:

  • Current-main picker label path lacks accepted auth providers: Current main formats /models provider headers by calling resolveModelAuthLabel with provider, config, session, and directory context only, so this path cannot prefer the OpenAI/Codex accepted-provider set before env-key fallback. (src/auto-reply/reply/commands-models.ts:394, 44c6ad7dce27)
  • Auth-label resolver already has the needed contract: resolveModelAuthLabel accepts acceptedProviderIds, resolves auth order across those providers, and only later falls back to resolveEnvApiKey, which matches the linked bug's desired fix boundary. (src/agents/model-auth-label.ts:44, 44c6ad7dce27)
  • Existing OpenAI/Codex helper supplies effective provider order: listOpenAIAuthProfileProvidersForAgentRuntime is the existing helper for deriving accepted OpenAI/Codex auth provider order from runtime policy and auth.order, so reusing it avoids a duplicate routing rule. (src/agents/openai-codex-routing.ts:125, 44c6ad7dce27)
  • PR diff wires the existing contract into the shared header path: The PR imports listOpenAIAuthProfileProvidersForAgentRuntime, resolves harness policy for the displayed provider, and passes the resulting acceptedProviderIds to resolveModelAuthLabel; the patch does not change model dispatch routing. (src/auto-reply/reply/commands-models.ts:388, 8ca2924adcbe)
  • Regression coverage targets both resolver and command path: The PR adds a resolver test proving accepted provider IDs beat OPENAI_API_KEY fallback and a /models openai command test proving the header receives acceptedProviderIds: ["openai-codex"]. (src/agents/model-auth-label.test.ts:123, 8ca2924adcbe)
  • Telegram surface uses the shared header formatter: The Telegram model-list callback calls formatModelsAvailableHeader, so the shared /models header path is the visible Telegram picker header path behind the linked report. (extensions/telegram/src/bot-handlers.runtime.ts:2351, 44c6ad7dce27)

Likely related people:

  • steipete: Recent history shows this person worked on effective /models runtime choices and Codex auth-order handling in the affected model command and auth-label surfaces. (role: recent area contributor; confidence: high; commits: 6171b4254d41, 9dedc4d95cfb, 246bb7f30f9b; files: src/auto-reply/reply/commands-models.ts, src/agents/model-auth-label.ts, src/agents/openai-codex-routing.ts)
  • shakkernerd: Recent commits in the same command/auth-label path aligned model auth display with workspace auth evidence and passed workspace scope into auth labels. (role: adjacent auth-label contributor; confidence: medium; commits: 3b4d2d88865a, b4ecc814c5b8; files: src/auto-reply/reply/commands-models.ts, src/agents/model-auth-label.ts)
  • ragesaq: Recent history on openai-codex-routing.ts includes OpenAI Codex runtime provider routing work adjacent to the helper reused by this PR. (role: OpenAI/Codex routing contributor; confidence: medium; commits: 58f1db1bc8eb; files: src/agents/openai-codex-routing.ts)
  • Takhoffman: Recent history includes model command behavior changes, and the replacement PR body records this person as the automerge requester for the source PR. (role: recent model command contributor and automerge requester; confidence: medium; commits: 58020ab759b4, 59e282527499, 07049c8eba24; files: src/auto-reply/reply/commands-models.ts)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 44c6ad7dce27.

@clawsweeper clawsweeper Bot temporarily deployed to qa-live-shared May 18, 2026 18:43 Inactive
@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane. and removed status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels May 18, 2026
@clawsweeper clawsweeper Bot merged commit bf93294 into main May 18, 2026
185 of 198 checks passed
@clawsweeper clawsweeper Bot deleted the clawsweeper/automerge-openclaw-openclaw-83697 branch May 18, 2026 18:49
@clawsweeper

clawsweeper Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=8ca2924adcbe412898c63546612ebeee0fbb8b28)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-18T18:48:59Z
Merge commit: bf9329486b94

What merged:

  • The PR passes the effective OpenAI/Codex auth provider set into /models provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
  • Reproducibility: yes. Current main lacks acceptedProviderIds in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:

  • PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

The automerge loop is complete.

Automerge progress:

  • 2026-05-18 18:48:45 UTC review passed 8ca2924adcbe (structured ClawSweeper verdict: pass (sha=8ca2924adcbe412898c63546612ebeee0fbb8...)
  • 2026-05-18 18:49:02 UTC merged 8ca2924adcbe (merged by ClawSweeper automerge)

@openclaw-mantis

openclaw-mantis Bot commented May 18, 2026

Copy link
Copy Markdown

Mantis Telegram Desktop Proof

Summary: Mantis captured native Telegram Desktop before/after GIF evidence with Convex-leased Telegram credentials.

Main screenshot This PR screenshot
Baseline native Telegram Desktop screenshot Candidate native Telegram Desktop screenshot
Main This PR
Baseline native Telegram Desktop proof GIF Candidate native Telegram Desktop proof GIF

Motion-trimmed clips:

Raw QA files: https://artifacts.openclaw.ai/mantis/telegram-desktop/pr-83726/run-26053179313-1/index.json

@github-actions github-actions Bot mentioned this pull request May 18, 2026
Open
eleqtrizit pushed a commit to eleqtrizit/openclaw that referenced this pull request May 18, 2026
@clawsweeper @yu-xin-c
@Takhoffman @eleqtrizit
fix(models): label picker auth via effective provider order (openclaw…
1398ee1 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (#83726)
13742aa 
Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924adcbe412898c63546612ebeee0fbb8b28.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924adcbe412898c63546612ebeee0fbb8b28
Review: openclaw/openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (#83726)
7c52565 
Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924adcbe412898c63546612ebeee0fbb8b28.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924adcbe412898c63546612ebeee0fbb8b28
Review: openclaw/openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @yu-xin-c
@Takhoffman @SebTardif
fix(models): label picker auth via effective provider order (openclaw…
a71812e 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @yu-xin-c
@Takhoffman @SebTardif
fix(models): label picker auth via effective provider order (openclaw…
b7c95ec 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @yu-xin-c
@Takhoffman @SebTardif
fix(models): label picker auth via effective provider order (openclaw…
12e8608 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (openclaw…
aa83360 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
galiniliev pushed a commit to galiniliev/openclaw that referenced this pull request May 25, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (openclaw…
ad0b3bc 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @yu-xin-c
@Takhoffman @SebTardif
fix(models): label picker auth via effective provider order (openclaw…
67446fb 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @yu-xin-c
@Takhoffman @SebTardif
fix(models): label picker auth via effective provider order (openclaw…
1c0b34a 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @yu-xin-c
@Takhoffman @SebTardif
fix(models): label picker auth via effective provider order (openclaw…
177ef69 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (openclaw…
6c55b42 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (openclaw…
e81454f 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
@clawsweeper @yu-xin-c @Takhoffman
fix(models): label picker auth via effective provider order (openclaw…
88fb724 
…#83726)

Summary:
- The PR passes the effective OpenAI/Codex auth provider set into `/models` provider-header labeling, adds focused regression tests, and records the user-facing fix in the changelog.
- Reproducibility: yes. Current main lacks `acceptedProviderIds` in the shared picker header path, and the source PR's Mantis baseline/candidate proof shows the visible Telegram header mismatch and after-fix OAuth label.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(models): label picker auth via effective provider order

Validation:
- ClawSweeper review passed for head 8ca2924.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ca2924
Review: openclaw#83726 (comment)

Co-authored-by: Stellar鱼 <2182712990@qq.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Approved-by: takhoffman
Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

agents Agent runtime and tooling clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge clawsweeper Tracked by ClawSweeper automation mantis: telegram-visible-proof Mantis should capture Telegram visible proof. P2 Normal backlog priority with limited blast radius. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: S status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: /model picker auth label ignores auth.order resolution

1 participant

@yu-xin-c