fix: improve logging for invalid embedding_dimensions configuration

- Add try-except around int() conversion in _embedding_kwargs to
  gracefully handle invalid embedding_dimensions config values
- Update get_dim() to return 0 when embedding_dimensions is not
  explicitly configured, so callers know dimensions weren't specified
  and can handle it accordingly
- Both methods now share consistent logic for reading the config

Signed-off-by: JiangNan <1394485448@qq.com>

.github/PULL_REQUEST_TEMPLATE.md

@@ -21,7 +21,23 @@
 <!--If merged, your code will serve tens of thousands of users! Please double-check the following items before submitting.-->
 <!--如果分支被合并，您的代码将服务于数万名用户！在提交前，请核查一下几点内容。-->
 
-- [ ] 😊 如果 PR 中有新加入的功能，已经通过 Issue / 邮件等方式和作者讨论过。/ If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
-- [ ] 👀 我的更改经过了良好的测试，**并已在上方提供了“验证步骤”和“运行截图”**。/ My changes have been well-tested, **and "Verification Steps" and "Screenshots" have been provided above**.
-- [ ] 🤓 我确保没有引入新依赖库，或者引入了新依赖库的同时将其添加到了 `requirements.txt` 和 `pyproject.toml` 文件相应位置。/ I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in `requirements.txt` and `pyproject.toml`.
-- [ ] 😮 我的更改没有引入恶意代码。/ My changes do not introduce malicious code.
+- [ ] 😊 如果 PR 中有新加入的功能，已经通过 Issue / 邮件等方式和作者讨论过。
+  / If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
+
+- [ ] 👀 我的更改经过了良好的测试，**并已在上方提供了“验证步骤”和“运行截图”**。
+  / My changes have been well-tested, **and "Verification Steps" and "Screenshots" have been provided above**.
+
+- [ ] 🤓 我确保没有引入新依赖库，或者引入了新依赖库的同时将其添加到 `requirements.txt` 和 `pyproject.toml` 文件相应位置。
+  / I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in `requirements.txt` and `pyproject.toml`.
+
+- [ ] 😮 我的更改没有引入恶意代码。
+  / My changes do not introduce malicious code.
+
+- [ ] ⚠️ 我已认真阅读并理解以上所有内容，确保本次提交符合规范。
+  / I have read and understood all the above and confirm this PR follows the rules.
+
+- [ ] 🚀 我确保本次开发**基于 dev 分支**，并将代码合并至**开发分支**（除非极其紧急，才允许合并到主分支）。
+   / I confirm that this development is **based on the dev branch** and will be merged into the **development branch**, unless it is extremely urgent to merge into the main branch.
+
+- [ ] ⚠️ 我**没有**认真阅读以上内容，直接提交。
+  / I **did not** read the above carefully before submitting.

.github/workflows/pr-checklist-check.yml

@@ -0,0 +1,45 @@
+name: PR Checklist Check
+
+on:
+  pull_request_target:
+    types: [opened, edited, reopened, synchronize]
+
+jobs:
+  check:
+    runs-on: ubuntu-latest
+
+    permissions:
+      pull-requests: write
+      issues: write
+
+    steps:
+      - name: Check checklist
+        id: check
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const body = context.payload.pull_request.body || "";
+            const regex = /-\s*\[\s*x\s*\].*没有.*认真阅读/i;
+            const bad = regex.test(body);
+            core.setOutput("bad", bad);
+
+      - name: Close PR
+        if: steps.check.outputs.bad == 'true'
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const pr = context.payload.pull_request;
+
+            await github.rest.issues.createComment({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              issue_number: pr.number,
+              body: `检测到你勾选了“我没有认真阅读”，PR 已关闭。`
+            });
+
+            await github.rest.pulls.update({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              pull_number: pr.number,
+              state: "closed"
+            });

astrbot/core/platform/sources/wecom_ai_bot/wecomai_adapter.py

