fix(commands): load opencode command dirs from aliases
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
@@ -0,0 +1,70 @@
|
|||||||
|
import { afterEach, beforeEach, describe, expect, it } from "bun:test"
|
||||||
|
import { mkdirSync, rmSync, writeFileSync } from "node:fs"
|
||||||
|
import { tmpdir } from "node:os"
|
||||||
|
import { join } from "node:path"
|
||||||
|
import { loadOpencodeGlobalCommands, loadOpencodeProjectCommands } from "./loader"
|
||||||
|
|
||||||
|
const TEST_DIR = join(tmpdir(), `claude-code-command-loader-${Date.now()}`)
|
||||||
|
|
||||||
|
function writeCommand(directory: string, name: string, description: string): void {
|
||||||
|
mkdirSync(directory, { recursive: true })
|
||||||
|
writeFileSync(
|
||||||
|
join(directory, `${name}.md`),
|
||||||
|
`---\ndescription: ${description}\n---\nRun ${name}.\n`,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
describe("claude-code command loader", () => {
|
||||||
|
let originalOpencodeConfigDir: string | undefined
|
||||||
|
|
||||||
|
beforeEach(() => {
|
||||||
|
mkdirSync(TEST_DIR, { recursive: true })
|
||||||
|
originalOpencodeConfigDir = process.env.OPENCODE_CONFIG_DIR
|
||||||
|
})
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
if (originalOpencodeConfigDir === undefined) {
|
||||||
|
delete process.env.OPENCODE_CONFIG_DIR
|
||||||
|
} else {
|
||||||
|
process.env.OPENCODE_CONFIG_DIR = originalOpencodeConfigDir
|
||||||
|
}
|
||||||
|
rmSync(TEST_DIR, { recursive: true, force: true })
|
||||||
|
})
|
||||||
|
|
||||||
|
it("#given a parent .opencode/commands directory #when loadOpencodeProjectCommands is called from child directory #then it loads the ancestor command", async () => {
|
||||||
|
// given
|
||||||
|
const projectDir = join(TEST_DIR, "project")
|
||||||
|
const childDir = join(projectDir, "apps", "desktop")
|
||||||
|
writeCommand(join(projectDir, ".opencode", "commands"), "ancestor", "Ancestor command")
|
||||||
|
|
||||||
|
// when
|
||||||
|
const commands = await loadOpencodeProjectCommands(childDir)
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(commands.ancestor?.description).toBe("(opencode-project) Ancestor command")
|
||||||
|
})
|
||||||
|
|
||||||
|
it("#given a .opencode/command directory #when loadOpencodeProjectCommands is called #then it loads the singular alias directory", async () => {
|
||||||
|
// given
|
||||||
|
writeCommand(join(TEST_DIR, ".opencode", "command"), "singular", "Singular command")
|
||||||
|
|
||||||
|
// when
|
||||||
|
const commands = await loadOpencodeProjectCommands(TEST_DIR)
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(commands.singular?.description).toBe("(opencode-project) Singular command")
|
||||||
|
})
|
||||||
|
|
||||||
|
it("#given a global .opencode/commands directory #when loadOpencodeGlobalCommands is called #then it loads the plural alias directory", async () => {
|
||||||
|
// given
|
||||||
|
const opencodeConfigDir = join(TEST_DIR, "opencode-config")
|
||||||
|
process.env.OPENCODE_CONFIG_DIR = opencodeConfigDir
|
||||||
|
writeCommand(join(opencodeConfigDir, "commands"), "global-plural", "Global plural command")
|
||||||
|
|
||||||
|
// when
|
||||||
|
const commands = await loadOpencodeGlobalCommands()
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(commands["global-plural"]?.description).toBe("(opencode) Global plural command")
|
||||||
|
})
|
||||||
|
})
|
||||||
@@ -3,7 +3,12 @@ import { join, basename } from "path"
|
|||||||
import { parseFrontmatter } from "../../shared/frontmatter"
|
import { parseFrontmatter } from "../../shared/frontmatter"
|
||||||
import { sanitizeModelField } from "../../shared/model-sanitizer"
|
import { sanitizeModelField } from "../../shared/model-sanitizer"
|
||||||
import { isMarkdownFile } from "../../shared/file-utils"
|
import { isMarkdownFile } from "../../shared/file-utils"
|
||||||
import { getClaudeConfigDir, getOpenCodeConfigDir } from "../../shared"
|
import {
|
||||||
|
findProjectOpencodeCommandDirs,
|
||||||
|
getClaudeConfigDir,
|
||||||
|
getOpenCodeCommandDirs,
|
||||||
|
getOpenCodeConfigDir,
|
||||||
|
} from "../../shared"
|
||||||
import { log } from "../../shared/logger"
|
import { log } from "../../shared/logger"
|
||||||
import type { CommandScope, CommandDefinition, CommandFrontmatter, LoadedCommand } from "./types"
|
import type { CommandScope, CommandDefinition, CommandFrontmatter, LoadedCommand } from "./types"
|
||||||
|
|
||||||
@@ -121,16 +126,21 @@ export async function loadProjectCommands(directory?: string): Promise<Record<st
|
|||||||
}
|
}
|
||||||
|
|
||||||
export async function loadOpencodeGlobalCommands(): Promise<Record<string, CommandDefinition>> {
|
export async function loadOpencodeGlobalCommands(): Promise<Record<string, CommandDefinition>> {
|
||||||
const configDir = getOpenCodeConfigDir({ binary: "opencode" })
|
const opencodeCommandDirs = getOpenCodeCommandDirs({ binary: "opencode" })
|
||||||
const opencodeCommandsDir = join(configDir, "command")
|
const allCommands = await Promise.all(
|
||||||
const commands = await loadCommandsFromDir(opencodeCommandsDir, "opencode")
|
opencodeCommandDirs.map((commandsDir) => loadCommandsFromDir(commandsDir, "opencode")),
|
||||||
return commandsToRecord(commands)
|
)
|
||||||
|
return commandsToRecord(allCommands.flat())
|
||||||
}
|
}
|
||||||
|
|
||||||
export async function loadOpencodeProjectCommands(directory?: string): Promise<Record<string, CommandDefinition>> {
|
export async function loadOpencodeProjectCommands(directory?: string): Promise<Record<string, CommandDefinition>> {
|
||||||
const opencodeProjectDir = join(directory ?? process.cwd(), ".opencode", "command")
|
const opencodeProjectDirs = findProjectOpencodeCommandDirs(directory ?? process.cwd())
|
||||||
const commands = await loadCommandsFromDir(opencodeProjectDir, "opencode-project")
|
const allCommands = await Promise.all(
|
||||||
return commandsToRecord(commands)
|
opencodeProjectDirs.map((commandsDir) =>
|
||||||
|
loadCommandsFromDir(commandsDir, "opencode-project"),
|
||||||
|
),
|
||||||
|
)
|
||||||
|
return commandsToRecord(allCommands.flat())
|
||||||
}
|
}
|
||||||
|
|
||||||
export async function loadAllCommands(directory?: string): Promise<Record<string, CommandDefinition>> {
|
export async function loadAllCommands(directory?: string): Promise<Record<string, CommandDefinition>> {
|
||||||
|
|||||||
Reference in New Issue
Block a user