Merged
Conversation
Change claude_args from single-line string to YAML block scalar (|) with each argument on its own line. This matches the format used by the reference project and prevents argument parsing issues.
docs: sync contributing and README
GitHub Actions multiline output syntax requires bare delimiter (EOF),
not quoted ('EOF').
github-actions
bot
added
area/ci
Review Summary by QodoFix workflow argument formatting and sync documentation with current setup
WalkthroughsDescription• Convert claude_args from single-line to YAML block scalar format across 7 workflow files • Fix GitHub Actions heredoc syntax by removing quotes from delimiter in duplicate check workflow • Update CONTRIBUTING.md and README.md with accurate environment requirements and setup instructions • Correct repository URLs from WhaleWhisper to whale-whisper and update localhost port from 5173 to 5174 • Add security vulnerability reporting guidelines and clarify code organization structure Diagramflowchart LR
A["Workflow Files<br/>7 files"] -- "Convert claude_args<br/>to block scalar" --> B["Multiline Format<br/>Arguments"]
C["Heredoc Syntax<br/>duplicate-check.yml"] -- "Remove quotes<br/>from EOF" --> D["Valid GitHub<br/>Actions Output"]
E["CONTRIBUTING.md<br/>README.md"] -- "Update environment<br/>requirements & URLs" --> F["Accurate Setup<br/>Instructions"]
B --> G["Consistent<br/>Workflow Config"]
D --> G
F --> G
File Changes1. .github/workflows/claude-ci-autofix.yml
|
Contributor
Greptile OverviewGreptile SummaryThis PR merges development branch updates that improve GitHub Actions workflows and documentation consistency. Key Changes:
All changes are maintenance updates that improve consistency and correctness without altering application logic. Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| .github/workflows/claude-issue-duplicate-check.yml | Fixed heredoc delimiter to enable variable expansion in GitHub Actions prompt |
| CONTRIBUTING.md | Updated Python version requirements, repository URLs, local dev port, and added security vulnerability reporting section |
| README.md | Updated Python version badge, repository URLs, local dev port, and project structure documentation |
Sequence Diagram
sequenceDiagram
participant Dev as Developer
participant GH as GitHub Actions
participant Claude as Claude Code Action
participant Repo as Repository
Note over Dev,Repo: Workflow Configuration Updates
Dev->>Repo: Push changes to workflow files
Note over Repo: Updated 7 Claude workflow files<br/>with multiline claude_args
Dev->>Repo: Push documentation updates
Note over Repo: Updated CONTRIBUTING.md & README.md<br/>with Python 3.10+ requirements
Note over GH,Claude: Issue Duplicate Check Workflow
GH->>GH: Issue opened trigger
GH->>GH: Load prompt with heredoc (EOF)
Note over GH: Variable expansion enabled<br/>$GITHUB_REPOSITORY, $ISSUE_NUMBER
GH->>Claude: Execute with expanded variables
Claude->>Repo: Search for duplicate issues
Claude->>GH: Label & comment if duplicate found
Note over GH,Claude: Other Claude Workflows
GH->>GH: Various triggers (PR, CI failure, mentions)
GH->>Claude: Execute with multiline args
Note over Claude: --model<br/>--max-turns<br/>--allowedTools
Claude->>Repo: Perform automated tasks
Claude->>GH: Create PRs, comments, reviews
Code Review by Qodo
1. claude_args newline risk
|
|
GitHub API calls via PR Size: S
No issues met the “report only if confidence >= 80” threshold in the changed hunks, so there are no inline review comments to post. REPO="${PR_REPO:-$GITHUB_REPOSITORY}"
PR="${PR_NUMBER}"
LATEST_COMMIT_SHA="e0a230a5b632e0945430be8b1fcd8954dd792e1e"
# Size label (if your runner has GitHub API access)
gh pr edit "$PR" --repo "$REPO" --add-label "size/S"
# Summary review
SUMMARY=$(cat <<'EOF'
## 🤖 Codex PR Review
No significant issues identified in this PR.
### PR Size: S
- **Lines changed**: 124
- **Files changed**: 9
### Review Coverage
- [x] Logic and correctness - Clean
- [x] Security (OWASP Top 10) - Clean
- [x] Error handling - Clean
- [x] Type safety - Clean
- [x] Documentation accuracy - Clean
- [x] Test coverage - Adequate
- [x] Code clarity - Good
---
*Automated review by Codex AI*
EOF
)
gh pr review "$PR" --repo "$REPO" --comment --body "$SUMMARY" |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review Summary
Clean PR that fixes CI workflow configuration issues and synchronizes documentation with the actual project state. All changes verified against the repository.
PR Size: S
Key findings:
- Heredoc delimiter fix (
claude-issue-duplicate-check.yml): The change from'EOF'toEOFis a genuine bug fix. The old code set the GitHub Actions multiline output delimiter to the literal string'EOF'(with quotes) viaecho "prompt<<'EOF'", but attempted to close it withecho "EOF"(without quotes). These delimiters don't match, so the multiline output would never terminate properly. claude_argsmultiline formatting: Converting from single-line strings to YAML block scalars (|) across 7 workflow files. This is a valid formatting improvement — theclaude_argsinput is a composite action parameter that accepts multiline strings.- Documentation updates: Verified against actual project state — Python
>= 3.10matchesbackend/pyproject.toml, port5174matchesfrontend/apps/web/vite.config.ts, repo URL casing matches the actual GitHub repository, directory structure matches the filesystem, and.github/SECURITY.mdexists as referenced.
Issues Found
| Category | Critical | High | Medium | Low |
|---|---|---|---|---|
| Logic and correctness | 0 | 0 | 0 | 0 |
| Security | 0 | 0 | 0 | 0 |
| Error handling | 0 | 0 | 0 | 0 |
| Type safety | 0 | 0 | 0 | 0 |
| Documentation accuracy | 0 | 0 | 0 | 0 |
| Test coverage | 0 | 0 | 0 | 0 |
| Code clarity | 0 | 0 | 0 | 0 |
Review Coverage
- Logic and correctness
- Security (OWASP Top 10)
- Error handling
- Type safety
- Documentation accuracy
- Test coverage
- Code clarity
Automated review by Claude AI
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
修复所有 Claude CI 工作流中
claude_args的格式问题（改为 YAML 多行格式），修复 heredoc 定界符引号问题，并同步 CONTRIBUTING.md 和 README.md 使其与项目当前实际状态一致。问题
claude_args使用单行字符串传参，不符合 action spec 的多行格式要求，可能导致参数解析异常。claude-issue-duplicate-check.yml中 heredoc 定界符带引号（'EOF'），导致变量替换行为不符合预期。关联 PR：
解决方案
claude_args从单行字符串改为 YAML 多行块标量（|），每个参数独占一行，提升可读性并符合 action 规范。claude-issue-duplicate-check.yml中 heredoc 定界符的多余引号。变更内容
CI 工作流修复
claude_args多行格式化（7 个文件）：claude-ci-autofix.yml、claude-issue-auto-response.yml、claude-issue-duplicate-check.yml、claude-mention-responder.yml、claude-pr-description.yml、claude-pr-review.yml、claude-review-responder.ymlclaude-issue-duplicate-check.yml中'EOF'→EOF文档同步（CONTRIBUTING.md + README.md）
WhaleWhisper.git→whale-whisper.git（含 Issues、Discussions 链接）uvicorn→uv run uvicornfrontend/apps/web、frontend/apps/desktop-tauri、backend/examples、backend/scripts、data/等实际结构pnpm run typecheck命令base-nlp→whale-whisper.github/SECURITY.md测试
自测方式
Checklist
由 Claude AI 自动生成