svn-agent-mcp 1.8.2 → 1.8.3

package/CHANGELOG.md

@@ -4,6 +4,25 @@ All notable changes to the SVN MCP are recorded here.
 
 ## [Unreleased]
 
+## [1.8.3] - 2026-09-07
+
+### Fixed
+
+- Refreshed compatible transitive dependencies in the development lockfile to resolve
+  the current fast-uri, qs, and browserslist audit findings.
+- Automatic added-file EOL normalization now works alongside ordinary changes in the same
+  explicit scope. EOL-only proof applies to repaired files, and the entire precommit is rerun.
+- EOL repair resolves exact file targets correctly when the caller uses a subdirectory cwd.
+- Routine mixed-revision state no longer adds a commit/precommit warning; diagnostic state
+  remains in full output and failures retain stale-path evidence.
+
+### Changed
+
+- Compact commit receipts omit SHA-256 lists; use `fields:["contentHashes"]` or full output.
+- Commit receipts carry line statistics from the revalidated precommit diff, without another
+  SVN call. Compact statistics include totals and at most 25 paths / 4 KiB of file entries;
+  `filesTruncated:true` signals omitted entries, available in full output.
+
 ## [1.8.2] - 2026-08-27
 
 ### Changed

package/README.md

@@ -2,7 +2,7 @@
 
 Strict SVN Model Context Protocol server for agent-safe status, diff, EOL diagnosis, precommit checks, and guarded SVN mutations.
 
-The implementation contract lives in `docs/SPEC.md`. The current source release is `1.8.2`; each source clone can prepare a local runtime under `releases/v1.8.2`, while npm installations run directly from package-root `dist/`.
+The implementation contract lives in `docs/SPEC.md`. The current source release is `1.8.3`; each source clone can prepare a local runtime under `releases/v1.8.3`, while npm installations run directly from package-root `dist/`.
 
 Requirements: Node.js 24.18.0 or newer, npm 11.16.0 or newer, Git, and access to the public npm registry. Windows uses the
 bundled VisualSVN Apache Subversion command-line package and dos2unix payload. On macOS and Linux, `svn`, `svnversion`, `svnadmin`,
@@ -243,11 +243,10 @@ Release workflows can pin `svn_update` with an exact `revision`; it still requir
 or `updateAll:true` and always postpones conflicts. Add `expectedRemoteHead` with a numeric revision
 to refuse if repository HEAD moved since the caller's probe. Use
 `svn_precommit requireUniformRevision:true` when a release handoff must not proceed from a
-mixed-revision working copy. The default remains backward compatible and reports mixed revisions
-without blocking ordinary precommit work.
+mixed-revision working copy. Ordinary precommit work does not warn about mixed revisions.
 
 Parallel agents can create a valid mixed-revision working copy. A commit receipt may report
-`workingCopyMixed:true` while the commit remains valid. `baseRevision:null` means no single base
+`working_copy_mixed:true` in full mode while the commit remains valid. `baseRevision:null` means no single base
 revision describes every committed path; use `baseRevisionRange` for the useful range. Treat mixed
 state as evidence, not failure by itself. Check the separate out-of-date and conflict diagnostics
 before deciding whether to stop.
@@ -388,7 +387,14 @@ Newly added text files have no `BASE`. For status `A`, the same safe mode uses a
 `svn:eol-style` or `.svn-mcp-policy.json normalizeEol` target, verifies valid UTF-8/no BOM and
 normalized-content identity, applies only that explicit file, and continues to issue the precommit
 token in the same call. An explicit file property takes precedence over repository fallback policy;
-the implementation never infers EOL policy from neighboring files.
+the implementation never infers EOL policy from neighboring files. Added files can be normalized
+alongside ordinary modifications in the same explicit scope, including calls from a subdirectory.
+
+Compact commit receipts omit per-file hashes; request `fields:["contentHashes"]` or
+`responseMode:"full"` when needed. Token-bound commits include `diffStat` from the validated
+precommit diff: total added/removed lines and up to 25 paths / 4 KiB of file entries. Larger
+scopes report `filesTruncated:true`; full output includes all available entries. Statistics
+describe the checked content diff, ignoring EOL, and `complete:false` means counts are partial.
 
 ## Commands
 

package/dist/response.js

