From 4723e1a4960604f820fab8ae87dd1e5a4fc7cf6d Mon Sep 17 00:00:00 2001
From: Nogic <24802730+nogic1008@users.noreply.github.com>
Date: Tue, 7 Feb 2023 04:20:55 +0000
Subject: [PATCH] refactor: move type definition

---
 __tests__/content/series.test.ts    |  3 ++-
 __tests__/content/songs.test.ts     |  2 +-
 __tests__/pages/songs/index.test.ts |  3 +--
 __tests__/test-utils.ts             | 16 +++++++++-------
 components/content/Charts.vue       |  2 +-
 components/content/SeriesList.vue   |  2 +-
 composables/useSoftwareData.ts      | 25 +++++++++++++++++++++----
 composables/useSoftwareList.ts      | 13 +++++++++----
 composables/useSongData.ts          | 14 +++++++++++---
 composables/useSongList.ts          |  9 ++++++---
 pages/songs/index.vue               |  3 +--
 src/content.ts                      | 28 ----------------------------
 12 files changed, 63 insertions(+), 57 deletions(-)
 delete mode 100644 src/content.ts

diff --git a/__tests__/content/series.test.ts b/__tests__/content/series.test.ts
index b45c0468a..834231c99 100644
--- a/__tests__/content/series.test.ts
+++ b/__tests__/content/series.test.ts
@@ -5,8 +5,9 @@ import { basename } from 'node:path'
 import { load } from 'js-yaml'
 import { describe, expect, test } from 'vitest'
 
+import type { Software } from '~~/composables/useSoftwareData'
+import { platforms, regionList } from '~~/composables/useSoftwareData'
 import seriesList from '~~/content/series/.seriesList.json'
-import { platforms, regionList, Software } from '~~/src/content'
 
 const dirpath = './content/series'
 
diff --git a/__tests__/content/songs.test.ts b/__tests__/content/songs.test.ts
index 6fcba0305..9763db10a 100644
--- a/__tests__/content/songs.test.ts
+++ b/__tests__/content/songs.test.ts
@@ -5,8 +5,8 @@ import { basename } from 'node:path'
 import { load } from 'js-yaml'
 import { describe, expect, test } from 'vitest'
 
+import type { Song } from '~~/composables/useSongData'
 import seriesList from '~~/content/series/.seriesList.json'
-import { Song } from '~~/src/content'
 
 const dirpath = './content/songs'
 
diff --git a/__tests__/pages/songs/index.test.ts b/__tests__/pages/songs/index.test.ts
index 8e1c3f762..758cf2957 100644
--- a/__tests__/pages/songs/index.test.ts
+++ b/__tests__/pages/songs/index.test.ts
@@ -4,7 +4,6 @@ import { ref } from 'vue'
 
 import useSongList from '~~/composables/useSongList'
 import SongList from '~~/pages/songs/index.vue'
-import { Song } from '~~/src/content'
 
 import { mountAsync, plugins } from '../../test-utils'
 
