sqlew 3.2.4 → 3.5.3

package/dist/tests/tasks.auto-pruning-safety.test.js

@@ -0,0 +1,196 @@
+/**
+ * Integration tests for v3.5.0 Auto-Pruning Safety Check
+ * Tests that the system blocks task transition when ALL watched files are non-existent
+ *
+ * Test Scenario: Zero Work Done Protection
+ * - Create task with 3 watched files
+ * - Make ALL 3 files non-existent
+ * - Trigger detectAndTransitionToReview()
+ * - Expected: Error thrown, task status unchanged, transaction rolled back
+ */
+import { describe, it, beforeEach, afterEach } from 'node:test';
+import assert from 'node:assert/strict';
+import Database from 'better-sqlite3';
+import { initializeSchema } from '../schema.js';
+import { runAllMigrations } from '../migrations/index.js';
+import { detectAndTransitionToReview } from '../utils/task-stale-detection.js';
+describe('Auto-pruning: Safety check when all files pruned', () => {
+    let db;
+    beforeEach(() => {
+        db = new Database(':memory:');
+        db.pragma('foreign_keys = ON');
+        initializeSchema(db);
+        runAllMigrations(db);
+    });
+    afterEach(() => {
+        db.close();
+    });
+    it('should block transition when all watched files are non-existent', async () => {
+        // 1. Create test task in 'in_progress' status
+        const taskId = createTestTask(db);
+        // 2. Add 3 non-existent files to watch list
+        // Using paths that will never exist in the test environment
+        const nonExistentFiles = [
+            '/tmp/never-created-file-1.ts',
+            '/tmp/never-created-file-2.ts',
+            '/tmp/never-created-file-3.ts'
+        ];
+        addWatchedFiles(db, taskId, nonExistentFiles);
+        // 3. Make task appear "stale" by backdating its updated_ts
+        // Set updated_ts to 10 minutes ago (older than default 3-minute idle threshold)
+        const tenMinutesAgo = Math.floor(Date.now() / 1000) - (10 * 60);
+        db.prepare(`
+      UPDATE t_tasks
+      SET updated_ts = ?
+      WHERE id = ?
+    `).run(tenMinutesAgo, taskId);
+        // 4. Verify task is in 'in_progress' before attempt
+        const beforeStatus = db.prepare(`
+      SELECT s.name
+      FROM t_tasks t
+      JOIN m_task_statuses s ON t.status_id = s.id
+      WHERE t.id = ?
+    `).get(taskId);
+        assert.strictEqual(beforeStatus.name, 'in_progress', 'Task should start in in_progress');
+        // 5. Verify watch list has all 3 files before attempt
+        const beforeWatchCount = db.prepare(`
+      SELECT COUNT(*) as count FROM t_task_file_links WHERE task_id = ?
+    `).get(taskId);
+        assert.strictEqual(beforeWatchCount.count, 3, 'Watch list should have 3 files before pruning attempt');
+        // 6. Attempt transition - should NOT throw at top level
+        // The error should be caught internally and logged
+        // Task should remain in in_progress without transitioning
+        const transitioned = await detectAndTransitionToReview(db);
+        // 7. Verify zero tasks were transitioned
+        assert.strictEqual(transitioned, 0, 'No tasks should have been transitioned');
+        // 8. Verify task status unchanged (remains in in_progress)
+        const afterStatus = db.prepare(`
+      SELECT s.name
+      FROM t_tasks t
+      JOIN m_task_statuses s ON t.status_id = s.id
+      WHERE t.id = ?
+    `).get(taskId);
+        assert.strictEqual(afterStatus.name, 'in_progress', 'Task should remain in in_progress after safety check');
+        // 9. Verify NO audit records created (transaction rollback)
+        const prunedCount = db.prepare(`
+      SELECT COUNT(*) as count FROM t_task_pruned_files WHERE task_id = ?
+    `).get(taskId);
+        assert.strictEqual(prunedCount.count, 0, 'Should have no pruned file records due to rollback');
+        // 10. Verify watch list is NOT empty (transaction rolled back)
+        const afterWatchCount = db.prepare(`
+      SELECT COUNT(*) as count FROM t_task_file_links WHERE task_id = ?
+    `).get(taskId);
+        assert.strictEqual(afterWatchCount.count, 3, 'Watch list should still have all 3 files after rollback');
+    });
+    it('should prune SOME non-existent files and continue (partial prune)', async () => {
+        // Create task
+        const taskId = createTestTask(db);
+        // Add mix of existent and non-existent files
+        // Use package.json as a file that definitely exists in the project root
+        const mixedFiles = [
+            'package.json', // exists
+            '/tmp/never-created-file-1.ts', // does not exist
+            '/tmp/never-created-file-2.ts' // does not exist
+        ];
+        addWatchedFiles(db, taskId, mixedFiles);
+        // Make task appear stale
+        const tenMinutesAgo = Math.floor(Date.now() / 1000) - (10 * 60);
+        db.prepare(`
+      UPDATE t_tasks
+      SET updated_ts = ?
+      WHERE id = ?
+    `).run(tenMinutesAgo, taskId);
+        // Verify initial watch count
+        const beforeWatchCount = db.prepare(`
+      SELECT COUNT(*) as count FROM t_task_file_links WHERE task_id = ?
+    `).get(taskId);
+        assert.strictEqual(beforeWatchCount.count, 3, 'Watch list should start with 3 files');
+        // Attempt transition - should proceed with partial pruning
+        await detectAndTransitionToReview(db);
+        // Verify some files were pruned
+        const prunedCount = db.prepare(`
+      SELECT COUNT(*) as count FROM t_task_pruned_files WHERE task_id = ?
+    `).get(taskId);
+        assert.strictEqual(prunedCount.count, 2, 'Should have 2 pruned file records (the non-existent ones)');
+        // Verify watch list now has only 1 file (the existing one)
+        const afterWatchCount = db.prepare(`
+      SELECT COUNT(*) as count FROM t_task_file_links WHERE task_id = ?
+    `).get(taskId);
+        assert.strictEqual(afterWatchCount.count, 1, 'Watch list should have 1 remaining file after partial prune');
+        // Verify remaining file is package.json
+        const remainingFile = db.prepare(`
+      SELECT f.path
+      FROM t_task_file_links tfl
+      JOIN m_files f ON tfl.file_id = f.id
+      WHERE tfl.task_id = ?
+    `).get(taskId);
+        assert.strictEqual(remainingFile.path, 'package.json', 'Remaining file should be package.json');
+    });
+    it('should handle task with no watched files gracefully', async () => {
+        // Create task without any watched files
+        const taskId = createTestTask(db);
+        // Make task appear stale
+        const tenMinutesAgo = Math.floor(Date.now() / 1000) - (10 * 60);
+        db.prepare(`
+      UPDATE t_tasks
+      SET updated_ts = ?
+      WHERE id = ?
+    `).run(tenMinutesAgo, taskId);
+        // Attempt transition - should skip this task
+        const transitioned = await detectAndTransitionToReview(db);
+        assert.strictEqual(transitioned, 0, 'Should not transition task with no watched files');
+        // Verify task status unchanged
+        const status = db.prepare(`
+      SELECT s.name
+      FROM t_tasks t
+      JOIN m_task_statuses s ON t.status_id = s.id
+      WHERE t.id = ?
+    `).get(taskId);
+        assert.strictEqual(status.name, 'in_progress', 'Task should remain in in_progress');
+    });
+    console.log('\n✅ All auto-pruning safety check tests passed!\n');
+});
+/**
+ * Helper: Create a test task in 'in_progress' status
+ */
+function createTestTask(db) {
+    // Create test agent
+    const agentId = db.prepare(`
+    INSERT INTO m_agents (name) VALUES (?) RETURNING id
+  `).get('test-agent');
+    // Get 'in_progress' status ID
+    const statusId = db.prepare(`
+    SELECT id FROM m_task_statuses WHERE name = 'in_progress'
+  `).get();
+    // Create task with updated_ts set to now (will be backdated in tests)
+    const taskId = db.prepare(`
+    INSERT INTO t_tasks (
+      title,
+      status_id,
+      priority,
+      assigned_agent_id,
+      created_by_agent_id,
+      created_ts,
+      updated_ts
+    )
+    VALUES (?, ?, 2, ?, ?, unixepoch(), unixepoch())
+    RETURNING id
+  `).get('Test task for auto-pruning', statusId.id, agentId.id, agentId.id);
+    return taskId.id;
+}
+/**
+ * Helper: Add watched files to a task
+ */
+function addWatchedFiles(db, taskId, filePaths) {
+    const insertFile = db.prepare(`
+    INSERT OR IGNORE INTO m_files (path) VALUES (?) RETURNING id
+  `);
+    const linkFile = db.prepare(`
+    INSERT INTO t_task_file_links (task_id, file_id) VALUES (?, ?)
+  `);
+    for (const filePath of filePaths) {
+        const fileResult = insertFile.get(filePath);
+        linkFile.run(taskId, fileResult.id);
+    }
+}
+//# sourceMappingURL=tasks.auto-pruning-safety.test.js.map

