Skip to content

Conversation

@danielroe
Copy link
Member

working on yet more improvements to the apis we use internally

@vercel
Copy link

vercel bot commented Feb 8, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 8, 2026 10:15pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 8, 2026 10:15pm
npmx-lunaria Ignored Ignored Feb 8, 2026 10:15pm

Request Review

@codecov
Copy link

codecov bot commented Feb 8, 2026

Codecov Report

❌ Patch coverage is 51.38889% with 70 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
app/composables/npm/useSearch.ts 36.17% 44 Missing and 16 partials ⚠️
app/composables/npm/useNpmSearch.ts 66.66% 5 Missing ⚠️
app/composables/npm/search-utils.ts 40.00% 3 Missing ⚠️
app/composables/npm/useOutdatedDependencies.ts 93.10% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@danielroe danielroe marked this pull request as ready for review February 8, 2026 22:18
@danielroe danielroe merged commit a851e1e into main Feb 8, 2026
20 checks passed
@danielroe danielroe deleted the fix/npm-api branch February 8, 2026 22:19
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 8, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This pull request introduces a new unified search infrastructure by creating a new useSearch composable alongside a lightweight server API endpoint for package metadata. It refactors the npm search composables to utilise server-side metadata lookups instead of direct registry calls, whilst replacing useNpmSearch with the new generalised useSearch hook across components and pages. The changes include new utilities for transforming package metadata into search results, updates to existing composables (useOrgPackages, useOutdatedDependencies, useUserPackages) to leverage the metadata-based approach, and supporting infrastructure updates such as new route caching rules and test fixture enhancements.

Possibly related PRs

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/npm-api

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

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