poi-plugin-quest-info-2 0.8.0 → 0.8.1

package/i18n/en-US.json

@@ -21,6 +21,8 @@
   "Locked": "Locked",
   "In Progress": "In Progress {{number}}",
   "Completed": "Completed",
+  "Already Completed": "Already Completed",
+  "Unknown": "Unknown",
   "TotalQuests": "Total {{number}} quests",
   "Sync with game": "Sync with game (Show in-game quests only)",
   "Version": "Version: {{version}}",

package/i18n/ja-JP.json

@@ -20,7 +20,9 @@
   "One-time": "単発",
   "Locked": "ロック中",
   "In Progress": "進行中 {{number}}",
-  "Completed": "完了",
+  "Completed": "達成",
+  "Already Completed": "完了",
+  "Unknown": "状態不明",
   "TotalQuests": "全て {{number}} 個の任務",
   "Sync with game": "ゲームと連動(インゲームクエストのみ表示する)",
   "Version": "バージョン: {{version}}",

package/i18n/ko-KR.json

@@ -5,7 +5,9 @@
   "All": "전체",
   "Locked": "잠김",
   "In Progress": "진행 중 {{number}}",
-  "Completed": "완료",
+  "Completed": "달성",
+  "Already Completed": "완료",
+  "Unknown": "알 수 없는 상태",
   "Version": "버전: {{version}}",
   "Data Version": "데이터 버전: {{version}}",
   "View source code on GitHub": "GitHub에서 소스코드를 알아보세요",

package/i18n/zh-CN.json

@@ -20,7 +20,9 @@
   "One-time": "单次",
   "Locked": "未解锁",
   "In Progress": "进行中 {{number}}",
-  "Completed": "已完成",
+  "Completed": "达成",
+  "Already Completed": "已完成",
+  "Unknown": "状态未知",
   "TotalQuests": "一共 {{number}} 个任务",
   "Sync with game": "与游戏同步（仅展示游戏内任务）",
   "Version": "版本: {{version}}",

package/i18n/zh-TW.json

@@ -20,7 +20,9 @@
   "One-time": "單次",
   "Locked": "未解鎖",
   "In Progress": "進行中 {{number}}",
-  "Completed": "已完成",
+  "Completed": "達成",
+  "Already Completed": "已完成",
+  "Unknown": "未知狀態",
   "TotalQuests": "一共 {{number}} 個任務",
   "Sync with game": "與遊戲同步",
   "Version": "版本: {{version}}",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poi-plugin-quest-info-2",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "private": false,
   "description": "show quest info",
   "homepage": "https://github.com/lawvs/poi-plugin-quest-2/",
@@ -34,6 +34,7 @@
     "update": "npm run downloadKcanotifyData && npm run downloadKcQuestsData && npm run genQuestData"
   },
   "dependencies": {
+    "moize": "^6.1.1",
     "react-use": "^17.3.1",
     "react-virtualized": "^9.22.3"
   },

package/src/__tests__/__snapshots__/questHelper.spec.ts.snap

