chore: remove misspelled keyword in lapack/base/iladl{c,r}#12368
Open
Planeshifter wants to merge 2 commits into
Open
chore: remove misspelled keyword in lapack/base/iladl{c,r}#12368Planeshifter wants to merge 2 commits into
lapack/base/iladl{c,r}#12368Planeshifter wants to merge 2 commits into
Conversation
Removes the misspelled `auxilary` keyword from the `keywords` array. The misspelled form appears in no sibling package; 26/28 routine packages in `lapack/base` carry no auxiliary-routine keyword at all, so removal aligns with the namespace convention (93% conformance).
Removes the misspelled `auxilary` keyword from the `keywords` array. The misspelled form appears in no sibling package; 26/28 routine packages in `lapack/base` carry no auxiliary-routine keyword at all, so removal aligns with the namespace convention (93% conformance).
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
Drops the misspelled
"auxilary"keyword from thepackage.jsonkeywordsarray in@stdlib/lapack/base/iladlcand@stdlib/lapack/base/iladlr. The misspelled form appears nowhere else in stdlib, and 26 of 28 sibling routine packages inlapack/basecarry noauxiliary-family keyword at all (93% namespace conformance for "absent"). Removal brings the two manifests in line with the rest of the namespace.Namespace summary
@stdlib/lapack/basesharedandxerbla(both path-export utilities, not function-routines)package.jsontop-level keys,scripts/enginesblobs, keyword sets, manifest shape, README##/###section sequence and presencesonnetagents): public signature,returnKind, validation prologue, error construction, JSDoc shape, runtime dependencies## C APIssection (86%), JSDocLDA/LDB/LDO/LDCtype (≥87%), JSDocMtype for matrix routines (83%), keyword core set (10 terms at 96–100%), error construction (100%formatamong packages that throw)returnKind(64% mode), JSDocNtype (58%), per-package validation-prologue sequencelapack/base/iladlcRemoves the misspelled keyword
"auxilary"frompackage.json. The correct spelling"auxiliary"is absent from 26 of 28 sibling packages inlapack/base, making the keyword itself non-conformant with namespace conventions. One-line deletion brings the manifest in line with the rest of the namespace.lapack/base/iladlrRemoves the misspelled keyword
"auxilary"frompackage.jsonto match the convention used by 26 of 28 other routines inlapack/base, which carry noauxiliary-family keyword. The correct spelling appears in neither this package noriladlc, so deletion is the conformant fix rather than correction.Related Issues
No.
Questions
No.
Other
Validation
package.jsonshape, README headings, manifest shape).sonnetsubagents in two batches of 14 (no JSON retries needed).Deliberately excluded from this PR
## C APIsREADME stub absent inclacgv,crot,zlacgv,zrot: stdlib-wide presence is only ~63% inblas/baseand 0% instats/base, failing the ≥90% ecosystem-wide gate for "add a missing section" fixes.lib/ndarray.js/benchmark/benchmark.ndarray.jsabsent indlamch,dlapy2,dlapy3: scalar-input routines have no strided variant; intentional.lib/base.jsabsent in 7 packages: requires extracting and rewiring the kernel — not a mechanical fix.@param {integer} LDA→{PositiveInteger}iniladlc/iladlr(87% namespace conformance, runtime guard enforcesLDA ≥ 1): correct on substance but Agent 2 flaggedneeds-humanbecausedocs/repl.txtdocuments the old type and is generator-owned per stdlib convention, so cannot be hand-edited in lockstep. Dropped to avoid leaving a half-state. Should be revisited once a REPL regen routine is in place.@param {PositiveInteger} M→{NonNegativeInteger}iniladlc/iladlr(83% namespace conformance for matrix routines): the per-namespace majority forNis only 58%, so changingMalone would leave the two annotations internally inconsistent against an identical kernel guard (if ( M <= 0 || N <= 0 ) return -1;). Dropped pending broader rework.xerblaandsharedexcluded entirely: both are JS-side path-export utilities, not routines.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was produced by Claude Code running a cross-package drift detection routine over
@stdlib/lapack/base. The routine extracted structural features (file tree,package.jsonshape, README sections) and semantic features (per-package via sonnet subagents) from all 28 routine packages, ran a three-agent adversarial validation pass (semantic, cross-reference, structural) on every candidate that exceeded the 75% majority threshold, and applied only the corrections that survived. Two larger drift findings (LDA type, M type) were dropped on validator review — see theOthersection above. The remaining keyword removal is mechanical and was applied verbatim.@stdlib-js/reviewers
Generated by Claude Code