Skip to content

docs: add review workflow to CLAUDE.md#13145

Merged
EurFelux merged 1 commit intomainfrom
chore/review
Mar 2, 2026
Merged

docs: add review workflow to CLAUDE.md#13145
EurFelux merged 1 commit intomainfrom
chore/review

Conversation

@EurFelux
Copy link
Copy Markdown
Collaborator

@EurFelux EurFelux commented Mar 2, 2026

What this PR does

Before this PR:
CLAUDE.md did not include guidance for AI assistants on how to review PRs. Assistants would run pnpm lint, pnpm test, and pnpm format locally during PR reviews, which is slow and unnecessary.

After this PR:
CLAUDE.md now includes a "Review Workflow" section that instructs AI assistants to check CI status directly via GitHub CLI (gh pr checks, gh pr view, gh run view --log-failed) instead of running checks locally.

Why we need it and why it was done in this way

The following tradeoffs were made:
N/A - this is a straightforward documentation addition.

The following alternatives were considered:
N/A

Breaking changes

None

Special notes for your reviewer

This only modifies CLAUDE.md — no code changes. The new section is placed between "Pull Request Workflow" and "Issue Workflow" for logical grouping.

Checklist

Release note

NONE

@EurFelux EurFelux merged commit cc0991f into main Mar 2, 2026
11 checks passed
@EurFelux EurFelux deleted the chore/review branch March 2, 2026 16:01
@kangfenmao kangfenmao mentioned this pull request Mar 3, 2026
4 tasks
EurFelux added a commit that referenced this pull request Mar 3, 2026
DeJeune added a commit that referenced this pull request Mar 4, 2026
### What this PR does

This is a release PR for **Cherry Studio v1.7.23**.

**Changes included:**
- Bump version from 1.7.22 to 1.7.23
- Update release notes with user-facing bug fixes

### Release Notes

<!--LANG:en-->
Cherry Studio 1.7.23 - Bug Fixes

🐛 Bug Fixes
- [Selection] Fix app crash on Windows when closing action window
- [MiniApp] Fix settings state synchronization and region filter
consistency
- [Plugin Browser] Make detail modal text selectable and improve search
experience
- [Tools] Fix approval card not showing for builtin and provider tools

### Included Commits

- fix(Selection): prevent Windows crash when closing transparent action
window (#13177)
- fix(renderer): synchronize miniapp settings state and respect region
filter (#13166)
- fix: improve plugin browser UX with three small fixes (#13153)
- fix: show approval card for builtin and provider tools (#13154)
- fix: support esc to close modal (#13159)
- fix: open external editor in new window instead of reusing existing
one (#13160)
- fix: render directory Select options with optionRender (#13152)
- refactor: replace static pnpm patch with postinstall script for
claude-agent-sdk (#13139)
- feat: add dev-only message data inspection button (#13142)
- docs: add review workflow to CLAUDE.md (#13145)
- chore(deps): upgrade @uiw/codemirror packages to 4.25.7 (#13149)
- fix(ci): skip CI on PR body/title edits, only re-run on base branch
changes (#13150)

### Review Checklist

- [ ] Review generated release notes in `electron-builder.yml`
- [ ] Verify version bump in `package.json`
- [ ] CI passes
- [ ] Merge to trigger release build

### Release note

```release-note
Cherry Studio 1.7.23 - Bug Fixes

🐛 Bug Fixes
- [Selection] Fix app crash on Windows when closing action window
- [MiniApp] Fix settings state synchronization and region filter consistency
- [Plugin Browser] Make detail modal text selectable and improve search experience
- [Tools] Fix approval card not showing for builtin and provider tools
```

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: suyao <[email protected]>
Co-authored-by: Claude Opus 4.6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants