@sqaoss/flowy 1.13.0 → 1.13.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sqaoss/flowy",
-  "version": "1.13.0",
+  "version": "1.13.1",
   "description": "Agentic persistent planning",
   "type": "module",
   "bin": {

package/server/src/resolvers.test.ts

@@ -1395,21 +1395,33 @@ describe('createResolvers', () => {
   })
 
   describe('Mutation.createEdge — edge cases', () => {
-    it('throws on duplicate edge', () => {
+    it('is idempotent: creating the same edge twice succeeds and yields one edge', () => {
       const p = create(resolvers, { type: 'project', title: 'P' })
       const f = create(resolvers, { type: 'feature', title: 'F' })
-      resolvers.Mutation.createEdge(null, {
+      const first = resolvers.Mutation.createEdge(null, {
         sourceId: f.id,
         targetId: p.id,
         relation: 'part_of',
       })
-      expect(() =>
-        resolvers.Mutation.createEdge(null, {
-          sourceId: f.id,
-          targetId: p.id,
-          relation: 'part_of',
-        }),
-      ).toThrow()
+      const second = resolvers.Mutation.createEdge(null, {
+        sourceId: f.id,
+        targetId: p.id,
+        relation: 'part_of',
+      })
+      // Both calls succeed and return the same canonical edge (same created_at).
+      expect(second).toMatchObject({
+        sourceId: f.id,
+        targetId: p.id,
+        relation: 'part_of',
+      })
+      expect(second.createdAt).toBe(first.createdAt)
+      // Exactly one row exists for the triple — no duplicate.
+      const { n } = db.raw
+        .query(
+          'SELECT COUNT(*) AS n FROM edges WHERE source_id = ? AND target_id = ? AND relation = ?',
+        )
+        .get(f.id, p.id, 'part_of') as { n: number }
+      expect(n).toBe(1)
     })
 
     it('throws when source node does not exist', () => {

package/server/src/resolvers.ts

@@ -778,25 +778,39 @@ export function createResolvers(db: Db, opts: ResolverOptions = {}) {
           throw validationError('A node cannot block itself')
         }
         const now = new Date().toISOString()
+        // Idempotent (F29): re-creating an existing edge is a no-op success, not
+        // a PK-violation error. ON CONFLICT DO NOTHING leaves the original row
+        // (and its created_at) untouched; we only audit a genuinely new edge.
         db.raw.transaction(() => {
-          db.raw.run(
-            'INSERT INTO edges (source_id, target_id, relation, created_at) VALUES (?, ?, ?, ?)',
+          const result = db.raw.run(
+            'INSERT INTO edges (source_id, target_id, relation, created_at) VALUES (?, ?, ?, ?) ON CONFLICT (source_id, target_id, relation) DO NOTHING',
             [args.sourceId, args.targetId, args.relation, now],
           )
-          // Record the edge against its source node so `auditLog(sourceId)`
-          // surfaces it. field = relation; newValue = the target it now links.
-          insertAudit(db, {
-            nodeId: args.sourceId,
-            action: 'create_edge',
-            field: args.relation,
-            newValue: args.targetId,
-          })
+          if (result.changes > 0) {
+            // Record the edge against its source node so `auditLog(sourceId)`
+            // surfaces it. field = relation; newValue = the target it now links.
+            insertAudit(db, {
+              nodeId: args.sourceId,
+              action: 'create_edge',
+              field: args.relation,
+              newValue: args.targetId,
+            })
+          }
         })()
+        // Read back the canonical row so the returned createdAt reflects the
+        // existing edge on a duplicate call, not the discarded `now`.
+        const edge = db.raw
+          .query(
+            'SELECT created_at FROM edges WHERE source_id = ? AND target_id = ? AND relation = ?',
+          )
+          .get(args.sourceId, args.targetId, args.relation) as {
+          created_at: string
+        }
         return {
           sourceId: args.sourceId,
           targetId: args.targetId,
           relation: args.relation,
-          createdAt: now,
+          createdAt: edge.created_at,
         }
       },