Skip to content

Polish impact review flow#12

Merged
AusafMo merged 1 commit into
mainfrom
codex/impact-review-flow
Jun 28, 2026
Merged

Polish impact review flow#12
AusafMo merged 1 commit into
mainfrom
codex/impact-review-flow

Conversation

@AusafMo

@AusafMo AusafMo commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • let the UI run LLM impact narration through config or environment overrides instead of a hardcoded browser provider
  • render single-record branch diffs as the same side-by-side paper diff used elsewhere
  • show existing open PR metadata instead of opening a duplicate PR form
  • make fenced LLM JSON parsing more robust

Tests

  • .venv/bin/ruff check src tests plugins/cfg_impact/cfg_impact
  • .venv/bin/python -m pytest

@AusafMo AusafMo force-pushed the codex/impact-review-flow branch from 1852bd4 to 70f88ed Compare June 28, 2026 14:41
@AusafMo AusafMo merged commit d96b3ae into main Jun 28, 2026
2 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