takt 0.1.4 → 0.1.6

package/dist/workflow/instruction-builder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"instruction-builder.d.ts","sourceRoot":"","sources":["../../src/workflow/instruction-builder.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAGtE;;GAEG;AACH,MAAM,WAAW,kBAAkB;IACjC,2BAA2B;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,+BAA+B;IAC/B,SAAS,EAAE,MAAM,CAAC;IAClB,iCAAiC;IACjC,aAAa,EAAE,MAAM,CAAC;IACtB,wBAAwB;IACxB,GAAG,EAAE,MAAM,CAAC;IACZ,8CAA8C;IAC9C,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,wCAAwC;IACxC,cAAc,CAAC,EAAE,aAAa,CAAC;IAC/B,4BAA4B;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AASD;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAC9B,IAAI,EAAE,YAAY,EAClB,OAAO,EAAE,kBAAkB,GAC1B,MAAM,CA4CR"}
+{"version":3,"file":"instruction-builder.d.ts","sourceRoot":"","sources":["../../src/workflow/instruction-builder.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAGtE;;GAEG;AACH,MAAM,WAAW,kBAAkB;IACjC,2BAA2B;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,0DAA0D;IAC1D,SAAS,EAAE,MAAM,CAAC;IAClB,iCAAiC;IACjC,aAAa,EAAE,MAAM,CAAC;IACtB,mFAAmF;IACnF,aAAa,EAAE,MAAM,CAAC;IACtB,wBAAwB;IACxB,GAAG,EAAE,MAAM,CAAC;IACZ,8CAA8C;IAC9C,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,wCAAwC;IACxC,cAAc,CAAC,EAAE,aAAa,CAAC;IAC/B,4BAA4B;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AASD;;;;;;;;;;;;GAYG;AACH,wBAAgB,gBAAgB,CAC9B,IAAI,EAAE,YAAY,EAClB,OAAO,EAAE,kBAAkB,GAC1B,MAAM,CA6CR"}

package/dist/workflow/instruction-builder.js

@@ -16,8 +16,9 @@ function escapeTemplateChars(str) {
  *
  * Supported placeholders:
  * - {task} - The main task/prompt
- * - {iteration} - Current iteration number
+ * - {iteration} - Current iteration number (workflow-wide turn count)
  * - {max_iterations} - Maximum iterations allowed
+ * - {step_iteration} - Current step's iteration number (how many times this step has been executed)
  * - {previous_response} - Output from previous step (if passPreviousResponse is true)
  * - {git_diff} - Current git diff output
  * - {user_inputs} - Accumulated user inputs
@@ -27,9 +28,10 @@ export function buildInstruction(step, context) {
     let instruction = step.instructionTemplate;
     // Replace {task}
     instruction = instruction.replace(/\{task\}/g, escapeTemplateChars(context.task));
-    // Replace {iteration} and {max_iterations}
+    // Replace {iteration}, {max_iterations}, and {step_iteration}
     instruction = instruction.replace(/\{iteration\}/g, String(context.iteration));
     instruction = instruction.replace(/\{max_iterations\}/g, String(context.maxIterations));
+    instruction = instruction.replace(/\{step_iteration\}/g, String(context.stepIteration));
     // Replace {previous_response}
     if (step.passPreviousResponse) {
         if (context.previousOutput) {

package/dist/workflow/instruction-builder.js.map

@@ -1 +1 @@
-{"version":3,"file":"instruction-builder.js","sourceRoot":"","sources":["../../src/workflow/instruction-builder.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAsBjD;;GAEG;AACH,SAAS,mBAAmB,CAAC,GAAW;IACtC,OAAO,GAAG,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;AACrD,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,gBAAgB,CAC9B,IAAkB,EAClB,OAA2B;IAE3B,IAAI,WAAW,GAAG,IAAI,CAAC,mBAAmB,CAAC;IAE3C,iBAAiB;IACjB,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,WAAW,EAAE,mBAAmB,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC;IAElF,2CAA2C;IAC3C,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,gBAAgB,EAAE,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC;IAC/E,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC;IAExF,8BAA8B;IAC9B,IAAI,IAAI,CAAC,oBAAoB,EAAE,CAAC;QAC9B,IAAI,OAAO,CAAC,cAAc,EAAE,CAAC;YAC3B,WAAW,GAAG,WAAW,CAAC,OAAO,CAC/B,wBAAwB,EACxB,mBAAmB,CAAC,OAAO,CAAC,cAAc,CAAC,OAAO,CAAC,CACpD,CAAC;QACJ,CAAC;aAAM,CAAC;YACN,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,wBAAwB,EAAE,EAAE,CAAC,CAAC;QAClE,CAAC;IACH,CAAC;IAED,qBAAqB;IACrB,MAAM,OAAO,GAAG,UAAU,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IACxC,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,eAAe,EAAE,OAAO,CAAC,CAAC;IAE5D,wBAAwB;IACxB,MAAM,aAAa,GAAG,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACpD,WAAW,GAAG,WAAW,CAAC,OAAO,CAC/B,kBAAkB,EAClB,mBAAmB,CAAC,aAAa,CAAC,CACnC,CAAC;IAEF,uBAAuB;IACvB,IAAI,OAAO,CAAC,SAAS,EAAE,CAAC;QACtB,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,iBAAiB,EAAE,OAAO,CAAC,SAAS,CAAC,CAAC;IAC1E,CAAC;IAED,wCAAwC;IACxC,IAAI,IAAI,CAAC,iBAAiB,EAAE,CAAC;QAC3B,WAAW,GAAG,GAAG,WAAW,OAAO,IAAI,CAAC,iBAAiB,EAAE,CAAC;IAC9D,CAAC;IAED,OAAO,WAAW,CAAC;AACrB,CAAC"}
+{"version":3,"file":"instruction-builder.js","sourceRoot":"","sources":["../../src/workflow/instruction-builder.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAwBjD;;GAEG;AACH,SAAS,mBAAmB,CAAC,GAAW;IACtC,OAAO,GAAG,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;AACrD,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,gBAAgB,CAC9B,IAAkB,EAClB,OAA2B;IAE3B,IAAI,WAAW,GAAG,IAAI,CAAC,mBAAmB,CAAC;IAE3C,iBAAiB;IACjB,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,WAAW,EAAE,mBAAmB,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC;IAElF,8DAA8D;IAC9D,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,gBAAgB,EAAE,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC;IAC/E,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC;IACxF,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC;IAExF,8BAA8B;IAC9B,IAAI,IAAI,CAAC,oBAAoB,EAAE,CAAC;QAC9B,IAAI,OAAO,CAAC,cAAc,EAAE,CAAC;YAC3B,WAAW,GAAG,WAAW,CAAC,OAAO,CAC/B,wBAAwB,EACxB,mBAAmB,CAAC,OAAO,CAAC,cAAc,CAAC,OAAO,CAAC,CACpD,CAAC;QACJ,CAAC;aAAM,CAAC;YACN,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,wBAAwB,EAAE,EAAE,CAAC,CAAC;QAClE,CAAC;IACH,CAAC;IAED,qBAAqB;IACrB,MAAM,OAAO,GAAG,UAAU,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IACxC,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,eAAe,EAAE,OAAO,CAAC,CAAC;IAE5D,wBAAwB;IACxB,MAAM,aAAa,GAAG,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACpD,WAAW,GAAG,WAAW,CAAC,OAAO,CAC/B,kBAAkB,EAClB,mBAAmB,CAAC,aAAa,CAAC,CACnC,CAAC;IAEF,uBAAuB;IACvB,IAAI,OAAO,CAAC,SAAS,EAAE,CAAC;QACtB,WAAW,GAAG,WAAW,CAAC,OAAO,CAAC,iBAAiB,EAAE,OAAO,CAAC,SAAS,CAAC,CAAC;IAC1E,CAAC;IAED,wCAAwC;IACxC,IAAI,IAAI,CAAC,iBAAiB,EAAE,CAAC;QAC3B,WAAW,GAAG,GAAG,WAAW,OAAO,IAAI,CAAC,iBAAiB,EAAE,CAAC;IAC9D,CAAC;IAED,OAAO,WAAW,CAAC;AACrB,CAAC"}

package/dist/workflow/state-manager.d.ts

@@ -10,6 +10,10 @@ import type { WorkflowEngineOptions } from './types.js';
  * Create initial workflow state from config and options.
  */
 export declare function createInitialState(config: WorkflowConfig, options: WorkflowEngineOptions): WorkflowState;
+/**
+ * Increment the iteration counter for a step and return the new value.
+ */
+export declare function incrementStepIteration(state: WorkflowState, stepName: string): number;
 /**
  * Add user input to state with truncation and limit handling.
  */

package/dist/workflow/state-manager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"state-manager.d.ts","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAKvF,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAC;AAExD;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,cAAc,EACtB,OAAO,EAAE,qBAAqB,GAC7B,aAAa,CAuBf;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAMtE;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,GAAG,SAAS,CAGjF;AAED;;GAEG;AACH,wBAAgB,eAAe,CAC7B,KAAK,EAAE,aAAa,EACpB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,aAAa,GACtB,IAAI,CAMN"}
+{"version":3,"file":"state-manager.d.ts","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAKvF,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAC;AAExD;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,cAAc,EACtB,OAAO,EAAE,qBAAqB,GAC7B,aAAa,CAwBf;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,aAAa,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAKrF;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAMtE;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,GAAG,SAAS,CAGjF;AAED;;GAEG;AACH,wBAAgB,eAAe,CAC7B,KAAK,EAAE,aAAa,EACpB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,aAAa,GACtB,IAAI,CAMN"}

package/dist/workflow/state-manager.js

@@ -27,9 +27,19 @@ export function createInitialState(config, options) {
         stepOutputs: new Map(),
         userInputs,
         agentSessions,
+        stepIterations: new Map(),
         status: 'running',
     };
 }
+/**
+ * Increment the iteration counter for a step and return the new value.
+ */
+export function incrementStepIteration(state, stepName) {
+    const current = state.stepIterations.get(stepName) ?? 0;
+    const next = current + 1;
+    state.stepIterations.set(stepName, next);
+    return next;
+}
 /**
  * Add user input to state with truncation and limit handling.
  */

package/dist/workflow/state-manager.js.map

@@ -1 +1 @@
-{"version":3,"file":"state-manager.js","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EACL,eAAe,EACf,gBAAgB,GACjB,MAAM,gBAAgB,CAAC;AAGxB;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAAsB,EACtB,OAA8B;IAE9B,kDAAkD;IAClD,MAAM,aAAa,GAAG,IAAI,GAAG,EAAkB,CAAC;IAChD,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC;QAC5B,KAAK,MAAM,CAAC,KAAK,EAAE,SAAS,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,eAAe,CAAC,EAAE,CAAC;YACzE,aAAa,CAAC,GAAG,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;QACtC,CAAC;IACH,CAAC;IAED,kDAAkD;IAClD,MAAM,UAAU,GAAG,OAAO,CAAC,iBAAiB;QAC1C,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,iBAAiB,CAAC;QAChC,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;QACL,YAAY,EAAE,MAAM,CAAC,IAAI;QACzB,WAAW,EAAE,MAAM,CAAC,WAAW;QAC/B,SAAS,EAAE,CAAC;QACZ,WAAW,EAAE,IAAI,GAAG,EAAE;QACtB,UAAU;QACV,aAAa;QACb,MAAM,EAAE,SAAS;KAClB,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAAC,KAAoB,EAAE,KAAa;IAC9D,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,IAAI,eAAe,EAAE,CAAC;QAC/C,KAAK,CAAC,UAAU,CAAC,KAAK,EAAE,CAAC,CAAC,gBAAgB;IAC5C,CAAC;IACD,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,gBAAgB,CAAC,CAAC;IACnD,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AACnC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAAoB;IACpD,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC;IACvD,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;AACrC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,eAAe,CAC7B,KAAoB,EACpB,QAAgB,EAChB,SAAiB,EACjB,QAAuB;IAEvB,KAAK,CAAC,WAAW,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IAE1C,IAAI,QAAQ,CAAC,SAAS,EAAE,CAAC;QACvB,KAAK,CAAC,aAAa,CAAC,GAAG,CAAC,SAAS,EAAE,QAAQ,CAAC,SAAS,CAAC,CAAC;IACzD,CAAC;AACH,CAAC"}
+{"version":3,"file":"state-manager.js","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EACL,eAAe,EACf,gBAAgB,GACjB,MAAM,gBAAgB,CAAC;AAGxB;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAAsB,EACtB,OAA8B;IAE9B,kDAAkD;IAClD,MAAM,aAAa,GAAG,IAAI,GAAG,EAAkB,CAAC;IAChD,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC;QAC5B,KAAK,MAAM,CAAC,KAAK,EAAE,SAAS,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,eAAe,CAAC,EAAE,CAAC;YACzE,aAAa,CAAC,GAAG,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;QACtC,CAAC;IACH,CAAC;IAED,kDAAkD;IAClD,MAAM,UAAU,GAAG,OAAO,CAAC,iBAAiB;QAC1C,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,iBAAiB,CAAC;QAChC,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;QACL,YAAY,EAAE,MAAM,CAAC,IAAI;QACzB,WAAW,EAAE,MAAM,CAAC,WAAW;QAC/B,SAAS,EAAE,CAAC;QACZ,WAAW,EAAE,IAAI,GAAG,EAAE;QACtB,UAAU;QACV,aAAa;QACb,cAAc,EAAE,IAAI,GAAG,EAAE;QACzB,MAAM,EAAE,SAAS;KAClB,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,sBAAsB,CAAC,KAAoB,EAAE,QAAgB;IAC3E,MAAM,OAAO,GAAG,KAAK,CAAC,cAAc,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IACxD,MAAM,IAAI,GAAG,OAAO,GAAG,CAAC,CAAC;IACzB,KAAK,CAAC,cAAc,CAAC,GAAG,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;IACzC,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAAC,KAAoB,EAAE,KAAa;IAC9D,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,IAAI,eAAe,EAAE,CAAC;QAC/C,KAAK,CAAC,UAAU,CAAC,KAAK,EAAE,CAAC,CAAC,gBAAgB;IAC5C,CAAC;IACD,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,gBAAgB,CAAC,CAAC;IACnD,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AACnC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAAoB;IACpD,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC;IACvD,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;AACrC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,eAAe,CAC7B,KAAoB,EACpB,QAAgB,EAChB,SAAiB,EACjB,QAAuB;IAEvB,KAAK,CAAC,WAAW,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IAE1C,IAAI,QAAQ,CAAC,SAAS,EAAE,CAAC;QACvB,KAAK,CAAC,aAAa,CAAC,GAAG,CAAC,SAAS,EAAE,QAAQ,CAAC,SAAS,CAAC,CAAC;IACzD,CAAC;AACH,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "takt",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "TAKT: Task Agent Koordination Tool - AI Agent Workflow Orchestration",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/resources/global/en/agents/default/architect.md

@@ -159,7 +159,68 @@ Prohibited patterns:
 | Hidden Dependencies | Child components implicitly calling APIs etc. |
 | Non-idiomatic | Custom implementation ignoring language/FW conventions |
 
-### 6. Unnecessary Backward Compatibility Code Detection
+### 6. Abstraction Level Evaluation
+
+**Conditional Branch Proliferation Detection:**
+
+| Pattern | Judgment |
+|---------|----------|
+| Same if-else pattern in 3+ places | Abstract with polymorphism → **REJECT** |
+| switch/case with 5+ branches | Consider Strategy/Map pattern |
+| Flag arguments changing behavior | Split into separate functions → **REJECT** |
+| Type-based branching (instanceof/typeof) | Replace with polymorphism → **REJECT** |
+| Nested conditionals (3+ levels) | Early return or extract → **REJECT** |
+
+**Abstraction Level Mismatch Detection:**
+
+| Pattern | Problem | Fix |
+|---------|---------|-----|
+| Low-level details in high-level processing | Hard to read | Extract details to functions |
+| Mixed abstraction levels in one function | Cognitive load | Align to same granularity |
+| DB operations mixed with business logic | Responsibility violation | Separate to Repository layer |
+| Config values mixed with processing logic | Hard to change | Externalize configuration |
+
+**Good Abstraction Examples:**
+
+```typescript
+// ❌ Proliferating conditionals
+function process(type: string) {
+  if (type === 'A') { /* process A */ }
+  else if (type === 'B') { /* process B */ }
+  else if (type === 'C') { /* process C */ }
+  // ...continues
+}
+
+// ✅ Abstract with Map pattern
+const processors: Record<string, () => void> = {
+  A: processA,
+  B: processB,
+  C: processC,
+};
+function process(type: string) {
+  processors[type]?.();
+}
+```
+
+```typescript
+// ❌ Mixed abstraction levels
+function createUser(data: UserData) {
+  // High level: business logic
+  validateUser(data);
+  // Low level: DB operation details
+  const conn = await pool.getConnection();
+  await conn.query('INSERT INTO users...');
+  conn.release();
+}
+
+// ✅ Aligned abstraction levels
+function createUser(data: UserData) {
+  validateUser(data);
+  await userRepository.save(data);  // Details hidden
+}
+```
+
+### 7. Unnecessary Backward Compatibility Code Detection
 
 **AI tends to leave unnecessary code "for backward compatibility." Don't overlook this.**
 
@@ -188,7 +249,7 @@ Code that should be kept:
 
 **Be suspicious when AI says "for backward compatibility."** Verify if it's really needed.
 
-### 7. Workaround Detection
+### 8. Workaround Detection
 
 **Don't overlook compromises made to "just make it work."**
 
@@ -203,7 +264,7 @@ Code that should be kept:
 
 **Always point these out.** Temporary fixes become permanent.
 
-### 8. Quality Attributes
+### 9. Quality Attributes
 
 | Attribute | Review Point |
 |-----------|--------------|
@@ -211,7 +272,7 @@ Code that should be kept:
 | Maintainability | Easy to modify and fix |
 | Observability | Logging and monitoring enabled |
 
-### 9. Big Picture
+### 10. Big Picture
 
 **Caution**: Don't get lost in minor "clean code" nitpicks.
 
@@ -222,7 +283,7 @@ Verify:
 - Does it align with business requirements
 - Is naming consistent with the domain
 
-### 10. Change Scope Assessment
+### 11. Change Scope Assessment
 
 **Check change scope and include in report (non-blocking).**
 
@@ -241,7 +302,7 @@ Verify:
 **Include as suggestions (non-blocking):**
 - If splittable, present splitting proposal
 
-### 11. Circular Review Detection
+### 12. Circular Review Detection
 
 When review count is provided (e.g., "Review count: 3rd"), adjust judgment accordingly.
 

package/resources/global/en/agents/default/coder.md

@@ -105,7 +105,45 @@ Perform self-check after implementation.
 | Boy Scout | Leave touched areas slightly improved |
 | Fail Fast | Detect errors early. Don't swallow them |
 
-**When in doubt**: Choose Simple. Abstraction can come later.
+**When in doubt**: Choose Simple.
+
+## Abstraction Principles
+
+**Before adding conditional branches, consider:**
+- Does this condition exist elsewhere? → Abstract with a pattern
+- Will more branches be added? → Use Strategy/Map pattern
+- Branching on type? → Replace with polymorphism
+
+```typescript
+// ❌ Adding more conditionals
+if (type === 'A') { ... }
+else if (type === 'B') { ... }
+else if (type === 'C') { ... }  // Yet another one
+
+// ✅ Abstract with Map
+const handlers = { A: handleA, B: handleB, C: handleC };
+handlers[type]?.();
+```
+
+**Align abstraction levels:**
+- Keep same granularity of operations within one function
+- Extract detailed processing to separate functions
+- Don't mix "what to do" with "how to do it"
+
+```typescript
+// ❌ Mixed abstraction levels
+function processOrder(order) {
+  validateOrder(order);           // High level
+  const conn = pool.getConnection(); // Low level detail
+  conn.query('INSERT...');        // Low level detail
+}
+
+// ✅ Aligned abstraction levels
+function processOrder(order) {
+  validateOrder(order);
+  saveOrder(order);  // Details hidden
+}
+```
 
 **Follow language/framework conventions:**
 - Be Pythonic in Python, Kotlin-like in Kotlin
@@ -134,6 +172,121 @@ Perform self-check after implementation.
 - Children don't modify state directly (notify parent via events)
 - State flows in one direction
 
+## Error Handling
+
+**Principle: Centralize error handling. Don't scatter try-catch everywhere.**
+
+```typescript
+// ❌ Try-catch everywhere
+async function createUser(data) {
+  try {
+    const user = await userService.create(data)
+    return user
+  } catch (e) {
+    console.error(e)
+    throw new Error('Failed to create user')
+  }
+}
+
+// ✅ Centralized handling at upper layer
+// Catch at Controller/Handler layer
+// Or use @ControllerAdvice / ErrorBoundary
+async function createUser(data) {
+  return await userService.create(data)  // Let exceptions propagate
+}
+```
+
+**Error handling placement:**
+
+| Layer | Responsibility |
+|-------|----------------|
+| Domain/Service layer | Throw exceptions on business rule violations |
+| Controller/Handler layer | Catch exceptions and convert to response |
+| Global handler | Handle common exceptions (NotFound, auth errors, etc.) |
+
+## Transformation Placement
+
+**Principle: Put conversion methods on DTOs.**
+
+```typescript
+// ✅ Request/Response DTOs have conversion methods
+interface CreateUserRequest {
+  name: string
+  email: string
+}
+
+function toUseCaseInput(req: CreateUserRequest): CreateUserInput {
+  return { name: req.name, email: req.email }
+}
+
+// Controller
+const input = toUseCaseInput(request)
+const output = await useCase.execute(input)
+return UserResponse.from(output)
+```
+
+**Conversion direction:**
+```
+Request → toInput() → UseCase/Service → Output → Response.from()
+```
+
+## Extraction Decisions
+
+**Rule of Three:**
+- 1st time: Write it inline
+- 2nd time: Don't extract yet (wait and see)
+- 3rd time: Consider extraction
+
+**Should extract:**
+- Same logic in 3+ places
+- Same style/UI pattern
+- Same validation logic
+- Same formatting logic
+
+**Should NOT extract:**
+- Similar but slightly different (forced generalization adds complexity)
+- Used in only 1-2 places
+- Based on "might use later" predictions
+
+```typescript
+// ❌ Over-generalization
+function formatValue(value, type, options) {
+  if (type === 'currency') { ... }
+  else if (type === 'date') { ... }
+  else if (type === 'percentage') { ... }
+}
+
+// ✅ Separate functions by purpose
+function formatCurrency(amount: number): string { ... }
+function formatDate(date: Date): string { ... }
+function formatPercentage(value: number): string { ... }
+```
+
+## Writing Tests
+
+**Principle: Structure tests with "Given-When-Then".**
+
+```typescript
+test('returns NotFound error when user does not exist', async () => {
+  // Given: non-existent user ID
+  const nonExistentId = 'non-existent-id'
+
+  // When: attempt to get user
+  const result = await getUser(nonExistentId)
+
+  // Then: NotFound error is returned
+  expect(result.error).toBe('NOT_FOUND')
+})
+```
+
+**Test priority:**
+
+| Priority | Target |
+|----------|--------|
+| High | Business logic, state transitions |
+| Medium | Edge cases, error handling |
+| Low | Simple CRUD, UI appearance |
+
 ## Prohibited
 
 - **Fallback value overuse** - Don't hide problems with `?? 'unknown'`, `|| 'default'`
@@ -143,4 +296,5 @@ Perform self-check after implementation.
 - **Direct object/array mutation** - Create new with spread operator
 - **console.log** - Don't leave in production code
 - **Hardcoded secrets**
+- **Scattered try-catch** - Centralize error handling at upper layer
 

package/resources/global/en/workflows/default.yaml

@@ -1,5 +1,15 @@
 # Default TAKT Workflow
 # Plan -> Coder -> Architect Review -> AI Review -> Security Review -> Supervisor Approval
+#
+# Template Variables:
+#   {iteration}      - Workflow-wide turn count (total steps executed across all agents)
+#   {max_iterations} - Maximum iterations allowed for the workflow
+#   {step_iteration} - Per-step iteration count (how many times THIS step has been executed)
+#   {task}           - Original user request
+#   {previous_response} - Output from the previous step
+#   {git_diff}       - Current uncommitted changes (git diff)
+#   {user_inputs}    - Accumulated user inputs during workflow
+#   {report_dir}     - Report directory name (e.g., "20250126-143052-task-summary")
 
 name: default
 description: Standard development workflow with planning and specialized reviews
@@ -55,7 +65,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: plan (Task Analysis)
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/00-plan.md
@@ -79,7 +90,7 @@ steps:
 
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -155,7 +166,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: implement
       - Report Directory: .takt/reports/{report_dir}/
       - Report Files:
@@ -174,7 +186,7 @@ steps:
 
       **Report output:** Output to the `Report Files` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Scope report format (create at implementation start):**
       ```markdown
@@ -279,7 +291,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: review (Architecture Review)
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/03-architect-review.md
@@ -299,7 +312,7 @@ steps:
 
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -382,7 +395,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: improve
 
       ## Architect Feedback (This is the latest instruction - prioritize this)
@@ -463,7 +477,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: ai_review (AI-Generated Code Review)
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/04-ai-review.md
@@ -485,7 +500,7 @@ steps:
 
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -564,7 +579,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: ai_fix
 
       ## AI Review Feedback (This is the latest instruction - prioritize this)
@@ -640,7 +656,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: security_review
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/05-security-review.md
@@ -662,7 +679,7 @@ steps:
 
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -744,7 +761,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: security_fix
 
       ## Security Review Feedback (This is the latest instruction - prioritize this)
@@ -813,7 +831,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: fix
 
       ## Architect Feedback (This is the latest instruction - prioritize this)
@@ -893,7 +912,8 @@ steps:
       ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: supervise (final verification)
       - Report Directory: .takt/reports/{report_dir}/
       - Report Files:
@@ -921,7 +941,7 @@ steps:
 
       **Report output:** Output to the `Report Files` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Validation report format:**
       ```markdown