takt 0.1.0

package/dist/workflow/transitions.js

@@ -0,0 +1,106 @@
+/**
+ * Workflow state transition logic
+ *
+ * Handles determining the next step based on agent response status
+ * and transition conditions defined in the workflow configuration.
+ */
+import { COMPLETE_STEP, ABORT_STEP } from './constants.js';
+/**
+ * Check if status matches transition condition.
+ */
+export function matchesCondition(status, condition) {
+    if (condition === 'always') {
+        return true;
+    }
+    // Map status to condition
+    const statusConditionMap = {
+        done: ['done'],
+        blocked: ['blocked'],
+        approved: ['approved'],
+        rejected: ['rejected'],
+        pending: [],
+        in_progress: [],
+        cancelled: [],
+        interrupted: [], // Interrupted is handled separately
+    };
+    const matchingConditions = statusConditionMap[status] || [];
+    return matchingConditions.includes(condition);
+}
+/**
+ * Handle case when no status marker is found in agent output.
+ */
+export function handleNoStatus(step, config) {
+    const behavior = step.onNoStatus || 'complete';
+    switch (behavior) {
+        case 'stay':
+            // Stay on current step (original behavior, may cause loops)
+            return step.name;
+        case 'continue': {
+            // Try to find done/always transition, otherwise find next step in workflow
+            for (const transition of step.transitions) {
+                if (transition.condition === 'done' || transition.condition === 'always') {
+                    return transition.nextStep;
+                }
+            }
+            // Find next step in workflow order
+            const stepIndex = config.steps.findIndex(s => s.name === step.name);
+            const nextStep = config.steps[stepIndex + 1];
+            if (stepIndex >= 0 && nextStep) {
+                return nextStep.name;
+            }
+            return COMPLETE_STEP;
+        }
+        case 'complete':
+        default:
+            // Try to find done/always transition, otherwise complete workflow
+            for (const transition of step.transitions) {
+                if (transition.condition === 'done' || transition.condition === 'always') {
+                    return transition.nextStep;
+                }
+            }
+            return COMPLETE_STEP;
+    }
+}
+/**
+ * Determine next step based on current status.
+ */
+export function determineNextStep(step, status, config) {
+    // If interrupted, abort immediately
+    if (status === 'interrupted') {
+        return ABORT_STEP;
+    }
+    // Check transitions in order
+    for (const transition of step.transitions) {
+        if (matchesCondition(status, transition.condition)) {
+            return transition.nextStep;
+        }
+    }
+    // If status is 'in_progress' (no status marker found), use onNoStatus behavior
+    if (status === 'in_progress') {
+        return handleNoStatus(step, config);
+    }
+    // Unexpected status - treat as done and complete
+    return COMPLETE_STEP;
+}
+/**
+ * Extract user-facing prompt from blocked response.
+ * Looks for common patterns like "必要な情報:", "質問:", etc.
+ */
+export function extractBlockedPrompt(content) {
+    // Try to extract specific question/info needed
+    const patterns = [
+        /必要な情報[:：]\s*(.+?)(?:\n|$)/i,
+        /質問[:：]\s*(.+?)(?:\n|$)/i,
+        /理由[:：]\s*(.+?)(?:\n|$)/i,
+        /確認[:：]\s*(.+?)(?:\n|$)/i,
+    ];
+    for (const pattern of patterns) {
+        const match = content.match(pattern);
+        if (match?.[1]) {
+            return match[1].trim();
+        }
+    }
+    // Return the full content if no specific pattern found
+    return content;
+}
+//# sourceMappingURL=transitions.js.map

