Skip to content

feat: add vue-player module - #1707

Open
VladChernyak wants to merge 3 commits into
nuxt:mainfrom
VladChernyak:feat/add-vue-player
Open

VladChernyak wants to merge 3 commits into
nuxt:mainfrom
VladChernyak:feat/add-vue-player

Conversation

@VladChernyak

Copy link
Copy Markdown

🔗 Linked issue

resolves #1490

📚 Description

Adds @vue-player/nuxt, the Nuxt 4 module for vue-player, a video player library for Vue.

vue-player supports HLS streaming (via optional hls.js), WebVTT subtitles, chapters and thumbnail previews on the timeline. The module registers the VideoPlayer component and the usePlayer composable as auto-imports, adds the player styles globally, and has a prefix option to avoid naming conflicts.

Docs: https://vue-player.vercel.app

@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ebaf4f7f-106b-42c1-97fc-3aa0be395ae5
Commits

Reviewing files that changed from the base of the PR and between 70515d7 and f9f1fca.

Files ignored due to path filters: 1
  • icons/vue-player.svg is excluded by !**/*.svg
Files selected for processing: 1
  • modules/vue-player.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

Walkthrough

The change adds a vue-player entry to the Nuxt module catalogue. The entry includes package and repository links, media categorisation, maintainer details, and a Nuxt compatibility requirement of >=4.0.0.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: Minimal · up to f9f1f

No actionable merge-blocking risk was identified.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title "feat: add vue-player module" matches the main change. The PR adds a vue-player Nuxt module catalogue entry in modules/vue-player.yml, which is the only reviewed file. The title is short, sp…
Description check Passed The description follows the template well. It links issue #1490 with "resolves #1490" and gives a clear Description section that explains what the module does, its key features, and the docs link. The…
Linked Issues check Passed The change adds the vue-player catalogue entry requested by issue #1490. It includes the package and repository links, media categorisation, maintainer details, and Nuxt compatibility >=4.0.0. The…
Out of Scope Changes check Passed The change is limited to modules/vue-player.yml. The added catalogue metadata and links directly support the module listing requested by issue #1490. No unrelated source, test, or documentation chan…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Full details: Title check

Explanation

The title "feat: add vue-player module" matches the main change. The PR adds a vue-player Nuxt module catalogue entry in modules/vue-player.yml, which is the only reviewed file. The title is short, specific, and follows a conventional commit style.

Full details: Description check

Explanation

The description follows the template well. It links issue #1490 with "resolves #1490" and gives a clear Description section that explains what the module does, its key features, and the docs link. The template's linked issue section is filled in. The Description section is filled in with substantive content. The template's authoring notes are HTML comments and do not need to be kept. The description does not state whether AI tools were used, and the template asks the author to write in their own voice. The check passes because the required sections are present and substantive.

Full details: Linked Issues check

Explanation

The change adds the vue-player catalogue entry requested by issue #1490. It includes the package and repository links, media categorisation, maintainer details, and Nuxt compatibility >=4.0.0. The issue describes the external module's auto-imports, styles, prefix option, and player features; this pull request lists that module and does not need to reimplement those external behaviours.

Full details: Out of Scope Changes check

Explanation

The change is limited to modules/vue-player.yml. The added catalogue metadata and links directly support the module listing requested by issue #1490. No unrelated source, test, or documentation changes are shown.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Module Listing Request]: vue-player

1 participant