smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: hashline improvements for spark

Browse Source 
- Redesigned hashline patch syntax from anchor-based (`A-B:`) to hunk-header format (`@@ A..B @@`) with unified-diff compatibility.
- Removed `autoDropPureInsertDuplicates` option and simplified apply behavior to preserve duplicated boundary and context lines.
- Changed repeat operator from `^A-B` to `&A..B` and range separator from `-` to `..` for consistency with hunk-header syntax.
- Added image resizing and dimension notes to eval tool output; improved write tool hashline header sanitation for legacy formats.
- Removed 521 lines of boundary-duplicate absorption code and simplified parser to auto-convert bare body rows and unified-diff contamination.
This commit is contained in:
can1357
2026-05-28 02:25:25 +02:00
parent b4238b10d3
commit 7dd00c015b
36 changed files with 1051 additions and 1837 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/test/edit-diff.test.ts
+4 -4
View File
@@ -236,12 +236,12 @@ describe("computeHashlineDiff", () => {
const line = "unchanged content";
await Bun.write(sourcePath, `${line}\n`);
// `1-1:` with the same line in the replace bucket is a true no-op: the edit
// `1 1` with the same line in the body is a true no-op: the edit
// fires through computeHashlineDiff but produces identical content.
const text = `${line}\n`;
const snapshotStore = new InMemorySnapshotStore();
const tag = snapshotStore.recordContiguous(sourcePath, 1, text.split("\n"), { fullText: text });
const input = `${formatHashlineHeader(sourcePath, tag)}\n1-1:\n|${line}\n`;
const input = `${formatHashlineHeader(sourcePath, tag)}\n1 1\n+${line}\n`;
const result = await computeHashlineDiff({ input }, tempDir, snapshotStore);
expect("error" in result).toBe(true);
if ("error" in result) {
@@ -254,7 +254,7 @@ describe("computeHashlineDiff", () => {
await Bun.write(sourcePath, "first\n");
const result = await computeHashlineDiff(
{ input: `${sourcePath}\nEOF:\n|second` },
{ input: `${sourcePath}\nEOF\n+second` },
tempDir,
new InMemorySnapshotStore(),
);
@@ -265,7 +265,7 @@ describe("computeHashlineDiff", () => {
});
test("returns a handled error when the source path is a local URL", async () => {
const result = await computeHashlineDiff(
{ input: "¶local://PLAN.md\nEOF:\n|x" },
{ input: "¶local://PLAN.md\nEOF\n+x" },
tempDir,
new InMemorySnapshotStore(),
);