feat: add comment

Author: CasLubbers

Dockerfile

@@ -42,5 +42,7 @@ RUN echo 'nouser:x:999:999::/home/nouser:/bin/sh' >> /etc/passwd
 USER node
 EXPOSE 8080
 
+# tini is used as PID 1 to reap zombie child processes (e.g. git-remote-http)
+# that Node.js would otherwise leave unreaped, causing PID exhaustion over time
 ENTRYPOINT ["/sbin/tini", "--"]
 CMD ["node", "--max-http-header-size", "16384", "dist/src/app.js"]

src/git/connect.ts

@@ -13,9 +13,7 @@ const env = cleanEnv({
 
 export default async function getLatestRemoteCommitSha(): Promise<string | undefined> {
   try {
-    // Timeout ensures the git-remote-http child process is killed after 30 seconds if the remote is unreachable,
-    // preventing hung processes from accumulating and exhausting system fork() resources.
-    const git = simpleGit({ timeout: { block: 30000 } })
+    const git = simpleGit()
     const repoUrl = new URL(env.GIT_REPO_URL)
     repoUrl.username = encodeURIComponent(env.GIT_USER)
     repoUrl.password = encodeURIComponent(env.GIT_PASSWORD)

src/utils/workloadUtils.ts

@@ -233,10 +233,7 @@ export class chartRepo {
     this.chartRepoUrl = chartRepoUrl
     this.gitUser = gitUser
     this.gitEmail = gitEmail
-    // Timeout kills the git child process if it produces no output for 30 seconds.
-    // This is safe for large repositories: active clones/fetches continuously emit progress,
-    // so the timer only triggers on truly hung processes (e.g. unresponsive remote).
-    this.git = simpleGit(this.localPath, { timeout: { block: 30000 } })
+    this.git = simpleGit(this.localPath)
   }
   async clone(branch: string = 'main') {
     await this.git.clone(this.chartRepoUrl, this.localPath, ['--branch', branch, '--single-branch', '--depth', '1'])