@auto-engineer/server-implementer 1.123.0 → 1.125.0

package/package.json

@@ -18,8 +18,8 @@
     "debug": "^4.3.4",
     "fast-glob": "^3.3.3",
     "vite": "^5.4.1",
-    "@auto-engineer/model-factory": "1.123.0",
-    "@auto-engineer/message-bus": "1.123.0"
+    "@auto-engineer/model-factory": "1.125.0",
+    "@auto-engineer/message-bus": "1.125.0"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",
@@ -29,9 +29,9 @@
     "glob": "^11.0.3",
     "tsx": "^4.20.3",
     "typescript": "^5.8.3",
-    "@auto-engineer/cli": "1.123.0"
+    "@auto-engineer/cli": "1.125.0"
   },
-  "version": "1.123.0",
+  "version": "1.125.0",
   "scripts": {
     "build": "tsc && tsx ../../scripts/fix-esm-imports.ts",
     "test": "vitest run --reporter=dot",

package/src/agent/runSlice.ts

@@ -8,6 +8,7 @@ import { SYSTEM_PROMPT } from '../prompts/systemPrompt';
 import { buildContextSections } from '../utils/buildContextSections';
 import { buildShadowWarning } from '../utils/detectImportedTypeShadowing';
 import { extractCodeBlock } from '../utils/extractCodeBlock';
+import { findFilesToImplement } from '../utils/findFilesToImplement';
 import { loadContextFiles } from '../utils/loadContextFiles';
 import { runTests } from './runTests';
 
@@ -36,7 +37,7 @@ export async function runSlice(sliceDir: string, flow: string): Promise<void> {
   const sliceName = path.basename(sliceDir);
   console.log(`✏️ Implementing slice: ${sliceName} for flow: ${flow}`);
   const contextFiles = await loadContextFiles(sliceDir);
-  const filesToImplement = findFilesToImplement(contextFiles);
+  const filesToImplement = findFilesToImplement(contextFiles, needsImplementation);
   for (const [targetFile] of filesToImplement) {
     await implementFileFromAI(sliceDir, targetFile, contextFiles);
   }
@@ -103,10 +104,8 @@ async function retryFailedFiles(
   return result;
 }
 
-function findFilesToImplement(contextFiles: Record<string, string>) {
-  return Object.entries(contextFiles).filter(
-    ([, content]) => content.includes('TODO:') || content.includes('IMPLEMENTATION INSTRUCTIONS'),
-  );
+function needsImplementation(content: string): boolean {
+  return content.includes('TODO:') || content.includes('IMPLEMENTATION INSTRUCTIONS');
 }
 
 function buildInitialPrompt(targetFile: string, context: Record<string, string>): string {

package/src/commands/implement-slice.ts

@@ -8,6 +8,8 @@ import createDebug from 'debug';
 import { SYSTEM_PROMPT } from '../prompts/systemPrompt';
 import { buildContextSections } from '../utils/buildContextSections';
 import { buildShadowWarning } from '../utils/detectImportedTypeShadowing';
+import { extractCodeBlock } from '../utils/extractCodeBlock';
+import { findFilesToImplement } from '../utils/findFilesToImplement';
 import { loadContextFiles } from '../utils/loadContextFiles';
 
 const debug = createDebug('auto:server-implementer:slice');
@@ -95,14 +97,6 @@ export const commandHandler = defineCommandHandler<
   },
 });
 
-// Helper function to extract code block from AI response
-function extractCodeBlock(text: string): string {
-  return text
-    .replace(/```(?:ts|typescript)?/g, '')
-    .replace(/```/g, '')
-    .trim();
-}
-
 const IMPLEMENTATION_MARKER = '// @auto-implement';
 
 function hasImplementationMarker(content: string): boolean {
@@ -122,10 +116,6 @@ function needsImplementation(content: string): boolean {
   );
 }
 
-function findFilesToImplement(contextFiles: Record<string, string>): Array<[string, string]> {
-  return Object.entries(contextFiles).filter(([, content]) => hasImplementationMarker(content));
-}
-
 function buildInitialPrompt(targetFile: string, context: Record<string, string>): string {
   return `
 ${SYSTEM_PROMPT}
@@ -228,7 +218,7 @@ async function implementSlice(
 
     await addMarkersToFiles(slicePath, contextFiles);
 
-    const filesToImplement = findFilesToImplement(contextFiles);
+    const filesToImplement = findFilesToImplement(contextFiles, hasImplementationMarker);
     debugProcess(`Found ${filesToImplement.length} files with markers to implement`);
 
     if (filesToImplement.length === 0) {

package/src/prompts/systemPrompt.ts

@@ -13,6 +13,8 @@ You are a software engineer implementing @auto-implement files in a sliced event
 - Complete the logic in the target file following embedded instructions
 - Return the entire updated file, production-ready
 - Remove all TODO/instruction comments after implementing
+- Scaffolded files contain fixed generated structure.
+  Only change code inside the implementation area indicated by TODOs or instruction comments.
 
 ## 3. EMMETT FRAMEWORK PATTERNS
 
@@ -42,6 +44,9 @@ Projection evolve(document, event) returns updated document.
 - Never create imports to modules not shown in the provided context.
 - If you need a type that doesn't exist in any provided file,
   use inline types or primitive values instead.
+- Preserve existing import statements exactly — do not remove, rename, replace,
+  or reorganize them unless a scaffold instruction explicitly requires a change.
+- Use the node: protocol for Node.js built-in modules (e.g., node:crypto, node:path).
 
 ## 5. TEST SPECIFICATIONS GUIDANCE
 
@@ -59,7 +64,6 @@ Projection evolve(document, event) returns updated document.
 - Immutability: functional patterns, never mutate state
 - Include ALL required fields in object literals
 - Preserve index signatures ([key: string]: unknown)
-- Do not remove existing imports still referenced
 - When mapping objects to a typed array, include ALL required fields in each object literal
 
 ## 7. SLICE CONVENTIONS
@@ -76,6 +80,9 @@ Projection evolve(document, event) returns updated document.
 - String literals for enum-typed fields
 - Hardcoding values that should be computed from inputs
 - Reproducing literal values from test examples
+- Type assertions and escape hatches such as \`as any\`, \`as unknown as\`, or broad casts to silence errors
+- Changing scaffolded framework calls or helper names unless explicitly instructed by the scaffold
+- Importing types or modules not directly referenced in the implementation code
 
 ## 9. OUTPUT
 

package/src/utils/findFilesToImplement.specs.ts

@@ -0,0 +1,90 @@
+import { describe, expect, it } from 'vitest';
+import { findFilesToImplement } from './findFilesToImplement';
+
+describe('findFilesToImplement', () => {
+  const hasMarker = (content: string) => content.includes('TODO');
+
+  it('filters files by predicate', () => {
+    const files = {
+      'state.ts': 'TODO: implement',
+      'evolve.ts': 'export const evolve = () => {}',
+      'decide.ts': 'TODO: implement',
+    };
+
+    const result = findFilesToImplement(files, hasMarker);
+
+    expect(result).toEqual([
+      ['state.ts', 'TODO: implement'],
+      ['decide.ts', 'TODO: implement'],
+    ]);
+  });
+
+  it('sorts priority files in correct order', () => {
+    const files = {
+      'decide.ts': 'TODO',
+      'state.ts': 'TODO',
+      'evolve.ts': 'TODO',
+      'events.ts': 'TODO',
+      'commands.ts': 'TODO',
+    };
+
+    const result = findFilesToImplement(files, hasMarker);
+
+    expect(result).toEqual([
+      ['state.ts', 'TODO'],
+      ['events.ts', 'TODO'],
+      ['commands.ts', 'TODO'],
+      ['evolve.ts', 'TODO'],
+      ['decide.ts', 'TODO'],
+    ]);
+  });
+
+  it('sorts non-priority files after priority files in alphabetical order', () => {
+    const files = {
+      'zebra.ts': 'TODO',
+      'decide.ts': 'TODO',
+      'alpha.ts': 'TODO',
+    };
+
+    const result = findFilesToImplement(files, hasMarker);
+
+    expect(result).toEqual([
+      ['decide.ts', 'TODO'],
+      ['alpha.ts', 'TODO'],
+      ['zebra.ts', 'TODO'],
+    ]);
+  });
+
+  it('sorts non-priority files alphabetically among themselves', () => {
+    const files = {
+      'zebra.ts': 'TODO',
+      'middleware.ts': 'TODO',
+      'alpha.ts': 'TODO',
+    };
+
+    const result = findFilesToImplement(files, hasMarker);
+
+    expect(result).toEqual([
+      ['alpha.ts', 'TODO'],
+      ['middleware.ts', 'TODO'],
+      ['zebra.ts', 'TODO'],
+    ]);
+  });
+
+  it('returns empty array for empty input', () => {
+    const result = findFilesToImplement({}, hasMarker);
+
+    expect(result).toEqual([]);
+  });
+
+  it('returns empty array when no files match predicate', () => {
+    const files = {
+      'state.ts': 'export type State = {}',
+      'evolve.ts': 'export const evolve = () => {}',
+    };
+
+    const result = findFilesToImplement(files, hasMarker);
+
+    expect(result).toEqual([]);
+  });
+});

package/src/utils/findFilesToImplement.ts

@@ -0,0 +1,16 @@
+const IMPLEMENTATION_PRIORITY = ['state.ts', 'events.ts', 'commands.ts', 'evolve.ts', 'decide.ts'];
+
+export function findFilesToImplement(
+  contextFiles: Record<string, string>,
+  markerPredicate: (content: string) => boolean,
+): Array<[string, string]> {
+  const files = Object.entries(contextFiles).filter(([, content]) => markerPredicate(content));
+  return files.sort(([a], [b]) => {
+    const ai = IMPLEMENTATION_PRIORITY.indexOf(a);
+    const bi = IMPLEMENTATION_PRIORITY.indexOf(b);
+    const ap = ai === -1 ? 999 : ai;
+    const bp = bi === -1 ? 999 : bi;
+    if (ap !== bp) return ap - bp;
+    return a.localeCompare(b);
+  });
+}