@@ -0,0 +1,399 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`questHelper should 101 locked quests match snapshot 1`] = `
+Object {
+  "102": true,
+  "103": true,
+  "104": true,
+  "105": true,
+  "106": true,
+  "107": true,
+  "108": true,
+  "109": true,
+  "110": true,
+  "1103": true,
+  "1104": true,
+  "1106": true,
+  "111": true,
+  "112": true,
+  "113": true,
+  "114": true,
+  "115": true,
+  "116": true,
+  "117": true,
+  "118": true,
+  "119": true,
+  "120": true,
+  "121": true,
+  "122": true,
+  "123": true,
+  "124": true,
+  "125": true,
+  "126": true,
+  "127": true,
+  "128": true,
+  "129": true,
+  "130": true,
+  "131": true,
+  "132": true,
+  "133": true,
+  "134": true,
+  "135": true,
+  "136": true,
+  "137": true,
+  "138": true,
+  "139": true,
+  "140": true,
+  "141": true,
+  "142": true,
+  "143": true,
+  "144": true,
+  "145": true,
+  "146": true,
+  "147": true,
+  "148": true,
+  "149": true,
+  "150": true,
+  "151": true,
+  "152": true,
+  "153": true,
+  "154": true,
+  "155": true,
+  "156": true,
+  "157": true,
+  "158": true,
+  "161": true,
+  "162": true,
+  "163": true,
+  "164": true,
+  "165": true,
+  "166": true,
+  "167": true,
+  "168": true,
+  "169": true,
+  "170": true,
+  "171": true,
+  "172": true,
+  "173": true,
+  "174": true,
+  "175": true,
+  "177": true,
+  "178": true,
+  "179": true,
+  "180": true,
+  "181": true,
+  "182": true,
+  "183": true,
+  "184": true,
+  "185": true,
+  "186": true,
+  "187": true,
+  "188": true,
+  "189": true,
+  "190": true,
+  "191": true,
+  "192": true,
+  "193": true,
+  "194": true,
+  "195": true,
+  "196": true,
+  "197": true,
+  "215": true,
+  "217": true,
+  "219": true,
+  "222": true,
+  "223": true,
+  "224": true,
+  "225": true,
+  "227": true,
+  "231": true,
+  "232": true,
+  "233": true,
+  "236": true,
+  "239": true,
+  "240": true,
+  "244": true,
+  "245": true,
+  "246": true,
+  "247": true,
+  "248": true,
+  "249": true,
+  "250": true,
+  "251": true,
+  "252": true,
+  "253": true,
+  "254": true,
+  "258": true,
+  "259": true,
+  "260": true,
+  "261": true,
+  "262": true,
+  "263": true,
+  "264": true,
+  "265": true,
+  "266": true,
+  "267": true,
+  "268": true,
+  "269": true,
+  "270": true,
+  "271": true,
+  "273": true,
+  "274": true,
+  "275": true,
+  "276": true,
+  "277": true,
+  "278": true,
+  "279": true,
+  "281": true,
+  "282": true,
+  "285": true,
+  "287": true,
+  "288": true,
+  "289": true,
+  "290": true,
+  "291": true,
+  "292": true,
+  "293": true,
+  "294": true,
+  "295": true,
+  "296": true,
+  "297": true,
+  "298": true,
+  "306": true,
+  "307": true,
+  "308": true,
+  "309": true,
+  "312": true,
+  "316": true,
+  "318": true,
+  "319": true,
+  "323": true,
+  "324": true,
+  "325": true,
+  "328": true,
+  "330": true,
+  "331": true,
+  "339": true,
+  "346": true,
+  "347": true,
+  "355": true,
+  "401": true,
+  "402": true,
+  "403": true,
+  "404": true,
+  "405": true,
+  "406": true,
+  "408": true,
+  "409": true,
+  "410": true,
+  "411": true,
+  "412": true,
+  "413": true,
+  "414": true,
+  "415": true,
+  "418": true,
+  "419": true,
+  "420": true,
+  "422": true,
+  "423": true,
+  "424": true,
+  "425": true,
+  "429": true,
+  "431": true,
+  "433": true,
+  "434": true,
+  "437": true,
+  "440": true,
+  "442": true,
+  "443": true,
+  "444": true,
+  "445": true,
+  "610": true,
+  "611": true,
+  "612": true,
+  "614": true,
+  "615": true,
+  "616": true,
+  "618": true,
+  "619": true,
+  "620": true,
+  "622": true,
+  "623": true,
+  "624": true,
+  "625": true,
+  "626": true,
+  "627": true,
+  "628": true,
+  "629": true,
+  "630": true,
+  "631": true,
+  "632": true,
+  "633": true,
+  "635": true,
+  "636": true,
+  "638": true,
+  "639": true,
+  "641": true,
+  "642": true,
+  "643": true,
+  "644": true,
+  "645": true,
+  "646": true,
+  "647": true,
+  "648": true,
+  "649": true,
+  "650": true,
+  "651": true,
+  "652": true,
+  "654": true,
+  "655": true,
+  "656": true,
+  "658": true,
+  "659": true,
+  "660": true,
+  "661": true,
+  "662": true,
+  "663": true,
+  "664": true,
+  "665": true,
+  "666": true,
+  "667": true,
+  "669": true,
+  "670": true,
+  "671": true,
+  "676": true,
+  "681": true,
+  "683": true,
+  "684": true,
+  "685": true,
+  "686": true,
+  "687": true,
+  "689": true,
+  "690": true,
+  "695": true,
+  "696": true,
+  "698": true,
+  "704": true,
+  "705": true,
+  "718": true,
+  "719": true,
+  "805": true,
+  "806": true,
+  "807": true,
+  "808": true,
+  "809": true,
+  "812": true,
+  "813": true,
+  "814": true,
+  "815": true,
+  "816": true,
+  "817": true,
+  "822": true,
+  "829": true,
+  "833": true,
+  "835": true,
+  "836": true,
+  "837": true,
+  "838": true,
+  "839": true,
+  "842": true,
+  "844": true,
+  "845": true,
+  "846": true,
+  "847": true,
+  "848": true,
+  "850": true,
+  "851": true,
+  "852": true,
+  "854": true,
+  "856": true,
+  "858": true,
+  "859": true,
+  "860": true,
+  "861": true,
+  "862": true,
+  "863": true,
+  "864": true,
+  "865": true,
+  "870": true,
+  "871": true,
+  "872": true,
+  "875": true,
+  "877": true,
+  "879": true,
+  "881": true,
+  "884": true,
+  "885": true,
+  "886": true,
+  "887": true,
+  "888": true,
+  "890": true,
+  "891": true,
+  "892": true,
+  "896": true,
+  "897": true,
+  "901": true,
+  "902": true,
+  "903": true,
+  "904": true,
+  "911": true,
+  "913": true,
+  "916": true,
+  "917": true,
+  "918": true,
+  "924": true,
+  "925": true,
+  "926": true,
+  "927": true,
+  "929": true,
+  "930": true,
+  "933": true,
+  "934": true,
+  "935": true,
+  "936": true,
+  "937": true,
+  "940": true,
+  "943": true,
+  "947": true,
+  "948": true,
+  "949": true,
+  "961": true,
+  "966": true,
+  "967": true,
+  "968": true,
+  "969": true,
+}
+`;
+
+exports[`questHelper should 196 getLockedQuest correct 1`] = `
+Object {
+  "197": true,
+  "346": true,
+  "892": true,
+  "930": true,
+}
+`;
+
+exports[`questHelper should 236 getCompletedQuest correct 1`] = `
+Object {
+  "101": true,
+  "102": true,
+  "103": true,
+  "105": true,
+  "106": true,
+  "107": true,
+  "113": true,
+  "119": true,
+  "121": true,
+  "124": true,
+  "125": true,
+  "131": true,
+  "133": true,
+  "136": true,
+  "152": true,
+  "227": true,
+  "235": true,
+  "273": true,
+  "307": true,
+}
+`;

