Skip to content

chore: remove unused StructuredPrompt.totalEstimatedTokens - #257

Open
detail-app[bot] wants to merge 1 commit into
devfrom
detail/dead-code/chore-remove-unused-structuredprompt-totalestimate-271048
Open

detail-app[bot] wants to merge 1 commit into
devfrom
detail/dead-code/chore-remove-unused-structuredprompt-totalestimate-271048

Conversation

@detail-app

@detail-app detail-app Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Remove the unused totalEstimatedTokens() method from StructuredPrompt. It had zero production and zero test callers anywhere in the codebase, and its sum-of-estimates role was already absorbed by PromptTruncator.truncate(), which computes its own reserved tokens directly from the segment accessors.

History


Dead Code PRs can be configured here.

@coderabbitai

coderabbitai Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 189991ca-7a57-440f-8b2d-71f223e1f0c5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

This branch has not been deployed

No deployments
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