@skillsgate/tui 0.1.10 → 0.1.13

package/src/views/server-skills.tsx

@@ -1,9 +1,15 @@
 import { useState, useEffect } from "react"
+import crypto from "node:crypto"
+import { spawnSync } from "node:child_process"
+import fs from "node:fs"
+import os from "node:os"
+import path from "node:path"
 import { useKeyboard } from "@opentui/react"
 import { useStore, useDispatch } from "../store/context.js"
 import { useDb } from "../db/context.js"
 import { colors } from "../utils/colors.js"
 import type { RemoteSkill } from "../db/skills.js"
+import { readRemoteFile, writeRemoteFile } from "../db/ssh.js"
 
 interface ServerSkillsViewProps {
   serverId: string
@@ -61,6 +67,57 @@ export function ServerSkillsView({ serverId }: ServerSkillsViewProps) {
       return
     }
 
+    if (key.name === "e" && selectedSkill && server) {
+      const editor = process.env.VISUAL || process.env.EDITOR || "vi"
+      const tempPath = path.join(
+        os.tmpdir(),
+        `skillsgate-remote-${selectedSkill.id}.md`,
+      )
+      const initialContent = selectedSkill.content ?? ""
+      try {
+        fs.writeFileSync(tempPath, initialContent, "utf-8")
+        spawnSync(editor, [tempPath], { stdio: "inherit" })
+        const nextContent = fs.readFileSync(tempPath, "utf-8")
+        if (nextContent !== initialContent) {
+          writeRemoteFile(server, selectedSkill.remotePath, nextContent)
+            .then(async () => {
+              const refreshed = await readRemoteFile(server, selectedSkill.remotePath)
+              const contentHash = crypto
+                .createHash("sha256")
+                .update(refreshed, "utf-8")
+                .digest("hex")
+              skills.updateContent(serverId, selectedSkill.remotePath, refreshed, contentHash)
+              setSkillList(skills.listByServer(serverId))
+              dispatch({
+                type: "SHOW_NOTIFICATION",
+                notification: { type: "success", message: `Saved "${selectedSkill.name}"` },
+              })
+            })
+            .catch((err) => {
+              dispatch({
+                type: "SHOW_NOTIFICATION",
+                notification: {
+                  type: "error",
+                  message: err instanceof Error ? err.message : String(err),
+                },
+              })
+            })
+        }
+      } catch (err) {
+        dispatch({
+          type: "SHOW_NOTIFICATION",
+          notification: {
+            type: "error",
+            message: err instanceof Error ? err.message : String(err),
+          },
+        })
+      } finally {
+        try {
+          fs.unlinkSync(tempPath)
+        } catch {}
+      }
+    }
+
     // Esc = go back (handled by layout)
   })
 
@@ -97,10 +154,10 @@ export function ServerSkillsView({ serverId }: ServerSkillsViewProps) {
         <box
           style={{
             width: "40%",
-            borderRight: true,
+            border: true,
             borderColor: colors.border,
             flexDirection: "column",
-          }}
+          } as any}
         >
           <box style={{ height: 1, paddingLeft: 1, backgroundColor: colors.bgAlt }}>
             <text fg={colors.textDim}>REMOTE SKILLS</text>
@@ -150,7 +207,7 @@ export function ServerSkillsView({ serverId }: ServerSkillsViewProps) {
 
           {/* Bottom hints */}
           <box style={{ height: 1, paddingLeft: 1, backgroundColor: colors.bgAlt }}>
-            <text fg={colors.textDim}>i=install locally  Esc=back</text>
+            <text fg={colors.textDim}>i=install locally  e=edit  Esc=back</text>
           </box>
         </box>
 
@@ -234,7 +291,7 @@ function RemoteSkillDetail({ skill }: RemoteSkillDetailProps) {
         </box>
 
         <text>{" "}</text>
-        <text fg={colors.textDim}>i=install locally  Esc=back to server list</text>
+        <text fg={colors.textDim}>i=install locally  e=edit  Esc=back to server list</text>
         <text fg={colors.border}>---</text>
 
         {/* Skill content */}

package/src/views/servers.tsx

@@ -237,10 +237,10 @@ export function ServersView({ onServerCountChange }: ServersViewProps) {
         <box
           style={{
             width: "50%",
-            borderRight: true,
+            border: true,
             borderColor: colors.border,
             flexDirection: "column",
-          }}
+          } as any}
         >
           {/* Header */}
           <box style={{ height: 1, paddingLeft: 1, backgroundColor: colors.bgAlt }}>

package/src/views/settings.tsx

@@ -34,12 +34,6 @@ const SETTING_DEFS: SettingDef[] = [
     options: ["dark", "light", "system"],
     defaultValue: "dark",
   },
-  {
-    key: "search.preferSemantic",
-    label: "Prefer semantic search",
-    type: "toggle",
-    defaultValue: true,
-  },
   {
     key: "telemetry.enabled",
     label: "Anonymous telemetry",

package/src/views/skill-detail.tsx

@@ -6,6 +6,7 @@ import { useKeyboard } from "@opentui/react"
 import { useStore, useDispatch } from "../store/context.js"
 import { useSkillActions } from "../data/use-skill-actions.js"
 import { ConfirmDialog } from "../components/confirm-dialog.js"
+import { fetchSkillContent } from "../data/api-client.js"
 import { colors, agentBadges as badgeMap } from "../utils/colors.js"
 import { agents } from "../../../cli/src/core/agents.js"
 
@@ -75,19 +76,15 @@ export function SkillDetailView() {
       return
     }
 
-    // Catalog skill: fetch content from API
-    const githubUrl = skill.metadata?.githubUrl as string | undefined
-    const urlPath = skill.metadata?.urlPath as string | undefined
-    if (githubUrl || urlPath) {
+    // Catalog skill: fetch SKILL.md content from GitHub
+    const source = skill.metadata?.source as string | undefined
+    const skillId = skill.metadata?.skillId as string | undefined
+    if (source && skillId) {
       setContentLoading(true)
-      const detailPath = urlPath
-        ? `/api/v1/skills/detail?path=${encodeURIComponent(urlPath)}`
-        : `/api/v1/skills/detail?path=${encodeURIComponent(skill.name)}`
-      fetch(`https://api.skillsgate.ai${detailPath}`)
-        .then(res => res.ok ? res.json() : null)
-        .then(data => {
-          if (data?.content) {
-            setContent(stripFrontmatter(data.content))
+      fetchSkillContent(source, skillId)
+        .then((md) => {
+          if (md) {
+            setContent(stripFrontmatter(md))
           } else {
             setContent(skill.description || "(No content available)")
           }
@@ -410,12 +407,12 @@ export function SkillDetailView() {
           width: "30%",
           flexDirection: "column",
           backgroundColor: colors.bgAlt,
-          borderLeft: true,
+          border: true,
           borderColor: colors.border,
           paddingLeft: 1,
           paddingRight: 1,
           paddingTop: 1,
-        }}
+        } as any}
       >
         {/* Skill name */}
         <text fg={colors.primary}>
@@ -424,8 +421,12 @@ export function SkillDetailView() {
         <text>{" "}</text>
 
         {/* Description */}
-        <text fg={colors.text}>{skill.description}</text>
-        <text>{" "}</text>
+        {skill.description ? (
+          <>
+            <text fg={colors.text}>{skill.description}</text>
+            <text>{" "}</text>
+          </>
+        ) : null}
 
         {/* Source */}
         <text fg={colors.textDim}>Source</text>