package/src/__tests__/questHelper.spec.ts

@@ -0,0 +1,36 @@
+import {
+  getCompletedQuest,
+  getLockedQuest,
+  getPostQuestIds,
+  getPreQuestIds,
+} from '../questHelper'
+
+describe('questHelper', () => {
+  test('should getPreQuestIds correct', () => {
+    expect(getPreQuestIds(101)).toEqual([])
+    expect(getPreQuestIds(102)).toEqual([101])
+    expect(getPreQuestIds(236)).toEqual([235, 273])
+  })
+
+  test('should getPostQuestIds correct', () => {
+    expect(getPostQuestIds(101)).toEqual([102])
+    expect(getPostQuestIds(105)).toEqual([106, 108, 254, 401, 612, 816])
+    expect(getPostQuestIds(140)).toEqual([])
+  })
+
+  test('should 101 no completed quest', () => {
+    expect(getCompletedQuest([101])).toEqual({})
+  })
+
+  test('should 236 getCompletedQuest correct', () => {
+    expect(getCompletedQuest([236])).toMatchSnapshot()
+  })
+
+  test('should 101 locked quests match snapshot', () => {
+    expect(getLockedQuest([101])).toMatchSnapshot()
+  })
+
+  test('should 196 getLockedQuest correct', () => {
+    expect(getLockedQuest([196])).toMatchSnapshot()
+  })
+})

