Erik Kotsiuba
d6db94a39d
Update Pi packages to 0.79.9 and align pi-ask with current TUI/RPC behavior. - guard custom ask/settings flows to TUI mode with a clear non-TUI fallback - reset config cache on extension reload so edited settings are picked up - clean up remote ask flow listeners on session shutdown - forward editor focus/invalidation and keep @ file autocomplete explicit - document the non-TUI contract and cover the compatibility fixes with tests
116 lines
2.9 KiB
TypeScript
116 lines
2.9 KiB
TypeScript
import assert from "node:assert/strict";
|
|
import { mkdtemp, rm } from "node:fs/promises";
|
|
import { tmpdir } from "node:os";
|
|
import { join } from "node:path";
|
|
import test from "node:test";
|
|
import { registerAskSettingsCommand } from "../src/ask-settings-command.ts";
|
|
|
|
function plainTheme() {
|
|
return {
|
|
fg(_color: string, text: string) {
|
|
return text;
|
|
},
|
|
bg(_color: string, text: string) {
|
|
return text;
|
|
},
|
|
};
|
|
}
|
|
|
|
test("registers /ask-settings and opens the shared settings overlay", async () => {
|
|
const agentDir = await mkdtemp(join(tmpdir(), "pi-ask-command-"));
|
|
process.env.PI_CODING_AGENT_DIR = agentDir;
|
|
|
|
const commands = new Map<
|
|
string,
|
|
{ handler: (args: string, ctx: any) => Promise<void> }
|
|
>();
|
|
registerAskSettingsCommand({
|
|
registerCommand(
|
|
name: string,
|
|
command: { handler: (args: string, ctx: any) => Promise<void> }
|
|
) {
|
|
commands.set(name, command);
|
|
},
|
|
} as never);
|
|
|
|
assert(commands.has("ask-settings"));
|
|
|
|
const customCalls: Array<{ options: unknown; lines: string[] }> = [];
|
|
await commands.get("ask-settings")?.handler("", {
|
|
mode: "tui",
|
|
ui: {
|
|
custom(callback: any, options: unknown) {
|
|
const done = () => {
|
|
// test callback intentionally unused
|
|
};
|
|
const tui = {
|
|
requestRender() {
|
|
// test render hook intentionally unused
|
|
},
|
|
};
|
|
const component = callback(tui, plainTheme(), undefined, done);
|
|
customCalls.push({ options, lines: component.render(72) });
|
|
return Promise.resolve();
|
|
},
|
|
},
|
|
});
|
|
|
|
assert.equal(customCalls.length, 1);
|
|
assert.deepEqual(customCalls[0]?.options, {
|
|
overlay: true,
|
|
overlayOptions: {
|
|
anchor: "center",
|
|
margin: 1,
|
|
maxHeight: "90%",
|
|
minWidth: 26,
|
|
width: 72,
|
|
},
|
|
});
|
|
const text = customCalls[0]?.lines.join("\n") ?? "";
|
|
assert(text.includes("@eko24ive/pi-ask"));
|
|
assert(text.includes("Edit this config file to customize"));
|
|
assert(text.includes("keymaps"));
|
|
assert(text.includes("notifications"));
|
|
assert(text.includes("extraction settings"));
|
|
|
|
delete process.env.PI_CODING_AGENT_DIR;
|
|
await rm(agentDir, { force: true, recursive: true });
|
|
});
|
|
|
|
test("/ask-settings notifies outside TUI instead of opening custom UI", async () => {
|
|
const commands = new Map<
|
|
string,
|
|
{ handler: (args: string, ctx: any) => Promise<void> }
|
|
>();
|
|
registerAskSettingsCommand({
|
|
registerCommand(
|
|
name: string,
|
|
command: { handler: (args: string, ctx: any) => Promise<void> }
|
|
) {
|
|
commands.set(name, command);
|
|
},
|
|
} as never);
|
|
|
|
const notifications: Array<{ message: string; type: string }> = [];
|
|
let customOpened = false;
|
|
await commands.get("ask-settings")?.handler("", {
|
|
mode: "rpc",
|
|
ui: {
|
|
custom() {
|
|
customOpened = true;
|
|
},
|
|
notify(message: string, type: string) {
|
|
notifications.push({ message, type });
|
|
},
|
|
},
|
|
});
|
|
|
|
assert.equal(customOpened, false);
|
|
assert.deepEqual(notifications, [
|
|
{
|
|
message: "/ask-settings requires interactive TUI mode.",
|
|
type: "error",
|
|
},
|
|
]);
|
|
});
|