opencode-copilot-account-switcher 0.1.2 → 0.1.3

package/README.md

@@ -12,13 +12,14 @@
 
 ## English
 
-Manage and switch between multiple **GitHub Copilot** accounts in **OpenCode**. This plugin only adds account switching and quota checks — it **uses the official `github-copilot` provider** and does **not** require model reconfiguration.
+Manage and switch between multiple **GitHub Copilot** accounts in **OpenCode**. This plugin adds account switching, quota checks, and an optional **Guided Loop Safety** mode — it **uses the official `github-copilot` provider** and does **not** require model reconfiguration.
 
 ## What You Get
 
 - **Multi-account support** — add multiple Copilot accounts and switch anytime
 - **Quota check** — view remaining quota per account
 - **Auth import** — import Copilot tokens from OpenCode auth storage
+- **Guided Loop Safety** — inject a stricter question-first Copilot policy with fewer report interruptions and fewer unnecessary subagent calls
 - **Zero model config** — no model changes required (official provider only)
 
 ---
@@ -52,7 +53,7 @@ Install the opencode-copilot-account-switcher plugin by following: https://raw.g
 3. **Login** to GitHub Copilot:
 
    ```bash
-   opencode auth login github-copilot
+   opencode auth login --provider github-copilot
    ```
 
 </details>
@@ -80,7 +81,7 @@ Install the opencode-copilot-account-switcher plugin by following: https://raw.g
 ### Verification
 
 ```bash
-opencode auth login github-copilot
+opencode auth login --provider github-copilot
 ```
 
 </details>
@@ -92,7 +93,7 @@ opencode auth login github-copilot
 Run inside the GitHub Copilot auth flow:
 
 ```bash
-opencode auth login github-copilot
+opencode auth login --provider github-copilot
 ```
 
 You will see an interactive menu (arrow keys + enter) with actions:
@@ -100,10 +101,13 @@ You will see an interactive menu (arrow keys + enter) with actions:
 - **Add account**
 - **Import from auth.json**
 - **Check quotas**
+- **Guided Loop Safety** — prompt-guided question-first reporting that requires `question` for user-facing reports when available, reduces report interruptions, and avoids unnecessary subagent calls
 - **Switch account**
 - **Remove account**
 - **Remove all**
 
+If you want stricter question-first reporting and fewer unnecessary subagent calls in GitHub Copilot sessions, enable Guided Loop Safety from the account menu.
+
 ---
 
 ## Storage
@@ -130,13 +134,14 @@ No. It uses the official provider and only adds account switching + quota checks
 
 ## 中文
 
-在 **OpenCode** 中管理并切换多个 **GitHub Copilot** 账号。本插件只提供**账号切换与配额查询**，**完全依赖官方 `github-copilot` provider**，无需修改模型配置。
+在 **OpenCode** 中管理并切换多个 **GitHub Copilot** 账号。本插件提供**账号切换、配额查询**以及可选的 **Guided Loop Safety** 模式，**完全依赖官方 `github-copilot` provider**，无需修改模型配置。
 
 ## 功能一览
 
 - **多账号管理** — 添加多个 Copilot 账号，随时切换
 - **配额查询** — 查看每个账号的剩余额度
 - **导入认证** — 可从 OpenCode 认证存储导入
+- **Guided Loop Safety** — 为 Copilot 注入更严格的 question-first 提示词规则，减少汇报打断，并避免不必要的子代理调用
 - **无需模型配置** — 使用官方 provider，无需改模型
 
 ---
@@ -170,7 +175,7 @@ No. It uses the official provider and only adds account switching + quota checks
 3. **登录 GitHub Copilot**：
 
    ```bash
-   opencode auth login github-copilot
+   opencode auth login --provider github-copilot
    ```
 
 </details>
@@ -198,7 +203,7 @@ No. It uses the official provider and only adds account switching + quota checks
 ### 验证
 
 ```bash
-opencode auth login github-copilot
+opencode auth login --provider github-copilot
 ```
 
 </details>
