Skip to content

fix(agents): skip wildcard catalog metadata refs#86524

Merged
giodl73-repo merged 2 commits into
openclaw:mainfrom
giodl73-repo:fix-agent-model-catalog-wildcard
May 26, 2026
Merged

fix(agents): skip wildcard catalog metadata refs#86524
giodl73-repo merged 2 commits into
openclaw:mainfrom
giodl73-repo:fix-agent-model-catalog-wildcard

Conversation

@giodl73-repo

@giodl73-repo giodl73-repo commented May 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Skips wildcard default model allowlist entries while building concrete model catalog metadata and configured models list rows.

Entries such as openai-codex/* are valid allowlist selectors, but they are not concrete model ids and do not have per-model alias metadata to resolve or display as configured rows. The catalog metadata builder and configured list resolver now ignore those wildcard keys instead of sending them through concrete model normalization.

Split out from #85817 so the policy-scoping PR remains policy-only.

Verification

  • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
  • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
  • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
  • git diff --check refs/remotes/upstream-safe/main...HEAD
  • git diff --check

Signed head: 01fbfe3ae08446dec418d14af119872a2cea1a23.

Real behavior proof

Behavior addressed: wildcard default model allowlist entries are treated as selectors, not concrete catalog model ids or configured model rows.

Real environment tested: WSL Ubuntu 24.04 checkout under /root/src/openclaw-86524, with a temp OpenClaw home at /tmp/openclaw-86524-proof.

Exact steps or command run after this patch: OPENCLAW_HOME=/tmp/openclaw-86524-proof OPENCLAW_DISABLE_AUTO_UPDATE=1 OPENCLAW_DISABLE_BUNDLED_PLUGINS=1 OPENCLAW_TEST_FAST=1 node --import tsx src/entry.ts models list --json

Evidence after fix: the temp config set agents.defaults.model to openai/gpt-5.5 and included both agents.defaults.models["openai-codex/*"] = {} and agents.defaults.models["openai/gpt-5.5"].alias = "Primary".

Observed result after fix: the redacted JSON summary was { "modelCount": 1, "keys": ["openai/gpt-5.5"], "wildcardRows": [], "configuredRows": [{ "key": "openai/gpt-5.5", "tags": ["default", "configured", "alias:Primary"], "aliases": [] }] }.

What was not tested: live provider catalog discovery with external provider credentials.

@openclaw-barnacle openclaw-barnacle Bot added agents Agent runtime and tooling size: XS maintainer Maintainer-authored PR labels May 25, 2026
@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge. Reviewed May 25, 2026, 9:06 PM ET / 01:06 UTC.

Summary
The PR skips agents.defaults.models keys ending in /* while building model catalog alias metadata and configured models list rows, with a focused regression test.

PR surface: Source +6, Tests +19. Total +25 across 3 files.

Reproducibility: yes. Source inspection shows current main treats every default model key as a configured row and alias metadata key while the visibility parser already classifies provider/* separately; the PR body gives a concrete after-fix CLI proof path.

Review metrics: 1 noteworthy metric.

  • Wildcard default selectors: 1 existing selector form changes display handling in 2 model-list paths. agents.defaults.models["provider/*"] is compatibility-sensitive because existing configs may notice different configured-row or alias metadata output after merge.

Merge readiness
Overall: 🦞 diamond lobster
Proof: 🦞 diamond lobster
Patch quality: 🦞 diamond lobster
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Risk before merge

  • Merging changes user-visible models list and catalog metadata behavior for existing configs that use wildcard default model selectors; maintainers should confirm that selector-only behavior is intended before landing.

Maintainer options:

  1. Confirm selector-only semantics (recommended)
    Have the model/config owner confirm that wildcard defaults are selectors only, then merge the focused fix with the existing proof.
  2. Document wildcard display semantics first
    If wildcard rows need a user-visible representation, update the docs and tests before changing configured-list output.
  3. Pause if wildcard aliases are intentional
    If aliases or rows on wildcard keys are meant to work, pause this PR and replace it with a design for provider-wide display metadata.

Next step before merge
No automated repair is needed; the remaining action is maintainer review because the protected label and compatibility-visible model config output change need a human land decision.

Security
Cleared: The diff only changes TypeScript model-list logic and a focused test; it does not touch dependencies, scripts, workflows, secrets, or code-download paths.

Review details

Best possible solution:

Land the focused skip if maintainers agree wildcard allowlist selectors should stay selection-only and not appear as concrete configured rows; otherwise define explicit wildcard display semantics before merging.

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

Yes. Source inspection shows current main treats every default model key as a configured row and alias metadata key while the visibility parser already classifies provider/* separately; the PR body gives a concrete after-fix CLI proof path.

Is this the best way to solve the issue?

Yes. The patch is the narrowest maintainable fix for the reported behavior because it leaves wildcard allowlist selection intact and only prevents wildcard selectors from being resolved as concrete display rows or alias metadata.

AGENTS.md: found and applied where relevant.

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

Label changes

Label changes:

  • add rating: 🦞 diamond lobster: Overall readiness is 🦞 diamond lobster; proof is 🦞 diamond lobster and patch quality is 🦞 diamond lobster.
  • remove rating: 🐚 platinum hermit: Current PR rating is rating: 🦞 diamond lobster, so this older rating label is no longer current.

Label justifications:

  • P2: This is a normal-priority focused bug fix for model catalog/configured-list output with limited blast radius.
  • merge-risk: 🚨 compatibility: The PR changes how existing wildcard model default config entries appear in user-facing model list/catalog output.
  • rating: 🦞 diamond lobster: Overall readiness is 🦞 diamond lobster; proof is 🦞 diamond lobster and patch quality is 🦞 diamond lobster.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (live_output): The PR body includes after-fix real CLI proof with a temp OpenClaw home and redacted JSON output showing no wildcard configured rows.
  • proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes after-fix real CLI proof with a temp OpenClaw home and redacted JSON output showing no wildcard configured rows.
Evidence reviewed

PR surface:

Source +6, Tests +19. Total +25 across 3 files.

View PR surface stats
Area Files Added Removed Net
Source 2 6 0 +6
Tests 1 19 0 +19
Docs 0 0 0 0
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 3 25 0 +25

What I checked:

Likely related people:

  • steipete: Peter Steinberger authored the shared model selection helper and allowlist parsing refactors that underpin this model visibility/config path. (role: feature-history owner; confidence: high; commits: 310d2db31241, 3eed32108187, dfc18b7a2b; files: src/agents/model-selection-shared.ts, src/commands/models/list.configured.ts)
  • UB: Current-main blame attributes the visible catalog metadata, wildcard parsing, and configured-row resolver bodies in this checkout to commit 48adcb1. (role: current-main snapshot introducer; confidence: medium; commits: 48adcb162c92; files: src/agents/model-selection-shared.ts, src/commands/models/list.configured.ts)
  • Deepflame: Deepflame recently changed the same model manifest/catalog context area in src/agents/model-selection-shared.ts on main. (role: recent area contributor; confidence: medium; commits: ea2496b00c5c; files: src/agents/model-selection-shared.ts)
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.

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.

@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from f3bc144 to 864653f Compare May 25, 2026 14:45
@giodl73-repo giodl73-repo marked this pull request as ready for review May 25, 2026 14:45
@clawsweeper clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. P2 Normal backlog priority with limited blast radius. labels May 25, 2026
@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper PR egg

✨ Hatched: 🥚 common Clockwork Branchling

Hatch command

Comment @clawsweeper hatch when this PR is hatchable.

Hatchability rules:

  • Merged PRs are hatchable.
  • Open PRs are hatchable when they are status: 👀 ready for maintainer look, status: 🚀 automerge armed, or labeled clawsweeper:automerge.
  • Closed unmerged PRs are hatchable only when one of those hatchable labels is still present in the durable record.

Rarity: 🥚 common.
Trait: sparkles near resolved comments.
Image traits: location release reef; accessory lint brush; palette sunrise gold and clean white; mood sparkly; pose stepping out of a freshly hatched shell; shell woven fiber shell; lighting gentle morning glow; background small green status lights.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Clockwork Branchling 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.
  • Hatchability usually comes from sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness. A merged PR is already final, so merge makes the egg hatchable independently.
  • 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.

@openclaw-barnacle openclaw-barnacle Bot added the commands Command implementations label May 25, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Follow-up for the proof gap:

  • New head: b4601c09d5f
  • Added a matching models list configured-row guard so agents.defaults.models["provider/*"] is treated only as a selector, not as a concrete configured model row.
  • Added focused regression coverage in src/commands/models/list.configured.test.ts.
  • Focused WSL validation passed:
    • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
    • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • git diff --check
  • Real CLI proof with temp OpenClaw home and config containing openai-codex/* plus openai/gpt-5.5 produced one model row, openai/gpt-5.5, and wildcardRows: [].

I updated the PR body with the exact proof command and redacted JSON summary.

@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. 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. and removed rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. labels May 25, 2026
@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from b4601c0 to cf53b81 Compare May 25, 2026 17:04

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Current-head refresh for #86524:

  • Rebases giodl73-repo:fix-agent-model-catalog-wildcard onto current upstream main; new head cf53b81717aa991676f03d70df59bf6e08e7af46.
  • Focused WSL validation passed:
    • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
    • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • git diff --check refs/remotes/upstream-safe/main...HEAD
  • The refreshed commits are SSH-signed with the giodl73 key.
  • The PR body still includes the real models list --json proof showing wildcard selector rows are omitted.

@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. labels May 25, 2026
@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from cf53b81 to a90eb0e Compare May 25, 2026 18:14

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Current-head refresh for #86524:

  • Rebases giodl73-repo:fix-agent-model-catalog-wildcard onto current upstream main; new head a90eb0e25486c553b215b8ce1868789210c414e1.
  • Focused WSL validation passed after rebase:
    • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
    • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • git diff --check refs/remotes/upstream-safe/main...HEAD
  • The PR body still includes the real models list --json proof showing wildcard selector rows are omitted.
  • The refreshed commits are SSH-signed with the giodl73 key.

No merge performed.

@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from a90eb0e to 8ac9707 Compare May 25, 2026 18:44

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Refreshed onto upstream main e844d1d6e56b31fa574b4c2970fdd6e59a9f2e83, which includes the upstream cron-service CI fix. New signed head is 8ac970726a0.

Focused validation after rebase passed:

  • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed

No merge performed.

@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. and removed rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. labels May 25, 2026
@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from 8ac9707 to da4b216 Compare May 25, 2026 20:07

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Resolved the current main conflict on #86524 and refreshed the branch.

  • Rebased giodl73-repo:fix-agent-model-catalog-wildcard onto upstream main fe14bcecee0.
  • New signed head: da4b21618bc0c702d91326a18a3a35d826b2f821.
  • Conflict resolution in src/agents/model-selection-shared.ts preserves upstream's alias-only metadata optimization and adds the wildcard selector skip before resolving concrete keys.
  • Focused WSL validation passed:
    • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
    • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
    • git diff --check refs/remotes/upstream-safe/main...HEAD

No merge performed.

@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from da4b216 to d428da7 Compare May 25, 2026 23:43

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Refreshed #86524 onto current upstream main; new signed head is d428da795ef00c28012e6a65cba4ef34704a5416.

Focused validation after the refresh passed:

  • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
  • ./node_modules/.bin/oxfmt --check ...
  • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json ...
  • git diff --check refs/remotes/upstream-safe/main...HEAD and git diff --check

The existing real models list --json proof remains in the PR body. No merge performed.

@clawsweeper

clawsweeper Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from d428da7 to 98c8c88 Compare May 26, 2026 00:38

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Refreshed #86524 onto current upstream main; new signed head is 98c8c88a241dddc98f936ff8d2b6448275cec3d6.

Focused WSL validation after the refresh passed:

  • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
  • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
  • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
  • git diff --check refs/remotes/upstream-safe/main...HEAD
  • git diff --check

The existing models list --json proof remains in the PR body. No merge performed.

@clawsweeper

clawsweeper Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@giodl73-repo giodl73-repo force-pushed the fix-agent-model-catalog-wildcard branch from 98c8c88 to 01fbfe3 Compare May 26, 2026 00:59

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Refreshed #86524 onto current upstream main; new signed head is 01fbfe3ae08446dec418d14af119872a2cea1a23.

Focused WSL validation after the refresh passed:

  • node scripts/run-vitest.mjs src/commands/models/list.configured.test.ts src/agents/model-catalog-visibility.test.ts --reporter=dot -> 3 files, 9 tests passed
  • ./node_modules/.bin/oxfmt --check src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
  • ./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/commands/models/list.configured.ts src/commands/models/list.configured.test.ts src/agents/model-selection-shared.ts src/agents/model-catalog-visibility.test.ts
  • git diff --check refs/remotes/upstream-safe/main...HEAD
  • git diff --check

The existing models list --json proof remains in the PR body. No merge performed.

@clawsweeper

clawsweeper Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. labels May 26, 2026
@giodl73-repo giodl73-repo merged commit 17ab9b9 into openclaw:main May 26, 2026
114 of 116 checks passed
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 26, 2026
* fix(agents): skip wildcard catalog metadata refs

* fix(models): skip wildcard configured rows
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
* fix(agents): skip wildcard catalog metadata refs

* fix(models): skip wildcard configured rows
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
* fix(agents): skip wildcard catalog metadata refs

* fix(models): skip wildcard configured rows
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
* fix(agents): skip wildcard catalog metadata refs

* fix(models): skip wildcard configured rows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling commands Command implementations maintainer Maintainer-authored PR merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. P2 Normal backlog priority with limited blast radius. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. size: XS status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant