@qduc/term2 0.32.0 → 0.32.2

package/dist/prompts/steering-notice.test.ts

@@ -1,36 +0,0 @@
-import { expect, it } from 'vitest';
-import { stripSteeringNotice, withSteeringNotice } from './steering-notice.js';
-
-it('steering distinguishes side questions, added constraints, replacement, and queued tasks', () => {
-  const text = withSteeringNotice('How far along are you?');
-  expect(text).toContain('before your next tool call');
-  expect(text).toContain('continue the pending work in the same response');
-  expect(text).toContain('Incorporate corrections and new constraints');
-  expect(text).toContain('pauses, cancels, or replaces');
-  expect(text).toContain('unrelated task');
-  expect(text).toContain('How far along are you?');
-});
-
-it.each(['How far along are you?', 'Keep the API compatible.\nAlso update the docs.', 'Stop.'])(
-  'steering round-trips user text: %s',
-  (text) => expect(stripSteeringNotice(withSteeringNotice(text))).toBe(text),
-);
-
-it('still strips the original notice from saved conversation messages', () => {
-  const legacy = `[Steering message: the user sent this while you were working, so it arrives mid-turn rather than as a new turn.
-- If it changes what you should be doing, change direction now and drop the superseded plan.
-- If it does not bear on the work in progress, treat it as the next task rather than an interruption: acknowledge it, finish what you are doing, and handle it after.]`;
-  expect(stripSteeringNotice(`${legacy}\n\nKeep the API compatible.`)).toBe('Keep the API compatible.');
-});
-
-it('leaves ordinary user messages unchanged', () => {
-  expect(stripSteeringNotice('No steering notice here.')).toBe('No steering notice here.');
-});
-
-it('strips the previous status-aware notice from saved conversation messages', () => {
-  const previous = `[Steering message: the user sent this while you were working, so it arrives mid-turn rather than as a new turn.
-- For status or side questions, answer briefly, then resume the active task.
-- Incorporate corrections and new constraints while preserving the active objective, unless the user explicitly pauses, cancels, or replaces it. Drop only superseded work.
-- For an unrelated task, acknowledge it, finish the active task, then handle it unless the user explicitly changes priority.]`;
-  expect(stripSteeringNotice(`${previous}\n\nWhat is done so far?`)).toBe('What is done so far?');
-});

package/dist/prompts/subagent-delegation.test.ts

