Skip to content

Docs: Outdated terminology in token meter docs comments#380

Open
tuanaiseo wants to merge 1 commit intostripe:mainfrom
tuanaiseo:contribai/docs/outdated-terminology-in-token-meter-docs
Open

Docs: Outdated terminology in token meter docs comments#380
tuanaiseo wants to merge 1 commit intostripe:mainfrom
tuanaiseo:contribai/docs/outdated-terminology-in-token-meter-docs

Conversation

@tuanaiseo
Copy link
Copy Markdown

Problem

The file-level comments refer to "token-billing" even though the package/module is now named token-meter. This creates confusion for contributors searching for the current module name and can propagate stale terminology into external docs.

Severity: low
File: llm/token-meter/types.ts

Solution

Update the header comment to use consistent naming, e.g. "Type definitions for token-meter".

Changes

  • llm/token-meter/types.ts (modified)

Testing

  • Existing tests pass
  • Manual review completed
  • No new warnings/errors introduced

The file-level comments refer to "token-billing" even though the package/module is now named `token-meter`. This creates confusion for contributors searching for the current module name and can propagate stale terminology into external docs.

Affected files: types.ts

Signed-off-by: tuanaiseo <221258316+tuanaiseo@users.noreply.github.com>
@cla-assistant
Copy link
Copy Markdown

cla-assistant bot commented Apr 11, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@cla-assistant
Copy link
Copy Markdown

cla-assistant bot commented Apr 11, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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