feat(hook): notify agent when no VCS detected on plan approval

When a plan is approved in a directory without version control,
include a message in the approval output so the agent can ask
the user to initialize git before making changes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: yslee-abutton

apps/hook/server/index.ts

@@ -63,7 +63,7 @@ import {
   startAnnotateServer,
   handleAnnotateServerReady,
 } from "@plannotator/server/annotate";
-import { type DiffType, getVcsContext, runVcsDiff } from "@plannotator/server/vcs";
+import { type DiffType, getVcsContext, runVcsDiff, detectVcs } from "@plannotator/server/vcs";
 import { parsePRUrl, checkPRAuth, fetchPR, getCliName, getCliInstallUrl, getMRLabel, getMRNumberLabel, getDisplayRepo } from "@plannotator/server/pr";
 import { writeRemoteShareLink } from "@plannotator/server/share-url";
 import { resolveMarkdownFile, hasMarkdownFiles } from "@plannotator/shared/resolve-file";
@@ -820,10 +820,19 @@ if (args[0] === "sessions") {
   // Cleanup
   server.stop();
 
+  // Check if working directory has version control (git or p4)
+  const noVcsMessage = await (async () => {
+    if (!result.approved) return "";
+    const provider = await detectVcs();
+    if (await provider.detect()) return "";
+    return "\n\n[Note: No version control detected in this directory. Before making changes, ask the user if they want to initialize git to enable change tracking.]";
+  })();
+
   // Output decision in the appropriate format for the harness
   if (isGemini) {
     if (result.approved) {
-      console.log(result.feedback ? JSON.stringify({ systemMessage: result.feedback }) : "{}");
+      const systemMessage = (result.feedback || "") + noVcsMessage;
+      console.log(systemMessage ? JSON.stringify({ systemMessage }) : "{}");
     } else {
       console.log(
         JSON.stringify({
@@ -853,6 +862,7 @@ if (args[0] === "sessions") {
             decision: {
               behavior: "allow",
               ...(updatedPermissions.length > 0 && { updatedPermissions }),
+              ...(noVcsMessage && { message: noVcsMessage.trim() }),
             },
           },
         })