Skip to content

docs: add billing and cost tracking manual#6

Merged
TheRealAgentK merged 3 commits intomasterfrom
na/add-billing-doc
Feb 24, 2026
Merged

docs: add billing and cost tracking manual#6
TheRealAgentK merged 3 commits intomasterfrom
na/add-billing-doc

Conversation

@NinosMan
Copy link
Copy Markdown
Contributor

@NinosMan NinosMan commented Feb 24, 2026

docs: Add billing and cost tracking manual

Description 📝

  • Purpose: Adds documentation for integration developers on how to use the SDK's billing and cost tracking features, covering the ActionResult class, supports_billing config, and cost reporting patterns.
  • Approach: A new manual page (docs/manual/billing.md) that walks through configuration, implementation, examples, best practices, and migration guidance.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Updates

👉 Added docs/manual/billing.md covering the billing & cost tracking workflow
👉 Documents supports_billing config field, ActionResult class andcost_usd usage
👉 Includes code examples for paid API integrations, best practices, and migration notes for existing integrations

Screenshots 📷

N/A — documentation only

Test plan 🧪

  • Review the Markdown renders correctly on GitHub
  • Verify code examples are syntactically correct and consistent with the SDK's ActionResult API

Author(s) to check 👓

  • Project and all contained modules builds successfully
  • Self-/dev-tested
  • Unit/UI/Automation/Integration tests provided where applicable
  • Code is written to standards
  • Appropriate documentation written (code comments, internal docs)

Reviewer(s) to check ✔️

  • Project and all contained modules builds successfully
  • Change has been dev-/reviewer-tested, where possible
  • Unit/UI/Automation/Integration tests provided where applicable
  • Code is written to standards
  • Appropriate documentation written (code comments, internal docs)

NinosMan and others added 2 commits February 25, 2026 10:41
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@NinosMan NinosMan marked this pull request as ready for review February 24, 2026 21:49
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6547018ddb

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread docs/manual/billing.md Outdated
Comment thread docs/manual/billing.md
Copy link
Copy Markdown
Collaborator

@TheRealAgentK TheRealAgentK left a comment

Choose a reason for hiding this comment

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

Nice, @NinosMan

LGTM.

@TheRealAgentK
Copy link
Copy Markdown
Collaborator

Actually, @NinosMan - might be useful to add a link to this new doc on the main README?

Remove the local ActionResult class redefinition from the billing doc
code snippet (PR feedback) and replace with import + field descriptions.
Add a link to the billing guide in the README documentation section.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@TheRealAgentK TheRealAgentK merged commit 43e0070 into master Feb 24, 2026
@TheRealAgentK TheRealAgentK deleted the na/add-billing-doc branch February 24, 2026 22:25
@TheRealAgentK TheRealAgentK mentioned this pull request Mar 30, 2026
5 tasks
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