refactor: make run() explicitly callable in tests, remove flaky setTimeout

Co-Authored-By: Niels Swimberghe <3382717+Swimburger@users.noreply.github.com>

Author: devin-ai-integration[bot]

dist/index.js

@@ -39766,7 +39766,10 @@ async function createPR(octokit, owner, repo, branchName, targetBranch, isFromFe
     core.info(`Pull request created: ${prResponse.data.html_url}`);
     return prResponse;
 }
-run();
+// Auto-invoke only when running as the action entry point (not when imported in tests)
+if (process.env.NODE_ENV !== "test") {
+    run();
+}
 
 
 /***/ }),

src/sync.test.ts

@@ -148,10 +148,8 @@ function setupMocks({
 async function importAndRun() {
     vi.resetModules();
 
-    await import("./sync");
-
-    // Wait for async operations to complete
-    await new Promise((resolve) => setTimeout(resolve, 200));
+    const { run } = await import("./sync");
+    await run();
 }
 
 beforeEach(() => {

src/sync.ts

@@ -693,4 +693,7 @@ async function createPR(
     return prResponse;
 }
 
-run();
+// Auto-invoke only when running as the action entry point (not when imported in tests)
+if (process.env.NODE_ENV !== "test") {
+    run();
+}