@sparkelf/dsh-dataops-managed 0.3.28 → 0.3.29

package/lib/index.js

@@ -1463,6 +1463,41 @@ function buildAiManagementTools(ctx, reviewApi, readScope, collectHub) {
 		if (!Array.isArray(parsed)) throw new Error(tool + ": " + label + " must be a JSON array");
 		return parsed;
 	};
+	/**
+	* Keep only the fields the DataOps execute DTO accepts. The plan tool returns
+	* `createSql`/`targetSchema`/`notes` beside `createTable`; if the model passes
+	* the whole plan (or rebuilds it after a table-name edit), the backend's
+	* `forbidNonWhitelisted` pipe rejects the call with a generic 400.
+	*/
+	const normalizeCreatePlan = (tool, plan) => {
+		if (plan === void 0) return void 0;
+		const tableName = typeof plan.tableName === "string" ? plan.tableName.trim() : "";
+		if (!tableName) throw new Error(tool + ": create_plan_json.tableName is required");
+		const rawColumns = Array.isArray(plan.columns) ? plan.columns : [];
+		if (!rawColumns.length) throw new Error(tool + ": create_plan_json.columns is required");
+		const columns = rawColumns.map((raw, index) => {
+			const column = raw && typeof raw === "object" && !Array.isArray(raw) ? raw : {};
+			const sourceKey = typeof column.sourceKey === "string" ? column.sourceKey : "";
+			const name = typeof column.name === "string" ? column.name : "";
+			const chineseName = typeof column.chineseName === "string" ? column.chineseName : "";
+			const dataType = typeof column.dataType === "string" ? column.dataType : "";
+			if (!sourceKey || !name || !chineseName || !dataType) throw new Error(tool + ": create_plan_json.columns[" + index + "] needs sourceKey/name/chineseName/dataType");
+			return {
+				sourceKey,
+				name,
+				chineseName,
+				dataType,
+				...typeof column.nullable === "boolean" ? { nullable: column.nullable } : {},
+				...typeof column.comment === "string" ? { comment: column.comment } : {}
+			};
+		});
+		return {
+			tableName,
+			...typeof plan.tableChineseName === "string" && plan.tableChineseName.trim() ? { tableChineseName: plan.tableChineseName.trim() } : {},
+			...typeof plan.tableComment === "string" && plan.tableComment.trim() ? { tableComment: plan.tableComment.trim() } : {},
+			columns
+		};
+	};
 	return [
 		{
 			name: "skill_list",
@@ -1949,7 +1984,7 @@ function buildAiManagementTools(ctx, reviewApi, readScope, collectHub) {
 			name: "dataops_collect_execute",
 			definition: defineTool({
 				name: "dataops_collect_execute",
-				description: "Execute one confirmed spreadsheet import: create a table (or write to an existing table), insert rows, and register the DataOps resource. This is the ONLY collection tool that mutates data. Call it only after the user explicitly clicked the plan's \"开始导入\" action; never call it while the plan is still being edited.",
+				description: "Execute one confirmed spreadsheet import: create a table (or write to an existing table), insert rows, and register the DataOps resource. This is the ONLY collection tool that mutates data. Call it only after the user explicitly confirms the read-only plan in natural language (e.g. 确认/开始吧); never call it while the plan is still being edited.",
 				parameters: {
 					session_id: {
 						type: "string",
@@ -1976,7 +2011,7 @@ function buildAiManagementTools(ctx, reviewApi, readScope, collectHub) {
 					},
 					create_plan_json: {
 						type: "string",
-						description: "JSON object returned as plan.createTable, edited by the user."
+						description: "Only plan.createTable (tableName/tableChineseName/tableComment/columns). Do not send createSql/targetSchema/notes; if the user edits the table name, call dataops_collect_plan again and pass its createTable."
 					},
 					mappings_json: {
 						type: "string",
@@ -2010,7 +2045,7 @@ function buildAiManagementTools(ctx, reviewApi, readScope, collectHub) {
 				},
 				execute: async (args, exec) => {
 					if (args.mode !== "CREATE_TABLE" && args.mode !== "EXISTING_TABLE") throw new Error("dataops_collect_execute: mode must be CREATE_TABLE or EXISTING_TABLE");
-					const createPlan = parseJsonObject("dataops_collect_execute", "create_plan_json", args.create_plan_json);
+					const createPlan = normalizeCreatePlan("dataops_collect_execute", parseJsonObject("dataops_collect_execute", "create_plan_json", args.create_plan_json));
 					const mappings = parseJsonArray("dataops_collect_execute", "mappings_json", args.mappings_json);
 					if (args.mode === "CREATE_TABLE" && createPlan === void 0) throw new Error("dataops_collect_execute: create_plan_json is required for CREATE_TABLE");
 					if (args.mode === "EXISTING_TABLE" && (!args.target_schema || !args.target_table)) throw new Error("dataops_collect_execute: target_schema and target_table are required for EXISTING_TABLE");
@@ -2433,10 +2468,10 @@ const CONNECTOR_STATE_PATH = join(process.env.DSH_HOME?.trim() || "/workspace/.d
 /**
 * Capability definitions before permission filtering.
 *
-* The set and its order follow `dataops/docs/dataops-connector-development-spec.md` §2.3,
-* whose credential `requiredFor` list is `approval.push`, `data.query`, and `admin.manage`.
-* Skill plaza and review queue are governed by `skill:*` permissions and stay out of the
-* connector's capability switches.
+* The set follows `dataops/docs/dataops-connector-development-spec.md` §2.3's credential list.
+* `approval.push` was renamed to `resource.manage` when the capability grew from approval push
+* into resource approval and authorization. Skill plaza and review queue are governed by
+* `skill:*` permissions and stay out of the connector's capability switches.
 */
 const CAPABILITIES = [
 	{

package/lib/types/client/DataOpsResourcesTab.d.ts

@@ -21,7 +21,11 @@ export interface DataOpsResourcesTabProps {
     };
     tab: ResourceTabRecord;
     visible?: boolean;
-    sendReference?: (text: string, sessionId?: string) => Promise<void>;
+    insertResourceReference?: (resource: {
+        id?: unknown;
+        name?: unknown;
+        table?: unknown;
+    }, reference: Record<string, unknown>, sessionId?: string) => void;
 }
 /** Render the embedded DataOps resource page and translate its parent messages. */
-export declare function DataOpsResourcesTab({ tab, scope, sendReference }: DataOpsResourcesTabProps): import("react").JSX.Element;
+export declare function DataOpsResourcesTab({ tab, scope, insertResourceReference }: DataOpsResourcesTabProps): import("react").JSX.Element;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sparkelf/dsh-dataops-managed",
-  "version": "0.3.28",
+  "version": "0.3.29",
   "description": "DataOps JWT intake and credential-backed MCP tools for managed DeepSeek Harness workspaces.",
   "type": "module",
   "main": "./lib/index.js",