sqlew 4.1.2 → 4.3.1

package/dist/tests/unit/hooks/plan-toml-parser.test.js

@@ -0,0 +1,316 @@
+/**
+ * Plan TOML Parser Unit Tests
+ *
+ * Tests for extracting and parsing [[decision]] and [[constraint]]
+ * from ```toml blocks in plan markdown files.
+ *
+ * @since v4.2.0
+ */
+import { describe, it } from 'node:test';
+import assert from 'node:assert';
+import { extractTomlBlocks, parseTomlBlock, parsePlanToml, } from '../../../cli/hooks/plan-toml-parser.js';
+describe('plan-toml-parser', () => {
+    describe('extractTomlBlocks', () => {
+        it('should extract single toml block', () => {
+            const content = `
+# Plan
+
+Some text here.
+
+\`\`\`toml
+[[decision]]
+key = "test"
+value = "value"
+\`\`\`
+
+More text.
+`;
+            const blocks = extractTomlBlocks(content);
+            assert.strictEqual(blocks.length, 1);
+            assert.ok(blocks[0].includes('[[decision]]'));
+        });
+        it('should extract multiple toml blocks', () => {
+            const content = `
+\`\`\`toml
+[[decision]]
+key = "first"
+value = "one"
+\`\`\`
+
+Some text between.
+
+\`\`\`toml
+[[decision]]
+key = "second"
+value = "two"
+\`\`\`
+`;
+            const blocks = extractTomlBlocks(content);
+            assert.strictEqual(blocks.length, 2);
+        });
+        it('should handle empty content', () => {
+            const blocks = extractTomlBlocks('');
+            assert.strictEqual(blocks.length, 0);
+        });
+        it('should ignore non-toml code blocks', () => {
+            const content = `
+\`\`\`javascript
+const x = 1;
+\`\`\`
+
+\`\`\`toml
+[[decision]]
+key = "test"
+value = "value"
+\`\`\`
+
+\`\`\`python
+print("hello")
+\`\`\`
+`;
+            const blocks = extractTomlBlocks(content);
+            assert.strictEqual(blocks.length, 1);
+        });
+        it('should be case-insensitive for toml tag', () => {
+            const content = `
+\`\`\`TOML
+[[decision]]
+key = "upper"
+value = "case"
+\`\`\`
+`;
+            const blocks = extractTomlBlocks(content);
+            assert.strictEqual(blocks.length, 1);
+        });
+    });
+    describe('parseTomlBlock - decisions', () => {
+        it('should parse decision with required fields only', () => {
+            const toml = `
+[[decision]]
+key = "api/auth"
+value = "Use JWT tokens"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.decisions.length, 1);
+            assert.strictEqual(result.decisions[0].key, 'api/auth');
+            assert.strictEqual(result.decisions[0].value, 'Use JWT tokens');
+        });
+        it('should parse decision with all optional fields', () => {
+            const toml = `
+[[decision]]
+key = "database/orm"
+value = "Knex query builder"
+status = "active"
+layer = "data"
+tags = ["database", "architecture"]
+rationale = "Explicit SQL control"
+alternatives = ["TypeORM", "Prisma"]
+tradeoffs = "Learning curve vs flexibility"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.decisions.length, 1);
+            const d = result.decisions[0];
+            assert.strictEqual(d.key, 'database/orm');
+            assert.strictEqual(d.value, 'Knex query builder');
+            assert.strictEqual(d.status, 'active');
+            assert.strictEqual(d.layer, 'data');
+            assert.deepStrictEqual(d.tags, ['database', 'architecture']);
+            assert.strictEqual(d.rationale, 'Explicit SQL control');
+            assert.deepStrictEqual(d.alternatives, ['TypeORM', 'Prisma']);
+            assert.strictEqual(d.tradeoffs, 'Learning curve vs flexibility');
+        });
+        it('should parse multiple decisions', () => {
+            const toml = `
+[[decision]]
+key = "first"
+value = "First decision"
+
+[[decision]]
+key = "second"
+value = "Second decision"
+
+[[decision]]
+key = "third"
+value = "Third decision"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.decisions.length, 3);
+        });
+        it('should skip invalid decisions (missing required fields)', () => {
+            const toml = `
+[[decision]]
+key = "valid"
+value = "This is valid"
+
+[[decision]]
+key = "missing-value"
+
+[[decision]]
+value = "missing-key"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.decisions.length, 1);
+            assert.strictEqual(result.decisions[0].key, 'valid');
+        });
+    });
+    describe('parseTomlBlock - constraints', () => {
+        it('should parse constraint with required fields only', () => {
+            const toml = `
+[[constraint]]
+text = "All API endpoints must validate JWT"
+category = "security"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.constraints.length, 1);
+            assert.strictEqual(result.constraints[0].text, 'All API endpoints must validate JWT');
+            assert.strictEqual(result.constraints[0].category, 'security');
+        });
+        it('should parse constraint with all optional fields', () => {
+            const toml = `
+[[constraint]]
+text = "No raw SQL strings"
+category = "code-style"
+priority = "high"
+layer = "data"
+tags = ["database", "security"]
+rationale = "Prevent SQL injection"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.constraints.length, 1);
+            const c = result.constraints[0];
+            assert.strictEqual(c.text, 'No raw SQL strings');
+            assert.strictEqual(c.category, 'code-style');
+            assert.strictEqual(c.priority, 'high');
+            assert.strictEqual(c.layer, 'data');
+            assert.deepStrictEqual(c.tags, ['database', 'security']);
+            assert.strictEqual(c.rationale, 'Prevent SQL injection');
+        });
+        it('should parse multiple constraints', () => {
+            const toml = `
+[[constraint]]
+text = "First rule"
+category = "security"
+
+[[constraint]]
+text = "Second rule"
+category = "performance"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.constraints.length, 2);
+        });
+        it('should skip invalid constraints (missing required fields)', () => {
+            const toml = `
+[[constraint]]
+text = "Valid constraint"
+category = "security"
+
+[[constraint]]
+text = "Missing category"
+
+[[constraint]]
+category = "missing-text"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.constraints.length, 1);
+            assert.strictEqual(result.constraints[0].text, 'Valid constraint');
+        });
+    });
+    describe('parseTomlBlock - mixed decisions and constraints', () => {
+        it('should parse both decisions and constraints from same block', () => {
+            const toml = `
+[[decision]]
+key = "auth/method"
+value = "JWT with refresh tokens"
+layer = "business"
+
+[[constraint]]
+text = "All endpoints must validate JWT"
+category = "security"
+priority = "critical"
+
+[[decision]]
+key = "database/engine"
+value = "PostgreSQL 15"
+layer = "data"
+
+[[constraint]]
+text = "Use parameterized queries only"
+category = "security"
+`;
+            const result = parseTomlBlock(toml);
+            assert.strictEqual(result.decisions.length, 2);
+            assert.strictEqual(result.constraints.length, 2);
+        });
+    });
+    describe('parsePlanToml - end-to-end', () => {
+        it('should parse complete plan markdown with toml blocks', () => {
+            const content = `
+# Implementation Plan
+
+## Overview
+This is a plan for implementing authentication.
+
+## Architectural Decisions & Constraints
+
+\`\`\`toml
+[[decision]]
+key = "auth/provider"
+value = "Custom JWT implementation"
+status = "active"
+layer = "business"
+tags = ["auth", "security"]
+rationale = "Full control over token lifecycle"
+
+[[constraint]]
+text = "JWT tokens must expire within 15 minutes"
+category = "security"
+priority = "critical"
+layer = "business"
+\`\`\`
+
+## Implementation Steps
+
+1. Create token service
+2. Add middleware
+
+## Additional Constraints
+
+\`\`\`toml
+[[constraint]]
+text = "Store refresh tokens in HTTP-only cookies"
+category = "security"
+priority = "high"
+\`\`\`
+`;
+            const result = parsePlanToml(content);
+            assert.strictEqual(result.decisions.length, 1);
+            assert.strictEqual(result.constraints.length, 2);
+            assert.strictEqual(result.decisions[0].key, 'auth/provider');
+            assert.strictEqual(result.constraints[0].text, 'JWT tokens must expire within 15 minutes');
+            assert.strictEqual(result.constraints[1].text, 'Store refresh tokens in HTTP-only cookies');
+        });
+        it('should handle plan with no toml blocks', () => {
+            const content = `
+# Simple Plan
+
+Just some text without any TOML.
+`;
+            const result = parsePlanToml(content);
+            assert.strictEqual(result.decisions.length, 0);
+            assert.strictEqual(result.constraints.length, 0);
+        });
+        it('should handle malformed toml gracefully', () => {
+            const content = `
+\`\`\`toml
+this is not valid toml
+{ broken json
+\`\`\`
+`;
+            // Should not throw, just return empty
+            const result = parsePlanToml(content);
+            assert.strictEqual(result.decisions.length, 0);
+            assert.strictEqual(result.constraints.length, 0);
+        });
+    });
+});
+//# sourceMappingURL=plan-toml-parser.test.js.map