@@ -1,112 +0,0 @@
-import { it, expect } from 'vitest';
-import { getSubagentDelegationAddendum } from './subagent-delegation.js';
-import { getAsyncSubagentDelegationAddendum } from './async-subagent-delegation.js';
-
-it('getSubagentDelegationAddendum returns non-empty delegation guidance', () => {
-  const result = getSubagentDelegationAddendum();
-
-  expect(typeof result).toBe('string');
-  expect(result.length > 0).toBe(true);
-  expect(result.includes('Delegating to subagents')).toBe(true);
-  expect(result.includes('run_subagent')).toBe(true);
-  expect(result).toContain('agent.run({ spec, worktree? })');
-});
-
-it('getSubagentDelegationAddendum does not encourage orchestrators to keep working directly', () => {
-  const result = getSubagentDelegationAddendum({ orchestratorMode: true });
-
-  expect(result.includes('Orchestrator mode')).toBe(true);
-  expect(result).toContain('Delegate when it provides meaningful leverage');
-  expect(result).not.toContain('directly inspect, edit, run commands, and test small or clear work');
-  expect(result).not.toContain('Delegate workspace inspection');
-});
-
-it('async delegation guidance does not tell the parent to find other work before yielding', () => {
-  const result = getAsyncSubagentDelegationAddendum({ orchestratorMode: true });
-
-  expect(result).not.toContain('other useful work to do first');
-  expect(result.toLowerCase()).toContain('end the current turn and wait for the completion notification');
-});
-
-it('getSubagentDelegationAddendum includes self-service fallback when orchestratorMode is false', () => {
-  const result = getSubagentDelegationAddendum({ orchestratorMode: false });
-
-  expect(result.includes('just do it yourself')).toBe(true);
-});
-
-it('keeps diagnosis and recommendations with the parent when delegating evidence collection', () => {
-  const result = getSubagentDelegationAddendum({ orchestratorMode: false });
-
-  expect(result).toContain('Explorer is an evidence collector, not a reasoning delegate');
-  expect(result).toContain(
-    "Do not pass the user's entire investigation, diagnosis, review, or planning task to explorer",
-  );
-  expect(result).toContain('You retain responsibility for hypotheses, causal analysis, judgments, and recommendations');
-});
-
-it('orchestrator mode protects outcome ownership and safe coordination', () => {
-  const result = getSubagentDelegationAddendum({ orchestratorMode: true });
-  expect(result).toContain('Delegation transfers execution, never outcome ownership');
-  expect(result).toContain('Avoid concurrent overlapping edits');
-  expect(result).not.toContain('Coordination checklist');
-  expect(result).not.toContain('First delegate investigation');
-});
-
-it('includes task framing guidance about autonomous workers', () => {
-  const result = getSubagentDelegationAddendum();
-  expect(result.includes('Task framing')).toBe(true);
-  expect(result.includes('autonomous agents')).toBe(true);
-  expect(result.includes('goal, relevant context, and constraints')).toBe(true);
-  expect(result.includes('not implementation steps')).toBe(true);
-});
-
-it('requires task-specific context without repeating automatically supplied context', () => {
-  const result = getSubagentDelegationAddendum();
-
-  expect(result).toContain(
-    'Do not repeat automatically supplied context: role instructions, generic tool guidance, worktree hygiene, environment metadata, root `AGENTS.md`, or skills catalog.',
-  );
-  expect(result).toContain(
-    'objective, task-specific scope, non-discoverable parent findings or decisions, constraints',
-  );
-  expect(result).toContain('deliverable or acceptance criteria, and validation when applicable');
-  expect(result).toContain('The subagent does not see your conversation or reasoning');
-});
-
-it('retains worker autonomy without mandatory pre-delegation ceremony', () => {
-  const result = getSubagentDelegationAddendum({ orchestratorMode: true });
-  expect(result.includes('one cohesive unit that can be understood, implemented, and verified')).toBe(true);
-  expect(result.includes('The orchestrator decides where execution units begin and end')).toBe(true);
-  expect(result).not.toContain('Before any `run_subagent` call, plan silently');
-});
-
-it('routes prior-session history digging to the librarian when session tools exist', () => {
-  const result = getSubagentDelegationAddendum({ memoryEnabled: true, sessionsEnabled: true });
-  expect(result).toContain('from prior sessions or several memories → `librarian`');
-  expect(result).toContain('- `librarian`:');
-});
-
-it('routes memory-only history to the librarian without promising transcripts', () => {
-  const result = getSubagentDelegationAddendum({ memoryEnabled: true, sessionsEnabled: false });
-  expect(result).toContain('several memories, or memory maintenance → `librarian`');
-  expect(result).not.toContain('prior-session transcripts');
-});
-
-it('does not advertise the librarian when persistent memory is disabled', () => {
-  const result = getSubagentDelegationAddendum({ memoryEnabled: false });
-  expect(result.includes('librarian')).toBe(false);
-});
-
-it('async control guidance distinguishes steering, cancellation, answers, and fresh-turn limits honestly', () => {
-  const result = getAsyncSubagentDelegationAddendum({ orchestratorMode: true, controlsEnabled: true });
-
-  expect(result).toContain('`tools.send_message(...)`');
-  expect(result).toContain('`tools.cancel_run(...)`');
-  expect(result).toContain('name or runId');
-  expect(result).toContain('`reply_to`');
-  expect(result).toContain('fresh session turn');
-  expect(result).toContain('maximum of three continuation segments');
-  expect(result).toContain('not SDK live injection');
-  expect(result).toContain('single point of contact');
-  expect(result).toContain('do NOT call `tools.get_subagent_result(...)` immediately');
-});

package/dist/prompts/subagents/worker-prompt.test.ts

