just serve: Fix inconsistencies

- add missing serve_api TARGETS file
- rename service client to align with server naming scheme
- fix inconsistencies in comments between implementation and protocol

1 files changed, 46 insertions, 0 deletions

diff --git a/src/buildtool/serve_api/remote/source_tree_client.hpp b/src/buildtool/serve_api/remote/source_tree_client.hpp
new file mode 100644
index 00000000..1b557163
--- /dev/null
+++ b/src/buildtool/serve_api/remote/source_tree_client.hpp
@@ -0,0 +1,46 @@
+// Copyright 2023 Huawei Cloud Computing Technology Co., Ltd.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#ifndef INCLUDED_SRC_BUILDTOOL_SERVE_API_SOURCE_TREE_CLIENT_HPP
+#define INCLUDED_SRC_BUILDTOOL_SERVE_API_SOURCE_TREE_CLIENT_HPP
+
+#include <memory>
+#include <string>
+
+#include "justbuild/just_serve/just_serve.grpc.pb.h"
+#include "src/buildtool/common/remote/port.hpp"
+#include "src/buildtool/logging/logger.hpp"
+
+/// Implements client side for SourceTree service defined in:
+/// src/buildtool/serve_api/serve_service/just_serve.proto
+class SourceTreeClient {
+  public:
+    SourceTreeClient(std::string const& server, Port port) noexcept;
+
+    /// \brief Retrieve the Git tree of a given commit, if known by the remote.
+    /// \param[in] commit_id Hash of the Git commit to look up.
+    /// \param[in] subdir Relative path of the tree inside commit.
+    /// \param[in] sync_tree Sync tree to the remote-execution end point
+    /// \returns The hash of the tree if commit found, nullopt otherwise.
+    [[nodiscard]] auto ServeCommitTree(std::string const& commit_id,
+                                       std::string const& subdir,
+                                       bool sync_tree)
+        -> std::optional<std::string>;
+
+  private:
+    std::unique_ptr<justbuild::just_serve::SourceTree::Stub> stub_;
+    Logger logger_{"RemoteSourceTreeClient"};
+};
+
+#endif  // INCLUDED_SRC_BUILDTOOL_SERVE_API_SOURCE_TREE_CLIENT_HPP