@@ -440,9 +440,16 @@ class WecomAIBotAdapter(Platform):
         )
 
     def _extract_session_id(self, message_data: dict[str, Any]) -> str:
-        """从消息数据中提取会话ID"""
-        user_id = message_data.get("from", {}).get("userid", "default_user")
-        return format_session_id("wecomai", user_id)
+        """从消息数据中提取会话ID
+        群聊使用 chatid，单聊使用 userid
+        """
+        chattype = message_data.get("chattype", "single")
+        if chattype == "group":
+            chat_id = message_data.get("chatid", "default_group")
+            return format_session_id("wecomai", chat_id)
+        else:
+            user_id = message_data.get("from", {}).get("userid", "default_user")
+            return format_session_id("wecomai", user_id)
 
     async def _enqueue_message(
         self,

astrbot/core/provider/manager.py

@@ -808,6 +808,8 @@ class ProviderManager:
             config.save_config()
             # load instance
             await self.load_provider(new_config)
+            # sync in-memory config for API queries (e.g., embedding provider list)
+            self.providers_config = astrbot_config["provider"]
 
     async def terminate(self) -> None:
         if self._mcp_init_task and not self._mcp_init_task.done():

astrbot/core/provider/sources/groq_source.py

@@ -13,3 +13,11 @@ class ProviderGroq(ProviderOpenAIOfficial):
     ) -> None:
         super().__init__(provider_config, provider_settings)
         self.reasoning_key = "reasoning"
+
+    def _finally_convert_payload(self, payloads: dict) -> None:
+        """Groq rejects assistant history items that include reasoning_content."""
+        super()._finally_convert_payload(payloads)
+        for message in payloads.get("messages", []):
+            if message.get("role") == "assistant":
+                message.pop("reasoning_content", None)
+                message.pop("reasoning", None)

astrbot/core/provider/sources/openai_embedding_source.py

@@ -40,25 +40,46 @@ class OpenAIEmbeddingProvider(EmbeddingProvider):
 
     async def get_embedding(self, text: str) -> list[float]:
         """获取文本的嵌入"""
+        kwargs = self._embedding_kwargs()
         embedding = await self.client.embeddings.create(
             input=text,
             model=self.model,
-            dimensions=self.get_dim(),
+            **kwargs,
         )
         return embedding.data[0].embedding
 
     async def get_embeddings(self, text: list[str]) -> list[list[float]]:
         """批量获取文本的嵌入"""
+        kwargs = self._embedding_kwargs()
         embeddings = await self.client.embeddings.create(
             input=text,
             model=self.model,
-            dimensions=self.get_dim(),
+            **kwargs,
         )
         return [item.embedding for item in embeddings.data]
 
+    def _embedding_kwargs(self) -> dict:
+        """构建嵌入请求的可选参数"""
+        kwargs = {}
+        if "embedding_dimensions" in self.provider_config:
+            try:
+                kwargs["dimensions"] = int(self.provider_config["embedding_dimensions"])
+            except (ValueError, TypeError):
+                logger.warning(
+                    f"embedding_dimensions in embedding configs is not a valid integer: '{self.provider_config['embedding_dimensions']}', ignored."
+                )
+        return kwargs
+
     def get_dim(self) -> int:
         """获取向量的维度"""
-        return int(self.provider_config.get("embedding_dimensions", 1024))
+        if "embedding_dimensions" in self.provider_config:
+            try:
+                return int(self.provider_config["embedding_dimensions"])
+            except (ValueError, TypeError):
+                logger.warning(
+                    f"embedding_dimensions in embedding configs is not a valid integer: '{self.provider_config['embedding_dimensions']}', ignored."
+                )
+        return 0
 
     async def terminate(self):
         if self.client:

astrbot/core/umop_config_router.py

@@ -25,12 +25,22 @@ class UmopConfigRouter:
         )
         self.umop_to_conf_id = sp_data
 