@@ -1,54 +0,0 @@
-import fs from 'node:fs';
-import path from 'node:path';
-import { it, expect } from 'vitest';
-
-const workerPrompt = fs.readFileSync(path.join(import.meta.dirname, 'worker.md'), 'utf-8');
-const explorerPrompt = fs.readFileSync(path.join(import.meta.dirname, 'explorer.md'), 'utf-8');
-
-it('worker prompt scales validation to behavioral impact while retaining required checks', () => {
-  const lower = workerPrompt.toLowerCase();
-  expect(lower).toContain('validation command');
-  expect(lower).toContain('project-required checks');
-  expect(lower).toContain('for inert changes such as docs or comments');
-  expect(lower).toContain('diff review is sufficient');
-  expect(lower).toContain('why no command was needed');
-  expect(lower).not.toContain('you still need to run a validation command');
-});
-
-it('worker prompt notes that validation and diff stat are auto-captured', () => {
-  const lower = workerPrompt.toLowerCase();
-  expect(lower).toContain('automatically captures');
-  expect(lower).toContain('diff stat');
-  expect(lower).toContain('do not need to paste the full output');
-});
-
-it('worker report distinguishes recorded checks before and after the edit', () => {
-  expect(workerPrompt).toContain('Before-edit validation:');
-  expect(workerPrompt).toContain('After-edit validation:');
-  expect(workerPrompt).toContain('tool-result history');
-  expect(workerPrompt).toContain('even if they were not required');
-});
-
-it('worker prompt mentions that shell-driven edits may not appear in diff stat', () => {
-  const lower = workerPrompt.toLowerCase();
-  expect(lower).toContain('shell-driven edits');
-  expect(lower).toContain('may not appear');
-});
-
-it('execution role prompts reserve ask_orchestrator for genuine blockers and never direct user contact', () => {
-  for (const prompt of [workerPrompt, explorerPrompt]) {
-    const lower = prompt.toLowerCase();
-    expect(lower).toContain('ask_orchestrator');
-    expect(lower).toContain('genuine blocker');
-    expect(lower).toContain('decision needed');
-    expect(lower).toContain('continue after');
-    expect(lower).toContain('do not contact the user');
-  }
-});
-
-it('explorer prompt names web tools directly instead of routing through unregistered run_code', () => {
-  expect(explorerPrompt).toContain('Use `web_search` to find relevant external documentation');
-  expect(explorerPrompt).toContain('Use `web_fetch` to retrieve the content of specific URLs');
-  expect(explorerPrompt).not.toContain('Inside `run_code`, use `tools.web_search`');
-  expect(explorerPrompt).not.toContain('Inside `run_code`, use `tools.web_fetch`');
-});

package/dist/prompts/unified-gpt-prompt.test.ts

@@ -1,53 +0,0 @@
-import { readFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { describe, expect, it } from 'vitest';
-import { buildPromptSpec } from './prompt-constructor.js';
-import { resolveProfile } from '../services/profiles/index.js';
-
-const standard = resolveProfile('builtin:standard');
-
-describe('unified main-agent GPT prompt', () => {
-  it.each([
-    'gpt-5.2',
-    'gpt-5.3-codex',
-    'gpt-5.4',
-    'gpt-5.4-mini',
-    'gpt-5.5',
-    'gpt-5.6-luna',
-    'gpt-6-astra',
-    'gpt-6-sol',
-    'gpt-6-luna',
-    'gpt-6-luna-codex',
-  ])('selects one base without GPT-only fragments for %s', (model) => {
-    const spec = buildPromptSpec({ model, profile: standard });
-    expect(spec.basePromptFile).toBe('gpt.md');
-    expect(spec.fragmentFiles).not.toContain('fragments/gpt-6.md');
-    expect(spec.fragmentFiles).not.toContain('fragments/skill-instruction-conflicts.md');
-  });
-
-  it.each(['gpt-4o', 'glm-5.3-flash', 'deepseek-flash', 'unknown-model'])(
-    'keeps the simpler generic base for %s',
-    (model) => {
-      const spec = buildPromptSpec({ model, profile: standard });
-      expect(spec.basePromptFile).toBe('simple_v4.md');
-      expect(spec.fragmentFiles).toContain('approval-model.md');
-    },
-  );
-
-  it('keeps the action, approval, verification, and skill-conflict rules in the base', () => {
-    const text = readFileSync(join(import.meta.dirname, 'gpt.md'), 'utf8');
-    expect(text).toContain('Default to action');
-    expect(text).toContain('approval layer');
-    expect(text).toContain('Run the checks appropriate to the change');
-    expect(text).toContain('Explicit user instructions override conflicting skill instructions');
-    expect(buildPromptSpec({ model: 'gpt-6-luna', profile: standard }).fragmentFiles).toContain('approval-model.md');
-  });
-
-  it('leaves non-GPT and lite profile routing unchanged', () => {
-    expect(buildPromptSpec({ model: 'claude-sonnet-4', profile: standard }).basePromptFile).toBe('anthropic.md');
-    expect(buildPromptSpec({ model: 'kimi-k2', profile: standard }).basePromptFile).toBe('kimi.md');
-    const lite = buildPromptSpec({ model: 'gpt-6-luna', profile: resolveProfile('builtin:lite') });
-    expect(lite.basePromptContent).toBeDefined();
-    expect(lite.fragmentFiles).toEqual([]);
-  });
-});