@odla-ai/ai 0.1.0 → 0.1.2

package/dist/index.cjs

@@ -1930,6 +1930,10 @@ function entityCrudSkill(opts) {
     return out;
   };
   const ok = (obj) => ({ content: JSON.stringify(obj) });
+  const existsById = async (id) => {
+    const res = await db.query({ [entity]: { $: {} } });
+    return (res[entity] ?? []).some((r) => String(r.id ?? "") === id);
+  };
   const listTool = {
     name: `list_${plural}`,
     description: `List ${plural}. Optionally filter by exact field values and/or limit the count.`,
@@ -1955,6 +1959,12 @@ function entityCrudSkill(opts) {
     description: `Create a new ${singular}.`,
     inputSchema: { type: "object", properties: fields, ...opts.required ? { required: opts.required } : {} },
     handler: async (input) => {
+      for (const req of opts.required ?? []) {
+        const v = input[req];
+        if (v === void 0 || v === null || v === "") {
+          return { content: `Cannot create ${singular}: field "${req}" is required.`, isError: true };
+        }
+      }
       const id = genId();
       const attrs = pickFields(input);
       if (stampAttr) attrs[stampAttr] = Date.now();
@@ -1967,7 +1977,10 @@ function entityCrudSkill(opts) {
     description: `Update fields of an existing ${singular} by ${idField}. Only the fields you pass change.`,
     inputSchema: {
       type: "object",
-      properties: { [idField]: { type: "string", description: `The ${singular}'s id.` }, ...fields },
+      properties: {
+        [idField]: { type: "string", description: `The exact ${idField} string from list_${plural} \u2014 NOT the item's position in a numbered list.` },
+        ...fields
+      },
       required: [idField]
     },
     handler: async (input) => {
@@ -1975,6 +1988,9 @@ function entityCrudSkill(opts) {
       if (!id) return { content: `Missing "${idField}".`, isError: true };
       const attrs = pickFields(input);
       if (Object.keys(attrs).length === 0) return { content: "No fields to update.", isError: true };
+      if (!await existsById(id)) {
+        return { content: `No ${singular} with ${idField} "${id}". Call list_${plural} for valid ids, then retry.`, isError: true };
+      }
       await db.transact([{ t: "update", ns: entity, id, attrs }]);
       return ok({ id, updated: true });
     }
@@ -1982,10 +1998,17 @@ function entityCrudSkill(opts) {
   const deleteTool = {
     name: `delete_${singular}`,
     description: `Delete a ${singular} by ${idField}.`,
-    inputSchema: { type: "object", properties: { [idField]: { type: "string" } }, required: [idField] },
+    inputSchema: {
+      type: "object",
+      properties: { [idField]: { type: "string", description: `The exact ${idField} string from list_${plural} \u2014 NOT the item's position in a numbered list.` } },
+      required: [idField]
+    },
     handler: async (input) => {
       const id = String(input[idField] ?? "");
       if (!id) return { content: `Missing "${idField}".`, isError: true };
+      if (!await existsById(id)) {
+        return { content: `No ${singular} with ${idField} "${id}". Call list_${plural} for valid ids.`, isError: true };
+      }
       const op = { t: "delete", ns: entity, id };
       await db.transact([op]);
       return ok({ id, deleted: true });