dsh-patch-edit-plus 0.1.0

package/lib/types/shellops.d.ts

@@ -0,0 +1,41 @@
+/**
+ * Shell seam for Delete / Move (the `ctx.fs` contract has no delete/rename).
+ *
+ * Security posture:
+ * - Command templates are CONSTANTS; paths travel only via the environment
+ *   (`DSH_PATCH_SOURCE` / `DSH_PATCH_TARGET`), never interpolated into the
+ *   command string — shell metacharacters in a path cannot inject commands.
+ * - Every run carries a `sandboxPolicy` so sandboxing executors fence it, and
+ *   the reported `ShellSandboxInfo` distinguishes "policy denied" from
+ *   "command failed".
+ * @module dsh-patch-edit-plus/shellops
+ */
+import type { Context } from '@deepseek-ai/cordis';
+import type { ToolRunContext } from '@deepseek-ai/dsh-tools';
+import type { FsTarget } from '@deepseek-ai/dsh-fs';
+import type { SandboxExecutionPolicy } from '@deepseek-ai/dsh-sandbox';
+import type { ShellExecutor } from '@deepseek-ai/dsh-shell';
+import type { ResolvedConfig } from './config.js';
+/** Constant POSIX templates; paths arrive via env, never via interpolation. */
+export declare const POSIX_REMOVE = "rm -f -- \"$DSH_PATCH_TARGET\"";
+export declare const POSIX_MOVE = "mv -f -- \"$DSH_PATCH_SOURCE\" \"$DSH_PATCH_TARGET\"";
+/** Constant PowerShell templates (Windows default). */
+export declare const PWSH_REMOVE = "Remove-Item -LiteralPath $env:DSH_PATCH_TARGET -Force";
+export declare const PWSH_MOVE = "Move-Item -LiteralPath $env:DSH_PATCH_SOURCE -Destination $env:DSH_PATCH_TARGET -Force";
+export type Dialect = 'posix' | 'pwsh';
+/** Resolve the configured dialect; `auto` picks pwsh on win32, posix elsewhere. */
+export declare function resolveDialect(cfg: ResolvedConfig): Dialect;
+/** The constant delete template for the resolved dialect (config override wins). */
+export declare function deleteTemplate(cfg: ResolvedConfig): string;
+/** The constant move template for the resolved dialect (config override wins). */
+export declare function moveTemplate(cfg: ResolvedConfig): string;
+/** Structurally resolve `ctx.shell` without assuming the service is mounted. */
+export declare function resolveShell(ctx: Context): ShellExecutor | undefined;
+/** True when Delete/Move can run at all under the current configuration. */
+export declare function shellReady(ctx: Context, cfg: ResolvedConfig): boolean;
+/** Throw the precise structured UNSUPPORTED error when Delete/Move cannot run. */
+export declare function assertShellReady(ctx: Context, cfg: ResolvedConfig): void;
+/** Remove one file via the sandbox-aware shell. */
+export declare function removeFile(ctx: Context, exec: ToolRunContext, cfg: ResolvedConfig, target: FsTarget, sandboxPolicy: SandboxExecutionPolicy | undefined): Promise<void>;
+/** Move (rename) one file via the sandbox-aware shell. */
+export declare function moveFile(ctx: Context, exec: ToolRunContext, cfg: ResolvedConfig, source: FsTarget, target: FsTarget, sandboxPolicy: SandboxExecutionPolicy | undefined): Promise<void>;

package/lib/types/tool.d.ts

@@ -0,0 +1,46 @@
+/**
+ * The single model-facing `apply_patch` tool definition.
+ *
+ * The description carries an explicit routing guide so the model reaches for
+ * the native `edit` tool on cheap single-point changes instead of paying for
+ * a patch, and reserves `apply_patch` for large-file small edits, cross-file
+ * batch edits, creation, deletion and renames.
+ * @module dsh-patch-edit-plus/tool
+ */
+import type { ToolDefinition } from '@deepseek-ai/dsh-tools';
+import type { Context } from '@deepseek-ai/cordis';
+import type { ResolvedConfig } from './config.js';
+import type { FileAction } from './parse/types.js';
+/** Per-file diff carried in the canonical value (replay-safe presentationMeta source). */
+export type ResultDiff = {
+    path: string;
+    oldText: string | null;
+    newText: string;
+    /** True when the diff was dropped because of `maxDiffBytes`. */
+    truncated?: boolean;
+};
+/** Canonical tool output value. */
+export type ApplyPatchResult = {
+    format: 'unified' | 'codex';
+    applied: boolean;
+    summary: string;
+    files: {
+        path: string;
+        action: FileAction;
+        from?: string;
+    }[];
+    stats: {
+        files: number;
+        added: number;
+        removed: number;
+    };
+    wallTimeMs: number;
+    diffs: ResultDiff[];
+};
+/**
+ * Build the (frozen) tool definition. The tool name is passed in resolved
+ * form so registration-side rename avoidance can mint variant names.
+ */
+export declare function buildApplyPatchTool(toolName: string, ctx: Context, cfg: ResolvedConfig): ToolDefinition;
+/** Model-facing text: grouped added → modified → deleted listing. */
+export declare function formatResultText(value: ApplyPatchResult): string;

