[25.1] Fix TERMINAL_STATES array has a duplicate state

The array has `skipped` listed twice. This was causing the bug where we would over-count jobs in the progress bar in the invocation view:

Fixes #21266

Author: ahmedhamidawan

client/src/api/jobs.ts

@@ -17,7 +17,7 @@ export type JobMessage =
 
 export const NON_TERMINAL_STATES = ["new", "queued", "running", "waiting", "paused", "resubmitted", "stop"];
 export const ERROR_STATES = ["error", "deleted", "deleting"];
-export const TERMINAL_STATES = ["ok", "skipped", "stop", "stopping", "skipped"].concat(ERROR_STATES);
+export const TERMINAL_STATES = ["ok", "skipped", "stop", "stopping"].concat(ERROR_STATES);
 
 interface JobDef {
     tool_id: string;