Skip to content

docs: 📝 Plugin doc updates - #858

Merged
evaline-ju merged 1 commit into
rossoctl:mainfrom
evaline-ju:doc-update
Sep 3, 2026
Merged

docs: 📝 Plugin doc updates#858
evaline-ju merged 1 commit into
rossoctl:mainfrom
evaline-ju:doc-update

Conversation

@evaline-ju

@evaline-ju evaline-ju commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Doc accuracy sweep against current plugin source without overlap with #857. Fixes: litellm-budget-track direction column to Both with placement guidance (reconciling PR #847 and #851); context-guru and cpex status Coming Soon → Opt-in; sparc deny_score_threshold example from an out-of-range 2.0 to 0.4; missing session-budget per-kind ceilings, ibac judge fields, and a stale client-registration reference.

Note: cpex was not updated - #856 was opened as a follow-up issue

Assisted-By: Claude (Anthropic AI) noreply@anthropic.com

Summary by CodeRabbit

  • Documentation
    • Documented optional IBAC settings for limiting judge responses and requesting JSON-formatted output.
    • Updated the plugin catalog to show context-guru and cpex as opt-in, and clarified bidirectional cost tracking for litellm-budget-track.
    • Clarified file-sourced value behavior during operator-managed Secret mounting.
    • Expanded session-budget documentation with per-kind token limits.
    • Corrected the SPARC score threshold example to use a valid normalized value.

Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: c23d6bd7-3fea-4897-ab5e-5e48e58c22a4

📥 Commits

Reviewing files that changed from the base of the PR and between 25b3a4a and 01f62f6.

📒 Files selected for processing (5)
  • authbridge/docs/ibac-plugin.md
  • authbridge/docs/plugin-catalog.md
  • authbridge/docs/plugin-reference.md
  • authbridge/docs/session-budget-plugin.md
  • authbridge/docs/sparc-plugin.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The PR updates plugin documentation for IBAC judge settings, session token limits, plugin catalog metadata, file-sourced values, and the SPARC threshold example.

Changes

Plugin documentation

Layer / File(s) Summary
Plugin configuration and validation documentation
authbridge/docs/ibac-plugin.md, authbridge/docs/session-budget-plugin.md
Documents IBAC judge token and JSON settings. Adds session-budget per-kind token limits and includes them in validation requirements.
Plugin catalog metadata
authbridge/docs/plugin-catalog.md
Updates plugin availability, direction, and LiteLLM response-cost tracking details.
Reference text and examples
authbridge/docs/plugin-reference.md, authbridge/docs/sparc-plugin.md
Updates the pod-boot Secret explanation and changes the SPARC threshold example to 0.4.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 01f62

Plugin configuration, catalog, and reference documentation now reflects current settings and examples. No current merge-readiness risk remains.

Suggested reviewers: huang195, esnible

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately identifies the changes as plugin documentation updates. It is concise and related to the main purpose of the pull request, although the emoji is unnecessary.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (5 skipped: 5 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@evaline-ju
evaline-ju merged commit 2b4cfbe into rossoctl:main Sep 3, 2026
23 checks passed
@github-project-automation github-project-automation Bot moved this from New/ToDo to Done in Rossoctl Issue Prioritization Sep 3, 2026
@evaline-ju
evaline-ju deleted the doc-update branch September 3, 2026 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants