From 45813ebbc0e8573102b2f9a3e1499342bcce5d8e Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Thu, 24 Aug 2023 12:10:15 +0800
Subject: [PATCH 01/18] feat: display tree by data

+ show discovered as highlight
+ BREAKING: cannot hightlight lines
---
 app/src/components/FileNode.vue  |  21 ++++++
 app/src/components/TreeChart.vue | 113 ++++++++++++++++---------------
 2 files changed, 80 insertions(+), 54 deletions(-)
 create mode 100644 app/src/components/FileNode.vue

diff --git a/app/src/components/FileNode.vue b/app/src/components/FileNode.vue
new file mode 100644
index 0000000..7039748
--- /dev/null
+++ b/app/src/components/FileNode.vue
@@ -0,0 +1,21 @@
+<template>
+  <a href="#" :class="nodeClass()">{{ node.discovered ? node.name : '???' }}</a>
+  <ul :class="nodeClass()" v-if="node.children && node.children.length > 0">
+    <li :class="nodeClass()" v-for="child in node.children" :key="child.path">
+      <FileNode :node="child" />
+    </li>
+  </ul>
+</template>
+
+<script setup>
+const props = defineProps({
+  node: {
+    type: Object,
+    required: true,
+  },
+})
+
+function nodeClass() {
+  return props.node.discovered ? 'discovered' : ''
+}
+</script>
diff --git a/app/src/components/TreeChart.vue b/app/src/components/TreeChart.vue
index f5ab0e6..2e0f82c 100644
--- a/app/src/components/TreeChart.vue
+++ b/app/src/components/TreeChart.vue
@@ -1,56 +1,64 @@
-<script setup></script>
+<script setup>
+import { ref } from 'vue'
+import FileNode from './FileNode.vue'
+
+const graph = ref({
+  path: '/',
+  name: '/',
+  type: 'folder',
+  discovered: false,
+  children: [
+    {
+      path: '/home',
+      name: 'home',
+      type: 'folder',
+      discovered: false,
+      children: [
+        {
+          path: '/home/rudeus',
+          name: 'rudeus',
+          type: 'folder',
+          discovered: true,
+          children: [
+            {
+              path: '/home/rudeus/secret',
+              name: 'secret',
+              type: 'file',
+              discovered: true,
+            },
+            {
+              path: '/home/rudeus/box',
+              name: 'box',
+              type: 'folder',
+              discovered: true,
+              children: [
+                {
+                  path: '/home/rudeus/box/scroll',
+                  name: 'scroll',
+                  type: 'file',
+                  discovered: false,
+                },
+              ],
+            },
+          ],
+        },
+      ],
+    },
+    {
+      path: '/etc',
+      name: 'etc',
+      type: 'folder',
+      discovered: false,
+    },
+  ],
+})
+</script>
 
 <template>
   <div class="tree w-max h-max">
     <ul>
       <li>
-        <a href="#">Parent</a>
-        <ul>
-          <li>
-            <a href="#">Child</a>
-            <ul>
-              <li>
-                <a href="#">Grand Child</a>
-              </li>
-            </ul>
-          </li>
-          <li>
-            <a href="#">Child</a>
-            <ul>
-              <li><a href="#">Grand Child</a></li>
-              <li>
-                <a href="#">Grand Child</a>
-                <ul>
-                  <li>
-                    <a href="#">Great Grand Child</a>
-                  </li>
-                  <li>
-                    <a href="#">Great Grand Child</a>
-                  </li>
-                  <li>
-                    <a href="#">Great Grand Child</a>
-                    <ul>
-                      <li><a>Grand Child</a></li>
-                      <li>
-                        <a>Grand Child</a>
-                        <ul>
-                          <li>
-                            <a>Great Grand Child</a>
-                          </li>
-                          <li>
-                            <a>Great Grand Child</a>
-                          </li>
-                        </ul>
-                      </li>
-                      <li><a>Grand Child</a></li>
-                    </ul>
-                  </li>
-                </ul>
-              </li>
-              <li><a href="#">Grand Child</a></li>
-            </ul>
-          </li>
-        </ul>
+        <FileNode :node="graph" />
       </li>
     </ul>
   </div>
@@ -149,17 +157,14 @@
   -moz-transition: all 0.5s;
 }
 