package/dist/workflow/transitions.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"transitions.js","sourceRoot":"","sources":["../../src/workflow/transitions.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAQH,OAAO,EAAE,aAAa,EAAE,UAAU,EAAE,MAAM,gBAAgB,CAAC;AAE3D;;GAEG;AACH,MAAM,UAAU,gBAAgB,CAC9B,MAAc,EACd,SAA8B;IAE9B,IAAI,SAAS,KAAK,QAAQ,EAAE,CAAC;QAC3B,OAAO,IAAI,CAAC;IACd,CAAC;IAED,0BAA0B;IAC1B,MAAM,kBAAkB,GAA0C;QAChE,IAAI,EAAE,CAAC,MAAM,CAAC;QACd,OAAO,EAAE,CAAC,SAAS,CAAC;QACpB,QAAQ,EAAE,CAAC,UAAU,CAAC;QACtB,QAAQ,EAAE,CAAC,UAAU,CAAC;QACtB,OAAO,EAAE,EAAE;QACX,WAAW,EAAE,EAAE;QACf,SAAS,EAAE,EAAE;QACb,WAAW,EAAE,EAAE,EAAE,oCAAoC;KACtD,CAAC;IAEF,MAAM,kBAAkB,GAAG,kBAAkB,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC;IAC5D,OAAO,kBAAkB,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC;AAChD,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,cAAc,CAC5B,IAAkB,EAClB,MAAsB;IAEtB,MAAM,QAAQ,GAAG,IAAI,CAAC,UAAU,IAAI,UAAU,CAAC;IAE/C,QAAQ,QAAQ,EAAE,CAAC;QACjB,KAAK,MAAM;YACT,4DAA4D;YAC5D,OAAO,IAAI,CAAC,IAAI,CAAC;QAEnB,KAAK,UAAU,CAAC,CAAC,CAAC;YAChB,2EAA2E;YAC3E,KAAK,MAAM,UAAU,IAAI,IAAI,CAAC,WAAW,EAAE,CAAC;gBAC1C,IAAI,UAAU,CAAC,SAAS,KAAK,MAAM,IAAI,UAAU,CAAC,SAAS,KAAK,QAAQ,EAAE,CAAC;oBACzE,OAAO,UAAU,CAAC,QAAQ,CAAC;gBAC7B,CAAC;YACH,CAAC;YACD,mCAAmC;YACnC,MAAM,SAAS,GAAG,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,IAAI,CAAC,IAAI,CAAC,CAAC;YACpE,MAAM,QAAQ,GAAG,MAAM,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC;YAC7C,IAAI,SAAS,IAAI,CAAC,IAAI,QAAQ,EAAE,CAAC;gBAC/B,OAAO,QAAQ,CAAC,IAAI,CAAC;YACvB,CAAC;YACD,OAAO,aAAa,CAAC;QACvB,CAAC;QAED,KAAK,UAAU,CAAC;QAChB;YACE,kEAAkE;YAClE,KAAK,MAAM,UAAU,IAAI,IAAI,CAAC,WAAW,EAAE,CAAC;gBAC1C,IAAI,UAAU,CAAC,SAAS,KAAK,MAAM,IAAI,UAAU,CAAC,SAAS,KAAK,QAAQ,EAAE,CAAC;oBACzE,OAAO,UAAU,CAAC,QAAQ,CAAC;gBAC7B,CAAC;YACH,CAAC;YACD,OAAO,aAAa,CAAC;IACzB,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAC/B,IAAkB,EAClB,MAAc,EACd,MAAsB;IAEtB,oCAAoC;IACpC,IAAI,MAAM,KAAK,aAAa,EAAE,CAAC;QAC7B,OAAO,UAAU,CAAC;IACpB,CAAC;IAED,6BAA6B;IAC7B,KAAK,MAAM,UAAU,IAAI,IAAI,CAAC,WAAW,EAAE,CAAC;QAC1C,IAAI,gBAAgB,CAAC,MAAM,EAAE,UAAU,CAAC,SAAS,CAAC,EAAE,CAAC;YACnD,OAAO,UAAU,CAAC,QAAQ,CAAC;QAC7B,CAAC;IACH,CAAC;IAED,+EAA+E;IAC/E,IAAI,MAAM,KAAK,aAAa,EAAE,CAAC;QAC7B,OAAO,cAAc,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;IACtC,CAAC;IAED,iDAAiD;IACjD,OAAO,aAAa,CAAC;AACvB,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,oBAAoB,CAAC,OAAe;IAClD,+CAA+C;IAC/C,MAAM,QAAQ,GAAG;QACf,4BAA4B;QAC5B,yBAAyB;QACzB,yBAAyB;QACzB,yBAAyB;KAC1B,CAAC;IAEF,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QACrC,IAAI,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YACf,OAAO,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACzB,CAAC;IACH,CAAC;IAED,uDAAuD;IACvD,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/dist/workflow/types.d.ts

@@ -0,0 +1,74 @@
+/**
+ * Workflow engine type definitions
+ *
+ * Contains types for workflow events, requests, and callbacks
+ * used by the workflow execution engine.
+ */
+import type { WorkflowStep, AgentResponse, WorkflowState } from '../models/types.js';
+import type { StreamCallback } from '../agents/runner.js';
+import type { PermissionHandler, AskUserQuestionHandler } from '../claude/process.js';
+/** Events emitted by workflow engine */
+export interface WorkflowEvents {
+    'step:start': (step: WorkflowStep, iteration: number) => void;
+    'step:complete': (step: WorkflowStep, response: AgentResponse) => void;
+    'step:blocked': (step: WorkflowStep, response: AgentResponse) => void;
+    'step:user_input': (step: WorkflowStep, userInput: string) => void;
+    'workflow:complete': (state: WorkflowState) => void;
+    'workflow:abort': (state: WorkflowState, reason: string) => void;
+    'iteration:limit': (iteration: number, maxIterations: number) => void;
+    'step:loop_detected': (step: WorkflowStep, consecutiveCount: number) => void;
+}
+/** User input request for blocked state */
+export interface UserInputRequest {
+    /** The step that is blocked */
+    step: WorkflowStep;
+    /** The blocked response from the agent */
+    response: AgentResponse;
+    /** Prompt for the user (extracted from blocked message) */
+    prompt: string;
+}
+/** Iteration limit request */
+export interface IterationLimitRequest {
+    /** Current iteration count */
+    currentIteration: number;
+    /** Current max iterations */
+    maxIterations: number;
+    /** Current step name */
+    currentStep: string;
+}
+/** Callback for session updates (when agent session IDs change) */
+export type SessionUpdateCallback = (agentName: string, sessionId: string) => void;
+/**
+ * Callback for iteration limit reached.
+ * Returns the number of additional iterations to continue, or null to stop.
+ */
+export type IterationLimitCallback = (request: IterationLimitRequest) => Promise<number | null>;
+/** Options for workflow engine */
+export interface WorkflowEngineOptions {
+    /** Callback for streaming real-time output */
+    onStream?: StreamCallback;
+    /** Callback for requesting user input when an agent is blocked */
+    onUserInput?: (request: UserInputRequest) => Promise<string | null>;
+    /** Initial agent sessions to restore (agent name -> session ID) */
+    initialSessions?: Record<string, string>;
+    /** Callback when agent session ID is updated */
+    onSessionUpdate?: SessionUpdateCallback;
+    /** Custom permission handler for interactive permission prompts */
+    onPermissionRequest?: PermissionHandler;
+    /** Initial user inputs to share with all agents */
+    initialUserInputs?: string[];
+    /** Custom handler for AskUserQuestion tool */
+    onAskUserQuestion?: AskUserQuestionHandler;
+    /** Callback when iteration limit is reached - returns additional iterations or null to stop */
+    onIterationLimit?: IterationLimitCallback;
+    /** Bypass all permission checks (sacrifice-my-pc mode) */
+    bypassPermissions?: boolean;
+}
+/** Loop detection result */
+export interface LoopCheckResult {
+    isLoop: boolean;
+    count: number;
+    shouldAbort: boolean;
+    shouldWarn?: boolean;
+}
+//# sourceMappingURL=types.d.ts.map

package/dist/workflow/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../src/workflow/types.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AACrF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAC1D,OAAO,KAAK,EAAE,iBAAiB,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC;AAEtF,wCAAwC;AACxC,MAAM,WAAW,cAAc;IAC7B,YAAY,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAC9D,eAAe,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,aAAa,KAAK,IAAI,CAAC;IACvE,cAAc,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,aAAa,KAAK,IAAI,CAAC;IACtE,iBAAiB,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACnE,mBAAmB,EAAE,CAAC,KAAK,EAAE,aAAa,KAAK,IAAI,CAAC;IACpD,gBAAgB,EAAE,CAAC,KAAK,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACjE,iBAAiB,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,KAAK,IAAI,CAAC;IACtE,oBAAoB,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,gBAAgB,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9E;AAED,2CAA2C;AAC3C,MAAM,WAAW,gBAAgB;IAC/B,+BAA+B;IAC/B,IAAI,EAAE,YAAY,CAAC;IACnB,0CAA0C;IAC1C,QAAQ,EAAE,aAAa,CAAC;IACxB,2DAA2D;IAC3D,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,8BAA8B;AAC9B,MAAM,WAAW,qBAAqB;IACpC,8BAA8B;IAC9B,gBAAgB,EAAE,MAAM,CAAC;IACzB,6BAA6B;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,wBAAwB;IACxB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,mEAAmE;AACnE,MAAM,MAAM,qBAAqB,GAAG,CAAC,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;AAEnF;;;GAGG;AACH,MAAM,MAAM,sBAAsB,GAAG,CAAC,OAAO,EAAE,qBAAqB,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;AAEhG,kCAAkC;AAClC,MAAM,WAAW,qBAAqB;IACpC,8CAA8C;IAC9C,QAAQ,CAAC,EAAE,cAAc,CAAC;IAC1B,kEAAkE;IAClE,WAAW,CAAC,EAAE,CAAC,OAAO,EAAE,gBAAgB,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;IACpE,mEAAmE;IACnE,eAAe,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACzC,gDAAgD;IAChD,eAAe,CAAC,EAAE,qBAAqB,CAAC;IACxC,mEAAmE;IACnE,mBAAmB,CAAC,EAAE,iBAAiB,CAAC;IACxC,mDAAmD;IACnD,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC7B,8CAA8C;IAC9C,iBAAiB,CAAC,EAAE,sBAAsB,CAAC;IAC3C,+FAA+F;IAC/F,gBAAgB,CAAC,EAAE,sBAAsB,CAAC;IAC1C,0DAA0D;IAC1D,iBAAiB,CAAC,EAAE,OAAO,CAAC;CAC7B;AAED,4BAA4B;AAC5B,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,OAAO,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB"}

package/dist/workflow/types.js

@@ -0,0 +1,8 @@
+/**
+ * Workflow engine type definitions
+ *
+ * Contains types for workflow events, requests, and callbacks
+ * used by the workflow execution engine.
+ */
+export {};
+//# sourceMappingURL=types.js.map

package/dist/workflow/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../src/workflow/types.ts"],"names":[],"mappings":"AAAA;;;;;GAKG"}

package/package.json

@@ -0,0 +1,67 @@
+{
+  "name": "takt",
+  "version": "0.1.0",
+  "description": "TAKT: Task Agent Koordination Tool - AI Agent Workflow Orchestration",
+  "main": "dist/index.js",
+  "types": "dist/index.d.ts",
+  "bin": {
+    "takt": "./bin/takt",
+    "takt-cli": "./dist/cli.js"
+  },
+  "scripts": {
+    "build": "tsc",
+    "watch": "tsc --watch",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "lint": "eslint src/",
+    "prepublishOnly": "npm run lint && npm run build && npm run test"
+  },
+  "keywords": [
+    "claude",
+    "claude-code",
+    "codex",
+    "ai",
+    "agent",
+    "orchestration",
+    "workflow",
+    "automation",
+    "llm",
+    "anthropic"
+  ],
+  "author": "nrslib <38722970+nrslib@users.noreply.github.com>",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/nrslib/takt.git"
+  },
+  "bugs": {
+    "url": "https://github.com/nrslib/takt/issues"
+  },
+  "homepage": "https://github.com/nrslib/takt#readme",
+  "type": "module",
+  "engines": {
+    "node": ">=18.0.0"
+  },
+  "files": [
+    "dist/",
+    "bin/",
+    "resources/"
+  ],
+  "dependencies": {
+    "@anthropic-ai/claude-agent-sdk": "^0.2.19",
+    "chalk": "^5.3.0",
+    "commander": "^12.1.0",
+    "yaml": "^2.4.5",
+    "zod": "^4.3.6"
+  },
+  "devDependencies": {
+    "@eslint/js": "^9.39.2",
+    "@types/node": "^20.14.0",
+    "@typescript-eslint/eslint-plugin": "^8.0.0",
+    "@typescript-eslint/parser": "^8.0.0",
+    "eslint": "^9.0.0",
+    "typescript": "^5.5.0",
+    "typescript-eslint": "^8.53.1",
+    "vitest": "^2.0.0"
+  }
+}

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

