Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions kompass.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
"pr/create": { "enabled": true },
"pr/fix": { "enabled": true },
"pr/review": { "enabled": true },
"reload": { "enabled": true },
"review": { "enabled": true },
"ship": { "enabled": true },
"rmslop": { "enabled": true },
Expand All @@ -45,7 +44,6 @@
"pr_sync": { "enabled": true },
"ticket_load": { "enabled": true },
"ticket_sync": { "enabled": true },
"reload": { "enabled": true },
},

"components": {
Expand Down
11 changes: 1 addition & 10 deletions kompass.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,6 @@
"pr/review": {
"$ref": "#/$defs/commandConfig"
},
"reload": {
"$ref": "#/$defs/commandConfig"
},
"review": {
"$ref": "#/$defs/commandConfig"
},
Expand Down Expand Up @@ -103,7 +100,6 @@
"pr/create",
"pr/fix",
"pr/review",
"reload",
"review",
"ship",
"rmslop",
Expand Down Expand Up @@ -131,7 +127,6 @@
"pr/create",
"pr/fix",
"pr/review",
"reload",
"review",
"ship",
"rmslop",
Expand Down Expand Up @@ -195,9 +190,6 @@
},
"ticket_load": {
"$ref": "#/$defs/toolConfig"
},
"reload": {
"$ref": "#/$defs/toolConfig"
}
}
},
Expand Down Expand Up @@ -433,8 +425,7 @@
"pr_load",
"pr_sync",
"ticket_sync",
"ticket_load",
"reload"
"ticket_load"
]
}
}
Expand Down
6 changes: 0 additions & 6 deletions packages/core/commands/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,6 @@ export const commandDefinitions: Record<string, CommandDefinition> = {
templatePath: "commands/learn.md",
subtask: false,
},
reload: {
description: "Reload the current OpenCode project configuration",
agent: "build",
templatePath: "commands/reload.md",
subtask: false,
},
"pr/create": {
description: "Summarize branch work and create a PR",
agent: "build",
Expand Down
34 changes: 0 additions & 34 deletions packages/core/commands/reload.md

This file was deleted.

2 changes: 0 additions & 2 deletions packages/core/kompass.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
"pr/create": { "enabled": true },
"pr/fix": { "enabled": true },
"pr/review": { "enabled": true },
"reload": { "enabled": true },
"review": { "enabled": true },
"ship": { "enabled": true },
"rmslop": { "enabled": true },
Expand All @@ -45,7 +44,6 @@
"pr_sync": { "enabled": true },
"ticket_load": { "enabled": true },
"ticket_sync": { "enabled": true },
"reload": { "enabled": true },
},

