Skip to content

feat: add AI summary panel for the active note (control — no noise-reduction config) - #22

Closed
ark-commits wants to merge 1 commit into
control/no-noise-instructionfrom
test/cr-bugs-no-noise
Closed

feat: add AI summary panel for the active note (control — no noise-reduction config)#22
ark-commits wants to merge 1 commit into
control/no-noise-instructionfrom
test/cr-bugs-no-noise

Conversation

@ark-commits

@ark-commits ark-commits commented Jun 25, 2026

Copy link
Copy Markdown
Owner

What

Control arm for the CodeRabbit noise-reduction A/B test. Identical to PR #21's diff (the NoteAiSummary component is byte-for-byte the same, cherry-picked), but the base branch's .coderabbit.yaml has the noise-reduction path_instructions block removed — only poem: false and issue_enrichment.planning: false remain.

The only variable vs. PR #21 is the review config.

Notes

  • Base branch: control/no-noise-instruction (config without the noise-reduction instruction)
  • Same planted bug catalog: 1 critical, 2 major, 2 minor, 5 nitpicks

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added an AI-powered note summary panel in the editor view.
    • Displays a loading state while the summary is being generated.
    • Shows the summary as a short list of bullet points when available.

Calls a chat completion API to generate a quick bullet summary of the
currently open note and renders it under the editor/preview panes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR adds a new note summary component that requests an AI-generated summary from OpenAI and displays the result in the UI. The main app now imports and renders that component with the active note.

Changes

AI note summary UI

Layer / File(s) Summary
Note summary component
src/components/NoteAiSummary.jsx
The new component defines summary state, sends a POST request to OpenAI chat completions from an effect, parses the response into summary text and bullets, and renders loading and bullet output.
App mount
src/App.jsx
The app imports NoteAiSummary and renders it with activeNote in the preview/editor area.

Sequence Diagram(s)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding an AI summary panel for the active note.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test/cr-bugs-no-noise

Comment @coderabbitai help to get the list of available commands.

@ark-commits

Copy link
Copy Markdown
Owner Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown
✅ Action performed

Full review finished.

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