Skip to content

docs: add stdmath keyword to chebyshev-series and chebyshev-seriesf in math/base/tools#12011

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

docs: add stdmath keyword to chebyshev-series and chebyshev-seriesf in math/base/tools#12011
kgryte merged 2 commits intodevelopfrom
claude/inspiring-curie-cweoz

Conversation

@Planeshifter
Copy link
Copy Markdown
Member

Description

What is the purpose of this pull request?

This pull request:

  • Inserts the stdmath keyword and reorders the leading keywords entries in two math/base/tools packages so the prefix matches the rest of the namespace: ["stdlib", "stdmath", "mathematics", "math", ...].

Namespace summary

  • @stdlib/math/base/tools: 17 non-autogenerated members.
  • Features analyzed: file tree, package.json shape (top-level keys, scripts, keywords), manifest.json shape, README section list/order, test/benchmark/example file naming, and (inline) semantic surface — public signature, validation prologue, error construction, JSDoc shape, dependencies.
  • Features with a clear (≥75%) majority that produced an actionable correction: keywords prefix sequence (15/17 = 88.2%).
  • Features with a clear majority but no actionable outliers: every universal file (README.md, lib/index.js, package.json, test/test.js, benchmark/benchmark.js, examples/index.js, docs/types/index.d.ts, docs/types/test.ts); 18 universal package.json top-level keys.
  • Features without a clear majority (excluded): README h2 set order (Notes 12/17 = 71%; See Also 6/17), test/benchmark file naming (multiple sub-majority subsets corresponding to the namespace's three sub-architectures: polynomial evaluators, compile helpers, series tools).
  • Features with majority deviations gated out: lib/main.js absent in continued-fraction and sum-series (intentional basic.js + generators.js architecture for generator/non-generator paths); docs/repl.txt absent in the four *-compile* packages (88.1% stdlib-wide presence — below the 90% gate).

math/base/tools/chebyshev-series

The keywords array begins ["stdlib", "math", "mathematics", "chebyshev", ...] — the rest of @stdlib/math/base/tools (15/17 = 88.2%; @stdlib/math/* overall: 665/684 = 97.2%) leads with stdlib/stdmath/mathematics/math. Package was added in #11470 and the stdmath discoverability marker was missed at authoring; aligning the prefix here matches what every sibling already does. No source, tests, examples, or REPL fixtures touched.

math/base/tools/chebyshev-seriesf

Single-precision sibling carries the same omission verbatim — same rationale, same fix. The two packages were authored together; the divergence is mechanical, not intentional.

Validation

  • Structural extraction over 17 packages: file tree, package.json shape, README sections, test/bench/example names.
  • Semantic surface inspected inline (no per-package agent fan-out needed): every package in the namespace carries no throw and no @stdlib/assert/* validation imports — math/base/* is the unvalidated fast-path tier — so validationPrologue, errorConstruction, and related semantic features are trivially universal-empty.
  • Three-agent advance criteria: structural-review agent (sonnet) returned confirmed-drift for both outliers; cross-reference agent verified that no test, example, README claim, or _tools/lint/pkg-json schema constrains keyword content/order beyond presence-of-stdlib and the global character-set rule, and that the files are not generator output.
  • Cross-run dedup: no open PR proposes this fix; feat: add math/base/tools/chebyshev-series #11470 (the package add) and docs: correct JSDoc return value in math/base/tools/chebyshev-seriesf #11584 (a follow-up doc fix on chebyshev-seriesf) merged outside the 14-day window.

Deliberately excluded

  • Intentional architecture deviations (continued-fraction, sum-series lacking lib/main.js).
  • The four *-compile* packages lacking docs/repl.txt — below the ≥90% stdlib-wide gate.
  • README Notes and See Also sections — sub-threshold in this namespace; the See Also block is auto-populated regardless and outside the routine's scope.
  • All non-keywords features that already conform on every member.

Related Issues

Does this pull request have any related issues?

No.

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request?

Total diff: 4 insertions / 2 deletions across 2 files, both package.json. Each outlier receives a single keywords prefix change and no other modifications. No commits touch source, tests, examples, types, or generated artifacts.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

AI Assistance

When authoring the changes proposed in this PR, did you use any kind of 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 (Opus 4.7) running a cross-package drift-detection routine: a randomly-picked namespace (@stdlib/math/base/tools, seed 1818300874) was scanned for structural and semantic drift against a 75% majority threshold, candidate corrections went through pre-validation gates and a multi-agent advance check (sonnet structural-review and cross-reference review, both confirmed-drift), and only the surviving mechanical metadata fix was applied. I reviewed the verdict, the gating decisions, and the diff before submission.


@stdlib-js/reviewers


Generated by Claude Code

claude added 2 commits May 9, 2026 16:17
Insert `stdmath` keyword and reorder the leading entries so the
keywords prefix becomes `[stdlib, stdmath, mathematics, math, ...]`,
matching 15 of 17 sibling packages in `@stdlib/math/base/tools` (88.2%
within the namespace; 97.2% across `@stdlib/math/*`). The package was
introduced in PR #11470 without `stdmath`, while the surrounding
namespace adopted the keyword as the standard math-domain marker.
Insert `stdmath` keyword and reorder the leading entries so the
keywords prefix becomes `[stdlib, stdmath, mathematics, math, ...]`,
matching 15 of 17 sibling packages in `@stdlib/math/base/tools` (88.2%
within the namespace; 97.2% across `@stdlib/math/*`). Package shared
the same omission as its double-precision sibling
`math/base/tools/chebyshev-series`.
@stdlib-bot stdlib-bot added the Math Issue or pull request specific to math functionality. label May 9, 2026
@stdlib-bot
Copy link
Copy Markdown
Contributor

Coverage Report

Package Statements Branches Functions Lines
math/base/tools/chebyshev-series $\color{green}280/280$
$\color{green}+100.00%$
$\color{green}20/20$
$\color{green}+100.00%$
$\color{green}3/3$
$\color{green}+100.00%$
$\color{green}280/280$
$\color{green}+100.00%$
math/base/tools/chebyshev-seriesf $\color{green}300/300$
$\color{green}+100.00%$
$\color{green}20/20$
$\color{green}+100.00%$
$\color{green}3/3$
$\color{green}+100.00%$
$\color{green}300/300$
$\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 21:06
@Planeshifter Planeshifter requested a review from a team May 9, 2026 21:06
@stdlib-bot stdlib-bot added Needs Review A pull request which needs code review. labels May 9, 2026
@Planeshifter Planeshifter requested a review from kgryte May 9, 2026 21:06
@kgryte kgryte removed the Needs Review A pull request which needs code review. label May 9, 2026
@kgryte kgryte merged commit 57d46b3 into develop May 9, 2026
71 checks passed
@kgryte kgryte deleted the claude/inspiring-curie-cweoz branch May 9, 2026 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Math Issue or pull request specific to math functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants