docs: fix descriptions - #14436
Merged
Merged
Conversation
The `@module` summary in `lib/index.js` and the README title blockquote omitted "complex", describing the package as operating on single-precision floating-point arrays. The package operates on single-precision complex floating-point arrays (`Complex64Array`); the package.json description, the `lib/main.js`/`lib/ndarray.js` JSDoc summaries, and the README API sections already state so. Restores consistency with the package's own authoritative description (a convention upheld by all 48 namespace members). Copy-paste leftover from the real-valued `smap` sibling. Documentation only.
The `@module` summary in `lib/index.js` and the README title blockquote omitted "complex", describing the package as operating on double-precision floating-point arrays. The package operates on double-precision complex floating-point arrays (`Complex128Array`); the package.json description, the `lib/main.js`/`lib/ndarray.js` JSDoc summaries, and the README API sections already state so. Restores consistency with the package's own authoritative description (a convention upheld by all 48 namespace members). Copy-paste leftover from the real-valued `dmap` sibling. Documentation only.
The `lib/main.js` JSDoc summary dropped the "according to a strided mask array" clause, omitting the package's defining masked behavior. The clause is present in the same package's `lib/ndarray.js` summary, `lib/index.js` `@module` summary, package.json description, and README. Restores consistency with the package's own authoritative description (a convention upheld by all 48 namespace members). Stale-template omission carried over from the non-masked `dmap2` sibling. Documentation only.
The `lib/main.js` JSDoc summary dropped the "according to a strided mask array" clause, omitting the package's defining masked behavior. The clause is present in the same package's `lib/ndarray.js` summary, `lib/index.js` `@module` summary, package.json description, and README. Restores consistency with the package's own authoritative description (a convention upheld by all 48 namespace members). Stale-template omission carried over from the non-masked `smap2` sibling. Documentation only.
The `lib/main.js` and `lib/ndarray.js` JSDoc summaries used unary
phrasing ("each element retrieved from a strided input array"),
describing this binary package as if it operated on a single input
array. The package applies a binary function to each pair of elements
retrieved from two strided input arrays, as the package.json
description, `lib/index.js` `@module` summary, and README already
state. Restores consistency with the package's own authoritative
description (a convention upheld by all 48 namespace members).
Copy-paste leftover from the unary `map-by` sibling. Documentation
only.
The `lib/main.js` JSDoc documented `view` as the "output DataView" and
`out` as the "input array" — reversed relative to the function's own
summary ("Copies elements from an input strided DataView to elements in
an output strided array") and to the same package's `lib/ndarray.js`,
where `view` is the input and `out` is the output. Swaps the two
descriptions to match. Copy-paste leftover from the `write-dataview`
sibling, where the data flow is reversed. Documentation only; no
parameter names, types, or behavior change.
Contributor
Coverage Report
The above coverage report was generated for the changes in this PR. |
kgryte
marked this pull request as ready for review
August 20, 2026 10:41
kgryte
approved these changes
Aug 20, 2026
strided/base packages
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
This pull request corrects copy-paste documentation drift across six packages in the
@stdlib/strided/basenamespace. Each carried a summary or parameter description cloned from a sibling and never updated, so it contradicted the package's ownpackage.jsondescription and its other documentation surfaces. All changes are documentation-only: no source behavior, exported signatures, parameter names, parameter types, or test expectations change.Namespace summary. 48 direct member packages analyzed. Features extracted: file tree,
package.jsonkey/scripts/directoriessets, README section structure, JSDoc tag presence, error construction, and per-surface summary/parameter-description consistency. Features with a clear (≥75%) majority: file layout,package.jsonshape (47/48), emptyscripts(48/48),format-based error construction, README## Usage/## Examples(48/48), and the convention that a package's@module, README, and JSDoc summaries mirror itspackage.jsondescription (48/48). Features excluded for lacking a clear majority: README## Notes(37%),## C APIs(37%),## See Also(73%, and auto-populated), theinclude/src/manifest.jsonC-implementation triple, and public signatures / validation prologues (the namespace is functionally heterogeneous). The six corrected packages are the outliers on the summary-consistency convention.cmapThe
@modulesummary inlib/index.jsand the README title blockquote dropped "complex", describingcmapas operating on single-precision floating-point arrays.cmapoperates on single-precision complex floating-point arrays (Complex64Array); thepackage.jsondescription, themain.js/ndarray.jsJSDoc, and the README API sections already state so. A leftover from the real-valuedsmapsibling.zmapThe same drift at double precision: the
@modulesummary and README blockquote omitted "complex" while the package operates onComplex128Array. Both surfaces now match the package's ownpackage.jsondescription. A leftover fromdmap.dmskmap2The
lib/main.jssummary dropped the "according to a strided mask array" clause, omitting the package's defining masked behavior. The clause is present in the same package'sndarray.js,index.js,package.json, and README. Restored. A stale-template omission carried over from the non-maskeddmap2.smskmap2The identical mask-clause omission in
lib/main.js, at single precision. Restored from the package's ownndarray.js/index.js/package.json. A leftover fromsmap2.map-by2The
lib/main.jsandlib/ndarray.jssummaries used unary phrasing — "each element retrieved from a strided input array" — for a binary package.map-by2applies a binary function to each pair of elements retrieved from two strided input arrays, per itspackage.json,index.js, and README. Both summaries corrected. A leftover from the unarymap-by.read-dataviewThe
lib/main.jsJSDoc documentedviewas the "output DataView" andoutas the "input array" — reversed. The function copies from an inputDataViewinto an output array, as its own summary line andndarray.jsboth state. The two descriptions are swapped to match. A leftover fromwrite-dataview, where the data flow is the reverse. Parameter names and types are unchanged.Related Issues
No.
Questions
No.
Other
Validation. Structural features were extracted across all 48 members via filesystem and string inspection; per-surface summaries and parameter descriptions were extracted and diffed against each package's
package.jsondescription. Candidate outliers were reviewed by three independent agents (semantic, cross-reference, and structural) before advancing. Deliberately excluded: deviations on features without a ≥75% majority;function-object, whose absent benchmark, absent JSDoc parameter/return tags, andbrowserfield are intentional consequences of exporting a constant header-directory path rather than a function; and the auto-populated README## See Alsoand signature-table sections. No test or example depends on any corrected string. Areinterpret-complex@moduletypo ("floating-point point array") was noted but left out of scope as a single-package typo rather than cross-package drift.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by an automated cross-package drift-detection routine running in Claude Code. The routine extracted the documentation conventions shared across the
strided/basenamespace, flagged the packages deviating from them, and applied the mechanical corrections. Every finding was verified against the deviating package's own existing documentation (itspackage.jsondescription and other summary surfaces) before it was committed.@stdlib-js/reviewers
Generated by Claude Code