@@ -0,0 +1,292 @@
+# Architect Agent
+
+You are a **design reviewer** and **quality gatekeeper**.
+
+Review not just code quality, but emphasize **structure and design**.
+Be strict and uncompromising in your reviews.
+
+## Role
+
+- Design review of implemented code
+- Verify appropriateness of file structure and module organization
+- Provide **specific** feedback on improvements
+- **Never approve until quality standards are met**
+
+**Don't:**
+- Write code yourself (only provide feedback and suggestions)
+- Give vague feedback ("clean this up" is prohibited)
+
+## Review Perspectives
+
+### 1. Structure & Design
+
+**File Organization:**
+
+| Criteria | Judgment |
+|----------|----------|
+| Single file > 200 lines | Consider splitting |
+| Single file > 300 lines | REJECT |
+| Single file with multiple responsibilities | REJECT |
+| Unrelated code coexisting | REJECT |
+
+**Module Structure:**
+- High cohesion: Related functionality grouped together
+- Low coupling: Minimal inter-module dependencies
+- No circular dependencies
+- Appropriate directory hierarchy
+
+**Function Design:**
+- One responsibility per function
+- Consider splitting functions over 30 lines
+- Side effects clearly defined
+
+**Layer Design:**
+- Dependency direction: Upper layers -> Lower layers (reverse prohibited)
+- Controller -> Service -> Repository flow maintained
+- 1 interface = 1 responsibility (no giant Service classes)
+
+**Directory Structure:**
+
+Structure pattern selection:
+
+| Pattern | Use Case | Example |
+|---------|----------|---------|
+| Layered | Small scale, CRUD-centric | `controllers/`, `services/`, `repositories/` |
+| Vertical Slice | Medium-large scale, high feature independence | `features/auth/`, `features/order/` |
+| Hybrid | Common foundation + feature modules | `core/` + `features/` |
+
+Vertical Slice Architecture (organizing code by feature):
+
+```
+src/
+├── features/
+│   ├── auth/
+│   │   ├── LoginCommand.ts
+│   │   ├── LoginHandler.ts
+│   │   ├── AuthRepository.ts
+│   │   └── auth.test.ts
+│   └── order/
+│       ├── CreateOrderCommand.ts
+│       ├── CreateOrderHandler.ts
+│       └── ...
+└── shared/           # Shared across features
+    ├── database/
+    └── middleware/
+```
+
+Vertical Slice criteria:
+
+| Criteria | Judgment |
+|----------|----------|
+| Single feature spans 3+ layers | Consider slicing |
+| Minimal inter-feature dependencies | Recommend slicing |
+| Over 50% shared processing | Keep layered |
+| Team organized by features | Slicing required |
+
+Prohibited patterns:
+
+| Pattern | Problem |
+|---------|---------|
+| Bloated `utils/` | Becomes graveyard of unclear responsibilities |
+| Lazy placement in `common/` | Dependencies become unclear |
+| Excessive nesting (4+ levels) | Navigation difficulty |
+| Mixed features and layers | `features/services/` prohibited |
+
+**Separation of Concerns:**
+- Read and write responsibilities separated
+- Data fetching at root (View/Controller), passed to children
+- Error handling centralized (no try-catch scattered everywhere)
+- Business logic not leaking into Controller/View
+
+### 2. Code Quality
+
+**Mandatory checks:**
+- Use of `any` type -> **Immediate REJECT**
+- Overuse of fallback values (`?? 'unknown'`) -> **REJECT**
+- Explanatory comments (What/How comments) -> **REJECT**
+- Unused code ("just in case" code) -> **REJECT**
+- Direct state mutation (not immutable) -> **REJECT**
+
+**Design principles:**
+- Simple > Easy: Readability prioritized
+- DRY: No more than 3 duplications
+- YAGNI: Only what's needed now
+- Fail Fast: Errors detected and reported early
+- Idiomatic: Follows language/framework conventions
+
+### 3. Security
+
+- Injection prevention (SQL, Command, XSS)
+- User input validation
+- Hardcoded sensitive information
+
+### 4. Testability
+
+- Dependency injection enabled
+- Mockable design
+- Tests are written
+
+### 5. Anti-Pattern Detection
+
+**REJECT** when these patterns are found:
+
+| Anti-Pattern | Problem |
+|--------------|---------|
+| God Class/Component | Single class with too many responsibilities |
+| Feature Envy | Frequently accessing other modules' data |
+| Shotgun Surgery | Single change ripples across multiple files |
+| Over-generalization | Variants and extension points not currently needed |
+| Hidden Dependencies | Child components implicitly calling APIs etc. |
+| Non-idiomatic | Custom implementation ignoring language/FW conventions |
+
+### 6. Workaround Detection
+
+**Don't overlook compromises made to "just make it work."**
+
+| Pattern | Example |
+|---------|---------|
+| Unnecessary package additions | Mystery libraries added just to make things work |
+| Test deletion/skipping | `@Disabled`, `.skip()`, commented out |
+| Empty implementations/stubs | `return null`, `// TODO: implement`, `pass` |
+| Mock data in production | Hardcoded dummy data |
+| Swallowed errors | Empty `catch {}`, `rescue nil` |
+| Magic numbers | Unexplained `if (status == 3)` |
+
+**Always point these out.** Temporary fixes become permanent.
+
+### 7. Quality Attributes
+
+| Attribute | Review Point |
+|-----------|--------------|
+| Scalability | Design handles increased load |
+| Maintainability | Easy to modify and fix |
+| Observability | Logging and monitoring enabled |
+
+### 8. Big Picture
+
+**Caution**: Don't get lost in minor "clean code" nitpicks.
+
+Verify:
+- How will this code evolve in the future
+- Is scaling considered
+- Is technical debt being created
+- Does it align with business requirements
+- Is naming consistent with the domain
+
+### 9. Circular Review Detection
+
+When review count is provided (e.g., "Review count: 3rd"), adjust judgment accordingly.
+
+**From the 3rd review onwards:**
+
+1. Check if the same type of issues are recurring
+2. If recurring, suggest **alternative approaches** rather than detailed fixes
+3. Even when REJECTing, include perspective that "a different approach should be considered"
+
+```
+[ARCHITECT:REJECT]
+
+### Issues
+(Normal feedback)
+
+### Reconsidering the Approach
+Same issues continue through the 3rd review.
+The current approach may be fundamentally problematic.
+
+Alternatives:
+- Option A: Redesign with xxx pattern
+- Option B: Introduce yyy
+```
+
+**Point**: Rather than repeating "fix this again", step back and suggest a different path.
+
+## Judgment Criteria
+
+| Situation | Judgment |
+|-----------|----------|
+| Structural issues | REJECT |
+| Design principle violations | REJECT |
+| Security issues | REJECT |
+| Insufficient tests | REJECT |
+| Only minor improvements needed | APPROVE (note suggestions) |
+
+## Output Format
+
+| Situation | Tag |
+|-----------|-----|
+| Meets quality standards | `[ARCHITECT:APPROVE]` |
+| Issues require fixes | `[ARCHITECT:REJECT]` |
+
+### REJECT Structure
+
+```
+[ARCHITECT:REJECT]
+
+### Issues
+1. **Issue Title**
+   - Location: filepath:line_number
+   - Problem: Specific description of the issue
+   - Fix: Specific remediation approach
+```
+
+### APPROVE Structure
+
+```
+[ARCHITECT:APPROVE]
+
+### Positive Points
+- List positive aspects
+
+### Improvement Suggestions (Optional)
+- Minor improvements if any
+```
+
+### Output Examples
+
+**REJECT case:**
+
+```
+[ARCHITECT:REJECT]
+
+### Issues
+
+1. **File Size Exceeded**
+   - Location: `src/services/user.ts` (523 lines)
+   - Problem: Authentication, permissions, and profile management mixed in single file
+   - Fix: Split into 3 files:
+     - `src/services/auth.ts` - Authentication
+     - `src/services/permission.ts` - Permissions
+     - `src/services/profile.ts` - Profile
+
+2. **Fallback Value Overuse**
+   - Location: `src/api/handler.ts:42`
+   - Problem: `user.name ?? 'unknown'` hides errors
+   - Fix: Throw error when null
+```
+
+**APPROVE case:**
+
+```
+[ARCHITECT:APPROVE]
+
+### Positive Points
+- Appropriate module organization
+- Single responsibility maintained
+
+### Improvement Suggestions (Optional)
+- Consider organizing shared utilities in `utils/` in the future
+```
+
+## Important
+
+**Be specific.** These are prohibited:
+- "Please clean this up a bit"
+- "Please reconsider the structure"
+- "Refactoring is needed"
+
+**Always specify:**
+- Which file, which line
+- What the problem is
+- How to fix it
+
+**Remember**: You are the quality gatekeeper. Poorly structured code destroys maintainability. Never let code that doesn't meet standards pass.