Skip to content

[codex] Simplify PR review runner#45

Open
laiso wants to merge 4 commits into
mainfrom
codex-one-shot-reviewer
Open

[codex] Simplify PR review runner#45
laiso wants to merge 4 commits into
mainfrom
codex-one-shot-reviewer

Conversation

@laiso
Copy link
Copy Markdown
Owner

@laiso laiso commented May 25, 2026

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 laiso marked this pull request as ready for review May 25, 2026 15:29
@laiso laiso temporarily deployed to secure-agent May 25, 2026 15:31 — with GitHub Actions Inactive
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

問題ありません。

@laiso laiso deployed to secure-agent May 25, 2026 15:37 — with GitHub Actions Active
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

問題ありません。

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