package/src/components/QuestCard/index.tsx

@@ -1,4 +1,4 @@
-import { Card, Elevation, H5, Text } from '@blueprintjs/core'
+import { Card, Elevation, H5 } from '@blueprintjs/core'
 import React, { forwardRef } from 'react'
 import type { StyledComponentProps } from 'styled-components'
 import { usePluginTranslation } from '../../poi/hooks'
@@ -7,6 +7,7 @@ import {
   guessQuestCategory,
   QUEST_STATUS,
 } from '../../questHelper'
+import { useQuestStatus } from '../../store/quest'
 import { QuestTag } from '../QuestTag'
 import {
   CardActionWrapper,
@@ -18,6 +19,7 @@ import {
   TagsWrapper,
 } from './styles'
 import { questIconMap, questStatusMap } from './utils'
+import { IN_POI } from '../../poi/env'
 
 export type QuestCardProps = {
   gameId: number
@@ -66,44 +68,30 @@ export const QuestCard = forwardRef<
   Card,
   // eslint-disable-next-line @typescript-eslint/ban-types
   QuestCardProps & StyledComponentProps<typeof Card, any, {}, never>
->(
-  (
-    {
-      gameId,
-      code,
-      name,
-      desc,
-      tip,
-      tip2,
-      status = QUEST_STATUS.DEFAULT,
-      ...props
-    },
-    ref
-  ) => {
-    const headIcon = questIconMap[guessQuestCategory(code).type]
-    const TailIcon = questStatusMap[status]
+>(({ gameId, code, name, desc, tip, tip2, ...props }, ref) => {
+  const status = useQuestStatus(gameId)
+  const headIcon = questIconMap[guessQuestCategory(code).type]
+  const TailIcon = questStatusMap[status]
+  const inPoi = IN_POI
 
-    return (
-      <FlexCard
-        ref={ref}
-        elevation={Elevation.ZERO}
-        interactive={false}
-        {...props}
-      >
-        <CardMedia src={headIcon}></CardMedia>
-        <CardBody>
-          <H5>{[code, name].filter((i) => i != undefined).join(' - ')}</H5>
-          <Text>{desc}</Text>
-          {tip2 && <b>{tip2}</b>}
-          {tip && <i>{tip}</i>}
+  return (
+    <FlexCard
+      ref={ref}
+      elevation={Elevation.ZERO}
+      interactive={false}
+      {...props}
+    >
+      <CardMedia src={headIcon}></CardMedia>
+      <CardBody>
+        <H5>{[code, name].filter((i) => i != undefined).join(' - ')}</H5>
+        <div>{desc}</div>
+        {tip2 && <b>{tip2}</b>}
+        {tip && <i>{tip}</i>}
 
-          <CardAction gameId={gameId}></CardAction>
-        </CardBody>
+        <CardAction gameId={gameId}></CardAction>
+      </CardBody>
 
-        <CardTail>
-          <TailIcon />
-        </CardTail>
-      </FlexCard>
-    )
-  }
-)
+      <CardTail>{inPoi && <TailIcon />}</CardTail>
+    </FlexCard>
+  )
+})

package/src/components/QuestCard/utils.tsx

@@ -50,6 +50,14 @@ export const questStatusMap: Record<QUEST_STATUS, React.FC> = {
       </Tooltip>
     )
   },
+  [QUEST_STATUS.UNKNOWN]: function AlreadyCompleted() {
+    const { t } = usePluginTranslation()
+    return (
+      <Tooltip content={t('Unknown')}>
+        <Icon icon={IconNames.HELP} iconSize={Icon.SIZE_LARGE}></Icon>
+      </Tooltip>
+    )
+  },
 }
 
 // transparent GIF pixel

package/src/components/QuestList.tsx

@@ -1,4 +1,5 @@
-import React, { useRef, useEffect, useCallback } from 'react'
+import React, { useCallback, useEffect, useRef } from 'react'
+import type { ListRowProps } from 'react-virtualized'
 // See https://github.com/bvaughn/react-virtualized
 import {
   AutoSizer,
@@ -7,13 +8,10 @@ import {
   List,
   ListRowRenderer,
 } from 'react-virtualized'
-import type { ListRowProps } from 'react-virtualized'
 import styled from 'styled-components'
-import { QUEST_STATUS } from '../questHelper'
+import { useIsQuestPluginTab } from '../poi/hooks'
 import type { UnionQuest } from '../questHelper'
 import { QuestCard } from './QuestCard'
-import { useIsQuestPluginTab } from '../poi/hooks'
-import { QUEST_API_STATE } from '../poi/types'
 
 const QuestListWrapper = styled.div`
   flex: 1;
@@ -26,28 +24,12 @@ const cache = new CellMeasurerCache({
   fixedWidth: true,
 })
 
-const questApiStateToQuestStatus = (
-  state: QUEST_API_STATE | undefined
-): QUEST_STATUS => {
-  switch (state) {
-    case QUEST_API_STATE.DEFAULT:
-      return QUEST_STATUS.DEFAULT
-    case QUEST_API_STATE.COMPLETED:
-      return QUEST_STATUS.COMPLETED
-    case QUEST_API_STATE.IN_PROGRESS:
-      return QUEST_STATUS.IN_PROGRESS
-    default:
-      return QUEST_STATUS.DEFAULT
-  }
-}
-
 const useQuestsRowRenderer = (quests: UnionQuest[]) => {
   const rowRenderer = useCallback(
     ({ key, index, style, parent }: ListRowProps) => {
       const quest = quests[index]
       const { gameId } = quest
       const { code, name, desc, memo, memo2, pre } = quest.docQuest
-      const questStatus = questApiStateToQuestStatus(quest.gameQuest?.api_state)
 
       return (
         <CellMeasurer
@@ -67,7 +49,6 @@ const useQuestsRowRenderer = (quests: UnionQuest[]) => {
               tip={memo}
               tip2={memo2}
               preQuest={pre}
-              status={questStatus}
             ></QuestCard>
           </div>
         </CellMeasurer>
@@ -78,7 +59,7 @@ const useQuestsRowRenderer = (quests: UnionQuest[]) => {
   return rowRenderer
 }
 
-export const QuestList: React.FC<{ quests: UnionQuest[] }> = ({ quests }) => {
+export const QuestList = ({ quests }: { quests: UnionQuest[] }) => {
   const activeTab = useIsQuestPluginTab()
   const listRef = useRef<List>(null)
   const rowRenderer: ListRowRenderer = useQuestsRowRenderer(quests)

package/src/poi/hooks.ts

@@ -1,9 +1,8 @@
-import { useState, useEffect } from 'react'
+import { useEffect, useState } from 'react'
 import { useTranslation } from 'react-i18next'
-import { observePluginStore, observePoiStore } from './store'
 import { name as PACKAGE_NAME } from '../../package.json'
+import { observePluginStore, observePoiStore } from './store'
 import { GameQuest, PoiQuestState, PoiState, QuestTab } from './types'
-import { createGlobalState } from 'react-use'
 
 export const activeQuestsSelector = (state: PoiState): PoiQuestState =>
   state?.info?.quests?.activeQuests ?? {}
@@ -25,10 +24,8 @@ export const usePluginTranslation = () => {
   return useTranslation(PACKAGE_NAME)
 }
 
-const useGlobalGameQuest = createGlobalState<GameQuest[]>([])
-
 export const useGameQuest = () => {
-  const [quests, setQuests] = useGlobalGameQuest()
+  const [quests, setQuests] = useState<GameQuest[]>([])
   useEffect(() => {
     const listener = (quests: GameQuest[] | null) => setQuests(quests ?? [])
     // See reducer.ts

package/src/questHelper.ts

@@ -1,3 +1,4 @@
+import moize from 'moize'
 import { QuestData } from '../build/kcanotifyGamedata'
 import { KcwikiQuestData } from '../build/kcQuestsData'
 import newQuestData from '../build/kcQuestsData/quests-scn-new.json'
@@ -217,10 +218,66 @@ export const getQuestIdByCode = (code: string) => {
   return null
 }
 
+export const getPreQuestIds = (gameId: number): number[] =>
+  getQuestPrePost(gameId)
+    .pre.map((code) => getQuestIdByCode(code))
+    .filter(Boolean) as number[]
+
+export const getPostQuestIds = (gameId: number): number[] =>
+  getQuestPrePost(gameId)
+    .post.map((code) => getQuestIdByCode(code))
+    .filter(Boolean) as number[]
+
+const calcQuestMap = (
+  inProgressQuests: number[],
+  next: (gameId: number) => number[]
+) => {
+  const map: Record<number, true> = {}
+  const queue: number[] = inProgressQuests.flatMap(next)
+  while (queue.length) {
+    const gameId = queue.shift()!
+    if (gameId in map) {
+      continue
+    }
+    map[gameId] = true
+
+    next(gameId).forEach((nextGameId) => {
+      queue.push(nextGameId)
+    })
+  }
+  return map
+}
+
+export const getCompletedQuest = moize((inProgressQuest: number[]) => {
+  const completedQuest = calcQuestMap(inProgressQuest, getPreQuestIds)
+  return completedQuest
+})
+
+export const getLockedQuest = moize((inProgressQuest: number[]) => {
+  const lockedQuest = calcQuestMap(inProgressQuest, getPostQuestIds)
+  return lockedQuest
+})
+
+export const questApiStateToQuestStatus = (
+  state: QUEST_API_STATE | undefined
+): QUEST_STATUS => {
+  switch (state) {
+    case QUEST_API_STATE.DEFAULT:
+      return QUEST_STATUS.DEFAULT
+    case QUEST_API_STATE.COMPLETED:
+      return QUEST_STATUS.COMPLETED
+    case QUEST_API_STATE.IN_PROGRESS:
+      return QUEST_STATUS.IN_PROGRESS
+    default:
+      return QUEST_STATUS.DEFAULT
+  }
+}
+
 export enum QUEST_STATUS {
   LOCKED,
   DEFAULT,
   IN_PROGRESS,
   COMPLETED,
   ALREADY_COMPLETED,
+  UNKNOWN,
 }

package/src/store/quest.ts

@@ -3,13 +3,15 @@ import { useGameQuest, usePluginTranslation } from '../poi/hooks'
 import {
   DocQuest,
   getCategory,
+  getCompletedQuest,
   getKcanotifyQuestData,
+  getLockedQuest,
   getQuestIdByCode,
-  getQuestPrePost,
+  questApiStateToQuestStatus,
   QUEST_STATUS,
+  UnionQuest,
 } from '../questHelper'
-import type { UnionQuest } from '../questHelper'
-import { useKcwikiData, checkIsKcwikiSupportedLanguages } from './kcwiki'
+import { checkIsKcwikiSupportedLanguages, useKcwikiData } from './kcwiki'
 import { useStore, useSyncWithGame } from './store'
 
 const DEFAULT_LANG = 'ja-JP'
@@ -98,63 +100,26 @@ export const useQuestByCode = (code: string): UnionQuest | null => {
   return null
 }
 
-const useCompletedQuest = () => {
-  const completedQuest: Record<number, true> = {}
-  const gameQuest = useGameQuest()
-  const queue: number[] = gameQuest.map((quest) => quest.api_no)
-  while (queue.length) {
-    const gameId = queue.shift()!
-    if (gameId in completedQuest) {
-      continue
-    }
-    completedQuest[gameId] = true
-
-    const prePostQuests = getQuestPrePost(gameId)
-    prePostQuests.pre.forEach((nextCode) => {
-      const nextGameId = getQuestIdByCode(nextCode)
-      if (nextGameId) {
-        queue.push(nextGameId)
-      }
-    })
-  }
-  return completedQuest
-}
-
-const useLockedQuest = () => {
-  const lockedQuest: Record<number, true> = {}
-  const gameQuest = useGameQuest()
-  const queue: number[] = gameQuest.map((quest) => quest.api_no)
-  while (queue.length) {
-    const gameId = queue.shift()!
-    if (gameId in lockedQuest) {
-      continue
-    }
-    lockedQuest[gameId] = true
-    const prePostQuests = getQuestPrePost(gameId)
-    prePostQuests.post.forEach((nextCode) => {
-      const nextGameId = getQuestIdByCode(nextCode)
-      if (nextGameId) {
-        queue.push(nextGameId)
-      }
-    })
-  }
-  return lockedQuest
-}
-
 export const useQuestStatus = (gameId: number | null) => {
-  const completedQuest = useCompletedQuest()
-  const lockedQuest = useLockedQuest()
+  const gameQuest = useGameQuest()
+  const gameQuestId = gameQuest.map((quest) => quest.api_no)
+  const completedQuest = getCompletedQuest(gameQuestId)
+  const lockedQuest = getLockedQuest(gameQuestId)
 
   if (!gameId) {
-    return QUEST_STATUS.DEFAULT
+    return QUEST_STATUS.UNKNOWN
   }
-  if (gameId in completedQuest) {
-    return QUEST_STATUS.ALREADY_COMPLETED
+  const theGameQuest = gameQuest.find((quest) => quest.api_no === gameId)
+  if (theGameQuest) {
+    return questApiStateToQuestStatus(theGameQuest.api_state)
   }
   if (gameId in lockedQuest) {
     return QUEST_STATUS.LOCKED
   }
-  return QUEST_STATUS.DEFAULT
+  if (gameId in completedQuest) {
+    return QUEST_STATUS.ALREADY_COMPLETED
+  }
+  return QUEST_STATUS.UNKNOWN
 }
 
 /**

package/tsconfig.json

@@ -44,7 +44,7 @@
     // "noImplicitReturns": true,             /* Report error when not all code paths in function return a value. */
     // "noFallthroughCasesInSwitch": true,    /* Report errors for fallthrough cases in switch statement. */
     // "noUncheckedIndexedAccess": true,      /* Include 'undefined' in index signature results */
-    "importsNotUsedAsValues": "error",
+    // "importsNotUsedAsValues": "error",
 
     /* Module Resolution Options */
     // "moduleResolution": "node",            /* Specify module resolution strategy: 'node' (Node.js) or 'classic' (TypeScript pre-1.6). */
@@ -72,5 +72,6 @@
     /* Advanced Options */
     "skipLibCheck": true /* Skip type checking of declaration files. */,
     "forceConsistentCasingInFileNames": true /* Disallow inconsistently-cased references to the same file. */
-  }
+  },
+  "exclude": ["./node_modules/**/*"]
 }