docs: align outliers in constants/time with namespace majority patterns#11866
Draft
Planeshifter wants to merge 3 commits intodevelopfrom
Draft
docs: align outliers in constants/time with namespace majority patterns#11866Planeshifter wants to merge 3 commits intodevelopfrom
constants/time with namespace majority patterns#11866Planeshifter wants to merge 3 commits intodevelopfrom
Conversation
…nventions Normalized the "Notes" wording in `README.md` and `docs/repl.txt` to match the long-form note used by 80% (12/15) of `constants/time/*` packages. The short-form variant in this package was the minority and the only DST/timezone disclaimer wording that diverged from the namespace convention.
…nventions Normalized the "Notes" wording in `README.md` and `docs/repl.txt` to match the long-form note used by 80% (12/15) of `constants/time/*` packages. The short-form variant in this package was the minority and the only DST/timezone disclaimer wording that diverged from the namespace convention.
…ions Normalized the "Notes" wording in `README.md` and `docs/repl.txt` to match the long-form note used by 80% (12/15) of `constants/time/*` packages. The short-form variant in this package was the minority and the only DST/timezone disclaimer wording that diverged from the namespace convention.
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
Aligning outliers in
constants/timewith namespace majority patterns (random namespace pick, seed20260501).Namespace summary
@stdlib/constants/timepackage.jsonshape, README sections,manifest.jsonshape, test/benchmark/example file naming, README anddocs/repl.txt"Notes" prosepackage.jsontop-level keys (100%), README H2 sectionsUsage/Notes/Examples(100%), test file layout (100%), example file layout (100%), and the long-form "Notes" wording inREADME.md/docs/repl.txt(80%, 12/15)docs/repl.txt"See Also" auto-population (variable, manually populated downstream), keyword arrays underpackage.json.keywords(varies by unit; no shared majority pattern), and the optionaltexderivation block inlib/index.js(10/15, below threshold and tracks whether the constant has a non-trivial derivation)Function-shape semantic features (
publicSignature,validationPrologue,errorConstruction,jsdocShape) are inapplicable here — every member exports a single integer constant, so per-package semantic-extraction agents were skipped and the analysis is structural-only. This is documented in the local report.Outlier corrections
@stdlib/constants/time/milliseconds-in-minute@stdlib/constants/time/milliseconds-in-minute: normalizes "Notes" prose inREADME.mdanddocs/repl.txtto the long variant ("inaccuracies due to daylight savings conventions, crossing timezones, or other complications with time and dates"), matching 12/15 (80%) ofconstants/time/*siblings.@stdlib/constants/time/milliseconds-in-second@stdlib/constants/time/milliseconds-in-second: normalizes "Notes" prose inREADME.mdanddocs/repl.txtto the long variant ("inaccuracies due to daylight savings conventions, crossing timezones, or other complications with time and dates"), matching 80% ofconstants/time/*siblings.@stdlib/constants/time/seconds-in-minute@stdlib/constants/time/seconds-in-minute: normalizes the "Notes" prose inREADME.mdanddocs/repl.txtto the long variant ("inaccuracies due to daylight savings conventions, crossing timezones, or other complications with time and dates"), matching 80% ofconstants/time/*siblings.Validation
Three independent validation agents reviewed each outlier before any patch was applied:
lib/index.jsand surrounding docs against majority siblings. Result for all three outliers:confirmed-drift. Sub-minute units likeseconds-in-hourandminutes-in-hourare equally DST-irrelevant yet still use the long note, so the short-form wording does not track a deliberate semantic boundary.test/test.jsfiles assert numeric equality only and do not depend on the Notes wording. Confirmed the short-form wording also appears inlib/node_modules/@stdlib/repl/help/data/data.{csv,json}, but those are downstream artifacts generated from each package'sdocs/repl.txtbyrepl/help/scripts/build.js. NoAUTOGENERATEDmarkers in any of the six edited files. Result for all three outliers:safe-to-fix.confirmed-drift.Deliberately excluded
<section class="related">blocks — these are auto-populated downstream; drift is upstream and out of scope for this routine.package.json.keywords— no shared majority pattern (each unit has its own abbreviation triplet, e.g.seconds/secs/sec, whilemilliseconds/msonly ships two variants).texderivation block inlib/index.js— present in 10/15 (67%, below threshold) and tracks whether the constant has a non-trivial multiplicative derivation; absence in leaf constants likehours-in-dayormonths-in-yearis intentional.Related Issues
None.
Questions
None.
Other
20260501(deterministic from today's date for reproducibility).philipp/drift-constants-time-2026-05-01.repl/help/data/{data.csv,data.json}will pick up the corrected wording the next timerepl/help/scripts/build.jsruns; not regenerated in this PR to keep the diff focused.~/drift-reports/drift-constants-time-2026-05-01.md.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by Claude Code running the cross-package API drift detection routine: a structural feature scan over the 15
constants/timepackages, three independent validation agents (semantic-review, cross-reference, structural-review) per outlier, and a final per-package PR-body refinement pass. Every patch is a textual normalization of the "Notes" wording to the namespace majority; no code, signatures, or test expectations were changed. A human will audit before promoting from draft.@stdlib-js/reviewers
Generated by Claude Code