@@ -210,7 +215,7 @@ opencode auth login github-copilot
 在 Copilot 认证流程中运行：
 
 ```bash
-opencode auth login github-copilot
+opencode auth login --provider github-copilot
 ```
 
 会出现交互式菜单（方向键 + 回车）：
@@ -218,10 +223,13 @@ opencode auth login github-copilot
 - **添加账号**
 - **从 auth.json 导入**
 - **检查配额**
+- **Guided Loop Safety 开关** — 通过提示词引导模型在可用时必须使用 `question` 做用户可见汇报、减少汇报打断，并避免不必要的子代理调用
 - **切换账号**
 - **删除账号**
 - **全部删除**
 
+如果你希望 GitHub Copilot 会话更严格地优先使用 `question` 工具汇报、减少汇报打断，并避免不必要的子代理调用，可以在账号菜单中开启 Guided Loop Safety。
+
 ---
 
 ## 存储位置

package/dist/index.d.ts

@@ -1 +1,3 @@
-export { CopilotAccountSwitcher } from "./plugin";
+export { CopilotAccountSwitcher } from "./plugin.js";
+export { applyMenuAction } from "./plugin-actions.js";
+export { buildPluginHooks } from "./plugin-hooks.js";

package/dist/index.js

@@ -1 +1,3 @@
-export { CopilotAccountSwitcher } from "./plugin";
+export { CopilotAccountSwitcher } from "./plugin.js";
+export { applyMenuAction } from "./plugin-actions.js";
+export { buildPluginHooks } from "./plugin-hooks.js";

package/dist/loop-safety-plugin.d.ts

@@ -0,0 +1,21 @@
+import type { Hooks } from "@opencode-ai/plugin";
+import { type StoreFile } from "./store.js";
+export declare const LOOP_SAFETY_POLICY = "Guided Loop Safety Policy\n- When the question tool is available and permitted in the current session, all user-facing reports must be delivered through the question tool.\n- The question tool is considered available and permitted when it appears in the active tool list and the current session has not denied its use.\n- Direct assistant text is allowed only when the question tool is unavailable, denied, or absent from the current session.\n- When reporting multiple related items, prefer a single question tool call with multiple well-grouped questions instead of multiple separate interruptions.\n- Group related items into clear question batches such as current progress, key findings, and next-step choices.\n- For long or complex reports, split the report into paginated or sequential question batches instead of overloading one large message.\n- Present the highest-priority information first and defer secondary details to later question batches when needed.\n- Even when no explicit decision is required, prefer brief question-tool status updates over direct assistant text whenever the tool is available.\n- Avoid unnecessary question frequency; combine small related updates when a single question call can cover them clearly.\n- Dispatching task or subagent work is expensive and should be avoided unless it materially improves the result.\n- Materially improves the result means clearly beneficial cases such as parallel analysis of independent areas; it does not include routine local searches, small file reads, or straightforward edits.\n- If task or subagent delegation is used, keep the number minimal and explain the reason briefly through the question tool when available.";
+export type ExperimentalChatSystemTransformHook = (input: {
+    sessionID: string;
+    model: {
+        providerID: string;
+    };
+}, output: {
+    system: string[];
+}) => Promise<void>;
+export type CopilotPluginHooks = Hooks & {
+    "experimental.chat.system.transform"?: ExperimentalChatSystemTransformHook;
+};
+export declare function isCopilotProvider(providerID: string): boolean;
+export declare function applyLoopSafetyPolicy(input: {
+    providerID: string;
+    enabled: boolean;
+    system: string[];
+}): string[];
+export declare function createLoopSafetySystemTransform(loadStore?: () => Promise<StoreFile | undefined>): ExperimentalChatSystemTransformHook;

package/dist/loop-safety-plugin.js

