From 8285a18f1850428bb4503bfa6ee09e3044eb3a7b Mon Sep 17 00:00:00 2001 From: "opencode-agent[bot]" <219766164+opencode-agent[bot]@users.noreply.github.com> Date: Fri, 31 Jul 2026 20:32:35 -0400 Subject: [PATCH] refactor(tui): remove unused revert diff parser (#39963) Co-authored-by: Kit Langton --- bun.lock | 1 - packages/tui/package.json | 1 - packages/tui/src/util/revert-diff.ts | 18 ----------- packages/tui/test/util/revert-diff.test.ts | 35 ---------------------- 4 files changed, 55 deletions(-) delete mode 100644 packages/tui/src/util/revert-diff.ts delete mode 100644 packages/tui/test/util/revert-diff.test.ts diff --git a/bun.lock b/bun.lock index b6277f3b650..215c55c66c7 100644 --- a/bun.lock +++ b/bun.lock @@ -88,7 +88,6 @@ "@tanstack/solid-query": "5.91.4", "@tanstack/solid-virtual": "catalog:", "@thisbeyond/solid-dnd": "0.7.5", - "diff": "catalog:", "effect": "catalog:", "fuzzysort": "catalog:", "ghostty-web": "github:anomalyco/ghostty-web#83c0a07b8628b748aed073b232cb4b52a6ca11c1", diff --git a/packages/tui/package.json b/packages/tui/package.json index 5fa63c5a76c..215bd69a89b 100644 --- a/packages/tui/package.json +++ b/packages/tui/package.json @@ -93,7 +93,6 @@ "@opentui/solid": "catalog:", "@solid-primitives/event-bus": "1.1.2", "clipboardy": "4.0.0", - "diff": "catalog:", "effect": "catalog:", "fuzzysort": "catalog:", "get-east-asian-width": "catalog:", diff --git a/packages/tui/src/util/revert-diff.ts b/packages/tui/src/util/revert-diff.ts deleted file mode 100644 index 6ee1737f0b8..00000000000 --- a/packages/tui/src/util/revert-diff.ts +++ /dev/null @@ -1,18 +0,0 @@ -import { parsePatch } from "diff" - -export function getRevertDiffFiles(diffText: string) { - if (!diffText) return [] - - try { - return parsePatch(diffText).map((patch) => { - const filename = [patch.newFileName, patch.oldFileName].find((item) => item && item !== "/dev/null") ?? "unknown" - return { - filename: filename.replace(/^[ab]\//, ""), - additions: patch.hunks.reduce((sum, hunk) => sum + hunk.lines.filter((line) => line.startsWith("+")).length, 0), - deletions: patch.hunks.reduce((sum, hunk) => sum + hunk.lines.filter((line) => line.startsWith("-")).length, 0), - } - }) - } catch { - return [] - } -} diff --git a/packages/tui/test/util/revert-diff.test.ts b/packages/tui/test/util/revert-diff.test.ts deleted file mode 100644 index fc4794e242d..00000000000 --- a/packages/tui/test/util/revert-diff.test.ts +++ /dev/null @@ -1,35 +0,0 @@ -import { describe, expect, test } from "bun:test" -import { getRevertDiffFiles } from "../../src/util/revert-diff" - -describe("revert diff", () => { - test("prefers the actual file path over /dev/null for added and deleted files", () => { - const files = getRevertDiffFiles(`diff --git a/new.txt b/new.txt -new file mode 100644 -index 0000000..3b18e51 ---- /dev/null -+++ b/new.txt -@@ -0,0 +1 @@ -+new content -diff --git a/old.txt b/old.txt -deleted file mode 100644 -index 3b18e51..0000000 ---- a/old.txt -+++ /dev/null -@@ -1 +0,0 @@ --old content -`) - - expect(files).toEqual([ - { - filename: "new.txt", - additions: 1, - deletions: 0, - }, - { - filename: "old.txt", - additions: 0, - deletions: 1, - }, - ]) - }) -})