Add skip build flag functionality

kleneway commented 1 month ago

When a new request comes in, the current process runs a build, updates the code, and then runs another build to ensure nothing broke. We need to add a flag (passed via the text of the GitHub issue) that skips the build. This will speed up the process and allow users to quickly try out JACoB without worrying about the semantics of how to properly build the project.

Requirements:

Modify checkAndCommit function:
- Update the checkAndCommit function in src/server/code/checkAndCommit.ts to accept a skipBuild parameter.
- Add logic to skip the build process if the skipBuild flag is set to true.
Update editFiles function:
- Modify the editFiles function in src/server/code/editFiles.ts to parse the issue body for the skip build flag and pass it to checkAndCommit.
Update getIssue function:
- Ensure the getIssue function in src/server/github/issue.ts retrieves the issue body correctly for parsing.
Parameter Interface:
- Update the CheckAndCommitParams interface to include the skipBuild parameter.

Files to Update:

src/server/code/checkAndCommit.ts
src/server/code/editFiles.ts
src/server/github/issue.ts
src/types.ts

Acceptance Criteria:

[ ] The checkAndCommit function should conditionally skip the build process based on the skipBuild flag.
[ ] The editFiles function should correctly parse the issue body for the skip build flag and pass it to checkAndCommit.
[ ] The getIssue function should retrieve the issue body for parsing.
[ ] The CheckAndCommitParams interface should include the skipBuild parameter.
[ ] Add tests to ensure the skip build flag works as expected.

Example Code Snippet:

// src/server/code/checkAndCommit.ts
export async function checkAndCommit({
  octokit,
  owner,
  repo,
  branch,
  message,
  files,
  issueNumber,
  baseBranch,
  skipBuild, // New parameter
}: CheckAndCommitParams): Promise<CheckAndCommitResult> {
  // ... (existing code)

  if (!skipBuild) {
    const buildResult = await buildAndTest({ cwd: repoPath });

    if (!buildResult.success) {
      console.log("Build failed");
      return {
        success: false,
        error: "Build failed",
        buildErrors: buildResult.errors,
      };
    }
  } else {
    console.log("Build skipped due to skipBuild flag");
  }

  // ... (rest of the function)
}

@jacob-ai-bot

Steps to Address Issue: 1. Update CheckAndCommitParams interface in src/types.ts to include skipBuild parameter.

Modify checkAndCommit function in src/server/code/checkAndCommit.ts to accept skipBuild parameter and implement logic to skip build process when flag is true.
Update editFiles function in src/server/code/editFiles.ts to parse issue body for skip build flag and pass it to checkAndCommit.
Ensure getIssue function in src/server/github/issue.ts correctly retrieves the issue body for parsing.
Add tests to verify the skip build functionality works as expected.

Files to Update: src/server/code/checkAndCommit.ts, src/server/code/editFiles.ts, src/server/github/issue.ts, src/types.ts

task assigned to: @jacob-ai-bot