Review/browser native - #17
Merged
Merged
Conversation
Adopt the Smashing Magazine 'ship less JavaScript' Baseline model: - add optional baseline field (widely/newly/limited) to the replacement DB, surfaced in table/markdown/json output alongside confidence - add packages: timeago.js, pluralize, numeral, accounting, humanize-duration, lodash.groupby, lodash.union/intersection/difference, and a UI Primitives cluster (a11y-dialog, focus-trap, body-scroll-lock, tippy.js) - document the three-question decision framework and progressive enhancement in SKILL.md; add a 'keep for now' Temporal note explaining why dayjs/date-fns are not flagged - bump skill version to 1.2.0
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
1 participant
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.
This pull request adds "Baseline" (browser support maturity) status to the browser-native dependency scanner skill, updates its documentation and reference guides, and surfaces this information in all user-facing outputs (table, markdown, and JSON). The goal is to help users distinguish between native API replacements that are safe to adopt broadly ("widely"), those that are newly available and require audience checks or fallbacks ("newly"), and those that are not ready for general use ("limited"). Several new packages and UI primitives are also added to the scanner database, and the documentation is updated to clarify how to interpret and act on Baseline status.
Enhancements to browser-native skill outputs:
Documentation and guidance updates:
SKILL.mdandreplacements-guide.mdare extensively updated to explain Baseline status, how to interpret it, and how to safely migrate dependencies based on both confidence and Baseline. Guidance is added for progressive enhancement and migration priorities. [1] [2] [3] [4] [5] [6] [7]Newly supported packages and primitives:
timeago.js,pluralize,numeral,accounting,humanize-duration,lodash.groupby,lodash.union/intersection/difference, and a cluster of UI primitives (a11y-dialog,focus-trap,body-scroll-lock,tippy.js). [1] [2]Skill metadata and changelog:
browser-nativeskill version to 1.2.0 and updates the skill description to mention Baseline. [1] [2]CHANGELOG.mdfor the repository, documenting all notable changes and skill versions.Linting and config:
.markdownlint.jsonto allow duplicate headings only among siblings, improving markdown linting flexibility.