@volter/supercode-frontend-vscode 0.1.40 → 0.1.42

package/README.md

@@ -155,7 +155,8 @@ The permission group offers **Ask for approval** and, on an answerable interacti
 A lifecycle host may start the conversations it creates in **Auto approve**: it records
 `defaultPermission: "autoApprove"` on each such conversation and declares `newChatPermission` for the new
 chat's picker (Model Editor does). A conversation without its own recorded default asks, so one that existed
-before a host began doing this keeps its behaviour. It applies to Claude Code and Codex where the runtime answers approvals
+before a host began doing this keeps its behaviour. A new blank chat does not inherit the previous conversation's permission choice.
+It applies to Claude Code and Codex where the runtime answers approvals
 through Chat, every other harness keeps its own prompts, and a person's explicit pick always
 wins. Only a pick that differs from the default (or replaces an earlier pick) is saved, so the
 default is never stored as a choice (a pick equal to the default follows the default if it later
@@ -403,7 +404,13 @@ the extension manifest beside those bytes with no build step of its own.
 Conversation creation uses `ChatSessionItemController.newChatSessionItemHandler`.
 Permissions use `getChatSessionInputState` and its change/disposal events, with
 per-conversation preferences persisted in workspace state. New untitled chats
-start with Ask; old untitled bindings are read only for backwards compatibility.
+start with the host's new-chat default and offer modes for the selected draft's harness,
+independently of the previous runtime. The paired native host exposes actual
+`initialSessionOptions` separately from its type-wide previous input state, so an
+explicit draft choice survives materialization. Draft input groups refresh after
+agent selection without overwriting the person's choice. Stock hosts without
+that optional context retain their previous-state contract; old untitled bindings
+are read only for backwards compatibility.
 Restored response cancellation interrupts the runtime and waits for acknowledgement.
 Session-list updates consistently prioritize pending input over busy status.
 

package/dist/extension.js

@@ -435,6 +435,16 @@ export async function activate(context) {
     // reports them, and never taken for the conversation the host manages.
     const attachedIds = new Set();
     const managed = () => frontends.all.find((connection) => !attachedIds.has(connection.id));
+    // The host runs one conversation at a time, so a running or waiting turn refuses a new or another chat. This is
+    // the sentence that refusal says to the person; each caller picks the one place it is shown (see its uses).
+    const busyRefusal = (active, action) => {
+        if (!active)
+            return;
+        if (active.session.pendingRequests.size)
+            return `The current chat is waiting for your answer. Answer it or cancel the turn before ${action}.`;
+        if (active.session.descriptor?.turn_state === 'busy')
+            return `A turn is still running in the current chat. Let it finish or cancel it before ${action}.`;
+    };
     // The attached runtime the person is looking at: the Chat names its harness, not the host's.
     let focusedAttached;
     let approvalAvailable = false;
@@ -496,7 +506,7 @@ export async function activate(context) {
         }));
     }
     const currentOptions = (resource) => {
-        const id = resource ? boundId(resource) : controls?.state.activeSession;
+        const id = resource ? boundId(resource) : activeDraft() ? undefined : controls?.state.activeSession;
         return { permissions: id ? approvalFor(id) : permission ?? newChatDefault() };
     };
     const savedConnections = new Map();
