From c06da04d3f5791b7fc86bff4d67095e5a343e295 Mon Sep 17 00:00:00 2001
From: Caleb Chase <caleb@chasecaleb.com>
Date: Sat, 30 Oct 2021 22:51:26 -0500
Subject: [PATCH] GitLab: implement create/update/delete methods

For #733
---
 .../gitlab_sync_backend_client.js             | 57 ++++++++++++++++---
 1 file changed, 49 insertions(+), 8 deletions(-)

diff --git a/src/sync_backend_clients/gitlab_sync_backend_client.js b/src/sync_backend_clients/gitlab_sync_backend_client.js
index 1a592719c..556d21cf6 100644
--- a/src/sync_backend_clients/gitlab_sync_backend_client.js
+++ b/src/sync_backend_clients/gitlab_sync_backend_client.js
@@ -201,12 +201,6 @@ export default (oauthClient) => {
   const getMoreDirectoryListing = async (additionalSyncBackendState) =>
     await fetchDirectory(additionalSyncBackendState.get('cursor'));
 
-  // FIXME stub
-  const updateFile = (path, contents) => new Promise((resolve) => resolve());
-
-  // FIXME stub
-  const createFile = (path, contents) => new Promise((resolve) => resolve());
-
   const getRawFile = async (path) => {
     const params = new URLSearchParams({
       ref: await getDefaultBranch(),
@@ -253,10 +247,57 @@ export default (oauthClient) => {
 
   // Parentheses are necessarily to await the actual promise. Yay, foot-guns.
   const getFileContents = async (path) => (await getRawFile(path)).contents;
+
+  const doCommit = async (action) => {
+    const capitalizedAction = action.action.charAt(0).toUpperCase() + action.action.slice(1);
+    const message =
+      `[organice] ${capitalizedAction} ${action.file_path}\n` +
+      'Automatic commit from organice app.';
+    // It's also possible to modify files using the files API instead of commits API. For this use
+    // case they're about equal, but I picked commits because it doesn't require non-standard
+    // encoding for file paths, whereas the files API requires URI encoding plus converting period
+    // characters to %2E. Also, the commits API is more flexible in case of future changes, since it
+    // allows modifying multiple files at a time.
+    //
+    // https://docs.gitlab.com/ee/api/commits.html#create-a-commit-with-multiple-files-and-actions
+    // https://docs.gitlab.com/ee/api/repository_files.html
+    await decoratedFetch(`${getProjectApi()}/repository/commits`, {
+      method: 'POST',
+      headers: {
+        'Content-Type': 'application/json',
+      },
+      body: JSON.stringify({
+        branch: await getDefaultBranch(),
+        commit_message: message,
+        // Organice only modifies a single file at a time, so only one action.
+        actions: [action],
+        stats: false,
+      }),
+    });
   };
 
-  // FIXME stub
-  const deleteFile = (path) => new Promise((resolve) => resolve());
+  const createFile = async (path, content) => {
+    await doCommit({
+      action: 'create',
+      file_path: path.replace(/^\//, ''),
+      content,
+    });
+  };
+
+  const updateFile = async (path, content) => {
+    await doCommit({
+      action: 'update',
+      file_path: path.replace(/^\//, ''),
+      content,
+    });
+  };
+
+  const deleteFile = async (path) => {
+    await doCommit({
+      action: 'delete',
+      file_path: path.replace(/^\//, ''),
+    });
+  };
 
   return {
     type: 'GitLab',