docs: updates docs for prompts plugin#2763
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughDocumentation updates for the Prompts plugin and Playground UI: header names changed to Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant PromptsPlugin as "Prompts Plugin"
participant Provider
participant Logs
Client->>PromptsPlugin: HTTP request (headers: `x-bf-prompt-id`?, `x-bf-prompt-version`?, body includes "stream"?, model params)
PromptsPlugin->>PromptsPlugin: resolve prompt (version header or versionNumber 0 => latest committed)
PromptsPlugin->>PromptsPlugin: merge committed prompt model params with request params (request wins)
PromptsPlugin->>Provider: forward injected messages + merged params (transport streaming controlled by Client "stream")
Provider-->>Client: model response (stream or non-stream)
PromptsPlugin->>Logs: emit conversation + selected prompt metadata (name, id, version)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
|
Confidence Score: 5/5Safe to merge — all changes are documentation only and accurately reflect the current codebase. All four changed files are documentation or a changelog entry. The header names, streaming behavior description, and context key names were verified against the current plugins/prompts/main.go source and are accurate. No P0 or P1 findings; the only open concerns (stale screenshot alt text, missing trailing newline) were already raised in prior review threads. No files require special attention. Important Files Changed
Reviews (5): Last reviewed commit: "docs: updates docs for prompts plugin" | Re-trigger Greptile |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
plugins/prompts/changelog.md (1)
1-1: Consider noting the breaking change nature in the changelog.The PR objectives describe this as a breaking change requiring clients to migrate from
bf-prompt-id→x-bf-prompt-idandbf-prompt-version→x-bf-prompt-version. A changelog entry typically benefits from explicitly flagging breaking changes for users scanning release notes.📝 Suggested enhancement
-- refactor: Updates prompt plugin flow to change the headers used, and better code quality +- **BREAKING**: Updates prompt plugin flow to use `x-bf-prompt-id` / `x-bf-prompt-version` headers (previously `bf-prompt-id` / `bf-prompt-version`), and improves code quality🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugins/prompts/changelog.md` at line 1, Update the changelog entry (the refactor line) to explicitly mark this as a BREAKING CHANGE and list the header renames: clients must migrate from bf-prompt-id → x-bf-prompt-id and bf-prompt-version → x-bf-prompt-version; mention any migration notes or examples to help users update clients and surface the breaking-change label prominently so it’s visible to readers scanning release notes.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@plugins/prompts/changelog.md`:
- Line 1: Update the changelog entry (the refactor line) to explicitly mark this
as a BREAKING CHANGE and list the header renames: clients must migrate from
bf-prompt-id → x-bf-prompt-id and bf-prompt-version → x-bf-prompt-version;
mention any migration notes or examples to help users update clients and surface
the breaking-change label prominently so it’s visible to readers scanning
release notes.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 2b795f1b-5a0a-435a-b877-a9510db25968
📒 Files selected for processing (4)
docs/features/observability/default.mdxdocs/features/prompt-repository/playground.mdxdocs/features/prompt-repository/prompts-plugin.mdxplugins/prompts/changelog.md
3fa3852 to
6fd8f22
Compare
1b845e1 to
e1f0e9a
Compare
6fd8f22 to
49c6229
Compare
e1f0e9a to
aa3b9fe
Compare
49c6229 to
27ce7d3
Compare
27ce7d3 to
e208a8a
Compare
aa3b9fe to
9a5e8bb
Compare
Merge activity
|
The base branch was changed.

Summary
Updates the Prompts plugin to use standardized
x-bf-prefixed headers and improves documentation clarity around parameter merging, streaming behavior, and observability integration.Changes
bf-prompt-id/bf-prompt-versiontox-bf-prompt-id/x-bf-prompt-versionfor consistencyPromptResolverinterface details and context key informationType of change
Affected areas
How to test
Test the updated prompt plugin headers:
Verify that the old
bf-prompt-idheaders no longer work and the newx-bf-prompt-idheaders function correctly.Screenshots/Recordings
N/A
Breaking changes
Breaking change: The prompt plugin now requires
x-bf-prompt-idandx-bf-prompt-versionheaders instead ofbf-prompt-idandbf-prompt-version.Migration: Update all client code to use the new header names:
bf-prompt-id→x-bf-prompt-idbf-prompt-version→x-bf-prompt-versionRelated issues
N/A
Security considerations
Header name changes do not introduce security implications. The functionality remains the same with standardized naming.
Checklist
docs/contributing/README.mdand followed the guidelines