scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-10-17 04:37:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix(web): show deleted Markdown files as diffs

Browse source
This commit is contained in:
Scott Werner 2026-10-15 13:49:23 -04:00
parent 3b1781022f
commit e87e0df84d
2 changed files with 17 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
apps/fabro-web/app/routes/run-files.render.test.tsx
View file

@ -258,6 +258,22 @@ describe("RunFiles rendering", () => {
expect(multiFileDiffCalls).toHaveLength(1);
});
test("shows deleted Markdown files as a diff instead of an empty preview", () => {
currentFilesPayload = makePayload(1);
currentFilesPayload.data[0] = {
change_kind: "deleted",
old_file: { name: "docs/plan.md", contents: "# Plan\n\nReview this\n" },
new_file: { name: "", contents: "" },
};
const renderer = renderRunFiles();
expect(renderer.root.findAllByProps({ "data-markdown-preview": "true" })).toHaveLength(0);
expect(multiFileDiffCalls).toHaveLength(1);
expect(multiFileDiffCalls[0].oldFile.contents).toBe("# Plan\n\nReview this\n");
expect(multiFileDiffCalls[0].newFile.contents).toBe("");
});
test("passes the selected URL scope to useRunFiles", () => {
currentFilesPayload = makePayload(1);

1
apps/fabro-web/app/routes/run-files.tsx
View file

@ -255,6 +255,7 @@ const RunFileRow = memo(function RunFileRow({
const oldPath = file.old_file.name || display;
const newPath = file.new_file.name || display;
const canPreviewMarkdown =
file.change_kind !== "deleted" &&
/\.md$/i.test(newPath) && newContents != null && !placeholder;
const [showMarkdownPreview, setShowMarkdownPreview] =
useState(canPreviewMarkdown);