+    @staticmethod
+    def _split_umo(umo: str) -> tuple[str, str, str] | None:
+        """将 UMO 拆分为 3 个部分，同时保留 session_id 中的 ':'"""
+        if not isinstance(umo, str):
+            return None
+        parts = umo.split(":", 2)
+        if len(parts) != 3:
+            return None
+        return parts[0], parts[1], parts[2]
+
     def _is_umo_match(self, p1: str, p2: str) -> bool:
         """判断 p2 umo 是否逻辑包含于 p1 umo"""
-        p1_ls = p1.split(":")
-        p2_ls = p2.split(":")
+        p1_ls = self._split_umo(p1)
+        p2_ls = self._split_umo(p2)
 
-        if len(p1_ls) != 3 or len(p2_ls) != 3:
+        if p1_ls is None or p2_ls is None:
             return False  # 非法格式
 
         return all(p == "" or fnmatch.fnmatchcase(t, p) for p, t in zip(p1_ls, p2_ls))
@@ -62,7 +72,7 @@ class UmopConfigRouter:
 
         """
         for part in new_routing:
-            if not isinstance(part, str) or len(part.split(":")) != 3:
+            if self._split_umo(part) is None:
                 raise ValueError(
                     "umop keys must be strings in the format [platform_id]:[message_type]:[session_id], with optional wildcards * or empty for all",
                 )
@@ -81,7 +91,7 @@ class UmopConfigRouter:
             ValueError: 如果 umo 格式不正确
 
         """
-        if not isinstance(umo, str) or len(umo.split(":")) != 3:
+        if self._split_umo(umo) is None:
             raise ValueError(
                 "umop must be a string in the format [platform_id]:[message_type]:[session_id], with optional wildcards * or empty for all",
             )