@@ -0,0 +1,39 @@
+import { readStoreSafe } from "./store.js";
+export const LOOP_SAFETY_POLICY = `Guided Loop Safety Policy
+- When the question tool is available and permitted in the current session, all user-facing reports must be delivered through the question tool.
+- The question tool is considered available and permitted when it appears in the active tool list and the current session has not denied its use.
+- Direct assistant text is allowed only when the question tool is unavailable, denied, or absent from the current session.
+- When reporting multiple related items, prefer a single question tool call with multiple well-grouped questions instead of multiple separate interruptions.
+- Group related items into clear question batches such as current progress, key findings, and next-step choices.
+- For long or complex reports, split the report into paginated or sequential question batches instead of overloading one large message.
+- Present the highest-priority information first and defer secondary details to later question batches when needed.
+- Even when no explicit decision is required, prefer brief question-tool status updates over direct assistant text whenever the tool is available.
+- Avoid unnecessary question frequency; combine small related updates when a single question call can cover them clearly.
+- Dispatching task or subagent work is expensive and should be avoided unless it materially improves the result.
+- Materially improves the result means clearly beneficial cases such as parallel analysis of independent areas; it does not include routine local searches, small file reads, or straightforward edits.
+- If task or subagent delegation is used, keep the number minimal and explain the reason briefly through the question tool when available.`;
+export function isCopilotProvider(providerID) {
+    return providerID === "github-copilot" || providerID === "github-copilot-enterprise";
+}
+export function applyLoopSafetyPolicy(input) {
+    if (!input.enabled)
+        return input.system;
+    if (!isCopilotProvider(input.providerID))
+        return input.system;
+    if (input.system.includes(LOOP_SAFETY_POLICY))
+        return input.system;
+    return [...input.system, LOOP_SAFETY_POLICY];
+}
+export function createLoopSafetySystemTransform(loadStore = readStoreSafe) {
+    return async (input, output) => {
+        const store = await loadStore().catch(() => undefined);
+        const next = applyLoopSafetyPolicy({
+            providerID: input.model.providerID,
+            enabled: store?.loopSafetyEnabled === true,
+            system: output.system,
+        });
+        if (next.length === output.system.length)
+            return;
+        output.system.push(LOOP_SAFETY_POLICY);
+    };
+}

package/dist/plugin-actions.d.ts

@@ -0,0 +1,7 @@
+import type { StoreFile } from "./store.js";
+import type { MenuAction } from "./ui/menu.js";
+export declare function applyMenuAction(input: {
+    action: MenuAction;
+    store: StoreFile;
+    writeStore: (store: StoreFile) => Promise<void>;
+}): Promise<boolean>;

package/dist/plugin-actions.js

@@ -0,0 +1,7 @@
+export async function applyMenuAction(input) {
+    if (input.action.type !== "toggle-loop-safety")
+        return false;
+    input.store.loopSafetyEnabled = input.store.loopSafetyEnabled !== true;
+    await input.writeStore(input.store);
+    return true;
+}

package/dist/plugin-hooks.d.ts

@@ -0,0 +1,6 @@
+import { type CopilotPluginHooks } from "./loop-safety-plugin.js";
+import { type StoreFile } from "./store.js";
+export declare function buildPluginHooks(input: {
+    auth: CopilotPluginHooks["auth"];
+    loadStore?: () => Promise<StoreFile | undefined>;
+}): CopilotPluginHooks;

package/dist/plugin-hooks.js

@@ -0,0 +1,8 @@
+import { createLoopSafetySystemTransform } from "./loop-safety-plugin.js";
+import { readStoreSafe } from "./store.js";
+export function buildPluginHooks(input) {
+    return {
+        auth: input.auth,
+        "experimental.chat.system.transform": createLoopSafetySystemTransform(input.loadStore ?? readStoreSafe),
+    };
+}

package/dist/plugin.js

@@ -1,8 +1,10 @@
 import { createInterface } from "node:readline/promises";
 import { stdin as input, stdout as output } from "node:process";