package/dist/tests/tasks.auto-pruning-safety.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"tasks.auto-pruning-safety.test.js","sourceRoot":"","sources":["../../src/tests/tasks.auto-pruning-safety.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAChE,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,QAAQ,MAAM,gBAAgB,CAAC;AACtC,OAAO,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AAChD,OAAO,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAC1D,OAAO,EAAE,2BAA2B,EAAE,MAAM,kCAAkC,CAAC;AAE/E,QAAQ,CAAC,kDAAkD,EAAE,GAAG,EAAE;IAChE,IAAI,EAAqB,CAAC;IAE1B,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,GAAG,IAAI,QAAQ,CAAC,UAAU,CAAC,CAAC;QAC9B,EAAE,CAAC,MAAM,CAAC,mBAAmB,CAAC,CAAC;QAC/B,gBAAgB,CAAC,EAAE,CAAC,CAAC;QACrB,gBAAgB,CAAC,EAAE,CAAC,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,KAAK,EAAE,CAAC;IACb,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,8CAA8C;QAC9C,MAAM,MAAM,GAAG,cAAc,CAAC,EAAE,CAAC,CAAC;QAElC,4CAA4C;QAC5C,4DAA4D;QAC5D,MAAM,gBAAgB,GAAG;YACvB,8BAA8B;YAC9B,8BAA8B;YAC9B,8BAA8B;SAC/B,CAAC;QACF,eAAe,CAAC,EAAE,EAAE,MAAM,EAAE,gBAAgB,CAAC,CAAC;QAE9C,2DAA2D;QAC3D,gFAAgF;QAChF,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,GAAG,CAAC,EAAE,GAAG,EAAE,CAAC,CAAC;QAChE,EAAE,CAAC,OAAO,CAAC;;;;KAIV,CAAC,CAAC,GAAG,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;QAE9B,oDAAoD;QACpD,MAAM,YAAY,GAAG,EAAE,CAAC,OAAO,CAAC;;;;;KAK/B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAqB,CAAC;QAEnC,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,IAAI,EAAE,aAAa,EAAE,kCAAkC,CAAC,CAAC;QAEzF,sDAAsD;QACtD,MAAM,gBAAgB,GAAG,EAAE,CAAC,OAAO,CAAC;;KAEnC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAsB,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,gBAAgB,CAAC,KAAK,EAAE,CAAC,EAAE,uDAAuD,CAAC,CAAC;QAEvG,wDAAwD;QACxD,mDAAmD;QACnD,0DAA0D;QAC1D,MAAM,YAAY,GAAG,MAAM,2BAA2B,CAAC,EAAE,CAAC,CAAC;QAE3D,yCAAyC;QACzC,MAAM,CAAC,WAAW,CAAC,YAAY,EAAE,CAAC,EAAE,wCAAwC,CAAC,CAAC;QAE9E,2DAA2D;QAC3D,MAAM,WAAW,GAAG,EAAE,CAAC,OAAO,CAAC;;;;;KAK9B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAqB,CAAC;QAEnC,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,IAAI,EAAE,aAAa,EAAE,sDAAsD,CAAC,CAAC;QAE5G,4DAA4D;QAC5D,MAAM,WAAW,GAAG,EAAE,CAAC,OAAO,CAAC;;KAE9B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAsB,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,KAAK,EAAE,CAAC,EAAE,oDAAoD,CAAC,CAAC;QAE/F,+DAA+D;QAC/D,MAAM,eAAe,GAAG,EAAE,CAAC,OAAO,CAAC;;KAElC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAsB,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,eAAe,CAAC,KAAK,EAAE,CAAC,EAAE,yDAAyD,CAAC,CAAC;IAC1G,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;QACjF,cAAc;QACd,MAAM,MAAM,GAAG,cAAc,CAAC,EAAE,CAAC,CAAC;QAElC,6CAA6C;QAC7C,wEAAwE;QACxE,MAAM,UAAU,GAAG;YACjB,cAAc,EAAE,SAAS;YACzB,8BAA8B,EAAE,iBAAiB;YACjD,8BAA8B,CAAE,iBAAiB;SAClD,CAAC;QACF,eAAe,CAAC,EAAE,EAAE,MAAM,EAAE,UAAU,CAAC,CAAC;QAExC,yBAAyB;QACzB,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,GAAG,CAAC,EAAE,GAAG,EAAE,CAAC,CAAC;QAChE,EAAE,CAAC,OAAO,CAAC;;;;KAIV,CAAC,CAAC,GAAG,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;QAE9B,6BAA6B;QAC7B,MAAM,gBAAgB,GAAG,EAAE,CAAC,OAAO,CAAC;;KAEnC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAsB,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,gBAAgB,CAAC,KAAK,EAAE,CAAC,EAAE,sCAAsC,CAAC,CAAC;QAEtF,2DAA2D;QAC3D,MAAM,2BAA2B,CAAC,EAAE,CAAC,CAAC;QAEtC,gCAAgC;QAChC,MAAM,WAAW,GAAG,EAAE,CAAC,OAAO,CAAC;;KAE9B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAsB,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,KAAK,EAAE,CAAC,EAAE,2DAA2D,CAAC,CAAC;QAEtG,2DAA2D;QAC3D,MAAM,eAAe,GAAG,EAAE,CAAC,OAAO,CAAC;;KAElC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAsB,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,eAAe,CAAC,KAAK,EAAE,CAAC,EAAE,6DAA6D,CAAC,CAAC;QAE5G,wCAAwC;QACxC,MAAM,aAAa,GAAG,EAAE,CAAC,OAAO,CAAC;;;;;KAKhC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAqB,CAAC;QAEnC,MAAM,CAAC,WAAW,CAAC,aAAa,CAAC,IAAI,EAAE,cAAc,EAAE,uCAAuC,CAAC,CAAC;IAClG,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,wCAAwC;QACxC,MAAM,MAAM,GAAG,cAAc,CAAC,EAAE,CAAC,CAAC;QAElC,yBAAyB;QACzB,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,GAAG,CAAC,EAAE,GAAG,EAAE,CAAC,CAAC;QAChE,EAAE,CAAC,OAAO,CAAC;;;;KAIV,CAAC,CAAC,GAAG,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;QAE9B,6CAA6C;QAC7C,MAAM,YAAY,GAAG,MAAM,2BAA2B,CAAC,EAAE,CAAC,CAAC;QAE3D,MAAM,CAAC,WAAW,CAAC,YAAY,EAAE,CAAC,EAAE,kDAAkD,CAAC,CAAC;QAExF,+BAA+B;QAC/B,MAAM,MAAM,GAAG,EAAE,CAAC,OAAO,CAAC;;;;;KAKzB,CAAC,CAAC,GAAG,CAAC,MAAM,CAAqB,CAAC;QAEnC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,IAAI,EAAE,aAAa,EAAE,mCAAmC,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,OAAO,CAAC,GAAG,CAAC,mDAAmD,CAAC,CAAC;AACnE,CAAC,CAAC,CAAC;AAEH;;GAEG;AACH,SAAS,cAAc,CAAC,EAAqB;IAC3C,oBAAoB;IACpB,MAAM,OAAO,GAAG,EAAE,CAAC,OAAO,CAAC;;GAE1B,CAAC,CAAC,GAAG,CAAC,YAAY,CAAmB,CAAC;IAEvC,8BAA8B;IAC9B,MAAM,QAAQ,GAAG,EAAE,CAAC,OAAO,CAAC;;GAE3B,CAAC,CAAC,GAAG,EAAoB,CAAC;IAE3B,sEAAsE;IACtE,MAAM,MAAM,GAAG,EAAE,CAAC,OAAO,CAAC;;;;;;;;;;;;GAYzB,CAAC,CAAC,GAAG,CACJ,4BAA4B,EAC5B,QAAQ,CAAC,EAAE,EACX,OAAO,CAAC,EAAE,EACV,OAAO,CAAC,EAAE,CACO,CAAC;IAEpB,OAAO,MAAM,CAAC,EAAE,CAAC;AACnB,CAAC;AAED;;GAEG;AACH,SAAS,eAAe,CAAC,EAAqB,EAAE,MAAc,EAAE,SAAmB;IACjF,MAAM,UAAU,GAAG,EAAE,CAAC,OAAO,CAAC;;GAE7B,CAAC,CAAC;IAEH,MAAM,QAAQ,GAAG,EAAE,CAAC,OAAO,CAAC;;GAE3B,CAAC,CAAC;IAEH,KAAK,MAAM,QAAQ,IAAI,SAAS,EAAE,CAAC;QACjC,MAAM,UAAU,GAAG,UAAU,CAAC,GAAG,CAAC,QAAQ,CAAmB,CAAC;QAC9D,QAAQ,CAAC,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,EAAE,CAAC,CAAC;IACtC,CAAC;AACH,CAAC"}