-.tree li a:hover,
-.tree li a:hover + ul li a {
+a.discovered {
   background: #c8e4f8;
   color: #000;
   border: 1px solid #94a0b4;
 }
 
-.tree li a:hover + ul li::after,
-.tree li a:hover + ul li::before,
-.tree li a:hover + ul::before,
-.tree li a:hover + ul ul::before {
+ul.discovered,
+li.discovered {
   border-color: #94a0b4;
 }
 </style>

From 54592831ee235cc4b58b448fd56c6fb78a133346 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Thu, 24 Aug 2023 12:16:03 +0800
Subject: [PATCH 02/18] feat: accept graph sent from server

---
 app/src/components/TreeChart.vue | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/app/src/components/TreeChart.vue b/app/src/components/TreeChart.vue
index 2e0f82c..19c8041 100644
--- a/app/src/components/TreeChart.vue
+++ b/app/src/components/TreeChart.vue
@@ -1,6 +1,7 @@
 <script setup>
-import { ref } from 'vue'
+import { onMounted, ref } from 'vue'
 import FileNode from './FileNode.vue'
+import socket from '../utils/socket.js'
 
 const graph = ref({
   path: '/',
@@ -52,6 +53,13 @@ const graph = ref({
     },
   ],
 })
+
+onMounted(() => {
+  socket.on('graph', (event) => {
+    // TODO: update graph
+    console.log(event)
+  })
+})
 </script>
 
 <template>

From 6c70a6eaeac528e8741422e9ec0dd2c4bd3da923 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Thu, 24 Aug 2023 12:26:09 +0800
Subject: [PATCH 03/18] feat: add box shadow to pwd node

---
 app/src/components/FileNode.vue  | 12 ++++++++++--
 app/src/components/TreeChart.vue |  8 +++++++-
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/app/src/components/FileNode.vue b/app/src/components/FileNode.vue
index 7039748..ee4636e 100644
--- a/app/src/components/FileNode.vue
+++ b/app/src/components/FileNode.vue
@@ -2,7 +2,7 @@
   <a href="#" :class="nodeClass()">{{ node.discovered ? node.name : '???' }}</a>
   <ul :class="nodeClass()" v-if="node.children && node.children.length > 0">
     <li :class="nodeClass()" v-for="child in node.children" :key="child.path">
-      <FileNode :node="child" />
+      <FileNode :node="child" :pwd="pwd" />
     </li>
   </ul>
 </template>
@@ -13,9 +13,17 @@ const props = defineProps({
     type: Object,
     required: true,
   },
+  pwd: {
+    type: String,
+    required: true,
+    default: '',
+  },
 })
 
 function nodeClass() {
-  return props.node.discovered ? 'discovered' : ''
+  const classes = []
+  if (props.node.discovered) classes.push('discovered')
+  if (props.node.path === props.pwd) classes.push('pwd')
+  return classes
 }
 </script>
diff --git a/app/src/components/TreeChart.vue b/app/src/components/TreeChart.vue
index 19c8041..9c2769a 100644
--- a/app/src/components/TreeChart.vue
+++ b/app/src/components/TreeChart.vue
@@ -3,6 +3,8 @@ import { onMounted, ref } from 'vue'
 import FileNode from './FileNode.vue'
 import socket from '../utils/socket.js'
 
+const pwd = ref('/home/rudeus')
+
 const graph = ref({
   path: '/',
   name: '/',
@@ -66,7 +68,7 @@ onMounted(() => {
   <div class="tree w-max h-max">
     <ul>
       <li>
-        <FileNode :node="graph" />
+        <FileNode :node="graph" :pwd="pwd" />
       </li>
     </ul>
   </div>
@@ -175,4 +177,8 @@ ul.discovered,
 li.discovered {
   border-color: #94a0b4;
 }
+
+a.pwd {
+  box-shadow: 0 0 10px rgba(255, 255, 255, 0.5); /* 水平偏移、垂直偏移、模糊半徑、顏色 */
+}
 </style>

From c3683ddaeadbf8c361e12c6467ffbdecf74005a3 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Mon, 28 Aug 2023 20:01:08 +0800
Subject: [PATCH 04/18] feat: import file-graph package

* Update Dockerfile structure, copy quests, packages to sub folder
---
 docker-compose.yml                |  5 ++--
 packages/file-graph/fileGraph.js  |  4 +++
 packages/file-graph/index.js      |  3 +++
 packages/file-graph/package.json  |  3 ++-
 server/Dockerfile                 | 10 ++++---
 server/Dockerfile.dev             |  6 +++--
 server/package.json               |  1 +
 server/src/game/commandHandler.js | 45 ++++++++++++++++++++++++++-----
 server/src/utils/quest.js         |  2 +-
 9 files changed, 64 insertions(+), 15 deletions(-)
 create mode 100644 packages/file-graph/index.js

diff --git a/docker-compose.yml b/docker-compose.yml
index e5b2474..d9fc5e2 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -21,8 +21,9 @@ services:
       - '3000:3000'
     volumes:
       - /var/run/docker.sock:/var/run/docker.sock
-      - ./server/src:/app/src
-      - ./server/swagger.yaml:/app/swagger.yaml
+      - ./server/src:/app/server/src
+      - ./server/swagger.yaml:/app/server/swagger.yaml
+      - ./packages:/app/packages
       - ./quests:/app/quests
     environment:
       - MONGO_URL=mongodb://db:27017/odyssey-test
diff --git a/packages/file-graph/fileGraph.js b/packages/file-graph/fileGraph.js
index 611fc4c..7bfda7d 100644
--- a/packages/file-graph/fileGraph.js
+++ b/packages/file-graph/fileGraph.js
@@ -54,4 +54,8 @@ export default class FileGraph {
       // Implement other cases if needed
     }
   }
+
+  toJSON() {
+    return this.root.toJSON()
+  }
 }
diff --git a/packages/file-graph/index.js b/packages/file-graph/index.js
new file mode 100644
index 0000000..4b1fcda
--- /dev/null
+++ b/packages/file-graph/index.js
@@ -0,0 +1,3 @@
+export { default as File } from './file.js'
+export { default as FileNode } from './fileNode.js'
+export { default as FileGraph } from './fileGraph.js'
diff --git a/packages/file-graph/package.json b/packages/file-graph/package.json
index 69b2535..59d0b82 100644
--- a/packages/file-graph/package.json
+++ b/packages/file-graph/package.json
@@ -1,7 +1,8 @@
 {
-  "name": "file-graph",
+  "name": "@linux-odyssey/file-graph",
   "version": "1.0.0",
   "main": "index.js",
+  "exports": "./index.js",
   "license": "MIT",
   "type": "module",
   "scripts": {
diff --git a/server/Dockerfile b/server/Dockerfile
index 6ee9ddd..078876b 100644
--- a/server/Dockerfile
+++ b/server/Dockerfile
@@ -1,13 +1,17 @@
 FROM node
 
 WORKDIR /app
-COPY server/package.json yarn.lock ./
+COPY ./package.json yarn.lock ./
+COPY server/package.json ./server/package.json
+COPY packages/file-graph/package.json ./packages/file-graph/package.json
 RUN yarn install --frozen-lockfile --production
 
-COPY server/src ./src
-COPY server/swagger.yaml .
+COPY ./server ./server
+COPY packages/file-graph ./packages/file-graph
 COPY quests ./quests
 
+WORKDIR /app/server
+
 ENV NODE_ENV=production
 ENV HOST=0.0.0.0
 ENV PORT=3000
diff --git a/server/Dockerfile.dev b/server/Dockerfile.dev
index 86186c2..738d4ea 100644
--- a/server/Dockerfile.dev
+++ b/server/Dockerfile.dev
@@ -1,7 +1,9 @@
 FROM node
 
 WORKDIR /app
-COPY server/package.json yarn.lock ./
+COPY package.json yarn.lock ./
+COPY server/package.json ./server/package.json
+COPY packages/file-graph/package.json ./packages/file-graph/package.json
 RUN yarn install --frozen-lockfile
 
 ENV NODE_ENV=development
@@ -9,4 +11,4 @@ ENV HOST=0.0.0.0
 ENV PORT=3000
 EXPOSE 3000
 
-CMD [ "yarn", "dev" ]
\ No newline at end of file
+CMD [ "yarn", "server" ]
\ No newline at end of file
diff --git a/server/package.json b/server/package.json
index 0fe6b1f..d30d738 100644
--- a/server/package.json
+++ b/server/package.json
@@ -10,6 +10,7 @@
     "clean": "node src/cli/clean.js"
   },
   "dependencies": {
+    "@linux-odyssey/file-graph": "1.0.0",
     "commander": "^11.0.0",
     "dockerode": "^3.3.5",
     "express": "^4.18.2",
diff --git a/server/src/game/commandHandler.js b/server/src/game/commandHandler.js
index 35335f3..caa7487 100644
--- a/server/src/game/commandHandler.js
+++ b/server/src/game/commandHandler.js
@@ -1,7 +1,10 @@
 import minimist from 'minimist'
+import { FileGraph } from '@linux-odyssey/file-graph'
 import Quest from '../models/quest.js'
 import { pushToSession } from '../api/socket.js'
 
+console.log(typeof FileGraph)
+
 // 檢查 pattern 是否符合 input
 const checkMatch = (pattern, input) => {
   if (pattern.length === 0) return true
@@ -11,6 +14,40 @@ const checkMatch = (pattern, input) => {
   })
 }
 
+async function discoverHandler(session, commandInput, additionalData) {
+  const graph = new FileGraph(session.graph)
+  graph.discover(additionalData.discover)
+  console.log('graph:', graph.toString())
+  session.graph = graph.toJSON()
+  await session.save()
+}
+
+const pushGraph = (session, commandInput, additionalData) => {
+  pushToSession(session.id, 'graph', {
+    pwd: commandInput.pwd,
+    discover: additionalData.discover,
+  })
+}
+
+const eventListenr = {
+  add: [],
+  remove: [],
+  discover: [discoverHandler, pushGraph],
+  pwd: [],
+}
+
+function handleEvent(session, commandInput, additionalData) {
+  const events = Object.keys(additionalData)
+  events.forEach((event) => {
+    const handlers = eventListenr[event]
+    if (handlers) {
+      handlers.forEach((handler) =>
+        handler(session, commandInput, additionalData)
+      )
+    }
+  })
+}
+
 async function commandHandler(session, commandInput, additionalData) {
   const quest = await Quest.findById(session.quest)
   const stage = quest.stages.find((s) => s.id === session.progress)
@@ -21,12 +58,8 @@ async function commandHandler(session, commandInput, additionalData) {
 
   const argv = minimist(commandInput.command.split(' '))
 
-  if ('discover' in additionalData) {
-    pushToSession(session.id, 'graph', {
-      pwd: commandInput.pwd,
-      discover: additionalData.discover,
-    })
-  } else if (argv._[0] === 'cd') {
+  handleEvent(session, commandInput, additionalData)
+  if (argv._[0] === 'cd') {
     pushToSession(session.id, 'graph', { pwd: commandInput.pwd })
   }
 
diff --git a/server/src/utils/quest.js b/server/src/utils/quest.js
index e8a2276..315528f 100644
--- a/server/src/utils/quest.js
+++ b/server/src/utils/quest.js
@@ -4,7 +4,7 @@ import yaml from 'yaml'
 
 import Quest from '../models/quest.js'
 
-const questDirectory = path.join(process.cwd(), 'quests')
+const questDirectory = path.join(process.cwd(), '..', 'quests')
 
 export default async function loadAndUpdateQuests() {
   const questNames = await fs.readdir(questDirectory)

From 3b8264e39e2e7e88d590467accadd59d080834de Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 12:08:36 +0800
Subject: [PATCH 05/18] refactor: refactor commandHandler to class

---
 .../src/api/controllers/commandController.js  |   6 +-
 server/src/game/commandHandler.js             | 143 +++++++++---------
 2 files changed, 79 insertions(+), 70 deletions(-)

diff --git a/server/src/api/controllers/commandController.js b/server/src/api/controllers/commandController.js
index 5526308..acfdeb2 100644
--- a/server/src/api/controllers/commandController.js
+++ b/server/src/api/controllers/commandController.js
@@ -2,7 +2,7 @@
 import Command from '../../models/command.js'
 import Session from '../../models/session.js'
 import { verifySessionJWT } from '../../utils/auth.js'
-import commandHandler from '../../game/commandHandler.js'
+import CommandHandler from '../../game/commandHandler.js'
 
 export async function newCommand(req, res) {
   console.log('new command:', req.body)
@@ -48,7 +48,9 @@ export async function newCommand(req, res) {
 
   await c.save()
 
-  const response = await commandHandler(session, c, additionalData)
+  const commandHandler = new CommandHandler(session, c, additionalData)
+
+  const response = await commandHandler.run()
   console.log(response)
 
   res.status(201).json(response)
diff --git a/server/src/game/commandHandler.js b/server/src/game/commandHandler.js
index caa7487..39da0f2 100644
--- a/server/src/game/commandHandler.js
+++ b/server/src/game/commandHandler.js
@@ -3,8 +3,6 @@ import { FileGraph } from '@linux-odyssey/file-graph'
 import Quest from '../models/quest.js'
 import { pushToSession } from '../api/socket.js'
 
-console.log(typeof FileGraph)
-
 // 檢查 pattern 是否符合 input
 const checkMatch = (pattern, input) => {
   if (pattern.length === 0) return true
@@ -14,78 +12,87 @@ const checkMatch = (pattern, input) => {
   })
 }
 
-async function discoverHandler(session, commandInput, additionalData) {
-  const graph = new FileGraph(session.graph)
-  graph.discover(additionalData.discover)
-  console.log('graph:', graph.toString())
-  session.graph = graph.toJSON()
-  await session.save()
-}
-
-const pushGraph = (session, commandInput, additionalData) => {
-  pushToSession(session.id, 'graph', {
-    pwd: commandInput.pwd,
-    discover: additionalData.discover,
-  })
-}
-
-const eventListenr = {
-  add: [],
-  remove: [],
-  discover: [discoverHandler, pushGraph],
-  pwd: [],
-}
-
-function handleEvent(session, commandInput, additionalData) {
-  const events = Object.keys(additionalData)
-  events.forEach((event) => {
-    const handlers = eventListenr[event]
-    if (handlers) {
-      handlers.forEach((handler) =>
-        handler(session, commandInput, additionalData)
-      )
+export default class CommandHandler {
+  constructor(session, commandInput, additionalData) {
+    this.session = session
+    this.commandInput = commandInput
+    this.additionalData = additionalData
+    this.eventListenr = {
+      discover: [this.discoverHandler, this.pushGraph],
     }
-  })
-}
-
-async function commandHandler(session, commandInput, additionalData) {
-  const quest = await Quest.findById(session.quest)
-  const stage = quest.stages.find((s) => s.id === session.progress)
-  if (!stage) {
-    console.error('stage not found', session.progress)
-    return {}
   }
 
-  const argv = minimist(commandInput.command.split(' '))
-
-  handleEvent(session, commandInput, additionalData)
-  if (argv._[0] === 'cd') {
-    pushToSession(session.id, 'graph', { pwd: commandInput.pwd })
+  handleEvent() {
+    const events = Object.keys(this.additionalData)
+    events.forEach((event) => {
+      const handlers = this.eventListeners[event]
+      if (handlers) {
+        handlers.forEach((handler) => handler())
+      }
+    })
   }
 
-  const commandMatch = checkMatch(stage.condition.command, commandInput.command)
-  const outputMatch = checkMatch(stage.condition.output, commandInput.output)
-  const errorMatch = checkMatch(stage.condition.error, commandInput.output)
-
-  if (!commandMatch || !outputMatch || !errorMatch) return {}
-
-  session.completion.push(session.progress)
-  session.hints.push(...stage.hints)
-
-  session.progress = stage.next
-  await session.save()
-
-  if (stage.next === 'END') {
-    session.status = 'finished'
-    session.finishedAt = new Date()
-    await session.save()
+  async discoverHandler() {
+    const graph = new FileGraph(this.session.graph)
+    graph.discover(this.additionalData.discover)
+    console.log('graph:', graph.toString())
+    this.session.graph = graph.toJSON()
+    await this.session.save()
   }
 
-  return {
-    responses: stage.responses,
-    hints: stage.hints,
-    end: stage.next === 'END',
+  pushGraph() {
+    pushToSession(this.session.id, 'graph', {
+      pwd: this.commandInput.pwd,
+      discover: this.additionalData.discover,
+    })
+  }
+
+  async run() {
+    const quest = await Quest.findById(this.session.quest)
+    const stage = quest.stages.find((s) => s.id === this.session.progress)
+    if (!stage) {
+      console.error('stage not found', this.session.progress)
+      return {}
+    }
+
+    const argv = minimist(this.commandInput.command.split(' '))
+
+    this.handleEvent()
+    if (argv._[0] === 'cd') {
+      pushToSession(this.session.id, 'graph', { pwd: this.commandInput.pwd })
+    }
+
+    const commandMatch = checkMatch(
+      stage.condition.command,
+      this.commandInput.command
+    )
+    const outputMatch = checkMatch(
+      stage.condition.output,
+      this.commandInput.output
+    )
+    const errorMatch = checkMatch(
+      stage.condition.error,
+      this.commandInput.output
+    )
+
+    if (!commandMatch || !outputMatch || !errorMatch) return {}
+
+    this.session.completion.push(this.session.progress)
+    this.session.hints.push(...stage.hints)
+
+    this.session.progress = stage.next
+    await this.session.save()
+
+    if (stage.next === 'END') {
+      this.session.status = 'finished'
+      this.session.finishedAt = new Date()
+      await this.session.save()
+    }
+
+    return {
+      responses: stage.responses,
+      hints: stage.hints,
+      end: stage.next === 'END',
+    }
   }
 }
-
-export default commandHandler

From 728399938a6af3658f060638121e229920b200b7 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:01:13 +0800
Subject: [PATCH 06/18] feat: update contains and add test cases

---
 packages/file-graph/file.js      |  7 +++--
 packages/file-graph/file.test.js | 49 ++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+), 2 deletions(-)
 create mode 100644 packages/file-graph/file.test.js

diff --git a/packages/file-graph/file.js b/packages/file-graph/file.js
index 986e280..983cbeb 100644
--- a/packages/file-graph/file.js
+++ b/packages/file-graph/file.js
@@ -2,7 +2,7 @@ import { basename } from './utils.js'
 
 export default class File {
   constructor({ path, type, discovered }) {
-    this.path = path
+    this.path = path.trimEnd('/')
     this.name = basename(path)
     this.type = type
     this.discovered = discovered
@@ -13,6 +13,9 @@ export default class File {
   }
 
   contains(file) {
-    return this.isDirectory() && file.path.startsWith(this.path)
+    if (!this.isDirectory()) return false
+    if (this.path === file.path) return true
+    const parentPath = this.path.endsWith('/') ? this.path : `${this.path}/`
+    return file.path.startsWith(parentPath)
   }
 }
diff --git a/packages/file-graph/file.test.js b/packages/file-graph/file.test.js
new file mode 100644
index 0000000..175ac73
--- /dev/null
+++ b/packages/file-graph/file.test.js
@@ -0,0 +1,49 @@
+import File from './file.js'
+
+describe('File', () => {
+  let file
+  beforeEach(() => {
+    file = new File({ path: '/home/user', type: 'folder', discovered: true })
+  })
+
+  const filesToCompare = [
+    {
+      name: 'direct children',
+      path: '/home/user/file1.txt',
+      answer: true,
+    },
+    {
+      name: 'nested children',
+      path: '/home/user/folder1/file1.txt',
+      answer: true,
+    },
+    {
+      name: 'outer file',
+      path: '/etc/file1.txt',
+      answer: false,
+    },
+    {
+      name: 'fake children',
+      path: '/home/user2/file2.txt',
+      answer: false,
+    },
+    {
+      name: 'same',
+      path: '/home/user',
+      answer: true,
+    },
+    {
+      name: 'same with trailing',
+      path: '/home/user/',
+      answer: true,
+    },
+  ]
+
+  filesToCompare.forEach(({ name, path, answer }) => {
+    test(`${name}: ${path}`, () => {
+      const fileToCompare = new File({ path, type: 'file', discovered: true })
+      console.log(fileToCompare)
+      expect(file.contains(fileToCompare)).toBe(answer)
+    })
+  })
+})

From cfb23347f804131cb12fb6f8649fec0d9f730a8a Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:26:14 +0800
Subject: [PATCH 07/18] feat: add load nested fileNode

---
 packages/file-graph/fileNode.js      | 16 +++++++++----
 packages/file-graph/fileNode.test.js | 36 ++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+), 4 deletions(-)

diff --git a/packages/file-graph/fileNode.js b/packages/file-graph/fileNode.js
index 96fc60c..d7d4e01 100644
--- a/packages/file-graph/fileNode.js
+++ b/packages/file-graph/fileNode.js
@@ -9,7 +9,8 @@ import { basename, dirname } from './utils.js'
 export default class FileNode extends File {
   constructor(file) {
     super(file)
-    this.children = file.children || []
+    this.children = []
+    if (file.children) this.children = file.children.map((f) => new FileNode(f))
   }
 
   addChild(fileInput, { makeParents } = {}) {
@@ -76,6 +77,8 @@ export default class FileNode extends File {
     // Merge the children of the provided file node into this node
     // If a child already exists, merge the children of that child
     // If a child does not exist, add it to this node
+
+    // If this is the same node, merge the children
     if (fileNode.path === this.path) {
       fileNode.children.forEach((child) => {
         const existingChild = this.children.find((c) => c.path === child.path)
@@ -92,12 +95,17 @@ export default class FileNode extends File {
           fileNode.children.some((c) => c.path === child.path)
         )
       }
-    } else if (this.contains(fileNode)) {
+    }
+    // If this contains the parent node
+    else if (this.contains(fileNode)) {
       const childNode = this.children.find((child) => child.contains(fileNode))
+      // If a child node exists, merge the provided node into the child node
       if (childNode) {
         childNode.merge(fileNode)
-      } else {
-        this.addChild(fileNode)
+      }
+      // If a child node does not exist, add the provided node as a child
+      else {
+        this.addChild(fileNode, { makeParents: true })
       }
     } else {
       throw new Error(
diff --git a/packages/file-graph/fileNode.test.js b/packages/file-graph/fileNode.test.js
index b58e38f..4621baf 100644
--- a/packages/file-graph/fileNode.test.js
+++ b/packages/file-graph/fileNode.test.js
@@ -219,3 +219,39 @@ describe('FileNode', () => {
     removeChildTest(files, fileToRemove, expected)
   })
 })
+
+test('load nested fileNode', () => {
+  const data = {
+    path: '/',
+    type: 'folder',
+    children: [
+      {
+        path: '/a',
+        type: 'file',
+      },
+      {
+        path: '/b',
+        type: 'folder',
+        children: [
+          {
+            path: '/b/c',
+            type: 'folder',
+            children: [
+              {
+                path: '/b/c/d',
+                type: 'file',
+              },
+            ],
+          },
+        ],
+      },
+    ],
+  }
+
+  const node = new FileNode(data)
+
+  console.log(node.toString())
+  expect(node.children.length).toBe(2)
+  expect(node.children[1].children.length).toBe(1)
+  expect(node.children[1].children[0].children.length).toBe(1)
+})

From 264d70fe3f612cd8c32986f5e7632e220d197d70 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:27:47 +0800
Subject: [PATCH 08/18] feat: add directly discover  tests

---
 packages/file-graph/fileGraph.js      | 16 ++-----
 packages/file-graph/fileGraph.test.js | 62 ++++++++++++++++++++-------
 2 files changed, 50 insertions(+), 28 deletions(-)

diff --git a/packages/file-graph/fileGraph.js b/packages/file-graph/fileGraph.js
index 7bfda7d..a2a924e 100644
--- a/packages/file-graph/fileGraph.js
+++ b/packages/file-graph/fileGraph.js
@@ -1,17 +1,13 @@
 import FileNode from './fileNode.js'
 
-export default class FileGraph {
-  constructor(root) {
-    this.root = new FileNode(root)
-  }
-
+export default class FileGraph extends FileNode {
   add(files) {
     // Logic to add the file node to the graph
     // You can implement the logic to find the appropriate location in the file graph
     // If the file node already exists, throw an error
 
     files.forEach((file) => {
-      this.root.addChild(file, { makeParents: true })
+      this.addChild(file, { makeParents: true })
     })
   }
 
@@ -21,7 +17,7 @@ export default class FileGraph {
     // If the file node doesn't exist, throw an error
 
     files.forEach((file) => {
-      this.root.removeChild(file)
+      this.removeChild(file)
     })
   }
 
@@ -36,7 +32,7 @@ export default class FileGraph {
       newNode.addChild(file, { makeParents: true })
     })
 
-    this.root.merge(newNode)
+    this.merge(newNode)
   }
 
   handleEvent(event) {
@@ -54,8 +50,4 @@ export default class FileGraph {
       // Implement other cases if needed
     }
   }
-
-  toJSON() {
-    return this.root.toJSON()
-  }
 }
diff --git a/packages/file-graph/fileGraph.test.js b/packages/file-graph/fileGraph.test.js
index 7f6447a..cff996a 100644
--- a/packages/file-graph/fileGraph.test.js
+++ b/packages/file-graph/fileGraph.test.js
@@ -30,9 +30,9 @@ describe('FileGraph', () => {
 
     fileGraph.add(filesToAdd)
 
-    console.log(fileGraph.root.toString())
-    expect(fileGraph.root.children.length).toBe(1)
-    expect(fileGraph.root.children[0].children[0].children.length).toBe(2)
+    console.log(fileGraph.toString())
+    expect(fileGraph.children.length).toBe(1)
+    expect(fileGraph.children[0].children[0].children.length).toBe(2)
   })
 
   test('remove files from graph', () => {
@@ -76,14 +76,44 @@ describe('FileGraph', () => {
 
     fileGraph.remove(filesToRemove)
 
-    console.log(fileGraph.root.toString())
-    expect(fileGraph.root.children[0].children[0].children.length).toBe(1)
-    expect(fileGraph.root.children[0].children[0].children[0].name).toBe(
-      'folder1'
+    console.log(fileGraph.toString())
+    expect(fileGraph.children[0].children[0].children.length).toBe(1)
+    expect(fileGraph.children[0].children[0].children[0].name).toBe('folder1')
+    expect(fileGraph.children[0].children[0].children[0].children.length).toBe(
+      0
+    )
+  })
+
+  test('discover', () => {
+    const fileGraph = new FileGraph(root)
+    const discoverFiles = [
+      {
+        path: '/home/user/folder1',
+        type: 'folder',
+        discovered: true,
+      },
+      {
+        path: '/home/user/folder1/file1.txt',
+        type: 'file',
+        discovered: true,
+      },
+      {
+        path: '/home/user/folder1/file2.txt',
+        type: 'file',
+        discovered: true,
+      },
+      {
+        path: '/home/user/folder1/file3.txt',
+        type: 'file',
+        discovered: true,
+      },
+    ]
+
+    fileGraph.discover(discoverFiles)
+    console.log(fileGraph.toString())
+    expect(fileGraph.children[0].children[0].children[0].children.length).toBe(
+      3
     )
-    expect(
-      fileGraph.root.children[0].children[0].children[0].children.length
-    ).toBe(0)
   })
 
   test('discover new files', () => {
@@ -113,7 +143,7 @@ describe('FileGraph', () => {
     ]
 
     fileGraph.add(filesToAdd)
-    console.log(fileGraph.root.toString())
+    console.log(fileGraph.toString())
 
     const newFiles = [
       {
@@ -140,10 +170,10 @@ describe('FileGraph', () => {
 
     fileGraph.discover(newFiles)
 
-    console.log(fileGraph.root.toString())
-    expect(fileGraph.root.children[0].children[0].children.length).toBe(2)
-    expect(
-      fileGraph.root.children[0].children[0].children[0].children.length
-    ).toBe(3)
+    console.log(fileGraph.toString())
+    expect(fileGraph.children[0].children[0].children.length).toBe(2)
+    expect(fileGraph.children[0].children[0].children[0].children.length).toBe(
+      3
+    )
   })
 })

From ee7cd083bd487add71129caa44ad387bebe388af Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:28:18 +0800
Subject: [PATCH 09/18] feat: handle discover in server

---
 server/src/game/commandHandler.js | 50 ++++++++++++++++---------------
 server/src/models/session.js      | 15 ++++++++++
 2 files changed, 41 insertions(+), 24 deletions(-)

diff --git a/server/src/game/commandHandler.js b/server/src/game/commandHandler.js
index 39da0f2..62eb626 100644
--- a/server/src/game/commandHandler.js
+++ b/server/src/game/commandHandler.js
@@ -16,37 +16,43 @@ export default class CommandHandler {
   constructor(session, commandInput, additionalData) {
     this.session = session
     this.commandInput = commandInput
+    this.argv = minimist(this.commandInput.command.split(' '))
+
     this.additionalData = additionalData
-    this.eventListenr = {
-      discover: [this.discoverHandler, this.pushGraph],
-    }
   }
 
   handleEvent() {
-    const events = Object.keys(this.additionalData)
-    events.forEach((event) => {
-      const handlers = this.eventListeners[event]
-      if (handlers) {
-        handlers.forEach((handler) => handler())
-      }
-    })
+    if (this.additionalData.discover) this.discoverHandler()
+  }
+
+  handleCommand() {
+    const command = this.argv._[0]
+    switch (command) {
+      case 'cd':
+        pushToSession(this.session.id, 'graph', {
+          pwd: this.commandInput.pwd,
+        })
+        break
+
+      case 'ls':
+        pushToSession(this.session.id, 'graph', {
+          pwd: this.commandInput.pwd,
+          discover: this.additionalData.discover,
+        })
+        break
+
+      default:
+        break
+    }
   }
 
   async discoverHandler() {
     const graph = new FileGraph(this.session.graph)
     graph.discover(this.additionalData.discover)
-    console.log('graph:', graph.toString())
-    this.session.graph = graph.toJSON()
+    this.session.graph = graph
     await this.session.save()
   }
 
-  pushGraph() {
-    pushToSession(this.session.id, 'graph', {
-      pwd: this.commandInput.pwd,
-      discover: this.additionalData.discover,
-    })
-  }
-
   async run() {
     const quest = await Quest.findById(this.session.quest)
     const stage = quest.stages.find((s) => s.id === this.session.progress)
@@ -55,12 +61,8 @@ export default class CommandHandler {
       return {}
     }
 
-    const argv = minimist(this.commandInput.command.split(' '))
-
     this.handleEvent()
-    if (argv._[0] === 'cd') {
-      pushToSession(this.session.id, 'graph', { pwd: this.commandInput.pwd })
-    }
+    this.handleCommand()
 
     const commandMatch = checkMatch(
       stage.condition.command,
diff --git a/server/src/models/session.js b/server/src/models/session.js
index 1ae266c..d56880d 100644
--- a/server/src/models/session.js
+++ b/server/src/models/session.js
@@ -1,4 +1,15 @@
 import { model, Schema } from 'mongoose'
+import { FileNode } from '@linux-odyssey/file-graph'
+
+const nodeSchema = new Schema({
+  path: String,
+  type: String,
+  discovered: Boolean,
+})
+
+nodeSchema.add({
+  children: [nodeSchema],
+})
 
 const Session = model(
   'Session',
@@ -28,6 +39,10 @@ const Session = model(
       progress: String,
       completion: [String],
       hints: [String],
+      graph: {
+        type: nodeSchema,
+        default: new FileNode({ path: '/', name: '/', type: 'folder' }),
+      },
     },
     { timestamps: true }
   )

From 638dff4be23542b477177a09a079b41ae371dd9d Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:28:36 +0800
Subject: [PATCH 10/18] refactor: move test scripts to project root

---
 package.json                     | 1 +
 packages/file-graph/package.json | 4 +---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/package.json b/package.json
index 3713418..8142c9d 100644
--- a/package.json
+++ b/package.json
@@ -17,6 +17,7 @@
     "cli": "yarn workspace client start",
     "lint": "eslint --ext .js,.vue .",
     "format": "prettier --ignore-path .prettierignore --write app server",
+    "test": "yarn node --experimental-vm-modules $(yarn bin jest)",
     "clean": "yarn workspace server clean"
   },
   "devDependencies": {
diff --git a/packages/file-graph/package.json b/packages/file-graph/package.json
index 59d0b82..2388c78 100644
--- a/packages/file-graph/package.json
+++ b/packages/file-graph/package.json
@@ -5,7 +5,5 @@
   "exports": "./index.js",
   "license": "MIT",
   "type": "module",
-  "scripts": {
-    "test": "yarn node --experimental-vm-modules $(yarn bin jest)"
-  }
+  "scripts": {}
 }

From 0bdcce5d21f367005a8e2dfba69cd9f4b792b6b4 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:51:55 +0800
Subject: [PATCH 11/18] feat: import file-graph package to app

* update docker images
---
 app/Dockerfile     | 12 ++++++++----
 app/Dockerfile.dev |  7 +++++--
 app/package.json   |  1 +
 docker-compose.yml |  3 ++-
 package.json       |  1 +
 5 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/app/Dockerfile b/app/Dockerfile
index dd77845..30a17f7 100644
--- a/app/Dockerfile
+++ b/app/Dockerfile
@@ -1,14 +1,18 @@
 # Build stage
 FROM node as build-stage
 WORKDIR /app
-COPY app/package.json yarn.lock ./
-RUN yarn install
-COPY app .
+COPY package.json yarn.lock ./
+COPY app/package.json ./app/package.json
+COPY packages/file-graph/package.json ./packages/file-graph/package.json
+RUN yarn install --frozen-lockfile
+
+COPY app ./app
+COPY packages/file-graph ./packages/file-graph
 RUN yarn build
 
 # Production stage
 FROM nginx:stable-alpine as production-stage
-COPY --from=build-stage /app/dist /usr/share/nginx/html
+COPY --from=build-stage /app/app/dist /usr/share/nginx/html
 COPY deployment/nginx.conf /etc/nginx/conf.d/default.conf
 EXPOSE 80
 CMD ["nginx", "-g", "daemon off;"]
diff --git a/app/Dockerfile.dev b/app/Dockerfile.dev
index 61d9568..a58a1cc 100644
--- a/app/Dockerfile.dev
+++ b/app/Dockerfile.dev
@@ -1,7 +1,10 @@
 FROM node
 
 WORKDIR /app
-COPY app/package.json yarn.lock ./
+COPY package.json yarn.lock ./
+COPY app/package.json ./app/package.json
+COPY packages/file-graph/package.json ./packages/file-graph/package.json
 RUN yarn install --frozen-lockfile
-COPY app .
+
+WORKDIR /app/app
 CMD [ "yarn", "dev:docker", "--host", "0.0.0.0" ]
\ No newline at end of file
diff --git a/app/package.json b/app/package.json
index d38cd0b..1f168c2 100644
--- a/app/package.json
+++ b/app/package.json
@@ -11,6 +11,7 @@
     "preview": "vite preview"
   },
   "dependencies": {
+    "@linux-odyssey/file-graph": "*",
     "@fortawesome/fontawesome-svg-core": "^6.4.0",
     "@fortawesome/free-brands-svg-icons": "^6.4.0",
     "@fortawesome/free-regular-svg-icons": "^6.4.0",
diff --git a/docker-compose.yml b/docker-compose.yml
index d9fc5e2..3492bab 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -9,7 +9,8 @@ services:
     networks:
       - internal
     volumes:
-      - ./app/src:/app/src
+      - ./app:/app/app
+      - ./packages/file-graph:/app/packages/file-graph
   backend:
     build:
       context: .
diff --git a/package.json b/package.json
index 8142c9d..cc4fd76 100644
--- a/package.json
+++ b/package.json
@@ -13,6 +13,7 @@
   "scripts": {
     "app": "yarn workspace app dev",
     "app:local": "yarn workspace app dev:local",
+    "build": "yarn workspace app build",
     "server": "yarn workspace server dev",
     "cli": "yarn workspace client start",
     "lint": "eslint --ext .js,.vue .",

From 1c017c98c796a4190474dc8befdf19225177d940 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 13:52:24 +0800
Subject: [PATCH 12/18] feat: receive discover event and display

---
 app/src/components/TreeChart.vue | 62 +++++++-------------------------
 1 file changed, 12 insertions(+), 50 deletions(-)

diff --git a/app/src/components/TreeChart.vue b/app/src/components/TreeChart.vue
index 9c2769a..72e2d96 100644
--- a/app/src/components/TreeChart.vue
+++ b/app/src/components/TreeChart.vue
@@ -1,65 +1,27 @@
 <script setup>
 import { onMounted, ref } from 'vue'
+import { FileGraph } from '@linux-odyssey/file-graph'
 import FileNode from './FileNode.vue'
 import socket from '../utils/socket.js'
 
 const pwd = ref('/home/rudeus')
 
-const graph = ref({
-  path: '/',
-  name: '/',
-  type: 'folder',
-  discovered: false,
-  children: [
-    {
-      path: '/home',
-      name: 'home',
-      type: 'folder',
-      discovered: false,
-      children: [
-        {
-          path: '/home/rudeus',
-          name: 'rudeus',
-          type: 'folder',
-          discovered: true,
-          children: [
-            {
-              path: '/home/rudeus/secret',
-              name: 'secret',
-              type: 'file',
-              discovered: true,
-            },
-            {
-              path: '/home/rudeus/box',
-              name: 'box',
-              type: 'folder',
-              discovered: true,
-              children: [
-                {
-                  path: '/home/rudeus/box/scroll',
-                  name: 'scroll',
-                  type: 'file',
-                  discovered: false,
-                },
-              ],
-            },
-          ],
-        },
-      ],
-    },
-    {
-      path: '/etc',
-      name: 'etc',
-      type: 'folder',
-      discovered: false,
-    },
-  ],
-})
+const graph = ref(
+  new FileGraph({
+    path: '/',
+    name: '/',
+    type: 'folder',
+    discovered: false,
+  })
+)
 
 onMounted(() => {
   socket.on('graph', (event) => {
     // TODO: update graph
     console.log(event)
+    if (event.discover) {
+      graph.value.discover(event.discover)
+    }
   })
 })
 </script>

From 3dae9e85522cb9576eb86c68fca7acc944b096f8 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 14:28:09 +0800
Subject: [PATCH 13/18] feat: store graph in session manager

---
 app/src/App.vue                  |  3 +++
 app/src/components/TreeChart.vue | 26 +++-----------------------
 app/src/utils/session.js         | 19 ++++++++++++++++++-
 server/src/models/session.js     | 10 ++++++++--
 4 files changed, 32 insertions(+), 26 deletions(-)

diff --git a/app/src/App.vue b/app/src/App.vue
index e6e3717..3c94c4f 100644
--- a/app/src/App.vue
+++ b/app/src/App.vue
@@ -20,6 +20,9 @@ onMounted(async () => {
     console.log('Disconnected from the server.')
   })
   socket.on('message', console.log)
+  socket.on('graph', (event) => {
+    sessionManager.handleGraphUpdate(event)
+  })
 })
 </script>
 
diff --git a/app/src/components/TreeChart.vue b/app/src/components/TreeChart.vue
index 72e2d96..207c33c 100644
--- a/app/src/components/TreeChart.vue
+++ b/app/src/components/TreeChart.vue
@@ -1,36 +1,16 @@
 <script setup>
-import { onMounted, ref } from 'vue'
-import { FileGraph } from '@linux-odyssey/file-graph'
+import { ref } from 'vue'
 import FileNode from './FileNode.vue'
-import socket from '../utils/socket.js'
+import sessionManager from '../utils/session.js'
 
 const pwd = ref('/home/rudeus')
-
-const graph = ref(
-  new FileGraph({
-    path: '/',
-    name: '/',
-    type: 'folder',
-    discovered: false,
-  })
-)
-
-onMounted(() => {
-  socket.on('graph', (event) => {
-    // TODO: update graph
-    console.log(event)
-    if (event.discover) {
-      graph.value.discover(event.discover)
-    }
-  })
-})
 </script>
 
 <template>
   <div class="tree w-max h-max">
     <ul>
       <li>
-        <FileNode :node="graph" :pwd="pwd" />
+        <FileNode :node="sessionManager.graph.value" :pwd="pwd" />
       </li>
     </ul>
   </div>
diff --git a/app/src/utils/session.js b/app/src/utils/session.js
index 80ab25c..a059833 100644
--- a/app/src/utils/session.js
+++ b/app/src/utils/session.js
@@ -1,10 +1,18 @@
 import { ref } from 'vue'
+import { FileGraph } from '@linux-odyssey/file-graph'
 import api from './api'
 
 class SessionManager {
   constructor(questId) {
     this.questId = questId
     this.session = ref(null)
+    this.graph = ref(
+      new FileGraph({
+        path: '/',
+        type: 'folder',
+        discovered: false,
+      })
+    )
   }
 
   getSession() {
@@ -14,6 +22,13 @@ class SessionManager {
   setSession(session) {
     console.log('Update session:', session)
     this.session.value = session
+    this.graph.value = new FileGraph(session.graph)
+  }
+
+  handleGraphUpdate(event) {
+    if (event.discover) {
+      this.graph.value.discover(event.discover)
+    }
   }
 
   async createSession() {
@@ -37,7 +52,8 @@ class SessionManager {
     if (sessions.length === 0) {
       return
     }
-    const session = sessions[sessions.length - 1]
+    const { _id: sessionId } = sessions[sessions.length - 1]
+    const { data: session } = await api.get(`/sessions/${sessionId}`)
     console.log(
       `Last Session ID: ${session._id}, Quest: ${session.quest}, Created At: ${session.createdAt}`
     )
@@ -49,6 +65,7 @@ class SessionManager {
     if (!this.session.value) {
       await this.createSession()
     }
+    console.log(this.session.value)
   }
 }
 
diff --git a/server/src/models/session.js b/server/src/models/session.js
index d56880d..27d7efa 100644
--- a/server/src/models/session.js
+++ b/server/src/models/session.js
@@ -4,7 +4,10 @@ import { FileNode } from '@linux-odyssey/file-graph'
 const nodeSchema = new Schema({
   path: String,
   type: String,
-  discovered: Boolean,
+  discovered: {
+    type: Boolean,
+    default: false,
+  },
 })
 
 nodeSchema.add({
@@ -41,7 +44,10 @@ const Session = model(
       hints: [String],
       graph: {
         type: nodeSchema,
-        default: new FileNode({ path: '/', name: '/', type: 'folder' }),
+        default: new FileNode({
+          path: '/',
+          type: 'folder',
+        }),
       },
     },
     { timestamps: true }

From 2bc94a33e2452dcd2c3f94488eb8d52fb0324413 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 15:56:23 +0800
Subject: [PATCH 14/18] fix: remove unmaintained test

---
 server/tests/api/sessionController.test.js | 56 ----------------------
 1 file changed, 56 deletions(-)
 delete mode 100644 server/tests/api/sessionController.test.js

diff --git a/server/tests/api/sessionController.test.js b/server/tests/api/sessionController.test.js
deleted file mode 100644
index 991647b..0000000
--- a/server/tests/api/sessionController.test.js
+++ /dev/null
@@ -1,56 +0,0 @@
-import jest from 'jest'
-import * as sessionController from '../../src/api/controllers/sessionController'
-import Session from '../../src/models/session'
-
-jest.mock('../../src/models/session')
-
-describe('Session Controller', () => {
-  let mockReq
-  let mockRes
-
-  beforeEach(() => {
-    mockReq = {
-      user: { id: 'userid' },
-      body: {},
-      params: {},
-    }
-    mockRes = {
-      status: jest.fn().mockReturnThis(),
-      json: jest.fn().mockReturnThis(),
-      end: jest.fn().mockReturnThis(),
-    }
-  })
-
-  test('getSessionList returns a list of sessions', async () => {
-    Session.find.mockResolvedValue(['session1', 'session2'])
-    await sessionController.getSessionList(mockReq, mockRes)
-    expect(mockRes.json).toHaveBeenCalledWith(['session1', 'session2'])
-  })
-
-  test('createSession creates a new session', async () => {
-    mockReq.body.exercise_id = 'exerciseid'
-    const mockSession = { user: 'userid', exercise: 'exerciseid' }
-    Session.mockReturnValue(mockSession)
-    mockSession.save = jest.fn().mockResolvedValue('newsess')
-    await sessionController.createSession(mockReq, mockRes)
-    expect(mockRes.status).toHaveBeenCalledWith(201)
-    expect(mockRes.json).toHaveBeenCalledWith('newsess')
-  })
-
-  test('getSessionById returns the correct session', async () => {
-    mockReq.params.session_id = 'sessionid'
-    const mockSession = { user: 'userid', _id: 'sessionid' }
-    Session.findOne.mockResolvedValue(mockSession)
-    await sessionController.getSessionById(mockReq, mockRes)
-    expect(mockRes.json).toHaveBeenCalledWith(mockSession)
-  })
-
-  test('deleteSessionById deletes a session', async () => {
-    mockReq.params.session_id = 'sessionid'
-    const mockSession = { user: 'userid', _id: 'sessionid' }
-    Session.findOneAndDelete.mockResolvedValue(mockSession)
-    await sessionController.deleteSessionById(mockReq, mockRes)
-    expect(mockRes.status).toHaveBeenCalledWith(204)
-    expect(mockRes.end).toHaveBeenCalled()
-  })
-})

From f360d783f8b7ddb9f87188abdf4bc807dda34569 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 16:13:59 +0800
Subject: [PATCH 15/18] feat: combine ci/cd workflow and add test step

---
 .github/workflows/ci_cd.yml  | 30 +++++++++++++++++++++++++-----
 .github/workflows/eslint.yml | 22 ----------------------
 2 files changed, 25 insertions(+), 27 deletions(-)
 delete mode 100644 .github/workflows/eslint.yml

diff --git a/.github/workflows/ci_cd.yml b/.github/workflows/ci_cd.yml
index c240cd8..080e10e 100644
--- a/.github/workflows/ci_cd.yml
+++ b/.github/workflows/ci_cd.yml
@@ -1,12 +1,32 @@
-name: CI/CD Workflow
+name: CI/CD
 
 on:
   push:
-    branches:
-      - main
 
 jobs:
-  build-and-push:
+  ci:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 20
+
+      - name: Install dependencies
+        run: yarn install
+
+      - name: Lint
+        run: yarn lint
+
+      - name: Test
+        run: yarn test
+
+  build:
+    needs: ci
+    if: github.ref == 'refs/heads/main'
     runs-on: ubuntu-latest
 
     steps:
@@ -44,7 +64,7 @@ jobs:
           tags: docker.io/${{ secrets.DOCKERHUB_USERNAME }}/linux-odyssey:helloworld
 
   deploy:
-    needs: build-and-push
+    needs: build
     runs-on: ubuntu-latest
 
     steps:
diff --git a/.github/workflows/eslint.yml b/.github/workflows/eslint.yml
deleted file mode 100644
index 01f475e..0000000
--- a/.github/workflows/eslint.yml
+++ /dev/null
@@ -1,22 +0,0 @@
-name: ESLint Checks
-
-on:
-  push:
-
-jobs:
-  lint:
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-
-      - uses: actions/setup-node@v3
-        with:
-          node-version: 20
-
-      - name: Install dependencies
-        run: yarn install
-
-      - name: Lint
-        run: yarn lint

From 6b94b1ae81cbf504380332e0b9cb822b6edc38df Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 16:28:29 +0800
Subject: [PATCH 16/18] fix: merge discovery state (with test)

* update fileNode.toString to add discovery state
---
 packages/file-graph/fileNode.js      | 16 ++++++++++-----
 packages/file-graph/fileNode.test.js | 29 ++++++++++++++++++++++++++++
 2 files changed, 40 insertions(+), 5 deletions(-)

diff --git a/packages/file-graph/fileNode.js b/packages/file-graph/fileNode.js
index d7d4e01..cf8f089 100644
--- a/packages/file-graph/fileNode.js
+++ b/packages/file-graph/fileNode.js
@@ -80,6 +80,9 @@ export default class FileNode extends File {
 
     // If this is the same node, merge the children
     if (fileNode.path === this.path) {
+      // Update the state of this node
+      this.discovered = this.discovered || fileNode.discovered
+
       fileNode.children.forEach((child) => {
         const existingChild = this.children.find((c) => c.path === child.path)
         if (existingChild) {
@@ -116,12 +119,15 @@ export default class FileNode extends File {
     }
   }
 
-  toString() {
-    const indent = '    '.repeat(this.path.split('/').length - 1)
-    const suffix = this.isDirectory() ? '/' : ''
+  toString(level = 0) {
+    let { name } = this
+    const indent = '    '.repeat(level)
+    name = `${indent}${name}`
+    if (this.isDirectory()) name += '/'
+    if (!this.discovered) name += `*`
     const childrenString = this.children
-      .map((child) => child.toString())
+      .map((child) => child.toString(level + 1))
       .join('')
-    return `${indent}${this.name}${suffix}\n${childrenString}`
+    return `${name}\n${childrenString}`
   }
 }
diff --git a/packages/file-graph/fileNode.test.js b/packages/file-graph/fileNode.test.js
index 4621baf..3413dc5 100644
--- a/packages/file-graph/fileNode.test.js
+++ b/packages/file-graph/fileNode.test.js
@@ -193,6 +193,35 @@ describe('FileNode', () => {
     mergeTest(files1, files2, expected)
   })
 
+  test('merge-hidden', () => {
+    const root = {
+      path: '/',
+      type: 'folder',
+    }
+    const files1 = [
+      root,
+      { path: '/a', type: 'file', discovered: true },
+      { path: '/.b', type: 'file', discovered: false },
+      { path: '/c', type: 'folder', discovered: true },
+      { path: '/c/.d', type: 'file', discovered: false },
+    ]
+
+    const files2 = [
+      { path: '/c', type: 'folder', discovered: true },
+      { path: '/c/.d', type: 'file', discovered: true },
+    ]
+
+    const expected = [
+      root,
+      { path: '/a', type: 'file', discovered: true },
+      { path: '/.b', type: 'file', discovered: false },
+      { path: '/c', type: 'folder', discovered: true },
+      { path: '/c/.d', type: 'file', discovered: true },
+    ]
+
+    mergeTest(files1, files2, expected)
+  })
+
   test('remove-child', () => {
     const root = {
       path: '/',

From 65bc838c8304426861e7de7ab539de77b1081bfa Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 16:33:02 +0800
Subject: [PATCH 17/18] feat: update pwd

---
 app/src/components/TreeChart.vue | 8 ++++----
 app/src/utils/session.js         | 4 ++++
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/app/src/components/TreeChart.vue b/app/src/components/TreeChart.vue
index 207c33c..93c77b3 100644
--- a/app/src/components/TreeChart.vue
+++ b/app/src/components/TreeChart.vue
@@ -1,16 +1,16 @@
 <script setup>
-import { ref } from 'vue'
 import FileNode from './FileNode.vue'
 import sessionManager from '../utils/session.js'
-
-const pwd = ref('/home/rudeus')
 </script>
 
 <template>
   <div class="tree w-max h-max">
     <ul>
       <li>
-        <FileNode :node="sessionManager.graph.value" :pwd="pwd" />
+        <FileNode
+          :node="sessionManager.graph.value"
+          :pwd="sessionManager.pwd.value"
+        />
       </li>
     </ul>
   </div>
diff --git a/app/src/utils/session.js b/app/src/utils/session.js
index a059833..b731d43 100644
--- a/app/src/utils/session.js
+++ b/app/src/utils/session.js
@@ -13,6 +13,7 @@ class SessionManager {
         discovered: false,
       })
     )
+    this.pwd = ref('')
   }
 
   getSession() {
@@ -29,6 +30,9 @@ class SessionManager {
     if (event.discover) {
       this.graph.value.discover(event.discover)
     }
+    if (event.pwd) {
+      this.pwd.value = event.pwd
+    }
   }
 
   async createSession() {

From 1582d8c62ab211b9a41ebc6c45fe8d07a6ba8d05 Mon Sep 17 00:00:00 2001
From: wancat <wancat@wancat.cc>
Date: Tue, 29 Aug 2023 21:10:17 +0800
Subject: [PATCH 18/18] fix: change workflow name

---
 .github/workflows/ci_cd.yml | 4 ++--
 server/package.json         | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/ci_cd.yml b/.github/workflows/ci_cd.yml
index 080e10e..5440a2e 100644
--- a/.github/workflows/ci_cd.yml
+++ b/.github/workflows/ci_cd.yml
@@ -24,7 +24,7 @@ jobs:
       - name: Test
         run: yarn test
 
-  build:
+  build-and-push:
     needs: ci
     if: github.ref == 'refs/heads/main'
     runs-on: ubuntu-latest
@@ -64,7 +64,7 @@ jobs:
           tags: docker.io/${{ secrets.DOCKERHUB_USERNAME }}/linux-odyssey:helloworld
 
   deploy:
-    needs: build
+    needs: build-and-push
     runs-on: ubuntu-latest
 
     steps:
diff --git a/server/package.json b/server/package.json
index d30d738..678672b 100644
--- a/server/package.json
+++ b/server/package.json
@@ -10,7 +10,7 @@
     "clean": "node src/cli/clean.js"
   },
   "dependencies": {
-    "@linux-odyssey/file-graph": "1.0.0",
+    "@linux-odyssey/file-graph": "*",
     "commander": "^11.0.0",
     "dockerode": "^3.3.5",
     "express": "^4.18.2",