Skip to content

chore: rename internal variable to main in two string packages#12009

Merged
kgryte merged 2 commits intodevelopfrom
claude/inspiring-curie-OgUYh
May 9, 2026
Merged

chore: rename internal variable to main in two string packages#12009
kgryte merged 2 commits intodevelopfrom
claude/inspiring-curie-OgUYh

Conversation

@Planeshifter
Copy link
Copy Markdown
Member

Description

This pull request:

  • Renames the internal variable in lib/index.js from a descriptive name to main in two @stdlib/string packages, matching the convention used in 54 of 56 (96.4%) sibling packages in the namespace. The change is local to lib/index.js and does not alter what is exported.

@stdlib/string/remove-utf8-bom

lib/index.js declared var removeUTF8BOM = require( './main.js' ); and module.exports = removeUTF8BOM;. Renamed both occurrences to main to match the namespace convention (96.4% conformance). No public-facing change.

@stdlib/string/utf16-to-utf8-array

lib/index.js declared var utf16ToUTF8Array = require( './main.js' ); and module.exports = utf16ToUTF8Array;. Renamed both occurrences to main to match the namespace convention (96.4% conformance). No public-facing change.

Related Issues

None.

Questions

No.

Other

No.

Checklist

AI Assistance

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

This PR was authored by Claude Code as part of a cross-package drift-detection routine. The routine extracted structural and semantic features from every package in @stdlib/string, identified the majority pattern per feature, and validated the surviving correction with three independent review agents before applying.


@stdlib-js/reviewers


Generated by Claude Code

claude added 2 commits May 9, 2026 12:17
…e-utf8-bom`

Rename the internal variable in `lib/index.js` from `removeUTF8BOM` to
`main` to align with the prevailing convention used in 54 of 56 packages
(96.4%) in the `@stdlib/string` namespace.

https://claude.ai/code/session_01XTKE9JQJfeRYxmMZQNTCfo
…-to-utf8-array`

Rename the internal variable in `lib/index.js` from `utf16ToUTF8Array` to
`main` to align with the prevailing convention used in 54 of 56 packages
(96.4%) in the `@stdlib/string` namespace.

https://claude.ai/code/session_01XTKE9JQJfeRYxmMZQNTCfo
@stdlib-bot
Copy link
Copy Markdown
Contributor

Coverage Report

Package Statements Branches Functions Lines
string/remove-utf8-bom $\color{green}107/107$
$\color{green}+100.00%$
$\color{green}7/7$
$\color{green}+100.00%$
$\color{green}1/1$
$\color{green}+100.00%$
$\color{green}107/107$
$\color{green}+100.00%$
string/utf16-to-utf8-array $\color{green}200/200$
$\color{green}+100.00%$
$\color{green}13/13$
$\color{green}+100.00%$
$\color{green}1/1$
$\color{green}+100.00%$
$\color{green}200/200$
$\color{green}+100.00%$

The above coverage report was generated for the changes in this PR.

@Planeshifter Planeshifter marked this pull request as ready for review May 9, 2026 20:59
@Planeshifter Planeshifter requested review from a team and kgryte May 9, 2026 20:59
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label May 9, 2026
@Planeshifter Planeshifter changed the title refactor: rename internal variable to main in two @stdlib/string packages chore: rename internal variable to main in two @stdlib/string packages May 9, 2026
@Planeshifter Planeshifter changed the title chore: rename internal variable to main in two @stdlib/string packages chore: rename internal variable to main in two string packages May 9, 2026
@kgryte kgryte removed the Needs Review A pull request which needs code review. label May 9, 2026
@kgryte kgryte merged commit b72a90b into develop May 9, 2026
45 checks passed
@kgryte kgryte deleted the claude/inspiring-curie-OgUYh branch May 9, 2026 22:54
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.

4 participants