tbenandClaude Opus 5.5
ddf431e866
- Add bun test suite: SQL generator and value unit tests, plus MariaDB integration tests gated on BUN_MARIADB_TEST_DB - schema: add --dry-run and require confirmation (or --yes) before dropping tables/columns; block unapproved destructive statements - Always AND targetId with the query, regardless of searchOperator - Initialize lazily on first call; init throws instead of exiting - Generate IN (?, ?, ...) lists for IN / NOT IN - Share value binding between insert and update (booleans, embeddings, function values, "" -> NULL for non-text columns) - Keep Buffers and convert BigInt/Date without a JSON round-trip - DbSQL: report success for empty SELECT results - Run update() in a transaction with row locking - Key the client cache on the full connection config - Consolidate identifier quoting, drop dead code, merge column lookup - Read CLI version from package.json, pin @types/bun, chmod CLI on compile - Rebuild dist Co-Authored-By: Claude Opus 5.5 <[email protected]>
61 lines
1.6 KiB
JavaScript
61 lines
1.6 KiB
JavaScript
import dbHandler from "../db-handler";
|
|
import _ from "lodash";
|
|
import sqlGenerator from "../../utils/sql-generator";
|
|
import { quoteIdentifier } from "../../utils/sql-identifier";
|
|
export default async function DbDelete({ table, query, targetId, config, }) {
|
|
let sqlObj = null;
|
|
try {
|
|
const finalQuery = query || {};
|
|
sqlObj = sqlGenerator({
|
|
tableName: table,
|
|
genObject: finalQuery,
|
|
targetId,
|
|
});
|
|
const whereClause = sqlObj.where;
|
|
if (!whereClause) {
|
|
return {
|
|
success: false,
|
|
msg: `No WHERE clause`,
|
|
debug: {
|
|
sqlObj,
|
|
},
|
|
};
|
|
}
|
|
let sql = `DELETE FROM ${quoteIdentifier(table)} ${whereClause}`;
|
|
sql += ` RETURNING *`;
|
|
sqlObj.string = sql;
|
|
sqlObj.values = sqlObj.whereValues;
|
|
const res = await dbHandler({
|
|
query: sql,
|
|
values: sqlObj.values,
|
|
config,
|
|
});
|
|
if (!res.success) {
|
|
return {
|
|
msg: "Database delete failed",
|
|
...res,
|
|
success: false,
|
|
debug: {
|
|
sqlObj,
|
|
},
|
|
};
|
|
}
|
|
return {
|
|
..._.omit(res, ["payload"]),
|
|
success: Boolean(res.insert_return?.last_insert_id),
|
|
debug: {
|
|
sqlObj,
|
|
},
|
|
};
|
|
}
|
|
catch (error) {
|
|
return {
|
|
success: false,
|
|
error: error.message,
|
|
debug: {
|
|
sqlObj,
|
|
},
|
|
};
|
|
}
|
|
}
|