style: align outliers in math/base/special/fast with namespace majority patterns#11856
Draft
Planeshifter wants to merge 1 commit intodevelopfrom
Draft
style: align outliers in math/base/special/fast with namespace majority patterns#11856Planeshifter wants to merge 1 commit intodevelopfrom
math/base/special/fast with namespace majority patterns#11856Planeshifter wants to merge 1 commit intodevelopfrom
Conversation
Relocated `examples/c/native/Makefile` to `examples/c/Makefile` and `examples/c/native/example.c` to `examples/c/example.c`, matching the flat C-example layout used by 15/16 (93.8%) sibling packages in the `math/base/special/fast` namespace. File contents unchanged; the empty `examples/c/native/` directory was removed.
Contributor
Coverage ReportNo coverage information available. |
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
Aligning outliers in
math/base/special/fastwith namespace majority patterns (random namespace pick, seed20260430).Namespace summary
math/base/special/fastpackage.jsonshape,manifest.jsonshape, README section sequences, test/benchmark/examples filenames, public signatures, return kind, validation prologue, error construction, JSDoc shape, dependency setsREADME.md,package.json,manifest.json,binding.gyp,lib/{index,main,native}.js,src/{Makefile,addon.c,main.c},docs/{repl.txt,types/index.d.ts,types/test.ts},benchmark/{benchmark.js,benchmark.native.js,c/native/{Makefile,benchmark.c}},examples/{c/{Makefile,example.c},index.js},include.gypi,test/{test.js,test.native.js}— all 16/16);package.jsontop-level keys (16/16);manifest.jsonkeys (16/16); README level-2 prefix[Usage, Notes, Examples, C APIs](16/16);returnKind: value(16/16); emptyvalidationPrologue(16/16, all packages use IEEE-754 / NaN conventions instead of throw-based validation);jsdocShape.hasExample: true(16/16).C APIs(10/16 = 62.5% haveSee Also; 5/16 have nothing; 1/16 hasReferences+See Also— under 75% threshold; theSee Alsosection is auto-populated by tooling); test fixtures presence (varies with whether the package needs reference data);publicSignaturearity (varies between 1-arg and 2-arg by mathematical purpose, no namespace-wide convention).math/base/special/fast/absfRelocates the C example files in
math/base/special/fast/absffrom the non-standardexamples/c/native/nesting to the flatexamples/c/layout — matching 15/16 (93.8%) of sibling packages in the namespace. File contents are unchanged; the nested directory was inherited drift, not a deliberate variation, as confirmed by byte-identical comparison with the siblingabspackage (modulo copyright year and float/double type differences). The emptynative/subdirectory has been removed.Validation
What was checked:
package.json/manifest.jsonshape, README section sequences, test/benchmark/examples filenames).absfpackage and the entiremath/base/special/fastnamespace for any reference to the oldexamples/c/native/path; none found. README does not link the path; tests/benchmarks/examples don't reference it.absf's build configuration is identical to other f-suffix siblings (atanhf,hypotf,maxf,minf), none of which use anative/subdirectory. The deviation has no semantic justification.What was deliberately excluded:
See Alsosections (under 75% threshold; section is auto-populated by tooling and shouldn't be hand-edited per the section comment).publicSignaturearity differences (1-arg vs 2-arg reflect mathematical purpose, not drift).alpha-max-plus-beta-minpackage'slib/factory.jsandtest/test.factory.js(intentional API extension, not drift).Related Issues
No.
Questions
No.
Other
Random namespace pick used seed
20260430(deterministic). 121 candidate namespaces were eligible (≥8 non-autogenerated members underlib/node_modules/@stdlib/); none of the eligible namespaces had >50% autogenerated members. Full per-feature drift report saved alongside the run.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by Claude Code as part of an automated cross-package API drift-detection routine: the namespace was selected at random (seed
20260430), structural and semantic features were extracted across all 16 non-autogenerated members, the drift finding was validated by two parallel review agents (opus cross-reference + sonnet structural-review), and the resulting one-package fix is a pure file relocation with byte-identical contents.@stdlib-js/reviewers
Generated by Claude Code