test(system-prompt): update assertions to match rewritten agent prompt

## Intent

Adapt unit tests that were asserting against the old five-sentence prompt to
reflect the new structured, meta-level system prompt.

## Key decisions

- system-prompt-resource.test.ts: replaced stale string checks ('You are an AI
  assistant that helps users develop software features', 'development plan') with
  assertions on the actual new content ('workflow-driven agent', 'plan_file_path',
  'clarifying question', etc.); removed the 1000-char upper bound since the prompt
  is now intentionally more comprehensive; third test now verifies all five named
  sections are present (Core loop, Before acting, Scope discipline, Subagent
  delegation, Task management)
- resume-workflow.test.ts: removed the 1000-char upper bound (prompt is now ~2250
  chars); updated the content assertion from 'workflows server' to
  'workflow-driven agent' and 'whats_next()'

Author: mrsimpson

packages/mcp-server/test/unit/resume-workflow.test.ts

@@ -60,10 +60,9 @@ describe('resume_workflow tool', () => {
     const result = await server.handleResumeWorkflow({});
 
     expect(result.system_prompt).toBeTypeOf('string');
-    // Streamlined prompt is ~400-600 chars (was 2000+ before)
-    expect(result.system_prompt.length).toBeGreaterThan(200);
-    expect(result.system_prompt.length).toBeLessThan(1000);
-    expect(result.system_prompt).toContain('workflows server');
+    expect(result.system_prompt.length).toBeGreaterThan(500);
+    expect(result.system_prompt).toContain('workflow-driven agent');
+    expect(result.system_prompt).toContain('whats_next()');
   });
 
   it('should exclude system prompt when requested', async () => {

packages/mcp-server/test/unit/system-prompt-resource.test.ts

@@ -30,18 +30,15 @@ describe('System Prompt Resource', () => {
     expect(data.text).toBeDefined();
     expect(typeof data.text).toBe('string');
 
-    // Verify content contains expected system prompt elements (streamlined version)
-    expect(data.text).toContain(
-      'You are an AI assistant that helps users develop software features'
-    );
-    expect(data.text).toContain('workflows server');
+    // Verify content contains expected system prompt elements
+    expect(data.text).toContain('You are a structured, workflow-driven agent');
     expect(data.text).toContain('whats_next()');
     expect(data.text).toContain('instructions');
-    expect(data.text).toContain('development plan');
+    expect(data.text).toContain('plan_file_path');
 
-    // Verify it's concise but not empty (streamlined prompt is ~400 chars)
-    expect(data.text.length).toBeGreaterThan(200);
-    expect(data.text.length).toBeLessThan(1000);
+    // Prompt is more comprehensive now — verify it's substantive but not unbounded
+    expect(data.text.length).toBeGreaterThan(500);
+    expect(data.text.length).toBeLessThan(5000);
   });
 
   it('should be workflow-independent and consistent', async () => {
@@ -70,14 +67,15 @@ describe('System Prompt Resource', () => {
     expect(result1.data!.text).toBe(result2.data!.text);
     expect(result2.data!.text).toBe(result3.data!.text);
 
-    // Verify the prompt contains standard elements (streamlined version)
-    expect(result1.data!.text).toContain('You are an AI assistant');
+    // Verify the prompt contains standard elements
+    expect(result1.data!.text).toContain(
+      'You are a structured, workflow-driven agent'
+    );
     expect(result1.data!.text).toContain('whats_next()');
-    expect(result1.data!.text).toContain('development');
     expect(result1.data!.text).toContain('instructions');
   });
 
-  it('should use streamlined system prompt', async () => {
+  it('should contain all major sections of the meta-level agent prompt', async () => {
     const handler = new SystemPromptResourceHandler();
 
     const result = await handler.handle(
@@ -87,16 +85,28 @@ describe('System Prompt Resource', () => {
 
     expect(result.success).toBe(true);
 
-    // The streamlined system prompt should be concise and focused
-    // It relies on tool responses for detailed phase instructions
-    expect(result.data!.text).toContain(
-      'You are an AI assistant that helps users develop software features'
-    );
-    expect(result.data!.text).toContain('whats_next()');
-    expect(result.data!.text).toContain('instructions');
-    expect(result.data!.text).toContain('development plan');
+    const text = result.data!.text;
+
+    // Core loop section
+    expect(text).toContain('## Core loop');
+    expect(text).toContain('whats_next()');
+    expect(text).toContain('plan_file_path');
+
+    // Before acting section
+    expect(text).toContain('## Before acting');
+    expect(text).toContain('clarifying question');
+
+    // Scope discipline section
+    expect(text).toContain('## Scope discipline');
+    expect(text).toContain('proceed_to_phase');
+
+    // Subagent delegation section
+    expect(text).toContain('## Subagent delegation');
+    expect(text).toContain('Capability hint');
+    expect(text).toContain('thinking-specialized subagent');
 
-    // Streamlined prompt should be concise (~400 chars vs old 2000+)
-    expect(result.data!.text.length).toBeLessThan(1000);
+    // Task management section
+    expect(text).toContain('## Task management');
+    expect(text).toContain('Do not use your own task management tools.');
   });
 });