Skip to content

Commit 2945431

Browse files
authored
chore: disabled autocommit (#41246)
## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Git" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!IMPORTANT] > 🟣 🟣 🟣 Your tests are running. > Tests running at: <https://github.com/appsmithorg/appsmith/actions/runs/17913609729> > Commit: 2055a6b > Workflow: `PR Automation test suite` > Tags: `@tag.Git` > Spec: `` > <hr>Mon, 22 Sep 2025 11:18:58 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Temporarily disabled Git auto-commit; auto-commit operations no longer execute. * Auto-commit status responses now consistently report IDLE with 0% progress. * Users may notice no automatic commits in linked repositories; manual commits unaffected. * No changes to public API signatures; only response behavior adjusted. * **Tests** * End-to-end test for Git autocommit is skipped to reflect disabled auto-commit. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 36c34db commit 2945431

2 files changed

Lines changed: 8 additions & 4 deletions

File tree

app/client/cypress/e2e/Regression/ClientSide/Git/GitAutocommit_spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ describe(
2626
],
2727
},
2828
function () {
29-
it("Check if autocommit progress bar is visible and network requests are properly called", function () {
29+
it.skip("Check if autocommit progress bar is visible and network requests are properly called", function () {
3030
agHelper.GenerateUUID();
3131
cy.get("@guid").then((uid) => {
3232
wsName = "GitAC-" + uid;

app/server/appsmith-server/src/main/java/com/appsmith/server/git/controllers/GitApplicationControllerCE.java

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
import com.appsmith.server.domains.GitArtifactMetadata;
1616
import com.appsmith.server.domains.GitAuth;
1717
import com.appsmith.server.dtos.AutoCommitResponseDTO;
18+
import com.appsmith.server.dtos.AutoCommitResponseDTO.AutoCommitResponse;
1819
import com.appsmith.server.dtos.BranchProtectionRequestDTO;
1920
import com.appsmith.server.dtos.GitAuthDTO;
2021
import com.appsmith.server.dtos.GitConnectDTO;
@@ -288,9 +289,12 @@ public Mono<ResponseDTO<List<String>>> getProtectedBranches(@PathVariable String
288289
artifactType = ArtifactType.APPLICATION,
289290
operation = GitRouteOperation.AUTO_COMMIT)
290291
public Mono<ResponseDTO<AutoCommitResponseDTO>> autoCommitApplication(@PathVariable String branchedApplicationId) {
291-
return autoCommitService
292-
.autoCommitApplication(branchedApplicationId)
293-
.map(data -> new ResponseDTO<>(HttpStatus.OK, data));
292+
// disabling autocommit till in-memory git has been incorporated in the auto-commit
293+
AutoCommitResponseDTO autoCommitResponseDTO = new AutoCommitResponseDTO();
294+
autoCommitResponseDTO.setAutoCommitResponse(AutoCommitResponse.IDLE);
295+
autoCommitResponseDTO.setProgress(0);
296+
297+
return Mono.just(autoCommitResponseDTO).map(data -> new ResponseDTO<>(HttpStatus.OK, data));
294298
}
295299

296300
@JsonView(Views.Public.class)

0 commit comments

Comments
 (0)