docs: rename E -> FLOAT32_E in @stdlib/constants/float32/e#13469
Draft
Planeshifter wants to merge 1 commit into
Draft
docs: rename E -> FLOAT32_E in @stdlib/constants/float32/e#13469Planeshifter wants to merge 1 commit into
E -> FLOAT32_E in @stdlib/constants/float32/e#13469Planeshifter wants to merge 1 commit into
Conversation
The `@stdlib/constants/float32/*` namespace uses a `FLOAT32_<NAME>` identifier convention consistently across README headings, `lib/index.js` variable declarations, TypeScript declarations, example scripts, and tests: 65 of 68 sibling packages (95.6%) conform. The `e` package was the sole outlier, using bare `E` throughout, having been ported from `constants/float64/e` where the namespace convention is unprefixed and never re-aligned. This mirrors the drift PR #12432 corrected in the sibling `nan` package (README heading `# NAN` -> `# FLOAT32_NAN`). Renames the internal identifier in `README.md`, `lib/index.js`, `docs/types/index.d.ts`, `docs/types/test.ts`, `examples/index.js`, and `test/test.js`. The exported numeric value is unchanged; `require(...)` consumers observe no difference (the exported binding was already publicly documented as `FLOAT32_E` via the `lib/index.js` `@example` line and via `namespace/pkg2alias/data.csv`).
Contributor
Coverage Report
The above coverage report was generated for the changes in this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Renames the internal identifier
E->FLOAT32_Ein@stdlib/constants/float32/eto match the namespace'sFLOAT32_<NAME>naming convention. Same class of drift and same fix as PR #12432 applied to the siblingnanpackage.Namespace summary
@stdlib/constants/float32— 68 packages (all non-autogenerated).package.jsontop-level / scripts / stdlib keys,manifest.jsonshape, README section list, and — via per-package agent — the exported constant's identifier across source, TypeScript declarations, examples, tests, and README headings.FLOAT32_<NAME>identifier convention holds in 65 of 68 packages (95.6%).See Alsosection (present in 67.6% of packages — below the 75% threshold; also gated as an auto-populated section).Outlier
@stdlib/constants/float32/eRenamed the exported identifier from
EtoFLOAT32_Eacrosslib/index.js,docs/types/index.d.ts,docs/types/test.ts,examples/index.js,test/test.js, andREADME.mdto match the namespace'sFLOAT32_<NAME>convention (95.6% of siblings already conform). Same drift, same fix asnanin #12432. No behavioral change — thelib/index.js@exampleline andnamespace/pkg2alias/data.csvalready documented the public alias asFLOAT32_E.Related Issues
None.
Questions
No.
Other
Validation
package.jsonandmanifest.jsonshapes, and README section lists compared across all 68 packages via filesystem walk.lib/index.js, examples, tests, docs, and README to isolate the naming-convention outlier.constants/float64/e(whose namespace convention is unprefixed) and not an intentional preservation of the mathematical symbol; an opus cross-reference pass confirmed no consumer accesses the constant by a property name and no test relies on the local binding name; a sonnet structural pass enumerated the specific line changes required and the tokens that must not be touched (mathematical_e_link,Euler,EXPORTS/MAINbanners, URL-encodedE_%28...%29, C macroSTDLIB_CONSTANT_FLOAT32_E, path strings).Deliberately excluded:
ninfandpinf, which use a localvbinding inlib/index.jsfor the ArrayBuffer bit-pattern trick but publicly exportFLOAT32_NINF/FLOAT32_PINF— a legitimate implementation shape, not drift.See Alsosection presence / absence (auto-populated, and below the 75% majority threshold).@seelink absences in 12 packages — non-mechanical; adding requires domain-specific reference selection.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by Claude Code on behalf of @Planeshifter as an automated cross-package drift-detection run over a randomly-selected stdlib namespace. The namespace
constants/float32was picked at random; structural and semantic features were extracted across all 68 packages; the sole naming-convention outlier was validated by three independent reviewer agents (two opus, one sonnet) before the identifier rename was applied in the primary worktree. A human will audit and promote the PR out of draft.@stdlib-js/reviewers
Generated by Claude Code