package/dist/tests/tasks.link-file-backward-compat.test.d.ts

@@ -0,0 +1,6 @@
+/**
+ * Unit tests for backward compatibility of deprecated task.link(link_type="file") (v3.4.1)
+ * Tests that the deprecated API still works while showing deprecation warnings
+ */
+export {};
+//# sourceMappingURL=tasks.link-file-backward-compat.test.d.ts.map

package/dist/tests/tasks.link-file-backward-compat.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tasks.link-file-backward-compat.test.d.ts","sourceRoot":"","sources":["../../src/tests/tasks.link-file-backward-compat.test.ts"],"names":[],"mappings":"AAAA;;;GAGG"}

package/dist/tests/tasks.link-file-backward-compat.test.js

@@ -0,0 +1,235 @@
+/**
+ * Unit tests for backward compatibility of deprecated task.link(link_type="file") (v3.4.1)
+ * Tests that the deprecated API still works while showing deprecation warnings
+ */
+import { describe, it, beforeEach } from 'node:test';
+import assert from 'node:assert/strict';
+import Database from 'better-sqlite3';
+import { initializeSchema } from '../schema.js';
+import { getOrCreateAgent, getOrCreateFile } from '../database.js';
+/**
+ * Test database instance
+ */
+let testDb;
+/**
+ * Create an in-memory test database
+ */
+function createTestDatabase() {
+    const db = new Database(':memory:');
+    db.pragma('foreign_keys = ON');
+    initializeSchema(db);
+    return db;
+}
+/**
+ * Helper: Create a test task
+ */
+function createTestTask(db, title) {
+    const agentId = getOrCreateAgent(db, 'test-agent');
+    const statusId = 1; // todo
+    const result = db.prepare(`
+    INSERT INTO t_tasks (title, status_id, priority, created_by_agent_id, assigned_agent_id)
+    VALUES (?, ?, 2, ?, ?)
+  `).run(title, statusId, agentId, agentId);
+    return result.lastInsertRowid;
+}
+/**
+ * Inline implementation of linkTask for testing (file link_type only)
+ */
+function linkTaskFile(db, params) {
+    if (!params.task_id) {
+        throw new Error('Parameter "task_id" is required');
+    }
+    if (params.target_id === undefined || params.target_id === null) {
+        throw new Error('Parameter "target_id" is required');
+    }
+    // Check if task exists
+    const taskExists = db.prepare('SELECT id FROM t_tasks WHERE id = ?').get(params.task_id);
+    if (!taskExists) {
+        throw new Error(`Task with id ${params.task_id} not found`);
+    }
+    // Deprecation warning (v3.4.1) - would appear in console
+    // console.warn(`⚠️  DEPRECATION WARNING: task.link(link_type="file") is deprecated as of v3.4.1.`);
+    const filePath = String(params.target_id);
+    const fileId = getOrCreateFile(db, filePath);
+    const stmt = db.prepare(`
+    INSERT OR IGNORE INTO t_task_file_links (task_id, file_id)
+    VALUES (?, ?)
+  `);
+    stmt.run(params.task_id, fileId);
+    return {
+        success: true,
+        task_id: params.task_id,
+        linked_to: 'file',
+        target: filePath,
+        deprecation_warning: 'task.link(link_type="file") is deprecated. Use task.create/update(watch_files) or watch_files action instead.',
+        message: `Task ${params.task_id} linked to file "${filePath}" (DEPRECATED API - use watch_files instead)`
+    };
+}
+describe('Backward compatibility: task.link(link_type="file")', () => {
+    beforeEach(() => {
+        testDb = createTestDatabase();
+    });
+    it('should still link file to task (backward compatible)', () => {
+        const taskId = createTestTask(testDb, 'Task for backward compat test');
+        const result = linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/index.ts'
+        });
+        assert.ok(result.success, 'Should succeed');
+        assert.strictEqual(result.task_id, taskId);
+        assert.strictEqual(result.linked_to, 'file');
+        assert.strictEqual(result.target, 'src/index.ts');
+    });
+    it('should include deprecation warning in response', () => {
+        const taskId = createTestTask(testDb, 'Task for deprecation warning test');
+        const result = linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/database.ts'
+        });
+        assert.ok(result.deprecation_warning, 'Should include deprecation warning');
+        assert.ok(result.deprecation_warning.includes('deprecated'), 'Warning should mention deprecation');
+        assert.ok(result.deprecation_warning.includes('watch_files'), 'Warning should suggest watch_files');
+    });
+    it('should create file link in database', () => {
+        const taskId = createTestTask(testDb, 'Task for DB link test');
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/schema.ts'
+        });
+        // Verify file link was created
+        const links = testDb.prepare(`
+      SELECT f.path
+      FROM t_task_file_links tfl
+      JOIN m_files f ON tfl.file_id = f.id
+      WHERE tfl.task_id = ?
+    `).all(taskId);
+        assert.strictEqual(links.length, 1);
+        assert.strictEqual(links[0].path, 'src/schema.ts');
+    });
+    it('should handle multiple file links', () => {
+        const taskId = createTestTask(testDb, 'Task for multiple links');
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/index.ts'
+        });
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/database.ts'
+        });
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/schema.ts'
+        });
+        // Verify all links exist
+        const links = testDb.prepare(`
+      SELECT f.path
+      FROM t_task_file_links tfl
+      JOIN m_files f ON tfl.file_id = f.id
+      WHERE tfl.task_id = ?
+      ORDER BY f.path
+    `).all(taskId);
+        assert.strictEqual(links.length, 3);
+        assert.strictEqual(links[0].path, 'src/database.ts');
+        assert.strictEqual(links[1].path, 'src/index.ts');
+        assert.strictEqual(links[2].path, 'src/schema.ts');
+    });
+    it('should be idempotent (duplicate links ignored)', () => {
+        const taskId = createTestTask(testDb, 'Task for idempotent test');
+        // Link same file twice
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/index.ts'
+        });
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/index.ts'
+        });
+        // Should only have one link
+        const links = testDb.prepare(`
+      SELECT f.path
+      FROM t_task_file_links tfl
+      JOIN m_files f ON tfl.file_id = f.id
+      WHERE tfl.task_id = ?
+    `).all(taskId);
+        assert.strictEqual(links.length, 1, 'Should not create duplicate links');
+    });
+    it('should work with new watch_files action on same task', () => {
+        const taskId = createTestTask(testDb, 'Task for mixed API test');
+        // Use old API
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/index.ts'
+        });
+        // Use new API (simulated by direct DB insert)
+        const fileId = getOrCreateFile(testDb, 'src/database.ts');
+        testDb.prepare('INSERT OR IGNORE INTO t_task_file_links (task_id, file_id) VALUES (?, ?)').run(taskId, fileId);
+        // Both files should be linked
+        const links = testDb.prepare(`
+      SELECT f.path
+      FROM t_task_file_links tfl
+      JOIN m_files f ON tfl.file_id = f.id
+      WHERE tfl.task_id = ?
+      ORDER BY f.path
+    `).all(taskId);
+        assert.strictEqual(links.length, 2);
+        assert.strictEqual(links[0].path, 'src/database.ts');
+        assert.strictEqual(links[1].path, 'src/index.ts');
+    });
+    it('should throw error for invalid task_id', () => {
+        assert.throws(() => {
+            linkTaskFile(testDb, {
+                task_id: 999,
+                target_id: 'src/index.ts'
+            });
+        }, /Task with id 999 not found/);
+    });
+    it('should handle various file path formats', () => {
+        const taskId = createTestTask(testDb, 'Task for path formats');
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'package.json'
+        });
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'src/tools/tasks.ts'
+        });
+        linkTaskFile(testDb, {
+            task_id: taskId,
+            target_id: 'docs/README.md'
+        });
+        const links = testDb.prepare(`
+      SELECT f.path
+      FROM t_task_file_links tfl
+      JOIN m_files f ON tfl.file_id = f.id
+      WHERE tfl.task_id = ?
+    `).all(taskId);
+        assert.strictEqual(links.length, 3);
+    });
+    it('should maintain same database schema as new API', () => {
+        const taskId1 = createTestTask(testDb, 'Task with old API');
+        const taskId2 = createTestTask(testDb, 'Task with new API');
+        // Old API
+        linkTaskFile(testDb, {
+            task_id: taskId1,
+            target_id: 'src/index.ts'
+        });
+        // New API (simulated)
+        const fileId = getOrCreateFile(testDb, 'src/index.ts');
+        testDb.prepare('INSERT OR IGNORE INTO t_task_file_links (task_id, file_id) VALUES (?, ?)').run(taskId2, fileId);
+        // Both should create identical links
+        const links1 = testDb.prepare(`
+      SELECT task_id, file_id
+      FROM t_task_file_links
+      WHERE task_id = ?
+    `).all(taskId1);
+        const links2 = testDb.prepare(`
+      SELECT task_id, file_id
+      FROM t_task_file_links
+      WHERE task_id = ?
+    `).all(taskId2);
+        // Same file_id should be used
+        assert.strictEqual(links1[0].file_id, links2[0].file_id, 'Should use same file ID');
+    });
+    console.log('\n✅ All backward compatibility tests passed!\n');
+});
+//# sourceMappingURL=tasks.link-file-backward-compat.test.js.map

