@qduc/term2 0.31.1 → 0.32.1

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([]);
-  });
-});