@kurokeita/add-skill 2.6.1 → 3.0.1

package/dist/skills/statusline-setup/mod/hooks/register.tsx

@@ -12,22 +12,21 @@ import {
 } from "./discover"
 import { fetchAzure, fetchGitHub, git, type Run } from "./fetch"
 
-const PANE = "work-items"
-const branches = atom({ plugin: "work-items", key: "branches" } as const, [])
-const work = atom({ plugin: "work-items", key: "work" } as const, null)
+const PANE = "work-pane"
+const branches = atom({ plugin: "work-pane", key: "branches" } as const, [])
+const work = atom({ plugin: "work-pane", key: "work" } as const, null)
 const isFetching = atom(
-	{ plugin: "work-items", key: "isFetching" } as const,
+	{ plugin: "work-pane", key: "isFetching" } as const,
 	false,
 )
-const summary = atom({ plugin: "work-items", key: "summary" } as const, null)
+const summary = atom({ plugin: "work-pane", key: "summary" } as const, null)
 const isSummarizing = atom(
-	{ plugin: "work-items", key: "isSummarizing" } as const,
+	{ plugin: "work-pane", key: "isSummarizing" } as const,
 	false,
 )
-const collapsed = atom({ plugin: "work-items", key: "collapsed" } as const, [])
+const collapsed = atom({ plugin: "work-pane", key: "collapsed" } as const, [])
 
 let isFetchRunning = false
-let isAutoOpenWanted = false
 let isAutoOpened = false
 // Which conversation this process shows, and whether a prompt was sent in it here.
 let watchedId = ""
@@ -83,7 +82,7 @@ async function refreshWork($: EngineInterface) {
 				prs: [],
 				items: [],
 				fetchedAt: Date.now(),
-				error: "origin is neither GitHub nor Azure DevOps.",
+				isUnsupported: true,
 			}))
 			return
 		}
@@ -145,26 +144,10 @@ async function summarizeIfMissing($: EngineInterface) {
 	if (!current || "error" in current) await summarize($)
 }
 
-// Opens the pane once in a resumed session, the first time it has a PR or item.
-async function autoOpen($: EngineInterface) {
-	if (!isAutoOpenWanted || isAutoOpened) return
-	await refreshWork($)
-	const current = await read($, work)
-	if (
-		isAutoOpened ||
-		!current ||
-		current.prs.length + current.items.length === 0
-	)
-		return
-	isAutoOpened = true
-	// No focus: the prompt keeps the keyboard. Ctrl+X Tab or a click focuses it to scroll.
-	await $.ui.open({ id: PANE, title: "Work", columns: 64 })
-	await summarizeIfMissing($)
-}
-
 // A conversation is resumed when it has turns no prompt of this process made:
 // --resume and --continue load them before session.start, while the picker and
 // an in-session /resume load them later, under a new session id.