@@ -676,9 +686,11 @@ export async function activate(context) {
             return existing.pending;
         const pending = applying.then(async () => {
             const active = managed();
-            if (active && active.id !== id && (active.session.descriptor?.turn_state === 'busy' || active.session.pendingRequests.size))
-                throw new Error('Finish or cancel the active turn before opening another conversation.');
             if (active && active.id !== id) {
+                // Thrown as is: the workbench shows a focus command's error, and a send writes it into the chat.
+                const refusal = busyRefusal(active, 'opening another conversation');
+                if (refusal)
+                    throw new Error(refusal);
                 // Capture the exact saved transcript while its runtime still belongs
                 // to the host. Subsequent background content reads need no activation.
                 await saveManagedHistory(active);
@@ -716,8 +728,12 @@ export async function activate(context) {
         if (!controls)
             throw new Error('This host does not expose harness or model configuration.');
         const active = managed();
-        if (active?.session.descriptor?.turn_state === 'busy' || active?.session.pendingRequests.size)
-            throw new Error('Finish or cancel the current turn before starting a new chat.');
+        // A send from a blank chat lands here, and the workbench does not show its error: say it, then refuse.
+        const refusal = busyRefusal(active, 'starting a new chat');
+        if (refusal) {
+            void vscode.window.showWarningMessage(refusal);
+            throw new Error(refusal);
+        }
         if (active)
             await saveManagedHistory(active);
         const state = await controls.select(selection);
@@ -728,9 +744,13 @@ export async function activate(context) {
     const openNewChat = async () => {
         if (!controls || !granted.has('chatSessionsProvider'))
             throw new Error('This host does not expose native conversation creation.');
-        const active = managed();
-        if (active?.session.descriptor?.turn_state === 'busy' || active?.session.pendingRequests.size)
-            throw new Error('Finish or cancel the current turn before starting a new chat.');
+        // Its callers (the new-chat pickers) swallow the cancellation, so the warning is the one message shown. An error
+        // would reach the person as a bare "Canceled" or not at all through the contributed-command boundary.
+        const refusal = busyRefusal(managed(), 'starting a new chat');
+        if (refusal) {
+            void vscode.window.showWarningMessage(refusal);
+            throw new vscode.CancellationError();
+        }
         // Code-OSS materializes this untitled session on first send, awaiting our
         // newChatSessionItemHandler before accepting/clearing the draft. Never boot
         // beside the old widget and reveal another resource after it accepts input.
@@ -791,20 +811,23 @@ export async function activate(context) {
     const chooseNewChat = async (pick) => {
         if (pendingDraft?.choosing)
             throw new Error('Finish choosing an agent before starting another chat.');
-        // Open before the picker: a send can beat its result across the extension-
-        // host boundary. Materialization awaits this exact draft's choice too.
-        const draft = await openNewChat();
-        const picking = (async () => {
-            const choice = await pick();
-            if (activeDraft() !== draft)
-                throw new vscode.CancellationError();
-            draft.choice = choice;
-            controlsChanged.fire();
-            if (choice.kind !== 'agent')
-                throw new vscode.CancellationError();
-        })();
-        draft.choosing = picking;
+        let draft;
+        let picking;
         try {
+            // Open before the picker: a send can beat its result across the extension-
+            // host boundary. Materialization awaits this exact draft's choice too.
+            // Inside the try: a busy refusal is a cancellation already explained.
+            const opened = draft = await openNewChat();
+            picking = (async () => {
+                const choice = await pick();
+                if (activeDraft() !== opened)
+                    throw new vscode.CancellationError();
+                opened.choice = choice;
+                controlsChanged.fire();
+                if (choice.kind !== 'agent')
+                    throw new vscode.CancellationError();
+            })();
+            opened.choosing = picking;
             await picking;
         }
         catch (error) {
@@ -812,9 +835,9 @@ export async function activate(context) {
                 throw error;
         }
         finally {
-            if (draft.choosing === picking)
+            if (draft && draft.choosing === picking)
                 draft.choosing = undefined;
-            if (draft.choice?.kind === 'signIn') {
+            if (draft?.choice?.kind === 'signIn') {
                 void refreshSetup().catch(error => log.warn(`Could not refresh sign-in: ${String(error)}`)).finally(pollSetup);
             }
         }
@@ -829,11 +852,16 @@ export async function activate(context) {
             if (controls?.state.setup && !controls.state.setup.ready)
                 return [];
             const id = resource ? boundId(resource) : undefined;
-            const active = resource ? (id ? frontends.get(id) : undefined) : managed();
-            const canApprove = active?.session.descriptor?.actions.respond && active.session.permissions.has('approve');
+            const active = id ? frontends.get(id) : undefined;
+            // A new draft's harness, not the previous runtime, owns its offered modes.
+            const harness = resource ? controls?.state.sessions.find(session => session.id === id)?.selection.harness : draftSelection()?.harness;
+            const canApprove = resource ? (active ? canAnswer(active) : ['claude-code', 'codex'].includes(harness ?? ''))
+                : (harness ? ['claude-code', 'codex'].includes(harness) : newChatDefault() === 'autoApprove' || ['claude-code', 'codex'].includes(controls?.state.selection.harness ?? ''));
+            // An unbound input is a new chat, not the active conversation left behind it.
+            const selectedPermission = resource ? currentOptions(resource).permissions : permission ?? newChatDefault();
             const permissionItems = [{ id: 'ask', name: canApprove ? 'Ask for approval' : 'Harness permissions', description: canApprove ? 'Show each approval requested by the harness.' : 'Approval controls are unavailable for this harness.' },
                 ...((canApprove || (!active && resource && currentOptions(resource).permissions === 'autoApprove')) ? [{ id: 'autoApprove', name: 'Auto approve', description: 'Approve the requests the harness would ask you about, including requests to run outside its sandbox. Its deny rules still apply.' }] : [])];
-            const groups = [{ id: 'permissions', name: 'Permissions', kind: 'permissions', items: permissionItems, selected: permissionItems.find(i => i.id === currentOptions(resource).permissions) }];
+            const groups = [{ id: 'permissions', name: 'Permissions', kind: 'permissions', items: permissionItems, selected: permissionItems.find(i => i.id === selectedPermission) ?? permissionItems[0] }];
             const selected = controls?.state.sessions.find(session => session.id === id);
             if (selected && ['claude-code', 'codex'].includes(selected.selection.harness)) {
                 // Native session-option commands remain reachable for nondelegating providers,
@@ -878,7 +906,8 @@ export async function activate(context) {
             applying = pending.then(() => undefined, () => undefined);
             const connection = await pending;
             // The picker shows the host's default until someone changes it; only a change is the person's choice.
-            const picked = turnPermission(inputState.groups.find(group => group.id === 'permissions')?.selected?.id);
+            const selected = inputState.groups.find(group => group.id === 'permissions')?.selected?.id;
+            const picked = selected === 'ask' || selected === 'autoApprove' ? selected : permission ?? newChatDefault();
             choosePermission(connection.id, picked, permission !== undefined ? null : newChatDefault());
             permission = undefined;
             if (pendingDraft === draft) {
@@ -967,17 +996,20 @@ export async function activate(context) {
             return { kind: 'agent', selection: { harness: harness.id, model: '', effort: '' } };
         });
     }));
-    context.subscriptions.push(vscode.commands.registerCommand('supercode.frontend.configureModel', async (resource, sessionId) => {
+    context.subscriptions.push(vscode.commands.registerCommand('supercode.frontend.configureModel', async (resource, _sessionId) => {
         if (!controls || focusedAttached)
             throw new Error('Model settings require a host-managed conversation.');
         await controls.refresh();
-        // The native session-options picker prepends its exact resource to command arguments.
-        const bound = resource ? boundId(resource) : undefined;
-        if (resource && (!bound || (sessionId && bound !== sessionId)))
-            throw new Error('Model settings have no matching conversation binding.');
+        // The native session-options picker prepends its widget's resource to command arguments, so the resource is the
+        // conversation the person clicked from. `sessionId` is not used: the workbench keeps one option-group list per
+        // session type, last written by whichever input state updated, so any chat's dropdown can carry another
+        // conversation's id. A resource with no binding (a blank draft) is no conversation; only the command palette,
+        // which passes no resource, falls back to the active one. Either way the choice opens a new chat.
+        const bound = resource ? boundId(resource) : controls.state.activeSession ?? undefined;
         // The conversation it was opened from sets the agent and the current values; with none open (a fresh project,
-        // the command palette) the person picks the agent first. Either way the choice opens a new chat.
-        const selected = controls.state.sessions.find(session => session.id === (bound ?? controls.state.activeSession));
+        // the command palette) the person picks the agent first. A running turn refuses the new chat before any picker
+        // opens, and says so (`openNewChat`).
+        const selected = controls.state.sessions.find(session => session.id === bound);
         // An agent that still needs sign-in is set up from the agent picker first, as there.
         const needsSignIn = new Set((controls.state.setup?.actions ?? []).filter(action => action.kind === 'login').map(action => action.harness));
         const configurable = controls.state.harnesses.filter(harness => CONFIGURABLE_HARNESSES.includes(harness.id) && !needsSignIn.has(harness.id));
@@ -1650,19 +1682,30 @@ function registerSessions(context, frontends, participant, granted, handler, his
     context.subscriptions.push(controller, frontends.onDidChange(refreshItems));
     if (controls) {
         const states = new Set();
-        controller.getChatSessionInputState = (resource, { previousInputState }) => {
-            const previous = previousInputState?.groups.find(group => group.id === 'permissions')?.selected?.id;
-            // Saved conversation preferences survive extension-host reloads. Native input state
-            // owns live changes; a blank conversation always starts with its own Ask default.
-            const groups = controls.options(resource).map(group => ({ ...group,
-                selected: group.items.find(item => item.id === (resource ? group.selected?.id : previous ?? group.selected?.id)) ?? group.selected,
-            }));
-            const state = controller.createChatSessionInputState(groups);
+        const draftPermissions = new WeakMap();
+        const applyOptions = (groups, options) => groups.map(group => {
+            const value = options.find(option => option.optionId === group.id)?.value;
+            const selected = group.items.find(item => item.id === value);
+            return selected ? { ...group, selected } : group;
+        });
+        controller.getChatSessionInputState = (resource, inputContext) => {
+            const groups = controls.options(resource);
+            // Native request options belong to this draft. previousInputState may instead
+            // be the type-wide last chat, so the paired host exposes them separately.
+            // Stock hosts without that context keep their existing previous-state contract.
+            const initial = inputContext.initialSessionOptions ?? inputContext.previousInputState?.groups.flatMap(group => group.selected ? [{ optionId: group.id, value: group.selected.id }] : []) ?? [];
+            const state = controller.createChatSessionInputState(resource ? groups : applyOptions(groups, initial));
+            const initialPermission = initial.find(option => option.optionId === 'permissions')?.value;
+            if (!resource && initialPermission)
+                draftPermissions.set(state, initialPermission);
             states.add(state);
             const changed = state.onDidChange(() => {
                 const target = state.sessionResource ?? resource;
+                const value = state.groups.find(group => group.id === 'permissions')?.selected?.id;
                 if (target)
-                    controls.setPermission(target, state.groups.find(group => group.id === 'permissions')?.selected?.id);
+                    controls.setPermission(target, value);
+                else if (value)
+                    draftPermissions.set(state, value);
             });
             const disposed = state.onDidDispose(() => { states.delete(state); changed.dispose(); disposed.dispose(); });
             return state;
@@ -1675,8 +1718,9 @@ function registerSessions(context, frontends, participant, granted, handler, his
         context.subscriptions.push(controls.changed(() => {
             refreshItems();
             for (const state of states) {
-                if (state.sessionResource)
-                    state.groups = controls.options(state.sessionResource);
+                const groups = controls.options(state.sessionResource);
+                const value = draftPermissions.get(state);
+                state.groups = state.sessionResource || !value ? groups : applyOptions(groups, [{ optionId: 'permissions', value }]);
             }
         }));
     }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@volter/supercode-frontend-vscode",
   "displayName": "Volter Harness",
-  "version": "0.1.40",
+  "version": "0.1.42",
   "type": "module",
   "description": "VS Code chat-view frontend for an existing Volter Harness SDK runtime",
   "publisher": "volter-ai-dev",