@@ -99,7 +109,7 @@ class UmopConfigRouter:
             ValueError: 当 umo 格式不正确时抛出
         """
 
-        if not isinstance(umo, str) or len(umo.split(":")) != 3:
+        if self._split_umo(umo) is None:
             raise ValueError(
                 "umop must be a string in the format [platform_id]:[message_type]:[session_id], with optional wildcards * or empty for all",
             )

docs/zh/deploy/astrbot/rainyun.md

@@ -41,4 +41,4 @@ AstrBot 已经上架至雨云的预装软件列表，支持**一键安装** Astr
 
 ![创建NAT端口映射规则](https://files.astrbot.app/docs/source/images/rainyun/image-2.png)
 
-然后，内网端口填写 `6185`，点击 `创建映射规则`，这样就可以通过 `http://IP:上面设置好的外网端口` 访问 AstrBot 的管理面板了。
+然后，内网端口填写 `6185`，点击 `创建映射规则`，这样就可以通过 `http://IP:上面设置好的外网端口` 访问 AstrBot 的管理面板了。如果无法打开，请点击`备用地址`，通过备用地址访问管理面板。

scripts/close_duplicate_plugin_publish_issues.py

@@ -1,196 +0,0 @@
-from __future__ import annotations
-
-import argparse
-import json
-import subprocess
-import sys
-from collections import defaultdict
-from dataclasses import dataclass
-from datetime import datetime
-
-
-@dataclass(frozen=True)
-class Issue:
-    number: int
-    title: str
-    created_at: datetime
-    url: str
-
-
-def parse_args() -> argparse.Namespace:
-    parser = argparse.ArgumentParser(
-        description=(
-            "Close duplicate open plugin-publish issues while keeping the latest one."
-        )
-    )
-    parser.add_argument(
-        "--repo",
-        default="AstrBotDevs/AstrBot",
-        help="GitHub repository in owner/name format.",
-    )
-    parser.add_argument(
-        "--label",
-        default="plugin-publish",
-        help="Issue label to target.",
-    )
-    parser.add_argument(
-        "--limit",
-        type=int,
-        default=1000,
-        help="Maximum number of open issues to inspect.",
-    )
-    parser.add_argument(
-        "--apply",
-        action="store_true",
-        help="Actually close duplicate issues. Defaults to dry-run.",
-    )
-    return parser.parse_args()
-
-
-def run_gh_command(args: list[str]) -> str:
-    try:
-        completed = subprocess.run(
-            args,
-            check=True,
-            capture_output=True,
-            text=True,
-        )
-    except FileNotFoundError as exc:
-        raise RuntimeError("GitHub CLI `gh` is not installed or not in PATH.") from exc
-    except subprocess.CalledProcessError as exc:
-        stderr = exc.stderr.strip()
-        stdout = exc.stdout.strip()
-        details = stderr or stdout or str(exc)
-        raise RuntimeError(f"`{' '.join(args)}` failed: {details}") from exc
-    return completed.stdout
-
-
-def load_open_issues(repo: str, label: str, limit: int) -> list[Issue]:
-    output = run_gh_command(
-        [
-            "gh",
-            "issue",
-            "list",
-            "--repo",
-            repo,
-            "--label",
-            label,
-            "--state",
-            "open",
-            "--limit",
-            str(limit),
-            "--json",
-            "number,title,createdAt,url",
-        ]
-    )
-    items = json.loads(output)
-    return [
-        Issue(
-            number=item["number"],
-            title=item["title"],
-            created_at=datetime.fromisoformat(item["createdAt"].replace("Z", "+00:00")),
-            url=item["url"],
-        )
-        for item in items
-    ]
-
-
-def normalize_title(title: str) -> str:
-    return " ".join(title.split()).strip()
-
-
-def find_duplicates(
-    issues: list[Issue],
-) -> list[tuple[Issue, list[Issue]]]:
-    grouped: dict[str, list[Issue]] = defaultdict(list)
-    for issue in issues:
-        grouped[normalize_title(issue.title)].append(issue)
-
-    duplicate_groups: list[tuple[Issue, list[Issue]]] = []
-    for group in grouped.values():
-        if len(group) < 2:
-            continue
-        ordered = sorted(
-            group,
-            key=lambda issue: (issue.created_at, issue.number),
-            reverse=True,
-        )
-        keep = ordered[0]
-        close_candidates = ordered[1:]
-        duplicate_groups.append((keep, close_candidates))
-
-    duplicate_groups.sort(
-        key=lambda item: (item[0].created_at, item[0].number),
-        reverse=True,
-    )
-    return duplicate_groups
-
-
-def print_plan(duplicate_groups: list[tuple[Issue, list[Issue]]], apply: bool) -> None:
-    action = "Will close" if apply else "Would close"
-    if not duplicate_groups:
-        print("No duplicate open issues found.")
-        return
-
-    total_to_close = sum(len(close_list) for _, close_list in duplicate_groups)
-    print(f"Found {len(duplicate_groups)} duplicate title groups.")
-    print(
-        f"{action} {total_to_close} issues and keep {len(duplicate_groups)} latest issues."
-    )
-
-    for keep, close_list in duplicate_groups:
-        print()
-        print(f'Keep  #{keep.number} [{keep.created_at.isoformat()}] "{keep.title}"')
-        print(f"      {keep.url}")
-        for issue in close_list:
-            print(
-                f'Close #{issue.number} [{issue.created_at.isoformat()}] "{issue.title}"'
-            )
-            print(f"      {issue.url}")
-
-
-def close_duplicates(
-    repo: str, duplicate_groups: list[tuple[Issue, list[Issue]]]
-) -> None:
-    for keep, close_list in duplicate_groups:
-        reason = (
-            f"Closing as duplicate of #{keep.number}. "
-            "Keeping the latest open issue with this title."
-        )
-        for issue in close_list:
-            print(f"Closing #{issue.number} as duplicate of #{keep.number}...")
-            run_gh_command(
-                [
-                    "gh",
-                    "issue",
-                    "close",
-                    str(issue.number),
-                    "--repo",
-                    repo,
-                    "--comment",
-                    reason,
-                ]
-            )
-
-
-def main() -> int:
-    args = parse_args()
-    try:
-        issues = load_open_issues(args.repo, args.label, args.limit)
-        duplicate_groups = find_duplicates(issues)
-        print_plan(duplicate_groups, apply=args.apply)
-        if args.apply and duplicate_groups:
-            print()
-            close_duplicates(args.repo, duplicate_groups)
-            print("Done.")
-        elif not args.apply:
-            print()
-            print("Dry-run only. Re-run with `--apply` to close the duplicates.")
-    except RuntimeError as exc:
-        print(str(exc), file=sys.stderr)
-        return 1
-    return 0
-
-
-if __name__ == "__main__":
-    raise SystemExit(main())

scripts/generate_changelog.py

@@ -1,253 +0,0 @@
-#!/usr/bin/env python3
-"""
-Auto-generate changelog from git commits using LLM.
-Usage: python scripts/generate_changelog.py [--version VERSION]
-"""
-
-import argparse
-import os
-import re
-import subprocess
-import sys
-from pathlib import Path
-
-
-def get_latest_tag():
-    """Get the latest git tag."""
-    result = subprocess.run(
-        ["git", "describe", "--tags", "--abbrev=0"],
-        capture_output=True,
-        text=True,
-        check=True,
-    )
-    return result.stdout.strip()
-
-
-def get_commits_since_tag(tag):
-    """Get all commit messages since the specified tag."""
-    result = subprocess.run(
-        ["git", "log", f"{tag}..HEAD", "--pretty=format:%H|%s|%b"],
-        capture_output=True,
-        text=True,
-        check=True,
-    )
-    commits = []
-    for line in result.stdout.strip().split("\n"):
-        if not line:
-            continue
-        parts = line.split("|", 2)
-        if len(parts) >= 2:
-            commit_hash = parts[0]
-            subject = parts[1]
-            body = parts[2] if len(parts) > 2 else ""
-            commits.append({"hash": commit_hash[:7], "subject": subject, "body": body})
-    return commits
-
-
-def extract_issue_number(text):
-    """Extract issue number from commit message."""
-    # Match #1234 or (#1234)
-    match = re.search(r"#(\d+)", text)
-    return match.group(1) if match else None
-
-
-def call_llm_for_changelog(commits, version):
-    """Call LLM to generate changelog from commits."""
-    try:
-        # Try to use OpenAI API or other LLM providers
-        import openai
-
-        # Build prompt
-        commits_text = "\n".join([f"- {c['subject']}" for c in commits])
-
-        prompt = f"""Based on the following git commit messages, generate a changelog document in BOTH Chinese and English.
-
-Commit messages:
-{commits_text}
-
-Please organize the changes into these categories:
-- 新增 (New Features)
-- 修复 (Bug Fixes)
-- 优化 (Improvements)
-- 其他 (Others)
-
-Format requirements:
-1. Start with Chinese version under "## What's Changed"
-2. Follow with English version under "## What's Changed (EN)"
-3. Use markdown format with proper bullet points
-4. Keep descriptions concise and user-friendly
-5. If a commit mentions an issue number (#1234), include it in the format ([#1234](https://github.com/AstrBotDevs/AstrBot/issues/1234))
-
-Example format:
-## What's Changed
-
-### 新增
-- 支持某某功能 ([#1234](https://github.com/AstrBotDevs/AstrBot/issues/1234))
-
-### 修复
-- 修复某某问题
-
-## What's Changed (EN)
-
-### New Features
-- Add support for something ([#1234](https://github.com/AstrBotDevs/AstrBot/issues/1234))
-
-### Bug Fixes
-- Fix something
-"""
-
-        client = openai.OpenAI(
-            api_key=os.getenv("OPENAI_API_KEY"),
-            base_url=os.getenv("OPENAI_BASE_URL", "https://api.openai.com/v1"),
-        )
-
-        response = client.chat.completions.create(
-            model=os.getenv("OPENAI_MODEL", "gpt-4"),
-            messages=[
-                {
-                    "role": "system",
-                    "content": "You are a helpful assistant that generates well-structured changelogs.",
-                },
-                {"role": "user", "content": prompt},
-            ],
-            temperature=0.3,
-        )
-
-        return response.choices[0].message.content
-
-    except ImportError:
-        print(
-            "Warning: openai package not installed. Install it with: pip install openai"
-        )
-        return generate_simple_changelog(commits)
-    except Exception as e:
-        print(f"Warning: Failed to call LLM API: {e}")
-        print("Falling back to simple changelog generation...")
-        return generate_simple_changelog(commits)
-
-
-def generate_simple_changelog(commits):
-    """Generate a simple changelog without LLM."""
-    sections = {
-        "feat": ("新增", "New Features", []),
-        "fix": ("修复", "Bug Fixes", []),
-        "perf": ("优化", "Improvements", []),
-        "docs": ("文档", "Documentation", []),
-        "refactor": ("重构", "Refactoring", []),
-        "test": ("测试", "Tests", []),
-        "chore": ("其他", "Chore", []),
-        "other": ("其他", "Others", []),
-    }
-
-    # Categorize commits by conventional commit type
-    for commit in commits:
-        subject = commit["subject"]
-        issue_num = extract_issue_number(subject)
-        issue_link = (
-            f" ([#{issue_num}](https://github.com/AstrBotDevs/AstrBot/issues/{issue_num}))"
-            if issue_num
-            else ""
-        )
-
-        # Detect conventional commit type
-        matched = False
-        for prefix in ["feat", "fix", "perf", "docs", "refactor", "test", "chore"]:
-            if subject.lower().startswith(f"{prefix}:") or subject.lower().startswith(
-                f"{prefix}("
-            ):
-                # Remove prefix for display
-                clean_subject = re.sub(
-                    r"^[a-z]+(\([^)]+\))?:\s*", "", subject, flags=re.IGNORECASE
-                )
-                sections[prefix][2].append(f"- {clean_subject}{issue_link}")
-                matched = True
-                break
-
-        if not matched:
-            sections["other"][2].append(f"- {subject}{issue_link}")
-
-    # Build Chinese version
-    changelog_zh = "## What's Changed\n\n"
-    for section_key in ["feat", "fix", "perf", "docs", "refactor", "test", "other"]:
-        zh_title, _, items = sections[section_key]
-        if items:
-            changelog_zh += f"### {zh_title}\n\n"
-            changelog_zh += "\n".join(items) + "\n\n"
-
-    # Build English version
-    changelog_en = "## What's Changed (EN)\n\n"
-    for section_key in ["feat", "fix", "perf", "docs", "refactor", "test", "other"]:
-        _, en_title, items = sections[section_key]
-        if items:
-            changelog_en += f"### {en_title}\n\n"
-            changelog_en += "\n".join(items) + "\n\n"
-
-    return changelog_zh + changelog_en
-
-
-def main() -> None:
-    parser = argparse.ArgumentParser(description="Generate changelog from git commits")
-    parser.add_argument(
-        "--version", help="Version number for the changelog (e.g., v4.13.3)"
-    )
-    parser.add_argument(
-        "--use-llm",
-        action="store_true",
-        help="Use LLM to generate changelog (requires OpenAI API key)",
-    )
-    args = parser.parse_args()
-
-    # Get latest tag
-    try:
-        latest_tag = get_latest_tag()
-        print(f"Latest tag: {latest_tag}")
-    except subprocess.CalledProcessError:
-        print("Error: No tags found in repository")
-        sys.exit(1)
-
-    # Get commits since tag
-    commits = get_commits_since_tag(latest_tag)
-    if not commits:
-        print(f"No commits found since {latest_tag}")
-        sys.exit(0)
-
-    print(f"Found {len(commits)} commits since {latest_tag}")
-
-    # Determine version
-    if args.version:
-        version = args.version
-    else:
-        # Auto-increment patch version
-        match = re.match(r"v(\d+)\.(\d+)\.(\d+)", latest_tag)
-        if match:
-            major, minor, patch = map(int, match.groups())
-            version = f"v{major}.{minor}.{patch + 1}"
-        else:
-            print(f"Warning: Could not parse version from tag {latest_tag}")
-            version = "vX.X.X"
-
-    print(f"Generating changelog for {version}...")
-
-    # Generate changelog
-    if args.use_llm:
-        changelog_content = call_llm_for_changelog(commits, version)
-    else:
-        changelog_content = generate_simple_changelog(commits)
-
-    # Save to file
-    changelog_dir = Path(__file__).parent.parent / "changelogs"
-    changelog_dir.mkdir(exist_ok=True)
-    changelog_file = changelog_dir / f"{version}.md"
-
-    with open(changelog_file, "w", encoding="utf-8") as f:
-        f.write(changelog_content)
-
-    print(f"\n✓ Changelog generated: {changelog_file}")
-    print("\nPreview:")
-    print("=" * 80)
-    print(changelog_content)
-    print("=" * 80)
-
-
-if __name__ == "__main__":
-    main()

tests/test_openai_source.py

@@ -2,6 +2,7 @@ from types import SimpleNamespace
 
 import pytest
 
+from astrbot.core.provider.sources.groq_source import ProviderGroq
 from astrbot.core.provider.sources.openai_source import ProviderOpenAIOfficial
 
 
@@ -32,6 +33,21 @@ def _make_provider(overrides: dict | None = None) -> ProviderOpenAIOfficial:
     )
 
 
+def _make_groq_provider(overrides: dict | None = None) -> ProviderGroq:
+    provider_config = {
+        "id": "test-groq",
+        "type": "groq_chat_completion",
+        "model": "qwen/qwen3-32b",
+        "key": ["test-key"],
+    }
+    if overrides:
+        provider_config.update(overrides)
+    return ProviderGroq(
+        provider_config=provider_config,
+        provider_settings={},
+    )
+
+
 @pytest.mark.asyncio
 async def test_handle_api_error_content_moderated_removes_images():
     provider = _make_provider(
@@ -198,6 +214,57 @@ def test_extract_error_text_candidates_truncates_long_response_text():
     )
 
 
+@pytest.mark.asyncio
+async def test_openai_payload_keeps_reasoning_content_in_assistant_history():
+    provider = _make_provider()
+    try:
+        payloads = {
+            "messages": [
+                {
+                    "role": "assistant",
+                    "content": [
+                        {"type": "think", "think": "step 1"},
+                        {"type": "text", "text": "final answer"},
+                    ],
+                }
+            ]
+        }
+
+        provider._finally_convert_payload(payloads)
+
+        assistant_message = payloads["messages"][0]
+        assert assistant_message["content"] == [{"type": "text", "text": "final answer"}]
+        assert assistant_message["reasoning_content"] == "step 1"
+    finally:
+        await provider.terminate()
+
+
+@pytest.mark.asyncio
+async def test_groq_payload_drops_reasoning_content_from_assistant_history():
+    provider = _make_groq_provider()
+    try:
+        payloads = {
+            "messages": [
+                {
+                    "role": "assistant",
+                    "content": [
+                        {"type": "think", "think": "step 1"},
+                        {"type": "text", "text": "final answer"},
+                    ],
+                }
+            ]
+        }
+
+        provider._finally_convert_payload(payloads)
+
+        assistant_message = payloads["messages"][0]
+        assert assistant_message["content"] == [{"type": "text", "text": "final answer"}]
+        assert "reasoning_content" not in assistant_message
+        assert "reasoning" not in assistant_message
+    finally:
+        await provider.terminate()
+
+
 @pytest.mark.asyncio
 async def test_handle_api_error_content_moderated_without_images_raises():
     provider = _make_provider(