+// A resumed conversation opens the pane once, whether or not it has a PR or item.
 async function checkResumed($: EngineInterface) {
 	const id = await $.session.id()
 	if (id !== watchedId) {
@@ -172,16 +155,17 @@ async function checkResumed($: EngineInterface) {
 		// A /clear starts its conversation here, though the transcript's /clear row counts as a turn.
 		hasPromptedHere = isClearPending
 		isClearPending = false
-		isAutoOpenWanted = false
 		isAutoOpened = false
 		// Another conversation now: its branches start over from here.
 		sessionStartedAt = Date.now()
 		await update($, branches, () => [])
 	}
-	if (isAutoOpenWanted || hasPromptedHere || (await $.session.turns()) === 0)
-		return
-	isAutoOpenWanted = true
-	await autoOpen($)
+	if (isAutoOpened || hasPromptedHere || (await $.session.turns()) === 0) return
+	isAutoOpened = true
+	void refreshWork($)
+	// No focus: the prompt keeps the keyboard. Ctrl+X Tab or a click focuses it to scroll.
+	await $.ui.open({ id: PANE, title: "Work", columns: 64 })
+	await summarizeIfMissing($)
 }
 
 function ago(at: number): string {
@@ -239,7 +223,6 @@ export const register: Register = (on) => {
 		// and moves its goal and steps on.
 		void (async () => {
 			await noteBranch($)
-			if (isAutoOpenWanted && !isAutoOpened) return autoOpen($)
 			if (await isPaneOpen($)) await Promise.all([refreshWork($), summarize($)])
 		})()
 
@@ -292,6 +275,7 @@ export const register: Register = (on) => {
 			)
 		const prs = current?.prs ?? []
 		const items = current?.items ?? []
+		const hasWork = !current?.isUnsupported
 
 		return (
 			<Box flexDirection="column" rowGap={1}>
@@ -331,78 +315,82 @@ export const register: Register = (on) => {
 					</Box>
 				)}
 
-				<Box flexDirection="column">
-					<Text>
-						<Text bold color="planMode">
-							Pull requests
-						</Text>
-						<Text dimColor>
-							{" "}
-							{prs.length}
-							{fetching ? " · updating…" : ""}
+				{hasWork && (
+					<Box flexDirection="column">
+						<Text>
+							<Text bold color="planMode">
+								Pull requests
+							</Text>
+							<Text dimColor>
+								{" "}
+								{prs.length}
+								{fetching ? " · updating…" : ""}
+							</Text>
 						</Text>
-					</Text>
-					{current?.error && <Text color="error">{current.error}</Text>}
-					{prs.length === 0 && !current?.error && (
-						<Text dimColor>None found for this session yet.</Text>
-					)}
-					{prs.map((pr) => (
-						<Box flexDirection="column">
-							<Box flexDirection="row" columnGap={1}>
-								<Button
-									key={`toggle-${pr.number}`}
-									plain
-									label={shut.includes(pr.number) ? "▸" : "▾"}
-									onPress={() => void toggle(pr)}
-								/>
-								<Text wrap="truncate-end">
-									<Link href={pr.url}>#{pr.number}</Link>
-									<Text color={STATE_COLOR[pr.state] ?? "text"}>
-										{" "}
-										{pr.state}
+						{current?.error && <Text color="error">{current.error}</Text>}
+						{prs.length === 0 && !current?.error && (
+							<Text dimColor>None found for this session yet.</Text>
+						)}
+						{prs.map((pr) => (
+							<Box flexDirection="column">
+								<Box flexDirection="row" columnGap={1}>
+									<Button
+										key={`toggle-${pr.number}`}
+										plain
+										label={shut.includes(pr.number) ? "▸" : "▾"}
+										onPress={() => void toggle(pr)}
+									/>
+									<Text wrap="truncate-end">
+										<Link href={pr.url}>#{pr.number}</Link>
+										<Text color={STATE_COLOR[pr.state] ?? "text"}>
+											{" "}
+											{pr.state}
+										</Text>
+										<Text dimColor> [{pr.branch}]</Text> {pr.title}
 									</Text>
-									<Text dimColor> [{pr.branch}]</Text> {pr.title}
-								</Text>
-							</Box>
-							{!shut.includes(pr.number) && (
-								<Text>
-									{"  "}
-									{pr.review && (
-										<Text
-											color={pr.review === "approved" ? "success" : "warning"}
-										>
-											{pr.review}{" "}
+								</Box>
+								{!shut.includes(pr.number) && (
+									<Text>
+										{"  "}
+										{pr.review && (
+											<Text
+												color={pr.review === "approved" ? "success" : "warning"}
+											>
+												{pr.review}{" "}
+											</Text>
+										)}
+										<Text color={CHECKS[pr.checks].color}>
+											{CHECKS[pr.checks].text}
 										</Text>
-									)}
-									<Text color={CHECKS[pr.checks].color}>
-										{CHECKS[pr.checks].text}
+										<Text dimColor> {pr.comments} comments</Text>
 									</Text>
-									<Text dimColor> {pr.comments} comments</Text>
-								</Text>
-							)}
-						</Box>
-					))}
-				</Box>
+								)}
+							</Box>
+						))}
+					</Box>
+				)}
 
-				<Box flexDirection="column">
-					<Text>
-						<Text bold color="planMode">
-							Work items
-						</Text>
-						<Text dimColor> {items.length}</Text>
-					</Text>
-					{items.length === 0 && <Text dimColor>None linked yet.</Text>}
-					{items.map((item) => (
-						<Text wrap="truncate-end">
-							<Link href={item.url}>#{item.id}</Link>
-							<Text color={STATE_COLOR[item.state] ?? "text"}>
-								{" "}
-								{item.state}
-							</Text>{" "}
-							{item.title}
+				{hasWork && (
+					<Box flexDirection="column">
+						<Text>
+							<Text bold color="planMode">
+								Work items
+							</Text>
+							<Text dimColor> {items.length}</Text>
 						</Text>
-					))}
-				</Box>
+						{items.length === 0 && <Text dimColor>None linked yet.</Text>}
+						{items.map((item) => (
+							<Text wrap="truncate-end">
+								<Link href={item.url}>#{item.id}</Link>
+								<Text color={STATE_COLOR[item.state] ?? "text"}>
+									{" "}
+									{item.state}
+								</Text>{" "}
+								{item.title}
+							</Text>
+						))}
+					</Box>
+				)}
 
 				<Box flexDirection="row" columnGap={1}>
 					<Button
@@ -411,11 +399,13 @@ export const register: Register = (on) => {
 						hotkey="r"
 						onPress={() => void Promise.all([refreshWork($), summarize($)])}
 					/>
-					<Text dimColor>
-						{current
-							? `PRs updated ${ago(current.fetchedAt)}`
-							: "PRs not fetched yet"}
-					</Text>
+					{hasWork && (
+						<Text dimColor>
+							{current
+								? `PRs updated ${ago(current.fetchedAt)}`
+								: "PRs not fetched yet"}
+						</Text>
+					)}
 				</Box>
 			</Box>
 		)

package/dist/skills/statusline-setup/mod/tests/work.test.ts

@@ -167,9 +167,9 @@ function fakeWorld(
 }
 
 const PANE = {
-	plugin: "work-items",
+	plugin: "work-pane",
 	component: "Pane",
-	requestId: "work-items",
+	requestId: "work-pane",
 	props: {
 		title: "Work",
 		isFocused: true,
@@ -231,6 +231,23 @@ test("Refresh skips the model while the session has no messages", async ($, on)
 	await ui.unmount()
 })
 
+test("a remote that is neither GitHub nor Azure DevOps hides PRs and work items", async ($, on) => {
+	fakeWorld(on, [], [message("hello")], (argv) =>
+		argv[0] === "git" && argv[1] === "remote"
+			? "git@gitlab.com:o/repo.git\n"
+			: undefined,
+	)
+	const ui = await $.ui.mount({ ...PANE, surface: "terminal" })
+	await ui.press({ key: "refresh" })
+	expect(
+		await ui.find({ type: "Text", text: /Ship the thing\./ }),
+	).toBeDefined()
+	for (const text of [/Pull requests/, /Work items/, /neither/, /^PRs /])
+		expect(await ui.find({ type: "Text", text })).toBeUndefined()
+	expect(await ui.find({ type: "Button", label: "Refresh" })).toBeDefined()
+	await ui.unmount()
+})
+
 function fakeSession(on: On, start: { id: string; turns: number }) {
 	const session = { ...start }
 	on("session.id", () => ({ value: session.id }))
@@ -284,6 +301,9 @@ test("a new session never opens the pane by itself, even after its own turns", a
 	session.turns = 1
 	await clock.advance(10_000)
 	expect(opened).toEqual([])
+	await $.command.run({ command: "work", args: "" })
+	await clock.settle()
+	expect(opened).toEqual(["work-pane"])
 })
 
 test("--resume opens the pane at start and summarizes once", async ($, on) => {
@@ -293,10 +313,10 @@ test("--resume opens the pane at start and summarizes once", async ($, on) => {
 	fakeSession(on, { id: "a", turns: 4 })
 	await $.session.start(START)
 	await clock.settle()
-	expect(opened).toEqual(["work-items"])
+	expect(opened).toEqual(["work-pane"])
 	expect(prompts.length).toBe(1)
 	await clock.advance(10_000)
-	expect(opened).toEqual(["work-items"])
+	expect(opened).toEqual(["work-pane"])
 })
 
 test("the picker and /resume open the pane once the resumed turns load", async ($, on) => {
@@ -310,12 +330,12 @@ test("the picker and /resume open the pane once the resumed turns load", async (
 	session.id = "resumed"
 	session.turns = 6
 	await clock.advance(5_000)
-	expect(opened).toEqual(["work-items"])
+	expect(opened).toEqual(["work-pane"])
 })
 
 test("each turn summarizes again while the pane is open, not once it closes", async ($, on) => {
 	const clock = mock.clock(on)
-	const opened = ["work-items"]
+	const opened = ["work-pane"]
 	const prompts = fakeWorld(on, opened)
 	on("turn.complete", (_$, e) => ({ text: e.answer }))
 	const turn = {
@@ -335,7 +355,7 @@ test("each turn summarizes again while the pane is open, not once it closes", as
 	expect(prompts.length).toBe(2)
 })
 
-test("a resumed session with nothing to show still summarizes each turn once /work opens the pane", async ($, on) => {
+test("a resumed session with nothing to show still opens the pane and summarizes each turn", async ($, on) => {
 	const clock = mock.clock(on)
 	const opened: string[] = []
 	const prompts = fakeWorld(on, opened, [message("hello")], (argv) =>
@@ -345,9 +365,7 @@ test("a resumed session with nothing to show still summarizes each turn once /wo
 	fakeSession(on, { id: "a", turns: 4 })
 	await $.session.start(START)
 	await clock.settle()
-	expect(opened).toEqual([])
-	await $.command.run({ command: "work", args: "" })
-	await clock.settle()
+	expect(opened).toEqual(["work-pane"])
 	const before = prompts.length
 	await $.turn.complete({
 		answer: "",

package/dist/skills/statusline-setup/mod/types/index.d.ts

@@ -15,6 +15,7 @@ export type Work = {
 	items: Item[]
 	fetchedAt: number
 	error?: string
+	isUnsupported?: boolean
 }
 export type Step = { text: string; status: "done" | "current" | "todo" }
 export type Summary =
@@ -23,7 +24,7 @@ export type Summary =
 
 declare module "claude-code" {
 	interface PluginState {
-		"work-items": {
+		"work-pane": {
 			branches: string[]
 			work: Work | null
 			isFetching: boolean

package/dist/skills/universalize-agents/reference/mapping.md

@@ -1,7 +1,8 @@
 # Platform mapping reference
 
-Reverse map used by the `universalize-agents` skill. Source of truth is
-`src/utils/paths.ts` in this repository — keep this file in sync with it.
+Reverse map used by the `universalize-agents` skill. The Skills rows come from
+`src/utils/paths.ts` in this repository; keep them in sync with it. The Agents
+and Commands rows are maintained here only.
 
 ## Source directories (discover from / symlink to)
 

package/package.json

@@ -1,10 +1,10 @@
 {
 	"name": "@kurokeita/add-skill",
-	"version": "2.6.1",
+	"version": "3.0.1",
 	"description": "CLI to install AI agent skills to various platforms",
 	"type": "module",
 	"bin": {
-		"ai-agent": "./dist/bin/cli.js"
+		"ai-agent": "dist/bin/cli.js"
 	},
 	"main": "dist/bin/cli.js",
 	"files": [
@@ -31,7 +31,7 @@
 	"author": "Kuro Keita",
 	"repository": {
 		"type": "git",
-		"url": "https://github.com/kurokeita/ai-agent.git"
+		"url": "git+https://github.com/kurokeita/ai-agent.git"
 	},
 	"license": "GPL-3.0",
 	"packageManager": "pnpm@10.28.0",