package/dist/tests/unit/hooks/plan-toml-parser.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan-toml-parser.test.js","sourceRoot":"","sources":["../../../../src/tests/unit/hooks/plan-toml-parser.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,MAAM,MAAM,aAAa,CAAC;AACjC,OAAO,EACL,iBAAiB,EACjB,cAAc,EACd,aAAa,GACd,MAAM,wCAAwC,CAAC;AAEhD,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;QACjC,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;YAC1C,MAAM,OAAO,GAAG;;;;;;;;;;;;CAYrB,CAAC;YACI,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;YAC1C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACrC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,OAAO,GAAG;;;;;;;;;;;;;;CAcrB,CAAC;YACI,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;YAC1C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;YACrC,MAAM,MAAM,GAAG,iBAAiB,CAAC,EAAE,CAAC,CAAC;YACrC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,MAAM,OAAO,GAAG;;;;;;;;;;;;;;CAcrB,CAAC;YACI,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;YAC1C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,OAAO,GAAG;;;;;;CAMrB,CAAC;YACI,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;YAC1C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,4BAA4B,EAAE,GAAG,EAAE;QAC1C,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;YACzD,MAAM,IAAI,GAAG;;;;CAIlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,GAAG,EAAE,UAAU,CAAC,CAAC;YACxD,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,EAAE,gBAAgB,CAAC,CAAC;QAClE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,IAAI,GAAG;;;;;;;;;;CAUlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,GAAG,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;YAC9B,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;YAC1C,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,EAAE,oBAAoB,CAAC,CAAC;YAClD,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,MAAM,EAAE,QAAQ,CAAC,CAAC;YACvC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;YACpC,MAAM,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,UAAU,EAAE,cAAc,CAAC,CAAC,CAAC;YAC7D,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,SAAS,EAAE,sBAAsB,CAAC,CAAC;YACxD,MAAM,CAAC,eAAe,CAAC,CAAC,CAAC,YAAY,EAAE,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAC;YAC9D,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,SAAS,EAAE,+BAA+B,CAAC,CAAC;QACnE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;YACzC,MAAM,IAAI,GAAG;;;;;;;;;;;;CAYlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACjD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;YACjE,MAAM,IAAI,GAAG;;;;;;;;;;CAUlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,8BAA8B,EAAE,GAAG,EAAE;QAC5C,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;YAC3D,MAAM,IAAI,GAAG;;;;CAIlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACjD,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,qCAAqC,CAAC,CAAC;YACtF,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QACjE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;YAC1D,MAAM,IAAI,GAAG;;;;;;;;CAQlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACjD,MAAM,CAAC,GAAG,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,EAAE,oBAAoB,CAAC,CAAC;YACjD,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,QAAQ,EAAE,YAAY,CAAC,CAAC;YAC7C,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;YACvC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;YACpC,MAAM,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC,CAAC;YACzD,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,SAAS,EAAE,uBAAuB,CAAC,CAAC;QAC3D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,MAAM,IAAI,GAAG;;;;;;;;CAQlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;YACnE,MAAM,IAAI,GAAG;;;;;;;;;;CAUlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACjD,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,kBAAkB,CAAC,CAAC;QACrE,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAChE,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;YACrE,MAAM,IAAI,GAAG;;;;;;;;;;;;;;;;;;;CAmBlB,CAAC;YACI,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,4BAA4B,EAAE,GAAG,EAAE;QAC1C,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;YAC9D,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAqCrB,CAAC;YACI,MAAM,MAAM,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC;YACtC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACjD,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,GAAG,EAAE,eAAe,CAAC,CAAC;YAC7D,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,0CAA0C,CAAC,CAAC;YAC3F,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,2CAA2C,CAAC,CAAC;QAC9F,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,OAAO,GAAG;;;;CAIrB,CAAC;YACI,MAAM,MAAM,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC;YACtC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,OAAO,GAAG;;;;;CAKrB,CAAC;YACI,sCAAsC;YACtC,MAAM,MAAM,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC;YACtC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tools/constraints/actions/activate.d.ts

