fix: add 'skipped_not_requested' to GitCommitStepStatus for non-commit actions

When the action is 'push' or 'create_pr', the commit step was hardcoded
to 'skipped_no_changes' which incorrectly implies the system tried to
commit but found nothing. Added 'skipped_not_requested' variant to
GitCommitStepStatus (matching the pattern used by push, PR, and branch
step schemas) and use it when the commit phase is not part of the action.

Author: cursoragent

apps/server/src/git/Layers/GitManager.test.ts

@@ -1388,7 +1388,7 @@ it.layer(GitManagerTestLayer)("GitManager", (it) => {
         action: "push",
       });
 
-      expect(result.commit.status).toBe("skipped_no_changes");
+      expect(result.commit.status).toBe("skipped_not_requested");
       expect(result.push.status).toBe("pushed");
       expect(result.pr.status).toBe("skipped_not_requested");
       expect(
@@ -1432,7 +1432,7 @@ it.layer(GitManagerTestLayer)("GitManager", (it) => {
         action: "create_pr",
       });
 
-      expect(result.commit.status).toBe("skipped_no_changes");
+      expect(result.commit.status).toBe("skipped_not_requested");
       expect(result.push.status).toBe("pushed");
       expect(result.push.setUpstream).toBe(true);
       expect(result.pr.status).toBe("created");
@@ -2727,7 +2727,7 @@ it.layer(GitManagerTestLayer)("GitManager", (it) => {
         },
       );
 
-      expect(result.commit.status).toBe("skipped_no_changes");
+      expect(result.commit.status).toBe("skipped_not_requested");
       expect(result.push.status).toBe("skipped_not_requested");
       expect(result.pr.status).toBe("created");
       expect(

apps/server/src/git/Layers/GitManager.ts

@@ -1592,7 +1592,7 @@ export const makeGitManager = Effect.fn("makeGitManager")(function* () {
                 ),
               ),
             )
-          : { status: "skipped_no_changes" as const };
+          : { status: "skipped_not_requested" as const };
 
         const push = wantsPush
           ? yield* progress

packages/contracts/src/git.ts

@@ -27,7 +27,11 @@ export const GitActionProgressKind = Schema.Literals([
 export type GitActionProgressKind = typeof GitActionProgressKind.Type;
 export const GitActionProgressStream = Schema.Literals(["stdout", "stderr"]);
 export type GitActionProgressStream = typeof GitActionProgressStream.Type;
-const GitCommitStepStatus = Schema.Literals(["created", "skipped_no_changes"]);
+const GitCommitStepStatus = Schema.Literals([
+  "created",
+  "skipped_no_changes",
+  "skipped_not_requested",
+]);
 const GitPushStepStatus = Schema.Literals([
   "pushed",
   "skipped_not_requested",