package/dist/tests/tasks.link-file-backward-compat.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"tasks.link-file-backward-compat.test.js","sourceRoot":"","sources":["../../src/tests/tasks.link-file-backward-compat.test.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,WAAW,CAAC;AACrD,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,QAAQ,MAAM,gBAAgB,CAAC;AACtC,OAAO,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AAChD,OAAO,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,gBAAgB,CAAC;AAGnE;;GAEG;AACH,IAAI,MAAoB,CAAC;AAEzB;;GAEG;AACH,SAAS,kBAAkB;IACzB,MAAM,EAAE,GAAG,IAAI,QAAQ,CAAC,UAAU,CAAC,CAAC;IACpC,EAAE,CAAC,MAAM,CAAC,mBAAmB,CAAC,CAAC;IAC/B,gBAAgB,CAAC,EAAE,CAAC,CAAC;IACrB,OAAO,EAAE,CAAC;AACZ,CAAC;AAED;;GAEG;AACH,SAAS,cAAc,CAAC,EAAgB,EAAE,KAAa;IACrD,MAAM,OAAO,GAAG,gBAAgB,CAAC,EAAE,EAAE,YAAY,CAAC,CAAC;IACnD,MAAM,QAAQ,GAAG,CAAC,CAAC,CAAC,OAAO;IAE3B,MAAM,MAAM,GAAG,EAAE,CAAC,OAAO,CAAC;;;GAGzB,CAAC,CAAC,GAAG,CAAC,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;IAE1C,OAAO,MAAM,CAAC,eAAyB,CAAC;AAC1C,CAAC;AAED;;GAEG;AACH,SAAS,YAAY,CAAC,EAAgB,EAAE,MAGvC;IACC,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;QACpB,MAAM,IAAI,KAAK,CAAC,iCAAiC,CAAC,CAAC;IACrD,CAAC;IAED,IAAI,MAAM,CAAC,SAAS,KAAK,SAAS,IAAI,MAAM,CAAC,SAAS,KAAK,IAAI,EAAE,CAAC;QAChE,MAAM,IAAI,KAAK,CAAC,mCAAmC,CAAC,CAAC;IACvD,CAAC;IAED,uBAAuB;IACvB,MAAM,UAAU,GAAG,EAAE,CAAC,OAAO,CAAC,qCAAqC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;IACzF,IAAI,CAAC,UAAU,EAAE,CAAC;QAChB,MAAM,IAAI,KAAK,CAAC,gBAAgB,MAAM,CAAC,OAAO,YAAY,CAAC,CAAC;IAC9D,CAAC;IAED,yDAAyD;IACzD,oGAAoG;IAEpG,MAAM,QAAQ,GAAG,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;IAC1C,MAAM,MAAM,GAAG,eAAe,CAAC,EAAE,EAAE,QAAQ,CAAC,CAAC;IAE7C,MAAM,IAAI,GAAG,EAAE,CAAC,OAAO,CAAC;;;GAGvB,CAAC,CAAC;IACH,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;IAEjC,OAAO;QACL,OAAO,EAAE,IAAI;QACb,OAAO,EAAE,MAAM,CAAC,OAAO;QACvB,SAAS,EAAE,MAAM;QACjB,MAAM,EAAE,QAAQ;QAChB,mBAAmB,EAAE,+GAA+G;QACpI,OAAO,EAAE,QAAQ,MAAM,CAAC,OAAO,oBAAoB,QAAQ,8CAA8C;KAC1G,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;IACnE,UAAU,CAAC,GAAG,EAAE;QACd,MAAM,GAAG,kBAAkB,EAAE,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,+BAA+B,CAAC,CAAC;QAEvE,MAAM,MAAM,GAAG,YAAY,CAAC,MAAM,EAAE;YAClC,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,gBAAgB,CAAC,CAAC;QAC5C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAC3C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC;QAC7C,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,mCAAmC,CAAC,CAAC;QAE3E,MAAM,MAAM,GAAG,YAAY,CAAC,MAAM,EAAE;YAClC,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,iBAAiB;SAC7B,CAAC,CAAC;QAEH,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,mBAAmB,EAAE,oCAAoC,CAAC,CAAC;QAC5E,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,mBAAmB,CAAC,QAAQ,CAAC,YAAY,CAAC,EAAE,oCAAoC,CAAC,CAAC;QACnG,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,mBAAmB,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,oCAAoC,CAAC,CAAC;IACtG,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,uBAAuB,CAAC,CAAC;QAE/D,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,eAAe;SAC3B,CAAC,CAAC;QAEH,+BAA+B;QAC/B,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC;;;;;KAK5B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAuB,CAAC;QAErC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,yBAAyB,CAAC,CAAC;QAEjE,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,iBAAiB;SAC7B,CAAC,CAAC;QAEH,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,eAAe;SAC3B,CAAC,CAAC;QAEH,yBAAyB;QACzB,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC;;;;;;KAM5B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAuB,CAAC;QAErC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,iBAAiB,CAAC,CAAC;QACrD,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,cAAc,CAAC,CAAC;QAClD,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,0BAA0B,CAAC,CAAC;QAElE,uBAAuB;QACvB,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,4BAA4B;QAC5B,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC;;;;;KAK5B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAuB,CAAC;QAErC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,mCAAmC,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,yBAAyB,CAAC,CAAC;QAEjE,cAAc;QACd,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,8CAA8C;QAC9C,MAAM,MAAM,GAAG,eAAe,CAAC,MAAM,EAAE,iBAAiB,CAAC,CAAC;QAC1D,MAAM,CAAC,OAAO,CAAC,0EAA0E,CAAC,CAAC,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAE/G,8BAA8B;QAC9B,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC;;;;;;KAM5B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAuB,CAAC;QAErC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,iBAAiB,CAAC,CAAC;QACrD,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,cAAc,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,CAAC,MAAM,CACX,GAAG,EAAE;YACH,YAAY,CAAC,MAAM,EAAE;gBACnB,OAAO,EAAE,GAAG;gBACZ,SAAS,EAAE,cAAc;aAC1B,CAAC,CAAC;QACL,CAAC,EACD,4BAA4B,CAC7B,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,MAAM,GAAG,cAAc,CAAC,MAAM,EAAE,uBAAuB,CAAC,CAAC;QAE/D,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,oBAAoB;SAChC,CAAC,CAAC;QAEH,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,MAAM;YACf,SAAS,EAAE,gBAAgB;SAC5B,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,MAAM,CAAC,OAAO,CAAC;;;;;KAK5B,CAAC,CAAC,GAAG,CAAC,MAAM,CAAuB,CAAC;QAErC,MAAM,CAAC,WAAW,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,EAAE,mBAAmB,CAAC,CAAC;QAC5D,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,EAAE,mBAAmB,CAAC,CAAC;QAE5D,UAAU;QACV,YAAY,CAAC,MAAM,EAAE;YACnB,OAAO,EAAE,OAAO;YAChB,SAAS,EAAE,cAAc;SAC1B,CAAC,CAAC;QAEH,sBAAsB;QACtB,MAAM,MAAM,GAAG,eAAe,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;QACvD,MAAM,CAAC,OAAO,CAAC,0EAA0E,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAEhH,qCAAqC;QACrC,MAAM,MAAM,GAAG,MAAM,CAAC,OAAO,CAAC;;;;KAI7B,CAAC,CAAC,GAAG,CAAC,OAAO,CAA2C,CAAC;QAE1D,MAAM,MAAM,GAAG,MAAM,CAAC,OAAO,CAAC;;;;KAI7B,CAAC,CAAC,GAAG,CAAC,OAAO,CAA2C,CAAC;QAE1D,8BAA8B;QAC9B,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,yBAAyB,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,OAAO,CAAC,GAAG,CAAC,gDAAgD,CAAC,CAAC;AAChE,CAAC,CAAC,CAAC"}

package/dist/tests/tasks.watch-files-action.test.d.ts

@@ -0,0 +1,6 @@
+/**
+ * Unit tests for Task watch_files action (v3.4.1)
+ * Tests the new watch_files action: watch, unwatch, list
+ */
+export {};
+//# sourceMappingURL=tasks.watch-files-action.test.d.ts.map

package/dist/tests/tasks.watch-files-action.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tasks.watch-files-action.test.d.ts","sourceRoot":"","sources":["../../src/tests/tasks.watch-files-action.test.ts"],"names":[],"mappings":"AAAA;;;GAGG"}