@@ -0,0 +1,46 @@
+/**
+ * Activate constraints (by tag or by ID)
+ *
+ * - activateConstraintsByTag: Bulk activate by tag (for plan-based workflow)
+ * - activateConstraint: Single constraint by ID (MCP action)
+ *
+ * @since v4.2.1
+ */
+import { DatabaseAdapter } from '../../../adapters/index.js';
+/**
+ * Activate constraints by tag response
+ */
+export interface ActivateConstraintsResponse {
+    success: boolean;
+    activated_count: number;
+}
+/**
+ * Activate constraint by ID response
+ */
+export interface ActivateConstraintByIdResponse {
+    success: boolean;
+    constraint_id: number;
+    message: string;
+}
+/**
+ * Activate all constraints matching a specific tag
+ *
+ * Used by queue-watcher to activate constraints when implementation starts.
+ * The tag typically contains the plan_id short form (8 chars).
+ *
+ * @param tag - Tag to match (e.g., plan_id short form)
+ * @param adapter - Optional database adapter (for testing)
+ * @returns Number of constraints activated
+ */
+export declare function activateConstraintsByTag(tag: string, adapter?: DatabaseAdapter): Promise<ActivateConstraintsResponse>;
+/**
+ * Activate a single constraint by ID
+ *
+ * @param params - Parameters containing constraint_id
+ * @param adapter - Optional database adapter (for testing)
+ * @returns Success response with constraint ID
+ */
+export declare function activateConstraint(params: {
+    constraint_id: number;
+}, adapter?: DatabaseAdapter): Promise<ActivateConstraintByIdResponse>;
+//# sourceMappingURL=activate.d.ts.map

package/dist/tools/constraints/actions/activate.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"activate.d.ts","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/activate.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAO7D;;GAEG;AACH,MAAM,WAAW,2BAA2B;IAC1C,OAAO,EAAE,OAAO,CAAC;IACjB,eAAe,EAAE,MAAM,CAAC;CACzB;AAED;;GAEG;AACH,MAAM,WAAW,8BAA8B;IAC7C,OAAO,EAAE,OAAO,CAAC;IACjB,aAAa,EAAE,MAAM,CAAC;IACtB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED;;;;;;;;;GASG;AACH,wBAAsB,wBAAwB,CAC5C,GAAG,EAAE,MAAM,EACX,OAAO,CAAC,EAAE,eAAe,GACxB,OAAO,CAAC,2BAA2B,CAAC,CAqCtC;AAED;;;;;;GAMG;AACH,wBAAsB,kBAAkB,CACtC,MAAM,EAAE;IAAE,aAAa,EAAE,MAAM,CAAA;CAAE,EACjC,OAAO,CAAC,EAAE,eAAe,GACxB,OAAO,CAAC,8BAA8B,CAAC,CA6CzC"}

package/dist/tools/constraints/actions/activate.js

