chore: update knip config & remove unused code (#2126)

Author: webpro

.github/workflows/ci.yml

@@ -199,7 +199,7 @@ jobs:
         run: pnpm knip
 
       - name: 🧹 Check for unused production code
-        run: pnpm knip --production
+        run: pnpm knip --production --exclude dependencies
 
   i18n:
     name: 🌐 i18n validation

app/components/Compare/ComparisonGrid.vue

@@ -1,7 +1,7 @@
 <script setup lang="ts">
 import type { ModuleReplacement } from 'module-replacements'
 
-export interface ComparisonGridColumn {
+interface ComparisonGridColumn {
   name: string
   version?: string
   /** Module replacement data for this package (if available) */

app/components/Package/ListControls.vue

@@ -1,5 +1,5 @@
 <script setup lang="ts">
-export type SortOption = 'downloads' | 'updated' | 'name-asc' | 'name-desc'
+type SortOption = 'downloads' | 'updated' | 'name-asc' | 'name-desc'
 
 const props = defineProps<{
   /** Current search/filter text */

app/components/Select/Field.vue

@@ -19,7 +19,7 @@ const SELECT_FIELD_LABEL_SIZES = {
 
 const model = defineModel<string | undefined>({ default: undefined })
 
-export interface SelectFieldProps extends SelectBaseProps {
+interface SelectFieldProps extends SelectBaseProps {
   items: { label: string; value: string; disabled?: boolean }[]
   size?: keyof typeof SELECT_FIELD_SIZES
   selectAttrs?: Omit<SelectBaseProps, 'size' | 'id'> &

app/composables/npm/useAlgoliaSearch.ts

@@ -107,7 +107,7 @@ function hitToSearchResult(hit: AlgoliaHit): NpmSearchResult {
   }
 }
 
-export interface AlgoliaSearchOptions {
+interface AlgoliaSearchOptions {
   size?: number
   from?: number
   filters?: string
@@ -121,7 +121,7 @@ export interface AlgoliaMultiSearchChecks {
   checkPackage?: string
 }
 
-export interface AlgoliaSearchWithSuggestionsResult {
+interface AlgoliaSearchWithSuggestionsResult {
   search: NpmSearchResponse
   orgExists: boolean
   userExists: boolean

app/composables/npm/useNpmSearch.ts

@@ -1,6 +1,6 @@
 import { emptySearchResponse, metaToSearchResult } from './search-utils'
 
-export interface NpmSearchOptions {
+interface NpmSearchOptions {
   size?: number
   from?: number
 }

app/composables/useVirtualInfiniteScroll.ts

@@ -11,7 +11,7 @@ export interface WindowVirtualizerHandle {
   ) => void
 }
 
-export interface UseVirtualInfiniteScrollOptions {
+interface UseVirtualInfiniteScrollOptions {
   /** Reference to the WindowVirtualizer component */
   listRef: Ref<WindowVirtualizerHandle | null>
   /** Current item count */

app/utils/atproto/likes.ts

@@ -2,15 +2,12 @@ import { FetchError } from 'ofetch'
 import { handleAuthError } from '~/utils/atproto/helpers'
 import type { PackageLikes } from '#shared/types/social'
 
-export type LikeResult = { success: true; data: PackageLikes } | { success: false; error: Error }
+type LikeResult = { success: true; data: PackageLikes } | { success: false; error: Error }
 
 /**
  * Like a package via the API
  */
-export async function likePackage(
-  packageName: string,
-  userHandle?: string | null,
-): Promise<LikeResult> {
+async function likePackage(packageName: string, userHandle?: string | null): Promise<LikeResult> {
   try {
     const result = await $fetch<PackageLikes>('/api/social/like', {
       method: 'POST',
@@ -28,10 +25,7 @@ export async function likePackage(
 /**
  * Unlike a package via the API
  */
-export async function unlikePackage(
-  packageName: string,
-  userHandle?: string | null,
-): Promise<LikeResult> {
+async function unlikePackage(packageName: string, userHandle?: string | null): Promise<LikeResult> {
   try {
     const result = await $fetch<PackageLikes>('/api/social/like', {
       method: 'DELETE',

app/utils/atproto/profile.ts

@@ -1,7 +1,7 @@
 import { FetchError } from 'ofetch'
 import { handleAuthError } from './helpers'
 
-export type UpdateProfileResult = {
+type UpdateProfileResult = {
   success: boolean
   error?: Error
 }

config/env.ts

@@ -6,8 +6,6 @@ import * as process from 'node:process'
 import { version as packageVersion } from '../package.json'
 import { getNextVersion } from '../scripts/next-version'
 
-export { packageVersion as version }
-
 /**
  * Environment variable `PULL_REQUEST` provided by Netlify.
  * @see {@link https://docs.netlify.com/build/configure-builds/environment-variables/#git-metadata}
@@ -17,7 +15,7 @@ export { packageVersion as version }
  *
  * Whether triggered by a GitHub PR
  */
-export const isPR = process.env.PULL_REQUEST === 'true' || !!process.env.VERCEL_GIT_PULL_REQUEST_ID
+const isPR = process.env.PULL_REQUEST === 'true' || !!process.env.VERCEL_GIT_PULL_REQUEST_ID
 
 /**
  * Environment variable `REVIEW_ID` provided by Netlify.
@@ -28,7 +26,7 @@ export const isPR = process.env.PULL_REQUEST === 'true' || !!process.env.VERCEL_
  *
  * Pull request number (if in a PR environment)
  */
-export const prNumber = process.env.REVIEW_ID || process.env.VERCEL_GIT_PULL_REQUEST_ID || null
+const prNumber = process.env.REVIEW_ID || process.env.VERCEL_GIT_PULL_REQUEST_ID || null
 
 /**
  * Environment variable `BRANCH` provided by Netlify.
@@ -39,7 +37,7 @@ export const prNumber = process.env.REVIEW_ID || process.env.VERCEL_GIT_PULL_REQ
  *
  * Git branch
  */
-export const gitBranch = process.env.BRANCH || process.env.VERCEL_GIT_COMMIT_REF
+const gitBranch = process.env.BRANCH || process.env.VERCEL_GIT_COMMIT_REF
 
 /**
  * Whether this is the canary environment (main.npmx.dev).
@@ -68,7 +66,7 @@ export const isCanary =
  *
  * Whether this is some sort of preview environment.
  */
-export const isPreview =
+const isPreview =
   isPR ||
   (process.env.CONTEXT && process.env.CONTEXT !== 'production') ||
   process.env.VERCEL_ENV === 'preview' ||
@@ -118,7 +116,7 @@ export const getProductionUrl = () =>
     : undefined
 
 const git = Git()
-export async function getGitInfo() {
+async function getGitInfo() {
   let branch
   try {
     branch = gitBranch || (await git.revparse(['--abbrev-ref', 'HEAD']))