-import { isTTY } from "./ui/ansi";
-import { showAccountActions, showMenu } from "./ui/menu";
-import { authPath, readAuth, readStore, writeStore } from "./store";
+import { applyMenuAction } from "./plugin-actions.js";
+import { buildPluginHooks } from "./plugin-hooks.js";
+import { isTTY } from "./ui/ansi.js";
+import { showAccountActions, showMenu } from "./ui/menu.js";
+import { authPath, readAuth, readStore, writeStore } from "./store.js";
 function now() {
     return Date.now();
 }
@@ -442,6 +444,32 @@ async function switchAccount(client, entry) {
     });
 }
 export const CopilotAccountSwitcher = async ({ client }) => {
+    const methods = [
+        {
+            type: "oauth",
+            label: "Manage GitHub Copilot accounts",
+            async authorize() {
+                const entry = await runMenu();
+                return {
+                    url: "",
+                    instructions: "",
+                    method: "auto",
+                    async callback() {
+                        if (!entry)
+                            return { type: "failed" };
+                        return {
+                            type: "success",
+                            provider: entry.enterpriseUrl ? "github-copilot-enterprise" : "github-copilot",
+                            refresh: entry.refresh,
+                            access: entry.access,
+                            expires: entry.expires,
+                            ...(entry.enterpriseUrl ? { enterpriseUrl: entry.enterpriseUrl } : {}),
+                        };
+                    },
+                };
+            },
+        },
+    ];
     async function runMenu() {
         const store = await readStore();
         const auth = await readAuth().catch(() => ({}));
@@ -533,11 +561,14 @@ export const CopilotAccountSwitcher = async ({ client }) => {
                     : undefined,
             }));
             const refresh = { enabled: store.autoRefresh === true, minutes: store.refreshMinutes ?? 15 };
-            const action = await showMenu(accounts, refresh, store.lastQuotaRefresh);
+            const action = await showMenu(accounts, refresh, store.lastQuotaRefresh, store.loopSafetyEnabled === true);
             if (action.type === "cancel") {
                 const active = store.active ? store.accounts[store.active] : undefined;
                 return active;
             }
+            if (await applyMenuAction({ action, store, writeStore })) {
+                continue;
+            }
             if (action.type === "add") {
                 const mode = await promptText("Add via device login? (y/n): ");
                 const useDevice = mode.toLowerCase() === "y" || mode.toLowerCase() === "yes";
@@ -673,35 +704,10 @@ export const CopilotAccountSwitcher = async ({ client }) => {
             }
         }
     }
-    return {
+    return buildPluginHooks({
         auth: {
             provider: "github-copilot",
-            methods: [
-                {
-                    type: "oauth",
-                    label: "Manage GitHub Copilot accounts",
-                    async authorize() {
-                        const entry = await runMenu();
-                        return {
-                            url: "",
-                            instructions: "",
-                            method: "auto",
-                            async callback() {
-                                if (!entry)
-                                    return { type: "failed" };
-                                return {
-                                    type: "success",
-                                    provider: entry.enterpriseUrl ? "github-copilot-enterprise" : "github-copilot",
-                                    refresh: entry.refresh,
-                                    access: entry.access,
-                                    expires: entry.expires,
-                                    ...(entry.enterpriseUrl ? { enterpriseUrl: entry.enterpriseUrl } : {}),
-                                };
-                            },
-                        };
-                    },
-                },
-            ],
+            methods,
         },
-    };
+    });
 };

package/dist/store.d.ts

@@ -54,9 +54,12 @@ export type StoreFile = {
     autoRefresh?: boolean;
     refreshMinutes?: number;
     lastQuotaRefresh?: number;
+    loopSafetyEnabled?: boolean;
 };
 export declare function storePath(): string;
 export declare function authPath(): string;