@@ -0,0 +1,105 @@
+/**
+ * Activate constraints (by tag or by ID)
+ *
+ * - activateConstraintsByTag: Bulk activate by tag (for plan-based workflow)
+ * - activateConstraint: Single constraint by ID (MCP action)
+ *
+ * @since v4.2.1
+ */
+import { getAdapter } from '../../../database.js';
+import { getProjectContext } from '../../../utils/project-context.js';
+import { normalizeParams, CONSTRAINT_ALIASES } from '../../../utils/param-normalizer.js';
+import connectionManager from '../../../utils/connection-manager.js';
+import { SQLITE_TRUE } from '../../../constants.js';
+/**
+ * Activate all constraints matching a specific tag
+ *
+ * Used by queue-watcher to activate constraints when implementation starts.
+ * The tag typically contains the plan_id short form (8 chars).
+ *
+ * @param tag - Tag to match (e.g., plan_id short form)
+ * @param adapter - Optional database adapter (for testing)
+ * @returns Number of constraints activated
+ */
+export async function activateConstraintsByTag(tag, adapter) {
+    const actualAdapter = adapter ?? getAdapter();
+    const knex = actualAdapter.getKnex();
+    try {
+        return await connectionManager.executeWithRetry(async () => {
+            const projectId = getProjectContext().getProjectId();
+            // Find constraints with matching tag that are inactive
+            const constraintIds = await knex('v4_constraints as c')
+                .join('v4_constraint_tags as ct', 'c.id', 'ct.constraint_id')
+                .join('v4_tags as t', 'ct.tag_id', 't.id')
+                .where('c.project_id', projectId)
+                .where('c.active', 0)
+                .where('t.name', tag)
+                .select('c.id')
+                .then(rows => rows.map(r => r.id));
+            if (constraintIds.length === 0) {
+                return { success: true, activated_count: 0 };
+            }
+            // Activate all matching constraints
+            await knex('v4_constraints')
+                .whereIn('id', constraintIds)
+                .where('project_id', projectId)
+                .update({ active: SQLITE_TRUE });
+            return {
+                success: true,
+                activated_count: constraintIds.length,
+            };
+        });
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Failed to activate constraints: ${message}`);
+    }
+}
+/**
+ * Activate a single constraint by ID
+ *
+ * @param params - Parameters containing constraint_id
+ * @param adapter - Optional database adapter (for testing)
+ * @returns Success response with constraint ID
+ */
+export async function activateConstraint(params, adapter) {
+    const actualAdapter = adapter ?? getAdapter();
+    const knex = actualAdapter.getKnex();
+    // Normalize aliases: id → constraint_id
+    const normalizedParams = normalizeParams(params, CONSTRAINT_ALIASES);
+    try {
+        return await connectionManager.executeWithRetry(async () => {
+            const projectId = getProjectContext().getProjectId();
+            // Check if constraint exists
+            const constraint = await knex('v4_constraints')
+                .where('id', normalizedParams.constraint_id)
+                .where('project_id', projectId)
+                .first();
+            if (!constraint) {
+                throw new Error(`Constraint not found: ${normalizedParams.constraint_id}`);
+            }
+            if (constraint.active === 1) {
+                return {
+                    success: true,
+                    constraint_id: normalizedParams.constraint_id,
+                    message: 'Constraint already active',
+                };
+            }
+            // Activate the constraint
+            await knex('v4_constraints')
+                .where('id', normalizedParams.constraint_id)
+                .where('project_id', projectId)
+                .update({ active: SQLITE_TRUE });
+            return {
+                success: true,
+                constraint_id: normalizedParams.constraint_id,
+                message: 'Constraint activated',
+            };
+        });
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Failed to activate constraint: ${message}`);
+    }
+}
+//# sourceMappingURL=activate.js.map

package/dist/tools/constraints/actions/activate.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"activate.js","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/activate.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAGH,OAAO,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AAClD,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,oCAAoC,CAAC;AACzF,OAAO,iBAAiB,MAAM,sCAAsC,CAAC;AACrE,OAAO,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAmBpD;;;;;;;;;GASG;AACH,MAAM,CAAC,KAAK,UAAU,wBAAwB,CAC5C,GAAW,EACX,OAAyB;IAEzB,MAAM,aAAa,GAAG,OAAO,IAAI,UAAU,EAAE,CAAC;IAC9C,MAAM,IAAI,GAAG,aAAa,CAAC,OAAO,EAAE,CAAC;IAErC,IAAI,CAAC;QACH,OAAO,MAAM,iBAAiB,CAAC,gBAAgB,CAAC,KAAK,IAAI,EAAE;YACzD,MAAM,SAAS,GAAG,iBAAiB,EAAE,CAAC,YAAY,EAAE,CAAC;YAErD,uDAAuD;YACvD,MAAM,aAAa,GAAG,MAAM,IAAI,CAAC,qBAAqB,CAAC;iBACpD,IAAI,CAAC,0BAA0B,EAAE,MAAM,EAAE,kBAAkB,CAAC;iBAC5D,IAAI,CAAC,cAAc,EAAE,WAAW,EAAE,MAAM,CAAC;iBACzC,KAAK,CAAC,cAAc,EAAE,SAAS,CAAC;iBAChC,KAAK,CAAC,UAAU,EAAE,CAAC,CAAC;iBACpB,KAAK,CAAC,QAAQ,EAAE,GAAG,CAAC;iBACpB,MAAM,CAAC,MAAM,CAAC;iBACd,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAY,CAAC,CAAC,CAAC;YAE/C,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAC/B,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,eAAe,EAAE,CAAC,EAAE,CAAC;YAC/C,CAAC;YAED,oCAAoC;YACpC,MAAM,IAAI,CAAC,gBAAgB,CAAC;iBACzB,OAAO,CAAC,IAAI,EAAE,aAAa,CAAC;iBAC5B,KAAK,CAAC,YAAY,EAAE,SAAS,CAAC;iBAC9B,MAAM,CAAC,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,CAAC;YAEnC,OAAO;gBACL,OAAO,EAAE,IAAI;gBACb,eAAe,EAAE,aAAa,CAAC,MAAM;aACtC,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,MAAM,IAAI,KAAK,CAAC,mCAAmC,OAAO,EAAE,CAAC,CAAC;IAChE,CAAC;AACH,CAAC;AAED;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,MAAiC,EACjC,OAAyB;IAEzB,MAAM,aAAa,GAAG,OAAO,IAAI,UAAU,EAAE,CAAC;IAC9C,MAAM,IAAI,GAAG,aAAa,CAAC,OAAO,EAAE,CAAC;IAErC,wCAAwC;IACxC,MAAM,gBAAgB,GAAG,eAAe,CAAC,MAAM,EAAE,kBAAkB,CAA8B,CAAC;IAElG,IAAI,CAAC;QACH,OAAO,MAAM,iBAAiB,CAAC,gBAAgB,CAAC,KAAK,IAAI,EAAE;YACzD,MAAM,SAAS,GAAG,iBAAiB,EAAE,CAAC,YAAY,EAAE,CAAC;YAErD,6BAA6B;YAC7B,MAAM,UAAU,GAAG,MAAM,IAAI,CAAC,gBAAgB,CAAC;iBAC5C,KAAK,CAAC,IAAI,EAAE,gBAAgB,CAAC,aAAa,CAAC;iBAC3C,KAAK,CAAC,YAAY,EAAE,SAAS,CAAC;iBAC9B,KAAK,EAAE,CAAC;YAEX,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,MAAM,IAAI,KAAK,CAAC,yBAAyB,gBAAgB,CAAC,aAAa,EAAE,CAAC,CAAC;YAC7E,CAAC;YAED,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAC5B,OAAO;oBACL,OAAO,EAAE,IAAI;oBACb,aAAa,EAAE,gBAAgB,CAAC,aAAa;oBAC7C,OAAO,EAAE,2BAA2B;iBACrC,CAAC;YACJ,CAAC;YAED,0BAA0B;YAC1B,MAAM,IAAI,CAAC,gBAAgB,CAAC;iBACzB,KAAK,CAAC,IAAI,EAAE,gBAAgB,CAAC,aAAa,CAAC;iBAC3C,KAAK,CAAC,YAAY,EAAE,SAAS,CAAC;iBAC9B,MAAM,CAAC,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,CAAC;YAEnC,OAAO;gBACL,OAAO,EAAE,IAAI;gBACb,aAAa,EAAE,gBAAgB,CAAC,aAAa;gBAC7C,OAAO,EAAE,sBAAsB;aAChC,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,MAAM,IAAI,KAAK,CAAC,kCAAkC,OAAO,EAAE,CAAC,CAAC;IAC/D,CAAC;AACH,CAAC"}

