[OliverJAsh]

No description provided.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9b7f8dc70b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[Copilot]

Pull request overview

Refactors the Lite workspace preview UI to reduce duplication when rendering per-file diffs across change/commit/branch previews.

Changes:

• Removes the editable prop from Hunk, FileDiff, and preview components, simplifying call sites.
• Introduces ChangesFileDiffList to centralize “fetch diff + render file header + render FileDiff” logic for multiple preview types.
• Simplifies commit message body rendering in CommitPreview (with a behavioral caveat noted in review comments).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
apps/lite/ui/src/routes/project/$id/workspace/route.tsx	Extracts shared file-diff list rendering into ChangesFileDiffList and wires it into Changes/Commit/Branch previews; removes editable plumbing.
apps/lite/ui/src/routes/project/$id/workspace/Item.ts	Adds missing /** @public */ annotation to ChangesSectionItem to align with other exported item types.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

ChangesFileDiffList accepts fileParent and stackId independently, but Commit sources require a stackId to build a commitFileItem. Consider tightening the props with a discriminated union so stackId is required when fileParent?._tag === "Commit" (or handle the missing stackId explicitly) to avoid silent behavior differences if it’s ever called with an inconsistent prop combination.

Suggested change

	Match.tag("Commit", ({ commitId }) =>
	stackId !== undefined
	? commitFileItem({
	stackId,
	commitId,
	treeChange: change,
	})
	: undefined,
	),
	Match.tag("Commit", ({ commitId }) => {
	assert(
	stackId !== undefined,
	"`ChangesFileDiffList` requires `stackId` when `fileParent._tag === \"Commit\"`"
	);
	return commitFileItem({
	stackId,
	commitId,
	treeChange: change,
	});
	}),

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.