feat: remove isFirst property from checkpoint system

- Removed isFirst from checkpoint schema
- Removed isFirst from CheckpointEventMap type
- Removed isFirst logic from ShadowCheckpointService
- Updated core checkpoints to not use isFirst
- Updated tests to remove isFirst references

This completes the removal of the Initial Checkpoint terminology distinction.

Author: roomote

src/core/checkpoints/index.ts

@@ -134,12 +134,12 @@ async function checkGitInstallation(
 			cline.checkpointServiceInitializing = false
 		})
 
-		service.on("checkpoint", ({ isFirst, fromHash: from, toHash: to }) => {
+		service.on("checkpoint", ({ fromHash: from, toHash: to }) => {
 			try {
 				provider?.postMessageToWebview({ type: "currentCheckpointUpdated", text: to })
 
 				cline
-					.say("checkpoint_saved", to, undefined, undefined, { isFirst, from, to }, undefined, {
+					.say("checkpoint_saved", to, undefined, undefined, { from, to }, undefined, {
 						isNonInteractive: true,
 					})
 					.catch((err) => {

src/services/checkpoints/ShadowCheckpointService.ts

@@ -215,14 +215,13 @@ export abstract class ShadowCheckpointService extends EventEmitter {
 			await this.stageAll(this.git)
 			const commitArgs = options?.allowEmpty ? { "--allow-empty": null } : undefined
 			const result = await this.git.commit(message, commitArgs)
-			const isFirst = this._checkpoints.length === 0
 			const fromHash = this._checkpoints[this._checkpoints.length - 1] ?? this.baseHash!
 			const toHash = result.commit || fromHash
 			this._checkpoints.push(toHash)
 			const duration = Date.now() - startTime
 
-			if (isFirst || result.commit) {
-				this.emit("checkpoint", { type: "checkpoint", isFirst, fromHash, toHash, duration })
+			if (result.commit) {
+				this.emit("checkpoint", { type: "checkpoint", fromHash, toHash, duration })
 			}
 
 			if (result.commit) {

src/services/checkpoints/__tests__/ShadowCheckpointService.spec.ts

@@ -693,7 +693,6 @@ describe.each([[RepoPerTaskCheckpointService, "RepoPerTaskCheckpointService"]])(
 				expect(eventData.type).toBe("checkpoint")
 				expect(eventData.toHash).toBe(result?.commit)
 				expect(typeof eventData.duration).toBe("number")
-				expect(typeof eventData.isFirst).toBe("boolean") // Can be true or false depending on checkpoint history
 			})
 
 			it("does not emit checkpoint event when no changes and allowEmpty=false", async () => {

src/services/checkpoints/types.ts

@@ -25,7 +25,6 @@ export interface CheckpointEventMap {
 	initialize: { type: "initialize"; workspaceDir: string; baseHash: string; created: boolean; duration: number }
 	checkpoint: {
 		type: "checkpoint"
-		isFirst: boolean
 		fromHash: string
 		toHash: string
 		duration: number

webview-ui/src/components/chat/checkpoints/schema.ts

@@ -1,7 +1,6 @@
 import { z } from "zod"
 
 export const checkpointSchema = z.object({
-	isFirst: z.boolean(),
 	from: z.string(),
 	to: z.string(),
 })