package/package.json

@@ -0,0 +1,103 @@
+{
+  "name": "dsh-patch-edit-plus",
+  "version": "0.1.0",
+  "description": "Patch-style file editing for DeepSeek Harness: one apply_patch tool accepting git/unified diff (default) and Codex apply_patch syntax (opt-in), with all-or-nothing application, read-before-write intent dance, sandbox-aware delete/move, and 0.1.2-rc.1 ~ 0.1.5-rc.2 compatibility.",
+  "type": "module",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/drscrewdriver/dsh-patch-edit-plus.git"
+  },
+  "main": "lib/index.js",
+  "types": "lib/types/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./lib/types/index.d.ts",
+      "default": "./lib/index.js"
+    },
+    "./package.json": "./package.json"
+  },
+  "files": [
+    "lib/",
+    "cordis.patch.yml",
+    "dsh.plugin.json",
+    "screenshots.json",
+    "README.md",
+    "README.zh.md",
+    "CHANGELOG.md",
+    "CHANGELOG.zh.md",
+    "LICENSE",
+    "assets/"
+  ],
+  "dsh": {
+    "bundle": {
+      "patch": "./cordis.patch.yml"
+    }
+  },
+  "scripts": {
+    "build": "tsc -p tsconfig.build.json",
+    "prepublishOnly": "npm run build",
+    "typecheck": "tsc -p tsconfig.json",
+    "typecheck:0.1.5": "node scripts/typecheck-0.1.5.mjs",
+    "lint": "eslint src tests scripts _smoke",
+    "lint:fix": "eslint src tests scripts _smoke --fix",
+    "test": "vitest run",
+    "verify:source": "node scripts/verify-source.mjs",
+    "smoke": "node _smoke/load-smoke.mjs",
+    "clean": "node -e \"require('node:fs').rmSync('lib', { recursive: true, force: true })\""
+  },
+  "engines": {
+    "node": ">=20",
+    "dsh": ">=0.1.2-rc.1 <0.2.0-0"
+  },
+  "license": "MIT",
+  "keywords": [
+    "dsh",
+    "dsh-plugin",
+    "deepseek-harness",
+    "apply-patch",
+    "unified-diff",
+    "codex",
+    "patch",
+    "edit"
+  ],
+  "peerDependencies": {
+    "@deepseek-ai/cordis": "^4.0.1",
+    "@deepseek-ai/dsh-fs": ">=0.1.2-rc.1",
+    "@deepseek-ai/dsh-sandbox": ">=0.1.2-rc.1",
+    "@deepseek-ai/dsh-shell": ">=0.1.2-rc.1",
+    "@deepseek-ai/dsh-tools": ">=0.1.2-rc.1"
+  },
+  "peerDependenciesMeta": {
+    "@deepseek-ai/cordis": {
+      "optional": true
+    },
+    "@deepseek-ai/dsh-fs": {
+      "optional": true
+    },
+    "@deepseek-ai/dsh-sandbox": {
+      "optional": true
+    },
+    "@deepseek-ai/dsh-shell": {
+      "optional": true
+    },
+    "@deepseek-ai/dsh-tools": {
+      "optional": true
+    }
+  },
+  "dependencies": {
+    "@deepseek-ai/schemastery": "^3.18.2"
+  },
+  "devDependencies": {
+    "@deepseek-ai/cordis": "^4.0.2",
+    "@deepseek-ai/dsh-fs": "0.1.2-rc.1",
+    "@deepseek-ai/dsh-sandbox": "0.1.2-rc.1",
+    "@deepseek-ai/dsh-shell": "0.1.2-rc.1",
+    "@deepseek-ai/dsh-tools": "0.1.2-rc.1",
+    "@eslint/js": "^9.0.0",
+    "@types/node": "^22.0.0",
+    "eslint": "^9.0.0",
+    "typescript": "^5.6.0",
+    "typescript-eslint": "^8.0.0",
+    "vitest": "^3.0.0"
+  }
+}

package/screenshots.json

@@ -0,0 +1,3 @@
+{
+  "screenshots": []
+}