package/dist/tools/constraints/actions/add.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"add.d.ts","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/add.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAkB7D,OAAO,KAAK,EACV,mBAAmB,EACnB,qBAAqB,EACtB,MAAM,aAAa,CAAC;AAErB;;;;;;GAMG;AACH,wBAAsB,aAAa,CACjC,MAAM,EAAE,mBAAmB,EAC3B,OAAO,CAAC,EAAE,eAAe,GACxB,OAAO,CAAC,qBAAqB,CAAC,CA6EhC"}
+{"version":3,"file":"add.d.ts","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/add.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAmB7D,OAAO,KAAK,EACV,mBAAmB,EACnB,qBAAqB,EACtB,MAAM,aAAa,CAAC;AAErB;;;;;;GAMG;AACH,wBAAsB,aAAa,CACjC,MAAM,EAAE,mBAAmB,EAC3B,OAAO,CAAC,EAAE,eAAe,GACxB,OAAO,CAAC,qBAAqB,CAAC,CA+FhC"}

package/dist/tools/constraints/actions/add.js

@@ -6,6 +6,7 @@ import { getAdapter, getLayerId, getOrCreateTag, getOrCreateCategoryId } from '.
 import { STRING_TO_PRIORITY, DEFAULT_PRIORITY, SQLITE_TRUE, STANDARD_LAYERS } from '../../../constants.js';
 import { validateCategory, validatePriority } from '../../../utils/validators.js';
 import { validateActionParams } from '../../../utils/parameter-validator.js';
+import { normalizeParams, CONSTRAINT_ALIASES } from '../../../utils/param-normalizer.js';
 import { parseStringArray } from '../../../utils/param-parser.js';
 import { getProjectContext } from '../../../utils/project-context.js';
 import connectionManager from '../../../utils/connection-manager.js';