"components": {
Expand Down
7 changes: 0 additions & 7 deletions packages/core/lib/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ export const DEFAULT_TOOL_NAMES = [
"pr_sync",
"ticket_sync",
"ticket_load",
"reload",
] as const;

export const DEFAULT_COMMAND_NAMES = [
Expand All @@ -31,7 +30,6 @@ export const DEFAULT_COMMAND_NAMES = [
"pr/create",
"pr/fix",
"pr/review",
"reload",
"review",
"ship",
"rmslop",
Expand Down Expand Up @@ -114,7 +112,6 @@ export interface KompassConfig {
"pr/create"?: CommandConfig;
"pr/fix"?: CommandConfig;
"pr/review"?: CommandConfig;
reload?: CommandConfig;
review?: CommandConfig;
ship?: CommandConfig;
rmslop?: CommandConfig;
Expand All @@ -140,7 +137,6 @@ export interface KompassConfig {
pr_sync?: ToolConfig;
ticket_sync?: ToolConfig;
ticket_load?: ToolConfig;
reload?: ToolConfig;
};
components?: {
"change-summary"?: ComponentConfig;
Expand Down Expand Up @@ -189,7 +185,6 @@ export interface MergedKompassConfig {
pr_sync: ToolConfig;
ticket_sync: ToolConfig;
ticket_load: ToolConfig;
reload: ToolConfig;
};
components: {
enabled: string[];
Expand Down Expand Up @@ -480,7 +475,6 @@ const defaultToolConfig: Record<ToolName, ToolConfig> = {
pr_sync: { enabled: true },
ticket_sync: { enabled: true },
ticket_load: { enabled: true },
reload: { enabled: true },
};

function getToggleEntry<T extends ToggleConfig>(
Expand Down Expand Up @@ -666,7 +660,6 @@ export function mergeWithDefaults(
pr_sync: { ...defaultToolConfig.pr_sync, ...config?.tools?.pr_sync },
ticket_sync: { ...defaultToolConfig.ticket_sync, ...config?.tools?.ticket_sync },
ticket_load: { ...defaultToolConfig.ticket_load, ...config?.tools?.ticket_load },
reload: { ...defaultToolConfig.reload, ...config?.tools?.reload },
},
components: {
enabled: getEnabledNames(
Expand Down
39 changes: 0 additions & 39 deletions packages/opencode/.opencode/commands/reload.md

This file was deleted.

6 changes: 0 additions & 6 deletions packages/opencode/.opencode/kompass.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,6 @@
"pr/review": {
"enabled": true
},
"reload": {
"enabled": true
},
"review": {
"enabled": true
},
Expand Down Expand Up @@ -94,9 +91,6 @@
},
"ticket_load": {
"enabled": true
},
"reload": {
"enabled": true
}
},
"defaults": {
Expand Down
24 changes: 0 additions & 24 deletions packages/opencode/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -216,27 +216,6 @@ export function removeSyntheticAgentHandoff(output: ChatMessageOutput): boolean
return true;
}

function createReloadTool(client: PluginInput["client"]) {
return tool({
description: "Reload the current OpenCode project cache",
args: {},
async execute(_, context) {
// Defer dispose so the tool returns before the session is torn down
setTimeout(() => {
void client.instance.dispose({ query: { directory: context.directory } }).catch((error) => {
console.error("[kompass] Failed to dispose instance during reload:", error);
});
}, 500);
return JSON.stringify({
scope: "project",
directory: context.directory,
status: "reload-requested",
nextLoad: "config, commands, agents, custom tools, and plugins rebuild on next access",
}, null, 2);
},
});
}

const opencodeToolCreators = {
changes_load($: PluginInput["$"], _: PluginInput["client"], __: MergedKompassConfig) {
const definition = createChangesLoadTool(asShell($));
Expand Down Expand Up @@ -345,9 +324,6 @@ const opencodeToolCreators = {
execute: (args, context) => definition.execute(args, context),
});
},
reload(_: PluginInput["$"], client: PluginInput["client"], __: MergedKompassConfig) {
return createReloadTool(client);
},
} as const;

export async function createOpenCodeTools(
Expand Down
2 changes: 0 additions & 2 deletions packages/opencode/kompass.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
"pr/create": { "enabled": true },
"pr/fix": { "enabled": true },
"pr/review": { "enabled": true },
"reload": { "enabled": true },
"review": { "enabled": true },
"ship": { "enabled": true },
"rmslop": { "enabled": true },
Expand All @@ -45,7 +44,6 @@
"pr_sync": { "enabled": true },
"ticket_load": { "enabled": true },
"ticket_sync": { "enabled": true },
"reload": { "enabled": true },
},

"components": {
Expand Down
16 changes: 0 additions & 16 deletions packages/opencode/test/commands-config.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ describe("applyCommandsConfig", () => {
const expectedCommands = [
"ask",
"branch",
"reload",
"pr/create",
"pr/review",
"pr/fix",
Expand All @@ -61,7 +60,6 @@ describe("applyCommandsConfig", () => {
assert.ok(cfg.command);
assert.equal(cfg.command!["pr/review"]?.agent, "reviewer");
assert.equal(cfg.command!["branch"]?.agent, "build");
assert.equal(cfg.command!["reload"]?.agent, "build");
assert.equal(cfg.command!["pr/create"]?.agent, "build");
assert.equal(cfg.command!["ticket/create"]?.agent, "build");
assert.equal(cfg.command!["ticket/plan"]?.agent, "planner");
Expand All @@ -72,7 +70,6 @@ describe("applyCommandsConfig", () => {
assert.equal(cfg.command!["ship"]?.agent, "navigator");
assert.equal(cfg.command!["todo"]?.agent, "navigator");
assert.ok(cfg.command!["pr/review"]?.description);
assert.ok(cfg.command!["reload"]?.template);
assert.ok(cfg.command!["dev"]?.template);
assert.ok(cfg.command!["branch"]?.template);
});
Expand Down Expand Up @@ -112,19 +109,6 @@ describe("applyCommandsConfig", () => {
assert.doesNotMatch(fixTemplate, /`pr_review`/);
});

test("rewrites project reload tool name with opencode prefix", async () => {
delete process.env.CI;
const cfg: { command?: Record<string, { template: string }> } = {};

await applyCommandsConfig(cfg as never, process.cwd());

assert.ok(cfg.command);
const reloadTemplate = cfg.command!["reload"].template;

assert.match(reloadTemplate, /`kompass_reload`/);
assert.doesNotMatch(reloadTemplate, /Call `reload` with no parameters\./);
});

test("rewrites tool references with configured aliases", async () => {
delete process.env.CI;
const tempDir = await mkdtemp(path.join(os.tmpdir(), "kompass-commands-"));
Expand Down
5 changes: 1 addition & 4 deletions packages/opencode/test/tool-registration.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,13 +67,11 @@ describe("createOpenCodeTools", () => {
assert.ok(tools.kompass_changes_load);
assert.ok(tools.kompass_pr_load);
assert.ok(tools.kompass_pr_sync);
assert.ok(tools.kompass_reload);
assert.ok(tools.kompass_ticket_load);
assert.ok(tools.kompass_ticket_sync);
assert.equal(tools.changes_load, undefined);
assert.equal(tools.pr_load, undefined);
assert.equal(tools.pr_sync, undefined);
assert.equal(tools.reload, undefined);
assert.equal(tools.ticket_load, undefined);
assert.equal(tools.ticket_sync, undefined);
});
Expand Down Expand Up @@ -106,9 +104,8 @@ describe("createOpenCodeTools", () => {
}) as never, createMockClient() as never, tempDir);

assert.ok(tools.custom_ticket_name);
assert.ok(tools.kompass_reload);
assert.equal(tools.kompass_ticket_sync, undefined);
assert.deepEqual(Object.keys(tools).sort(), ["custom_ticket_name", "kompass_reload"]);
assert.deepEqual(Object.keys(tools).sort(), ["custom_ticket_name"]);
} finally {
await rm(tempDir, { recursive: true, force: true });
}
Expand Down
Loading
Loading