Skip to content

Navigation Menu

Sign in
Sign up

[codex] Simplify PR review runner - #45

Closed
laiso wants to merge 6 commits into
main from
codex-one-shot-reviewer
Closed

[codex] Simplify PR review runner #45
laiso wants to merge 6 commits into
main from
codex-one-shot-reviewer

Conversation

@laiso

@laiso laiso commented May 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace the PR review agent loop with a one-shot diff review pipeline
  • Fetch and filter gh pr diff before a single model call
  • Add size/file limits and skip comments for oversized diffs

Why

This lowers review cost by avoiding repeated tool-driven LLM turns and repeated diff context uploads.

Validation

  • bun x tsc --noEmit
  • bun test

laiso marked this pull request as ready for review May 25, 2026 15:29

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

問題ありません。

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

問題ありません。

laiso deployed to secure-agent May 27, 2026 17:03 — with GitHub Actions Active

laiso commented May 27, 2026

Copy link
Copy Markdown
Owner Author

Closing this dogfood PR for now after removing session-work-summary.md from the diff.

@laiso laiso closed this May 27, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

レビュー

Blocking: なし

差分上、重大な指摘は見つかりませんでした。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /