From 4458b1ea418dd19ced833fdc34b929e7f6add5db Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 22 Mar 2026 16:25:25 +0000
Subject: [PATCH 1/2] Initial plan


From 4e73ac4cf4f5e7dceb145beef7cc2a7c5e9e7035 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 22 Mar 2026 16:41:59 +0000
Subject: [PATCH 2/2] fix: exclude .git directory from push_repo_memory size
 check

The memory directory (/tmp/gh-aw/repo-memory/{id}) is a shallow git
clone created by clone_repo_memory_branch.sh. The scanDir function
in safe_outputs_handlers.cjs did not skip the .git directory, causing
git pack files to accumulate across runs and inflate the reported
"total memory size", eventually causing push_repo_memory to fail.

Fix scanDir to skip .git directories so only actual memory files are
counted toward the size limit.

Adds a test that simulates a large .git pack file alongside tiny memory
files and confirms the handler correctly reports success.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/f6471e53-0dd8-4c5f-a6c4-ada8411df77c
---
 actions/setup/js/safe_outputs_handlers.cjs    |  6 +++++
 .../setup/js/safe_outputs_handlers.test.cjs   | 23 +++++++++++++++++++
 2 files changed, 29 insertions(+)

diff --git a/actions/setup/js/safe_outputs_handlers.cjs b/actions/setup/js/safe_outputs_handlers.cjs
index d631684f1ee..6632b9626e9 100644
--- a/actions/setup/js/safe_outputs_handlers.cjs
+++ b/actions/setup/js/safe_outputs_handlers.cjs
@@ -558,6 +558,12 @@ function createHandlers(server, appendSafeOutput, config = {}) {
     function scanDir(dirPath, relativePath) {
       const entries = fs.readdirSync(dirPath, { withFileTypes: true });
       for (const entry of entries) {
+        // Skip .git directory to avoid counting git metadata as memory content.
+        // The memory directory is a git clone, so .git may contain pack files that
+        // grow with each commit and must not be counted toward the memory size limit.
+        if (entry.isDirectory() && entry.name === ".git") {
+          continue;
+        }
         const fullPath = path.join(dirPath, entry.name);
         const relPath = relativePath ? path.join(relativePath, entry.name) : entry.name;
         if (entry.isDirectory()) {
diff --git a/actions/setup/js/safe_outputs_handlers.test.cjs b/actions/setup/js/safe_outputs_handlers.test.cjs
index 657d2483838..016bfaecaf3 100644
--- a/actions/setup/js/safe_outputs_handlers.test.cjs
+++ b/actions/setup/js/safe_outputs_handlers.test.cjs
@@ -643,5 +643,28 @@ describe("safe_outputs_handlers", () => {
       // The nested file path should appear correctly
       expect(data.error).toContain("exceeds the allowed limit");
     });
+
+    it("should exclude .git directory from size calculation", () => {
+      // Simulate the real scenario: memory directory is a git clone.
+      // The .git directory can accumulate pack files across runs.
+      // With max_patch_size = 500 bytes (effective limit = 600 bytes), actual memory
+      // files are small but .git directory content is large — must not count toward limit.
+      const h = makeHandlersWithMemory({ max_patch_size: 500, max_file_size: 1024 * 1024 });
+      fs.mkdirSync(memoryDir, { recursive: true });
+      // Small memory files (well within limit)
+      fs.writeFileSync(path.join(memoryDir, "memory.json"), "x".repeat(100));
+      fs.writeFileSync(path.join(memoryDir, "state.json"), "x".repeat(100));
+      // Simulate a large .git directory (pack files accumulate with each run)
+      const gitDir = path.join(memoryDir, ".git");
+      const packDir = path.join(gitDir, "objects", "pack");
+      fs.mkdirSync(packDir, { recursive: true });
+      fs.writeFileSync(path.join(packDir, "pack-abc123.pack"), "x".repeat(30000));
+      // Total without .git: 200 bytes (within 600 byte limit)
+      // Total with .git: 30200 bytes (would exceed limit if .git were included)
+      const result = h.pushRepoMemoryHandler({ memory_id: "default" });
+      const data = JSON.parse(result.content[0].text);
+      expect(data.result).toBe("success");
+      expect(data.message).toContain("validation passed");
+    });
   });
 });