Skip to content

fix(site): use first-match-wins for multipart primary selection#519

Merged
decepulis merged 1 commit intomainfrom
codex/api-docs-builder-primary-first-match
Feb 12, 2026
Merged

fix(site): use first-match-wins for multipart primary selection#519
decepulis merged 1 commit intomainfrom
codex/api-docs-builder-primary-first-match

Conversation

@decepulis
Copy link
Collaborator

Summary

  • apply first-match-wins logic for multipart primary selection
  • keep behavior non-fatal while ensuring only one part is marked primary
  • add warning log when additional primary candidates are skipped

Validation

  • pnpm -C site test scripts/api-docs-builder/src/tests
  • pnpm -C site api-docs

@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
vjs-10-demo-react Ignored Ignored Feb 12, 2026 4:14pm

Request Review

@netlify
Copy link

netlify bot commented Feb 12, 2026

Deploy Preview for vjs10-site ready!

Name Link
🔨 Latest commit f66ba36
🔍 Latest deploy log https://app.netlify.com/projects/vjs10-site/deploys/698dfc4bf8b4de0008f02efd
😎 Deploy Preview https://deploy-preview-519--vjs10-site.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Contributor

📦 Bundle Size Report

Package Size Diff %
@videojs/element 1.78 kB 0 B ░░░░░░░░ 0%
@videojs/store 2.23 kB 0 B ░░░░░░░░ 0%
@videojs/core 3.53 kB 0 B ░░░░░░░░ 0%
@videojs/html 7.42 kB 0 B ░░░░░░░░ 0%
@videojs/react 6.43 kB 0 B ░░░░░░░░ 0%
@videojs/utils 2.13 kB 0 B ░░░░░░░░ 0%

Total: 23.51 kB · 0 B · 0%


Subpath Breakdown

@videojs/element
Subpath Base PR Diff %
. 845 B 845 B 0 B 0%
./context 977 B 977 B 0 B 0%
total 1.78 kB 1.78 kB 0 B 0%
@videojs/store
Subpath Base PR Diff %
. 1.32 kB 1.32 kB 0 B 0%
./html 612 B 612 B 0 B 0%
./react 318 B 318 B 0 B 0%
total 2.23 kB 2.23 kB 0 B 0%
@videojs/core
Subpath Base PR Diff %
. 1003 B 1003 B 0 B 0%
./dom 2.55 kB 2.55 kB 0 B 0%
total 3.53 kB 3.53 kB 0 B 0%
@videojs/html
Subpath Base PR Diff %
. 7.42 kB 7.42 kB 0 B 0%
@videojs/react
Subpath Base PR Diff %
. 6.43 kB 6.43 kB 0 B 0%
@videojs/utils
Subpath Base PR Diff %
./array 152 B 152 B 0 B 0%
./dom 583 B 583 B 0 B 0%
./events 259 B 259 B 0 B 0%
./function 213 B 213 B 0 B 0%
./object 164 B 164 B 0 B 0%
./predicate 296 B 296 B 0 B 0%
./time 517 B 517 B 0 B 0%
total 2.13 kB 2.13 kB 0 B 0%

ℹ️ How to interpret

Each package shows its own code size with workspace and peer dependencies externalized.
Sizes are minified + brotli, measured via size-limit with esbuild.

Icon Meaning
No change
🔺 Increased ≤ 10%
🔴 Increased > 10%
🔽 Decreased
🆕 New (no baseline)

Run pnpm size locally to check current sizes.

@decepulis decepulis merged commit d5e41bf into main Feb 12, 2026
10 checks passed
@decepulis decepulis deleted the codex/api-docs-builder-primary-first-match branch February 12, 2026 16:16
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.

1 participant