Skip to content

Polish demo impact and PR flow#11

Closed
AusafMo wants to merge 1 commit into
mainfrom
codex/yc-demo-impact-flow
Closed

Polish demo impact and PR flow#11
AusafMo wants to merge 1 commit into
mainfrom
codex/yc-demo-impact-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 demo env overrides instead of a hardcoded browser provider
  • render single-record branch diffs as the same side-by-side paper diff used elsewhere and show existing open PR metadata
  • make fenced LLM JSON parsing more robust and add a YC demo runbook

Tests

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

@AusafMo

AusafMo commented Jun 28, 2026

Copy link
Copy Markdown
Owner Author

Closing in favor of #12 with the neutral branch name codex/impact-review-flow.

@AusafMo AusafMo closed this Jun 28, 2026
@AusafMo AusafMo deleted the codex/yc-demo-impact-flow branch June 28, 2026 14:39
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