@@ -18,50 +19,65 @@ import connectionManager from '../../../utils/connection-manager.js';
  */
 export async function addConstraint(params, adapter) {
     const actualAdapter = adapter ?? getAdapter();
+    // Normalize aliases: text → constraint_text
+    const normalizedParams = normalizeParams(params, CONSTRAINT_ALIASES);
     try {
         return await connectionManager.executeWithRetry(async () => {
             // Fail-fast project_id validation (Constraint #29)
             const projectId = getProjectContext().getProjectId();
             // Validate parameters
-            validateActionParams('constraint', 'add', params);
+            validateActionParams('constraint', 'add', normalizedParams);
             // Validate category
-            validateCategory(params.category);
+            validateCategory(normalizedParams.category);
             // Validate priority if provided
-            const priorityStr = params.priority || 'medium';
+            const priorityStr = normalizedParams.priority || 'medium';
             validatePriority(priorityStr);
             const priority = STRING_TO_PRIORITY[priorityStr] || DEFAULT_PRIORITY;
             // Validate and get layer ID if provided
             let layerId = null;
-            if (params.layer) {
-                if (!STANDARD_LAYERS.includes(params.layer)) {
+            if (normalizedParams.layer) {
+                if (!STANDARD_LAYERS.includes(normalizedParams.layer)) {
                     throw new Error(`Invalid layer. Must be one of: ${STANDARD_LAYERS.join(', ')}`);
                 }
-                layerId = await getLayerId(actualAdapter, params.layer);
+                layerId = await getLayerId(actualAdapter, normalizedParams.layer);
                 if (!layerId) {
-                    throw new Error(`Layer not found: ${params.layer}`);
+                    throw new Error(`Layer not found: ${normalizedParams.layer}`);
                 }
             }
             // Use transaction for multi-table insert
             const result = await actualAdapter.transaction(async (trx) => {
                 // Get or create category
-                const categoryId = await getOrCreateCategoryId(actualAdapter, params.category, trx);
+                const categoryId = await getOrCreateCategoryId(actualAdapter, normalizedParams.category, trx);
+                // Duplicate check: skip if same text + category already exists
+                const existing = await trx('v4_constraints')
+                    .where({
+                    constraint_text: normalizedParams.constraint_text,
+                    category_id: categoryId,
+                    project_id: projectId
+                })
+                    .first();
+                if (existing) {
+                    return { constraintId: existing.id, alreadyExists: true };
+                }
                 // Note: Agent tracking removed in v4.0 (created_by param kept for API compatibility but not stored)
                 // Calculate timestamp
                 const ts = Math.floor(Date.now() / 1000);
                 // Insert constraint with project_id (agent_id removed in v4.0)
+                // v4.2.1: Support active parameter for plan-based workflow
+                const activeValue = normalizedParams.active === false ? 0 : SQLITE_TRUE;
                 const [constraintId] = await trx('v4_constraints').insert({
                     category_id: categoryId,
                     layer_id: layerId,
-                    constraint_text: params.constraint_text,
+                    constraint_text: normalizedParams.constraint_text,
                     priority: priority,
-                    active: SQLITE_TRUE,
+                    active: activeValue,
                     ts: ts,
                     project_id: projectId
                 });
                 // Insert m_tags if provided
-                if (params.tags && params.tags.length > 0) {
+                if (normalizedParams.tags && normalizedParams.tags.length > 0) {
                     // Parse tags (handles both arrays and JSON strings from MCP)
-                    const tags = parseStringArray(params.tags);
+                    const tags = parseStringArray(normalizedParams.tags);
                     for (const tagName of tags) {
                         const tagId = await getOrCreateTag(actualAdapter, projectId, tagName, trx); // v3.7.3: pass projectId
                         await trx('v4_constraint_tags').insert({
@@ -70,11 +86,12 @@ export async function addConstraint(params, adapter) {
                         });
                     }
                 }
-                return { constraintId: Number(constraintId) };
+                return { constraintId: Number(constraintId), alreadyExists: false };
             });
             return {
                 success: true,
                 constraint_id: result.constraintId,
+                already_exists: result.alreadyExists,
             };
         });
     }

package/dist/tools/constraints/actions/add.js.map

@@ -1 +1 @@
-{"version":3,"file":"add.js","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/add.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,OAAO,EACL,UAAU,EACV,UAAU,EACV,cAAc,EACd,qBAAqB,EACtB,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EACL,kBAAkB,EAClB,gBAAgB,EAChB,WAAW,EACX,eAAe,EAChB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,MAAM,8BAA8B,CAAC;AAClF,OAAO,EAAE,oBAAoB,EAAE,MAAM,uCAAuC,CAAC;AAC7E,OAAO,EAAE,gBAAgB,EAAE,MAAM,gCAAgC,CAAC;AAClE,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,iBAAiB,MAAM,sCAAsC,CAAC;AAMrE;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,MAA2B,EAC3B,OAAyB;IAEzB,MAAM,aAAa,GAAG,OAAO,IAAI,UAAU,EAAE,CAAC;IAE9C,IAAI,CAAC;QACH,OAAO,MAAM,iBAAiB,CAAC,gBAAgB,CAAC,KAAK,IAAI,EAAE;YACzD,mDAAmD;YACnD,MAAM,SAAS,GAAG,iBAAiB,EAAE,CAAC,YAAY,EAAE,CAAC;YAErD,sBAAsB;YACtB,oBAAoB,CAAC,YAAY,EAAE,KAAK,EAAE,MAAM,CAAC,CAAC;YAElD,oBAAoB;YACpB,gBAAgB,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;YAElC,gCAAgC;YAChC,MAAM,WAAW,GAAG,MAAM,CAAC,QAAQ,IAAI,QAAQ,CAAC;YAChD,gBAAgB,CAAC,WAAW,CAAC,CAAC;YAC9B,MAAM,QAAQ,GAAG,kBAAkB,CAAC,WAAW,CAAC,IAAI,gBAAgB,CAAC;YAErE,wCAAwC;YACxC,IAAI,OAAO,GAAkB,IAAI,CAAC;YAClC,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC;gBACjB,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,MAAM,CAAC,KAAY,CAAC,EAAE,CAAC;oBACnD,MAAM,IAAI,KAAK,CAAC,kCAAkC,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;gBAClF,CAAC;gBACD,OAAO,GAAG,MAAM,UAAU,CAAC,aAAa,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;gBACxD,IAAI,CAAC,OAAO,EAAE,CAAC;oBACb,MAAM,IAAI,KAAK,CAAC,oBAAoB,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;gBACtD,CAAC;YACH,CAAC;YAED,yCAAyC;YACzC,MAAM,MAAM,GAAG,MAAM,aAAa,CAAC,WAAW,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE;gBAC3D,yBAAyB;gBACzB,MAAM,UAAU,GAAG,MAAM,qBAAqB,CAAC,aAAa,EAAE,MAAM,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC;gBAEpF,oGAAoG;gBAEpG,sBAAsB;gBACtB,MAAM,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;gBAEzC,+DAA+D;gBAC/D,MAAM,CAAC,YAAY,CAAC,GAAG,MAAM,GAAG,CAAC,gBAAgB,CAAC,CAAC,MAAM,CAAC;oBACxD,WAAW,EAAE,UAAU;oBACvB,QAAQ,EAAE,OAAO;oBACjB,eAAe,EAAE,MAAM,CAAC,eAAe;oBACvC,QAAQ,EAAE,QAAQ;oBAClB,MAAM,EAAE,WAAW;oBACnB,EAAE,EAAE,EAAE;oBACN,UAAU,EAAE,SAAS;iBACtB,CAAC,CAAC;gBAEH,4BAA4B;gBAC5B,IAAI,MAAM,CAAC,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;oBAC1C,6DAA6D;oBAC7D,MAAM,IAAI,GAAG,gBAAgB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;oBAC3C,KAAK,MAAM,OAAO,IAAI,IAAI,EAAE,CAAC;wBAC3B,MAAM,KAAK,GAAG,MAAM,cAAc,CAAC,aAAa,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,CAAC,CAAC,CAAE,yBAAyB;wBACtG,MAAM,GAAG,CAAC,oBAAoB,CAAC,CAAC,MAAM,CAAC;4BACrC,aAAa,EAAE,MAAM,CAAC,YAAY,CAAC;4BACnC,MAAM,EAAE,KAAK;yBACd,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;gBAED,OAAO,EAAE,YAAY,EAAE,MAAM,CAAC,YAAY,CAAC,EAAE,CAAC;YAChD,CAAC,CAAC,CAAC;YAEH,OAAO;gBACL,OAAO,EAAE,IAAI;gBACb,aAAa,EAAE,MAAM,CAAC,YAAY;aACnC,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,MAAM,IAAI,KAAK,CAAC,6BAA6B,OAAO,EAAE,CAAC,CAAC;IAC1D,CAAC;AACH,CAAC"}
+{"version":3,"file":"add.js","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/add.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,OAAO,EACL,UAAU,EACV,UAAU,EACV,cAAc,EACd,qBAAqB,EACtB,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EACL,kBAAkB,EAClB,gBAAgB,EAChB,WAAW,EACX,eAAe,EAChB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,MAAM,8BAA8B,CAAC;AAClF,OAAO,EAAE,oBAAoB,EAAE,MAAM,uCAAuC,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,oCAAoC,CAAC;AACzF,OAAO,EAAE,gBAAgB,EAAE,MAAM,gCAAgC,CAAC;AAClE,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,iBAAiB,MAAM,sCAAsC,CAAC;AAMrE;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,MAA2B,EAC3B,OAAyB;IAEzB,MAAM,aAAa,GAAG,OAAO,IAAI,UAAU,EAAE,CAAC;IAE9C,4CAA4C;IAC5C,MAAM,gBAAgB,GAAG,eAAe,CAAC,MAAM,EAAE,kBAAkB,CAAwB,CAAC;IAE5F,IAAI,CAAC;QACH,OAAO,MAAM,iBAAiB,CAAC,gBAAgB,CAAC,KAAK,IAAI,EAAE;YACzD,mDAAmD;YACnD,MAAM,SAAS,GAAG,iBAAiB,EAAE,CAAC,YAAY,EAAE,CAAC;YAErD,sBAAsB;YACtB,oBAAoB,CAAC,YAAY,EAAE,KAAK,EAAE,gBAAgB,CAAC,CAAC;YAE5D,oBAAoB;YACpB,gBAAgB,CAAC,gBAAgB,CAAC,QAAQ,CAAC,CAAC;YAE5C,gCAAgC;YAChC,MAAM,WAAW,GAAG,gBAAgB,CAAC,QAAQ,IAAI,QAAQ,CAAC;YAC1D,gBAAgB,CAAC,WAAW,CAAC,CAAC;YAC9B,MAAM,QAAQ,GAAG,kBAAkB,CAAC,WAAW,CAAC,IAAI,gBAAgB,CAAC;YAErE,wCAAwC;YACxC,IAAI,OAAO,GAAkB,IAAI,CAAC;YAClC,IAAI,gBAAgB,CAAC,KAAK,EAAE,CAAC;gBAC3B,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,gBAAgB,CAAC,KAAY,CAAC,EAAE,CAAC;oBAC7D,MAAM,IAAI,KAAK,CAAC,kCAAkC,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;gBAClF,CAAC;gBACD,OAAO,GAAG,MAAM,UAAU,CAAC,aAAa,EAAE,gBAAgB,CAAC,KAAK,CAAC,CAAC;gBAClE,IAAI,CAAC,OAAO,EAAE,CAAC;oBACb,MAAM,IAAI,KAAK,CAAC,oBAAoB,gBAAgB,CAAC,KAAK,EAAE,CAAC,CAAC;gBAChE,CAAC;YACH,CAAC;YAED,yCAAyC;YACzC,MAAM,MAAM,GAAG,MAAM,aAAa,CAAC,WAAW,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE;gBAC3D,yBAAyB;gBACzB,MAAM,UAAU,GAAG,MAAM,qBAAqB,CAAC,aAAa,EAAE,gBAAgB,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC;gBAE9F,+DAA+D;gBAC/D,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,gBAAgB,CAAC;qBACzC,KAAK,CAAC;oBACL,eAAe,EAAE,gBAAgB,CAAC,eAAe;oBACjD,WAAW,EAAE,UAAU;oBACvB,UAAU,EAAE,SAAS;iBACtB,CAAC;qBACD,KAAK,EAAE,CAAC;gBACX,IAAI,QAAQ,EAAE,CAAC;oBACb,OAAO,EAAE,YAAY,EAAE,QAAQ,CAAC,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC;gBAC5D,CAAC;gBAED,oGAAoG;gBAEpG,sBAAsB;gBACtB,MAAM,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;gBAEzC,+DAA+D;gBAC/D,2DAA2D;gBAC3D,MAAM,WAAW,GAAG,gBAAgB,CAAC,MAAM,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC;gBACxE,MAAM,CAAC,YAAY,CAAC,GAAG,MAAM,GAAG,CAAC,gBAAgB,CAAC,CAAC,MAAM,CAAC;oBACxD,WAAW,EAAE,UAAU;oBACvB,QAAQ,EAAE,OAAO;oBACjB,eAAe,EAAE,gBAAgB,CAAC,eAAe;oBACjD,QAAQ,EAAE,QAAQ;oBAClB,MAAM,EAAE,WAAW;oBACnB,EAAE,EAAE,EAAE;oBACN,UAAU,EAAE,SAAS;iBACtB,CAAC,CAAC;gBAEH,4BAA4B;gBAC5B,IAAI,gBAAgB,CAAC,IAAI,IAAI,gBAAgB,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;oBAC9D,6DAA6D;oBAC7D,MAAM,IAAI,GAAG,gBAAgB,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC;oBACrD,KAAK,MAAM,OAAO,IAAI,IAAI,EAAE,CAAC;wBAC3B,MAAM,KAAK,GAAG,MAAM,cAAc,CAAC,aAAa,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,CAAC,CAAC,CAAE,yBAAyB;wBACtG,MAAM,GAAG,CAAC,oBAAoB,CAAC,CAAC,MAAM,CAAC;4BACrC,aAAa,EAAE,MAAM,CAAC,YAAY,CAAC;4BACnC,MAAM,EAAE,KAAK;yBACd,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;gBAED,OAAO,EAAE,YAAY,EAAE,MAAM,CAAC,YAAY,CAAC,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC;YACtE,CAAC,CAAC,CAAC;YAEH,OAAO;gBACL,OAAO,EAAE,IAAI;gBACb,aAAa,EAAE,MAAM,CAAC,YAAY;gBAClC,cAAc,EAAE,MAAM,CAAC,aAAa;aACrC,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,MAAM,IAAI,KAAK,CAAC,6BAA6B,OAAO,EAAE,CAAC,CAAC;IAC1D,CAAC;AACH,CAAC"}

package/dist/tools/constraints/actions/deactivate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"deactivate.d.ts","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/deactivate.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAM7D,OAAO,KAAK,EACV,0BAA0B,EAC1B,4BAA4B,EAC7B,MAAM,aAAa,CAAC;AAErB;;;;;;;GAOG;AACH,wBAAsB,oBAAoB,CACxC,MAAM,EAAE,0BAA0B,EAClC,OAAO,CAAC,EAAE,eAAe,GACxB,OAAO,CAAC,4BAA4B,CAAC,CAmCvC"}
+{"version":3,"file":"deactivate.d.ts","sourceRoot":"","sources":["../../../../src/tools/constraints/actions/deactivate.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAO7D,OAAO,KAAK,EACV,0BAA0B,EAC1B,4BAA4B,EAC7B,MAAM,aAAa,CAAC;AAErB;;;;;;;GAOG;AACH,wBAAsB,oBAAoB,CACxC,MAAM,EAAE,0BAA0B,EAClC,OAAO,CAAC,EAAE,eAAe,GACxB,OAAO,CAAC,4BAA4B,CAAC,CAsCvC"}

package/dist/tools/constraints/actions/deactivate.js

@@ -4,6 +4,7 @@
  */
 import { getAdapter } from '../../../database.js';
 import { validateActionParams } from '../../../utils/parameter-validator.js';
+import { normalizeParams, CONSTRAINT_ALIASES } from '../../../utils/param-normalizer.js';
 import { getProjectContext } from '../../../utils/project-context.js';
 import connectionManager from '../../../utils/connection-manager.js';
 import { SQLITE_FALSE } from '../../../constants.js';
@@ -18,23 +19,25 @@ import { SQLITE_FALSE } from '../../../constants.js';
 export async function deactivateConstraint(params, adapter) {
     const actualAdapter = adapter ?? getAdapter();
     const knex = actualAdapter.getKnex();
+    // Normalize aliases: id → constraint_id
+    const normalizedParams = normalizeParams(params, CONSTRAINT_ALIASES);
     try {
         return await connectionManager.executeWithRetry(async () => {
             // Fail-fast project_id validation (Constraint #29)
             const projectId = getProjectContext().getProjectId();
             // Validate parameters
-            validateActionParams('constraint', 'deactivate', params);
+            validateActionParams('constraint', 'deactivate', normalizedParams);
             // Check if constraint exists in current project
             const constraint = await knex('v4_constraints')
-                .where({ id: params.constraint_id, project_id: projectId })
+                .where({ id: normalizedParams.constraint_id, project_id: projectId })
                 .select('id', 'active')
                 .first();
             if (!constraint) {
-                throw new Error(`Constraint not found: ${params.constraint_id}`);
+                throw new Error(`Constraint not found: ${normalizedParams.constraint_id}`);
             }
             // Update constraint to inactive (idempotent) with project_id filter
             await knex('v4_constraints')
-                .where({ id: params.constraint_id, project_id: projectId })
+                .where({ id: normalizedParams.constraint_id, project_id: projectId })
                 .update({ active: SQLITE_FALSE });
             return {
                 success: true,