@@ -26,7 +25,7 @@ describe('pages/songs/index.vue', () => {
       bpm: 180,
       series: ['1st-jp', '2nd']
     }
-  ] as Song[]
+  ] as Awaited<ReturnType<typeof useSongList>>['songs']['value']
   const global = { plugins, stubs: { NuxtLink: RouterLinkStub, Series: true } }
 
   beforeEach(() => {
diff --git a/__tests__/test-utils.ts b/__tests__/test-utils.ts
index 1e62d5c1a..af2e1125d 100644
--- a/__tests__/test-utils.ts
+++ b/__tests__/test-utils.ts
@@ -4,10 +4,6 @@ import { flushPromises, mount } from '@vue/test-utils'
 import type { DefineComponent, Plugin } from 'vue'
 import { defineComponent, h, Suspense } from 'vue'
 
-import type { SoftwareParsedContent } from '~~/composables/useSoftwareData'
-import type { SoftwareListData } from '~~/composables/useSoftwareList'
-import type { SongParsedContent } from '~~/composables/useSongData'
-
 export const plugins: (Plugin | [Plugin, ...unknown[]])[] = [
   [Oruga, bulmaConfig]
 ]
@@ -34,7 +30,9 @@ export const mountAsync = async <T extends DefineComponent<{}, {}, {}, any>>(
   return wrapper
 }
 
-export const mockSoftware: SoftwareParsedContent = {
+export const mockSoftware: Awaited<
+  ReturnType<typeof useSoftwareData>
+>['software']['value'] = {
   slug: '1st-jp',
   title: 'Dance Dance Revolution',
   platform: 'Play Station',
@@ -50,7 +48,9 @@ export const mockSoftware: SoftwareParsedContent = {
   _id: 'content:series:1st-jp.md'
 }
 
-export const mockSoftwareList: SoftwareListData[] = [
+export const mockSoftwareList: Awaited<
+  ReturnType<typeof useSoftwareList>
+>['softwareList']['value'] = [
   {
     slug: '1st-jp',
     title: 'Dance Dance Revolution',
@@ -67,7 +67,9 @@ export const mockSoftwareList: SoftwareListData[] = [
   }
 ]
 
-export const mockSong: SongParsedContent = {
+export const mockSong: Awaited<
+  ReturnType<typeof useSongData>
+>['song']['value'] = {
   slug: 'butterfly',
   name: 'butterfly',
   artist: 'smile. dk',
diff --git a/components/content/Charts.vue b/components/content/Charts.vue
index 96ceeeb58..87c60c38f 100644
--- a/components/content/Charts.vue
+++ b/components/content/Charts.vue
@@ -13,7 +13,7 @@
 </template>
 
 <script lang="ts" setup>
-import type { Software } from '~~/src/content'
+import type { Software } from '~~/composables/useSoftwareData'
 
 interface Props {
   difficulties: Software['difficulties']
diff --git a/components/content/SeriesList.vue b/components/content/SeriesList.vue
index a6028a289..bb3ec2d76 100644
--- a/components/content/SeriesList.vue
+++ b/components/content/SeriesList.vue
@@ -7,8 +7,8 @@
 </template>
 
 <script lang="ts" setup>
+import type { Song } from '~~/composables/useSongData'
 import seriesList from '~~/content/series/.seriesList.json'
-import type { Song } from '~~/src/content'
 
 interface SeriesListProps {
   series: Song['series']
diff --git a/composables/useSoftwareData.ts b/composables/useSoftwareData.ts
index 1a79e06f9..cf91a637f 100644
--- a/composables/useSoftwareData.ts
+++ b/composables/useSoftwareData.ts
@@ -1,16 +1,33 @@
 import type { ParsedContent } from '@nuxt/content/dist/runtime/types'
 import type { Ref } from 'vue'
 
-import type { Software } from '~~/src/content'
+import type seriesList from '~~/content/series/.seriesList.json'
 
-export type SoftwareParsedContent = ParsedContent & Software
+export const regionList = ['JP', 'US', 'EU', 'None'] as const
+
+export const platforms = [
+  'Play Station',
+  'Play Station 2',
+  'Dreamcast',
+  'GAME BOY COLOR',
+  'Wii'
+] as const
+
+export interface Software {
+  slug: keyof typeof seriesList
+  title: string
+  platform: (typeof platforms)[number]
+  region: (typeof regionList)[number]
+  launched: string
+  difficulties: { [key in number]: { name: string; class: string } }
+}
 
 export default async function (slug: string) {
   const { data } = await useAsyncData(`/software/${slug}`, () =>
-    queryContent<SoftwareParsedContent>()
+    queryContent<Software & ParsedContent>()
       .where({ _type: 'markdown', slug })
       .findOne()
   )
 
-  return { software: data as Ref<SoftwareParsedContent> }
+  return { software: data as Ref<Software & ParsedContent> }
 }
diff --git a/composables/useSoftwareList.ts b/composables/useSoftwareList.ts
index 3489ae43b..2763fe0ad 100644
--- a/composables/useSoftwareList.ts
+++ b/composables/useSoftwareList.ts
@@ -1,15 +1,20 @@
 import type { Ref } from 'vue'
 
-import type { Software } from '~~/src/content'
+import type { Software } from '~~/composables/useSoftwareData'
 
-const keys = ['slug', 'title', 'platform', 'region', 'launched'] as const
-export type SoftwareListData = Pick<Software, (typeof keys)[number]>
+export type SoftwareListData = Omit<Software, 'difficulties'>
 
 export default async function () {
   const { data } = await useAsyncData('/software', () =>
     queryContent<Software>('series')
       .sort({ launched: 1 })
-      .only([...keys])
+      .only<(keyof Software)[]>([
+        'slug',
+        'title',
+        'platform',
+        'region',
+        'launched'
+      ])
       .find()
   )
 
diff --git a/composables/useSongData.ts b/composables/useSongData.ts
index ae27fc2c4..f750029d8 100644
--- a/composables/useSongData.ts
+++ b/composables/useSongData.ts
@@ -1,16 +1,24 @@
 import type { ParsedContent } from '@nuxt/content/dist/runtime/types'
 import type { Ref } from 'vue'
 
-import type { Song } from '~~/src/content'
+import type seriesList from '~~/content/series/.seriesList.json'
+
+export interface Song {
+  slug: string
+  name: string
+  artist: string
+  bpm: number | string
+  series: (keyof typeof seriesList)[]
+}
 
 export type SongParsedContent = ParsedContent & Song
 
 export default async function (slug: string) {
   const { data } = await useAsyncData(`/songs/${slug}`, () =>
-    queryContent<SongParsedContent>()
+    queryContent<ParsedContent & Song>()
       .where({ _type: 'markdown', slug: { $eq: slug } })
       .findOne()
   )
 
-  return { song: data as Ref<SongParsedContent> }
+  return { song: data as Ref<ParsedContent & Song> }
 }
diff --git a/composables/useSongList.ts b/composables/useSongList.ts
index f4330c738..950114001 100644
--- a/composables/useSongList.ts
+++ b/composables/useSongList.ts
@@ -1,11 +1,14 @@
 import type { Ref } from 'vue'
 
-import type { Song } from '~~/src/content'
+import type { Song } from '~~/composables/useSongData'
 
-export default async function (key: string, ...keys: (keyof Song)[]) {
+export default async function <T extends keyof Song>(
+  key: string,
+  ...keys: T[]
+) {
   const { data } = await useAsyncData(key, () =>
     queryContent<Song>('songs').sort({ name: 1 }).only(keys).find()
   )
 
-  return { songs: data as Ref<Pick<Song, (typeof keys)[number]>[]> }
+  return { songs: data as Ref<Pick<Song, T>[]> }
 }
diff --git a/pages/songs/index.vue b/pages/songs/index.vue
index 6deedfb72..27fa412e8 100644
--- a/pages/songs/index.vue
+++ b/pages/songs/index.vue
@@ -17,7 +17,7 @@
       <OTableColumn v-slot="props" label="Series">
         <div class="buttons">
           <NuxtLink
-            v-for="slug in (props.row as Song).series"
+            v-for="slug in (props.row as (typeof songs[number])).series"
             :key="slug"
             class="button is-small"
             :to="`/series/${slug}`"
@@ -45,7 +45,6 @@
 <script lang="ts" setup>
 import useSongList from '~~/composables/useSongList'
 import seriesList from '~~/content/series/.seriesList.json'
-import type { Song } from '~~/src/content'
 
 useHead({ title: 'Song List' })
 
diff --git a/src/content.ts b/src/content.ts
deleted file mode 100644
index a67d3b227..000000000
--- a/src/content.ts
+++ /dev/null
@@ -1,28 +0,0 @@
-import type seriesList from '~~/content/series/.seriesList.json'
-
-export const regionList = ['JP', 'US', 'EU', 'None'] as const
-
-export const platforms = [
-  'Play Station',
-  'Play Station 2',
-  'Dreamcast',
-  'GAME BOY COLOR',
-  'Wii'
-] as const
-
-export interface Software {
-  slug: keyof typeof seriesList
-  title: string
-  platform: (typeof platforms)[number]
-  region: (typeof regionList)[number]
-  launched: string
-  difficulties: { [key in number]: { name: string; class: string } }
-}
-
-export interface Song {
-  slug: string
-  name: string
-  artist: string
-  bpm: number | string
-  series: (keyof typeof seriesList)[]
-}