@@ -105,7 +105,7 @@ function shapePayload(tool, payload, mode, request) {
     }
     if (compactMode && tool === "svn_commit"
         && (payload.operation === "safe_commit" || payload.operation === "safe_commit_detail")) {
-        return compactSafeCommit(payload);
+        return compactSafeCommit(payload, request);
     }
     if (compactMode && (tool === "svn_prepare_commit" || (tool === "svn_commit" && payload.operation === "prepare_commit"))) {
         return compactPrepareCommit(payload, request);
@@ -241,6 +241,7 @@ function receiptPayload(tool, payload, request) {
         assignDefined(result, "scopeCheckUnavailableReason", payload.scope_check_unavailable_reason);
     }
     if (tool === "svn_commit") {
+        Object.assign(result, compactCommitDiffStat(payload));
         assignDefined(result, "postStatusClean", payload.post_status_clean);
         assignDefined(result, "postStatusScope", payload.post_status_scope);
         const postStatusPaths = stringArray(payload.post_status_paths).slice(0, 100);
@@ -332,6 +333,7 @@ function applyFieldProjection(tool, payload, shaped, request) {
         revision: payload.revision,
         revisionRange: payload.revision_range,
         mixedRevision: payload.mixed_revision,
+        contentHashes: recordArray(payload.content_hashes).slice(0, 100),
         remoteHeadRevision: payload.remote_head_revision !== undefined
             ? payload.remote_head_revision
             : payload.observed_remote_head,
@@ -360,6 +362,10 @@ function applyFieldProjection(tool, payload, shaped, request) {
         && projected.conflictCount === undefined) {
         projected.conflictCount = source.conflictCount;
     }
+    if (fields.includes("contentHashes") && recordArray(payload.content_hashes).length > 100) {
+        projected.contentHashCount = recordArray(payload.content_hashes).length;
+        projected.contentHashesTruncated = true;
+    }
     return { ok: true, ...projected };
 }
 const ROOT_RELATIVE_PATH_ARRAY_KEYS = new Set([
@@ -1169,8 +1175,9 @@ function compactPrecommit(payload, request) {
             ? { rollbackFailedPaths, rollbackFailedPathCount: allRollbackFailedPaths.length,
                 ...(allRollbackFailedPaths.length > rollbackFailedPaths.length ? { rollbackFailedPathsTruncated: true } : {}) }
             : {}),
-        mixedRevision,
-        ...(mixedRevision && payload.revision_range ? { revisionRange: payload.revision_range } : {}),
+        ...(!payload.ok || verdict === "REVISION_NORMALIZATION_NEEDED"
+            ? { mixedRevision, ...(payload.revision_range ? { revisionRange: payload.revision_range } : {}) }
+            : {}),
         ...(payload.remediation ? { remediation: payload.remediation } : {}),
         ...(nextAction ? { nextAction } : {}),
         ...(guardFailures.length > 0 ? { guardFailures } : {}),
@@ -1271,7 +1278,31 @@ function compactPrepareReceipt(payload) {
         ...(!payload.ok && payload.note ? { note: payload.note } : {})
     };
 }
-function compactSafeCommit(payload) {
+function compactCommitDiffStat(payload) {
+    if (!payload.diff_stat || typeof payload.diff_stat !== "object")
+        return {};
+    const stat = payload.diff_stat;
+    const allFiles = recordArray(stat.files);
+    const files = [];
+    let bytes = 0;
+    for (const file of allFiles.slice(0, 25)) {
+        const size = Buffer.byteLength(JSON.stringify(file), "utf8");
+        if (bytes + size > 4096)
+            break;
+        files.push(file);
+        bytes += size;
+    }
+    return { diffStat: {
+            source: stat.source,
+            complete: stat.complete,
+            added: stat.added,
+            removed: stat.removed,
+            files,
+            fileCount: allFiles.length,
+            ...(files.length < allFiles.length ? { filesTruncated: true } : {})
+        } };
+}
+function compactSafeCommit(payload, request = {}) {
     if (payload.operation === "safe_commit_detail") {
         return {
             ok: payload.ok,
@@ -1306,6 +1337,9 @@ function compactSafeCommit(payload) {
         ...(payload.detail_expires_at ? { detailExpiresAt: payload.detail_expires_at } : {}),
         ...(payload.detail_cursor ? { detailCursor: payload.detail_cursor } : {}),
         ...(payload.code ? { code: payload.code } : {}),
+        ...compactCommitDiffStat(payload),
+        ...(stringArray(request.fields).includes("contentHashes") && payload.content_hashes !== undefined
+            ? { contentHashes: recordArray(payload.content_hashes).slice(0, 100) } : {}),
         ...compactAutoEol(payload),
         ...(!payload.ok && payload.note ? { note: payload.note } : {})
     };
@@ -1412,7 +1446,6 @@ function compactMutation(tool, payload, request) {
         assignDefined(receipt, "baseRevisionRange", payload.base_revision_range);
         assignDefined(receipt, "remoteHeadRevision", payload.remote_head_revision);
         assignDefined(receipt, "eolVerdict", payload.eol_verdict);
-        assignDefined(receipt, "workingCopyMixed", payload.working_copy_mixed);
         assignDefined(receipt, "postStatusScope", payload.post_status_scope);
         const postStatusPaths = stringArray(payload.post_status_paths).slice(0, 100);
         if (payload.post_status_paths !== undefined)
@@ -1422,7 +1455,8 @@ function compactMutation(tool, payload, request) {
             receipt.postStatusPathsTruncated = true;
         assignDefined(receipt, "workingCopyClean", payload.working_copy_clean);
         assignDefined(receipt, "precommitToken", payload.precommit_token);
-        if (payload.content_hashes !== undefined) {
+        Object.assign(receipt, compactCommitDiffStat(payload));
+        if (payload.content_hashes !== undefined && stringArray(request.fields).includes("contentHashes")) {
             receipt.contentHashes = recordArray(payload.content_hashes).slice(0, 100);
         }
         if (committedPaths.length > 100) {