Skip to content

docs: remove resolve_batch() references after SDK dead-code removal#2000

Open
praisonai-triage-agent[bot] wants to merge 1 commit into
mainfrom
claude/issue-1999-20260714-1800
Open

docs: remove resolve_batch() references after SDK dead-code removal#2000
praisonai-triage-agent[bot] wants to merge 1 commit into
mainfrom
claude/issue-1999-20260714-1800

Conversation

@praisonai-triage-agent

@praisonai-triage-agent praisonai-triage-agent Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Fixes #1999

Summary

Removes documentation references to the resolve_batch() helper deleted upstream as dead code (PraisonAI@27c9de4). resolve() remains the single source of truth for the parameter-precedence ladder.

Changes

  • Deleted docs/sdk/reference/praisonaiagents/functions/resolve_batch.mdx
  • Removed the resolve_batch() card from docs/sdk/reference/praisonaiagents/modules/param_resolver.mdx (now 16 function cards)
  • Removed the nav entry from docs.json

Verification

  • grep -r resolve_batch docs/ returns no results
  • docs.json parses as valid JSON (siblings correctly comma-terminated)
  • No broken cross-links to ../functions/resolve_batch remain

Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Removed the resolve_batch function from the SDK reference navigation.
    • Removed its dedicated reference page and corresponding module documentation entry.

…ixes #1999)

Co-authored-by: MervinPraison <MervinPraison@users.noreply.github.com>
@MervinPraison

Copy link
Copy Markdown
Owner

@coderabbitai review

@MervinPraison

Copy link
Copy Markdown
Owner

/review

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@MervinPraison

Copy link
Copy Markdown
Owner

@gemini review this PR

@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@MervinPraison MervinPraison added pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI pipeline/blocked:ci Blocked: CI not green on HEAD pipeline/blocked:manual-review Blocked: requires manual review and removed pipeline/blocked:ci Blocked: CI not green on HEAD labels Jul 14, 2026
@coderabbitai

coderabbitai Bot commented Jul 14, 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: Pro

Run ID: 98c2d858-35af-4dd9-a214-88d87879f3dc

📥 Commits

Reviewing files that changed from the base of the PR and between 8eb721b and 2668c0d.

📒 Files selected for processing (3)
  • docs.json
  • docs/sdk/reference/praisonaiagents/functions/resolve_batch.mdx
  • docs/sdk/reference/praisonaiagents/modules/param_resolver.mdx
💤 Files with no reviewable changes (3)
  • docs/sdk/reference/praisonaiagents/functions/resolve_batch.mdx
  • docs/sdk/reference/praisonaiagents/modules/param_resolver.mdx
  • docs.json

📝 Walkthrough

Walkthrough

The SDK reference removes the deleted resolve_batch function page, its module card, and its navigation entry.

Changes

SDK reference cleanup

Layer / File(s) Summary
Remove stale SDK references
docs/sdk/reference/praisonaiagents/functions/resolve_batch.mdx, docs/sdk/reference/praisonaiagents/modules/param_resolver.mdx, docs.json
Deletes the resolve_batch reference page and removes its module card and navigation entry.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 PR's docs cleanup for removing resolve_batch references.
Linked Issues check ✅ Passed The changes remove the resolve_batch page, module card, and nav entry as required by #1999.
Out of Scope Changes check ✅ Passed The PR stays within the affected generated SDK docs and introduces no unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-1999-20260714-1800

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.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request removes the documentation and references for the resolve_batch function, including deleting its dedicated documentation file and removing references to it in docs.json and param_resolver.mdx. I have no feedback to provide as there are no review comments.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@MervinPraison MervinPraison added pipeline/blocked:ci Blocked: CI not green on HEAD and removed pipeline/blocked:ci Blocked: CI not green on HEAD labels Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI pipeline/blocked:manual-review Blocked: requires manual review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: remove resolve_batch() references after SDK dead-code removal

1 participant