-export declare function readStore(): Promise<StoreFile>;
+export declare function parseStore(raw: string): StoreFile;
+export declare function readStore(filePath?: string): Promise<StoreFile>;
+export declare function readStoreSafe(filePath?: string): Promise<StoreFile | undefined>;
 export declare function readAuth(filePath?: string): Promise<Record<string, AccountEntry>>;
 export declare function writeStore(store: StoreFile): Promise<void>;

package/dist/store.js

@@ -12,12 +12,12 @@ export function authPath() {
     const dataDir = xdgData ?? path.join(os.homedir(), ".local", "share");
     return path.join(dataDir, "opencode", authFile);
 }
-export async function readStore() {
-    const file = storePath();
-    const raw = await fs.readFile(file, "utf-8").catch(() => "");
+export function parseStore(raw) {
     const data = raw ? JSON.parse(raw) : { accounts: {} };
     if (!data.accounts)
         data.accounts = {};
+    if (data.loopSafetyEnabled !== true)
+        data.loopSafetyEnabled = false;
     for (const [name, entry] of Object.entries(data.accounts)) {
         const info = entry;
         if (!info.name)
@@ -25,6 +25,26 @@ export async function readStore() {
     }
     return data;
 }
+export async function readStore(filePath = storePath()) {
+    const raw = await fs.readFile(filePath, "utf-8").catch((error) => {
+        if (error.code === "ENOENT")
+            return "";
+        throw error;
+    });
+    return parseStore(raw);
+}
+export async function readStoreSafe(filePath = storePath()) {
+    try {
+        const raw = await fs.readFile(filePath, "utf-8");
+        return parseStore(raw);
+    }
+    catch (error) {
+        const issue = error;
+        if (issue.code === "ENOENT")
+            return parseStore("");
+        return undefined;
+    }
+}
 export async function readAuth(filePath) {
     const dataFile = path.join(xdgData ?? path.join(os.homedir(), ".local", "share"), "opencode", authFile);
     const configFile = path.join(xdgConfig ?? path.join(os.homedir(), ".config"), "opencode", authFile);

package/dist/ui/confirm.js

@@ -1,4 +1,4 @@
-import { select } from "./select";
+import { select } from "./select.js";
 export async function confirm(message, defaultYes = false) {
     const items = defaultYes
         ? [

package/dist/ui/menu.d.ts

@@ -1,3 +1,4 @@
+import { type MenuItem } from "./select.js";
 export type AccountStatus = "active" | "expired" | "unknown";
 export interface AccountInfo {
     name: string;
@@ -52,6 +53,8 @@ export type MenuAction = {
     type: "toggle-refresh";
 } | {
     type: "set-interval";
+} | {
+    type: "toggle-loop-safety";
 } | {
     type: "switch";
     account: AccountInfo;
@@ -63,8 +66,17 @@ export type MenuAction = {
 } | {
     type: "cancel";
 };
+export declare function buildMenuItems(input: {
+    accounts: AccountInfo[];
+    refresh?: {
+        enabled: boolean;
+        minutes: number;
+    };
+    lastQuotaRefresh?: number;
+    loopSafetyEnabled: boolean;
+}): MenuItem<MenuAction>[];
 export declare function showMenu(accounts: AccountInfo[], refresh?: {
     enabled: boolean;
     minutes: number;
-}, lastQuotaRefresh?: number): Promise<MenuAction>;
+}, lastQuotaRefresh?: number, loopSafetyEnabled?: boolean): Promise<MenuAction>;
 export declare function showAccountActions(account: AccountInfo): Promise<"switch" | "remove" | "back">;

package/dist/ui/menu.js

@@ -1,6 +1,6 @@
-import { ANSI } from "./ansi";
-import { select } from "./select";
-import { confirm } from "./confirm";
+import { ANSI } from "./ansi.js";
+import { select } from "./select.js";
+import { confirm } from "./confirm.js";
 function formatRelativeTime(timestamp) {
     if (!timestamp)
         return "never";
@@ -25,9 +25,9 @@ function getStatusBadge(status) {
         return `${ANSI.red}[expired]${ANSI.reset}`;
     return "";
 }
-export async function showMenu(accounts, refresh, lastQuotaRefresh) {
-    const quotaHint = lastQuotaRefresh ? `last ${formatRelativeTime(lastQuotaRefresh)}` : undefined;
-    const items = [
+export function buildMenuItems(input) {
+    const quotaHint = input.lastQuotaRefresh ? `last ${formatRelativeTime(input.lastQuotaRefresh)}` : undefined;
+    return [
         { label: "Actions", value: { type: "cancel" }, kind: "heading" },
         { label: "Add account", value: { type: "add" }, color: "cyan", hint: "device login or manual" },
         { label: "Import from auth.json", value: { type: "import" }, color: "cyan" },
@@ -35,15 +35,21 @@ export async function showMenu(accounts, refresh, lastQuotaRefresh) {
         { label: "Refresh identity", value: { type: "refresh-identity" }, color: "cyan" },
         { label: "Check models", value: { type: "check-models" }, color: "cyan" },
         {
-            label: refresh?.enabled ? "Disable auto refresh" : "Enable auto refresh",
+            label: input.refresh?.enabled ? "Disable auto refresh" : "Enable auto refresh",
             value: { type: "toggle-refresh" },
             color: "cyan",
-            hint: refresh ? `${refresh.minutes}m` : undefined,
+            hint: input.refresh ? `${input.refresh.minutes}m` : undefined,
         },
         { label: "Set refresh interval", value: { type: "set-interval" }, color: "cyan" },
+        {
+            label: input.loopSafetyEnabled ? "Disable guided loop safety" : "Enable guided loop safety",
+            value: { type: "toggle-loop-safety" },
+            color: "cyan",
+            hint: "Prompt-guided: fewer report interruptions, fewer unnecessary subagents",
+        },
         { label: "", value: { type: "cancel" }, separator: true },
         { label: "Accounts", value: { type: "cancel" }, kind: "heading" },
-        ...accounts.map((account) => {
+        ...input.accounts.map((account) => {
             const statusBadge = getStatusBadge(account.status);
             const currentBadge = account.isCurrent ? ` ${ANSI.cyan}*${ANSI.reset}` : "";
             const format = (s) => s?.unlimited ? "∞" : s?.remaining !== undefined && s?.entitlement !== undefined ? `${s.remaining}/${s.entitlement}` : "?";
@@ -69,6 +75,14 @@ export async function showMenu(accounts, refresh, lastQuotaRefresh) {
         { label: "Danger zone", value: { type: "cancel" }, kind: "heading" },
         { label: "Remove all accounts", value: { type: "remove-all" }, color: "red" },
     ];
+}
+export async function showMenu(accounts, refresh, lastQuotaRefresh, loopSafetyEnabled = false) {
+    const items = buildMenuItems({
+        accounts,
+        refresh,
+        lastQuotaRefresh,
+        loopSafetyEnabled,
+    });
     while (true) {
         const result = await select(items, {
             message: "GitHub Copilot accounts",

package/dist/ui/select.js

@@ -1,4 +1,4 @@
-import { ANSI, isTTY, parseKey } from "./ansi";
+import { ANSI, isTTY, parseKey } from "./ansi.js";
 const ESCAPE_TIMEOUT_MS = 50;
 const ANSI_REGEX = new RegExp("\\x1b\\[[0-9;]*m", "g");
 const ANSI_LEADING_REGEX = new RegExp("^\\x1b\\[[0-9;]*m");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-copilot-account-switcher",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "GitHub Copilot account switcher plugin for OpenCode",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -32,6 +32,7 @@
   ],
   "scripts": {
     "build": "tsc -p tsconfig.build.json",
+    "test": "npm run build && node --test",
     "typecheck": "tsc --noEmit",
     "prepublishOnly": "npm run build"
   },