Skip to content

Document run and audit allowance behavior - #736

Open
SiddarthAA wants to merge 1 commit into
mainfrom
codex/quota-limits-docs
Open

Document run and audit allowance behavior#736
SiddarthAA wants to merge 1 commit into
mainfrom
codex/quota-limits-docs

Conversation

@SiddarthAA

@SiddarthAA SiddarthAA commented Aug 21, 2026

Copy link
Copy Markdown
Member

Documents the customer-visible behavior when run or audit allowances are reached, including independent enforcement and recovery through an administrator.

Summary by CodeRabbit

  • Documentation
    • Clarified that audit allowances are separate from run allowances.
    • Explained how exhausted audit allowances affect scheduled and manual audits without interrupting event ingestion.
    • Added guidance on run allowance limits, retained runs, visibility, and quota change requests.

Hermes review

Field Value
Status Approved
Reviewed commit 78877f2c27c8171ac70ce29511a9fb9076b5d316
Policy revision 1d8f31d926828f3bae215c58f5b35baa44acbff0
Model gpt-5.6-terra
Duration 282s
Updated 2026-08-21T18:49:31.411746440+00:00

Summary

Documentation-only change reviewed in full. No actionable defects found.

Changes

  • Documents that audit allowances are independent and suppress scheduled and manual audits when exhausted.
  • Documents run allowance counting, retention, visibility, dependent-processing behavior, and administrator recovery.

Validation

  • Skipped docker run ... oven/bun:latest sh -lc 'bun install --frozen-lockfile --ignore-scripts && bun run validate:mdx' — No validation command is centrally configured. The relevant repository MDX validator was attempted in an isolated container, but its clean dependency install remained blocked on package-fetch tasks; this harness limitation cannot be resolved by an author response. (0s)

Findings

None.

Open questions

None.

Policy overrides

None.

@github-actions

Copy link
Copy Markdown
Contributor

Thanks @SiddarthAA for your contribution to Failproof AI! 🙌

We'd love to discuss your PR and welcome you to our community.

Discord: https://discord.befailproof.ai/
Reddit: https://www.reddit.com/r/failproofai/

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3a43c510-bdd2-43ce-aafe-99f389712f81

📥 Commits

Reviewing files that changed from the base of the PR and between 653f116 and 78877f2.

📒 Files selected for processing (2)
  • docs/agenteye/audits.mdx
  • docs/agenteye/sessions.mdx

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The documentation adds notices for audit allowances and run allowances. The notices describe exhaustion behavior, retained runs, event ingestion, visibility, and administrator actions.

Changes

Quota documentation

Layer / File(s) Summary
Allowance behavior notices
docs/agenteye/audits.mdx, docs/agenteye/sessions.mdx
The documentation explains audit allowance exhaustion, run allowance counting, over-limit run visibility, continued event ingestion, and administrator quota changes.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 78877

This documentation-only change clarifies allowance behavior without changing runtime behavior. No actionable merge-blocking risk remains.

Suggested reviewers: niveditjain

Poem

I’m a small rabbit with notes in my pack,
Audit limits now show the correct track.
Runs stay recorded when quotas are tight,
Existing sessions remain in sight.
Admins can reset the gate—
Clear docs make limits easy to state.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the change but omits the required template sections, change-type selection, and validation checklist. Add the required Description, Type of Change, and Checklist sections, and mark the Documentation option and applicable validation checks.
✅ Passed checks (4 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 files. (2 skipped: 2 unsupported.)
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.
Title check ✅ Passed The title clearly and concisely describes the documentation changes to run and audit allowance behavior.

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.

@hermes-exosphere

Copy link
Copy Markdown
Contributor

Hermes

Status Reviewing
Verdict Not reviewed yet
Head 78877f2c27c8
Rounds 0 of 5

No summary yet.

What this changes

No component map for this revision.

Rounds

No review has finished on this pull request yet.

Findings

Nothing raised yet.


@hermes-exosphere help lists every command. This comment is maintained in place — I rewrite it after each review rather than posting a new one.

@hermes-exosphere

Copy link
Copy Markdown
Contributor

Hermes

Status Reviewed
Verdict Approved
Head 78877f2c27c8
Rounds 0 of 5

Documentation-only change reviewed in full. No actionable defects found.

What this changes

flowchart LR
    n0Auditexecution["~ Audit execution"]
    n1RuningestionandSessions["~ Run ingestion and Sessions"]
    n2Allowanceadministration["Allowance administration"]
    n0Auditexecution -- "independent allowances" --> n1RuningestionandSessions
    n0Auditexecution -- "reset or increase" --> n2Allowanceadministration
    n1RuningestionandSessions -- "reset or increase" --> n2Allowanceadministration
Loading

Rounds

Round Reviewed Commits in this round Verdict
0 78877f2c27c8 78877f2c27c8 Approved

Findings

Nothing raised yet.


@hermes-exosphere help lists every command. This comment is maintained in place — I rewrite it after each review rather than posting a new one.

@hermes-exosphere hermes-exosphere left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hermes found no blocking issues in this revision.

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.

2 participants