refactor(pdk): extract shared scope filtering logic

- create utils/commit.ts for shared scope extraction functions
- replace duplicate scope filtering logic in github.ts and ai-changelog.ts
- add comprehensive testing documentation

Author: ulivz

infra/pdk/docs/filter-scopes-testing.md

@@ -0,0 +1,111 @@
+# PDK Filter Scopes 功能测试指南
+
+## 功能概述
+
+PDK 的 `--filter-scopes` 参数用于在生成 changelog 时过滤特定 scope 的 commit。默认情况下支持全部 scope，可通过 CLI 参数指定需要包含的 scope。
+
+## 测试场景
+
+### 1. 基本功能测试
+
+#### 测试默认行为（包含所有 scope）
+```bash
+# 生成包含所有 scope 的 changelog
+pdk changelog --from-tag=v1.0.0 --to-tag=v1.1.0
+# 或
+pdk release --filter-scopes=""
+```
+
+#### 测试指定 scope 过滤
+```bash
+# 只生成特定 scope 的 changelog
+pdk changelog --from-tag=v1.0.0 --to-tag=v1.1.0 --filter-scopes=tars,agent
+# 或
+pdk release --filter-scopes=tars,agent
+```
+
+### 2. AI 生成 changelog 测试
+
+#### 使用 AI 生成 + scope 过滤
+```bash
+pdk changelog --use-ai --filter-scopes=tars,agent --provider=azure-openai --model=gpt-4o
+# 或
+pdk release --use-ai --filter-scopes=tars,agent --provider=azure-openai --model=gpt-4o
+```
+
+### 3. 实际项目测试
+
+#### 在 multimodal 项目中测试
+```bash
+cd multimodal
+# 使用预配置的 scope 过滤
+pnpm release:ai:dryrun
+```
+
+#### 在 infra 项目中测试
+```bash
+cd infra
+# 测试包含所有 scope（适合 infra monorepo）
+pdk changelog --filter-scopes=""
+# 或指定 infra 相关 scope
+pdk changelog --filter-scopes=infra,pdk,build
+```
+
+## 验证方法
+
+### 1. 检查 commit 过滤效果
+创建测试 commit：
+```bash
+feat(tars): add new feature
+feat(agent): improve performance  
+feat(infra): update build system
+chore: update dependencies
+```
+
+使用不同 filter 参数验证：
+```bash
+# 应包含所有 commit
+pdk changelog --filter-scopes=""
+
+# 只包含 tars 和 agent 的 feat
+pdk changelog --filter-scopes="tars,agent"
+
+# 只包含 infra 相关
+pdk changelog --filter-scopes="infra,pdk,build"
+```
+
+### 2. 验证 AI 生成结果
+```bash
+# AI 生成时应用 scope 过滤
+pdk changelog --use-ai --filter-scopes="tars,agent" --dry-run
+```
+
+### 3. 验证 GitHub Release 生成
+```bash
+# 生成 GitHub Release 时应用 scope 过滤
+pdk release --create-github-release --filter-scopes="tars,agent" --dry-run
+```
+
+## 常见问题排查
+
+### 1. 空结果
+- 检查 tag 是否存在：`git tag --list`
+- 检查 commit 格式是否符合 conventional commits
+- 验证 scope 名称是否正确
+
+### 2. 过滤不生效
+- 确认 scope 名称大小写敏感
+- 检查是否有空格或特殊字符
+- 验证 commit message 格式：`type(scope): description`
+
+### 3. AI 生成异常
+- 检查 API key 和配置
+- 验证网络连接
+- 查看详细错误日志
+
+## 最佳实践
+
+1. **项目配置**：在 package.json 中预定义常用的 scope 组合
+2. **CI/CD 集成**：在发布流程中使用固定的 scope 过滤
+3. **文档维护**：保持 scope 命名规范的一致性
+4. **测试覆盖**：定期验证不同 scope 组合的生成效果

infra/pdk/src/utils/ai-changelog.ts

@@ -1,6 +1,7 @@
 import { existsSync, readFileSync, writeFileSync } from 'fs';
 import { execa } from 'execa';
 import { logger } from './logger';
+import { shouldIncludeCommitByScope } from './commit';
 import { AgentModel } from '@tarko/model-provider';
 
 interface CommitEntry {
@@ -103,26 +104,9 @@ export class AIChangelogGenerator {
 
       // Apply scope filter if provided
       if (filterScopes && filterScopes.length > 0) {
-        return commits.filter((commit) => {
-          const match = commit.message.match(/^(\w+)(\([^)]+\))?:\s*(.+)$/);
-          if (match) {
-            const [, , scopeStr] = match;
-            
-            // Extract scope from scopeStr (remove parentheses)
-            let scope = '';
-            if (scopeStr) {
-              const scopeMatch = scopeStr.match(/^\(([^)]+)\)$/);
-              scope = scopeMatch ? scopeMatch[1] : '';
-            }
-            
-            // Include if no scope or scope matches filter
-            if (!scope || filterScopes.includes(scope) || filterScopes.includes('all')) {
-              return true;
-            }
-            return false;
-          }
-          return true; // Include non-conventional commits
-        });
+        return commits.filter((commit) => 
+          shouldIncludeCommitByScope(commit.message, filterScopes)
+        );
       }
 
       return commits;

infra/pdk/src/utils/commit.ts

@@ -0,0 +1,45 @@
+/**
+ * Git commit utilities
+ */
+
+/**
+ * Extract scope from conventional commit message
+ * @param commitMessage - The commit message to extract scope from
+ * @returns The extracted scope or empty string if no scope found
+ */
+export function extractScopeFromCommit(commitMessage: string): string {
+  const match = commitMessage.match(/^(\w+)(\([^)]+\))?:\s*(.+)$/);
+  if (!match) {
+    return '';
+  }
+
+  const [, , scopeStr] = match;
+  
+  if (!scopeStr) {
+    return '';
+  }
+
+  const scopeMatch = scopeStr.match(/^\(([^)]+)\)$/);
+  return scopeMatch ? scopeMatch[1] : '';
+}
+
+/**
+ * Check if a commit should be included based on scope filters
+ * @param commitMessage - The commit message to check
+ * @param filterScopes - Array of scopes to include (empty array means include all)
+ * @returns True if the commit should be included
+ */
+export function shouldIncludeCommitByScope(
+  commitMessage: string,
+  filterScopes?: string[],
+): boolean {
+  // If no filter provided, include all commits
+  if (!filterScopes || filterScopes.length === 0) {
+    return true;
+  }
+
+  const scope = extractScopeFromCommit(commitMessage);
+  
+  // Include if no scope or scope matches filter
+  return !scope || filterScopes.includes(scope) || filterScopes.includes('all');
+}

infra/pdk/src/utils/github.ts

@@ -8,6 +8,7 @@
  */
 import { execa } from 'execa';
 import { logger } from './logger';
+import { shouldIncludeCommitByScope } from './commit';
 
 // Username mapping for commit authors to correct GitHub usernames
 const USERNAME_MAP: Record<string, string> = {
@@ -138,20 +139,11 @@ export async function generateReleaseNotes(
     commits.forEach((commit) => {
       const match = commit.subject.match(/^(\w+)(\([^)]+\))?:\s*(.+)$/);
       if (match) {
-        const [, type, scopeStr] = match;
-        
-        // Extract scope from scopeStr (remove parentheses)
-        let scope = '';
-        if (scopeStr) {
-          const scopeMatch = scopeStr.match(/^\(([^)]+)\)$/);
-          scope = scopeMatch ? scopeMatch[1] : '';
-        }
+        const [, type] = match;
 
         // Apply scope filter if provided
-        if (filterScopes && filterScopes.length > 0 && scope) {
-          if (!filterScopes.includes(scope) && !filterScopes.includes('all')) {
-            return; // Skip this commit
-          }
+        if (!shouldIncludeCommitByScope(commit.subject, filterScopes)) {
+          return; // Skip this commit
         }
 
         if (type in groups) {