Skip to content

feat(miniapp): refine built-in PR review workflow#771

Merged
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/miniapp-pr-review-plan
May 18, 2026
Merged

feat(miniapp): refine built-in PR review workflow#771
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/miniapp-pr-review-plan

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

Summary

  • Refine the built-in PR Review MiniApp into a focused review workflow with source management kept out of the primary surface.
  • Improve startup sync visibility, PR queue focus, diff navigation, manual comment editing, and AI review progress states.
  • Add guarded lifecycle actions, safer review draft replacement, published-review context reuse, and contract tests for the built-in app assets.

Validation

  • node --check src\crates\core\src\miniapp\builtin\assets\pr-review\ui.js
  • rustfmt --edition 2021 --check src\crates\core\src\miniapp\builtin\mod.rs
  • git diff --check -- src\crates\core\src\miniapp\builtin\assets\pr-review\style.css src\crates\core\src\miniapp\builtin\assets\pr-review\ui.js src\crates\core\src\miniapp\builtin\mod.rs
  • cargo test -p bitfun-core miniapp::builtin::tests -- --nocapture
  • cargo check -p bitfun-core
  • pnpm run lint:web
  • pnpm run type-check:web
  • pnpm --dir src/web-ui exec vitest run --fileParallelism=false

@limityan limityan force-pushed the yanzhn/miniapp-pr-review-plan branch 3 times, most recently from 1e4bff7 to a887f45 Compare May 18, 2026 09:42
@limityan limityan marked this pull request as ready for review May 18, 2026 09:47
@limityan limityan force-pushed the yanzhn/miniapp-pr-review-plan branch from a887f45 to 9e27a15 Compare May 18, 2026 10:04
@limityan limityan force-pushed the yanzhn/miniapp-pr-review-plan branch from 9e27a15 to f683660 Compare May 18, 2026 10:22
@limityan limityan merged commit ad907dd into GCWing:main May 18, 2026
4 checks passed
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.

1 participant