Merge branch 'main' into nicu1989_uplift_docs_as_code

Signed-off-by: Nicolae Dicu <nicolae.dicu.ext@qorix.ai>

Author: nicu1989

.bazelversion

@@ -1 +1 @@
-8.3.0
+8.3.0

.github/workflows/check.yml

@@ -80,10 +80,13 @@ jobs:
           sudo apt-get update
           sudo apt-get install -y lcov
 
-      - name: Extract Coverage for kvs.cpp
+      - name: Extract Coverage for CPP Files
         run: |
           REPORT=$(find "$(bazel info output_path)" -name _coverage_report.dat | head -n1)
           lcov \
             --rc branch_coverage=1 \
-            --extract "$REPORT" "src/cpp/src/kvs.cpp" \
+            --extract "$REPORT" '*.cpp' -o "${REPORT}.cpp" \
             --output-file kvs_coverage.info
+
+      - name: Bazel Benchmark
+        run: bazel run -c opt //:bm_kvs_cpp

.github/workflows/copyright.yml

@@ -20,5 +20,3 @@ on:
 jobs:
   copyright-check:
     uses: eclipse-score/cicd-workflows/.github/workflows/copyright.yml@main
-    with:
-      bazel-target: "run //:copyright.check"

.github/workflows/docs.yml

@@ -38,5 +38,5 @@ jobs:
       id-token: write
 
     with:
-      bazel run //:docs -- --github_user=${{ github.repository_owner }} --github_repo=${{ github.event.repository.name }}
+      bazel-target: "//:docs -- --github_user=${{ github.repository_owner }} --github_repo=${{ github.event.repository.name }}"
       retention-days: 3

.github/workflows/format.yml

@@ -22,6 +22,4 @@ on:
 
 jobs:
   formatting-check:
-    uses: qorix-group/cicd-workflows/.github/workflows/format.yml@dcalavrezo_local_cache
-    with:
-      bazel-target: "test //:format.check" # optional, this is the default
+    uses: eclipse-score/cicd-workflows/.github/workflows/format.yml@main

BUILD

@@ -10,13 +10,25 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 # *******************************************************************************
+
 load("@score_cr_checker//:cr_checker.bzl", "copyright_checker")
 load("@score_dash_license_checker//:dash.bzl", "dash_license_checker")
+load("@score_docs_as_code//:docs.bzl", "docs")
 load("@score_format_checker//:macros.bzl", "use_format_targets")
 load("@score_starpls_lsp//:starpls.bzl", "setup_starpls")
 load("//:project_config.bzl", "PROJECT_CONFIG")
 load("@score_docs_as_code//:docs.bzl", "docs")
 
+# Creates all documentation targets:
+# - `:docs` for building documentation at build-time
+docs(
+    data = [
+        "@score_platform//:needs_json",
+        "@score_process//:needs_json",
+    ],
+    source_dir = "docs",
+)
+
 setup_starpls(
     name = "starpls_server",
     visibility = ["//visibility:public"],
@@ -46,7 +58,7 @@ use_format_targets()
 
 alias(
     name = "kvs_cpp",
-    actual = "//src/cpp:kvs_cpp",
+    actual = "//src/cpp/src:kvs_cpp",
     visibility = ["//visibility:public"],
 )
 

MODULE.bazel

@@ -28,14 +28,14 @@ python.toolchain(
 use_repo(python)
 
 # Add GoogleTest dependency
-bazel_dep(name = "googletest", version = "1.14.0")
+bazel_dep(name = "googletest", version = "1.15.0")
 bazel_dep(name = "google_benchmark", version = "1.9.4")
 
 # Rust rules for Bazel
 bazel_dep(name = "rules_rust", version = "0.56.0")
 
 # Checker rule for CopyRight checks/fixs
-bazel_dep(name = "score_cr_checker", version = "0.2.2")
+bazel_dep(name = "score_cr_checker", version = "0.3.1")
 
 # C/C++ rules for Bazel
 bazel_dep(name = "rules_cc", version = "0.1.1")
@@ -59,13 +59,19 @@ bazel_dep(name = "score_dash_license_checker", version = "0.1.2")
 
 # Format checker
 bazel_dep(name = "score_format_checker", version = "0.1.1")
-bazel_dep(name = "aspect_rules_lint", version = "1.0.3")
-bazel_dep(name = "buildifier_prebuilt", version = "7.3.1")
+bazel_dep(name = "aspect_rules_lint", version = "1.4.4")
+bazel_dep(name = "buildifier_prebuilt", version = "8.2.0.2")
 
 #docs-as-code
 bazel_dep(name = "score_docs_as_code", version = "1.0.1")
-bazel_dep(name = "score_process", version = "1.1.0")
+git_override(
+    module_name = "score_docs_as_code",
+    commit = "13ba715a95cfe85158b60d7f4748ba8e28895d8c",
+    remote = "https://github.com/eclipse-score/docs-as-code.git",
+)
+
 bazel_dep(name = "score_platform", version = "0.3.0")
+bazel_dep(name = "score_process", version = "1.1.0")
 
 # Module deps
 
@@ -97,6 +103,6 @@ archive_override(
 bazel_dep(name = "score-baselibs", version = "0.0.0")
 git_override(
     module_name = "score-baselibs",
-    commit = "ae349b99cafc1e79d98c0391a851fc5664c04ebc",
+    commit = "46923f5c4f302bd9feae0261588687aaf32e3c5c",
     remote = "https://github.com/eclipse-score/baselibs.git",
 )

docs/conf.py

@@ -1,5 +1,5 @@
 # *******************************************************************************
-# Copyright (c) 2024 Contributors to the Eclipse Foundation
+# Copyright (c) 2025 Contributors to the Eclipse Foundation
 #
 # See the NOTICE file(s) distributed with this work for additional
 # information regarding copyright ownership.
@@ -19,11 +19,12 @@
 
 # -- Project information -----------------------------------------------------
 # https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information
-project = "Persistency"
-project_url = "https://eclipse-score.github.io/inc_mw_per"
-project_prefix = "PERSISTENCY_"
-version = "0.1.0"
+
+project = "Score Persistency"
+project_url = "https://eclipse-score.github.io/inc_mw_per/"
+project_prefix = "PER_"
 author = "S-CORE"
+version = "0.1.0"
 
 # -- General configuration ---------------------------------------------------
 # https://www.sphinx-doc.org/en/master/usage/configuration.html#general-configuration
@@ -32,6 +33,7 @@
 extensions = [
     "sphinx_design",
     "sphinx_needs",
+    "myst_parser",
     "sphinxcontrib.plantuml",
     "score_plantuml",
     "score_metamodel",
@@ -40,16 +42,26 @@
     "score_layout",
 ]
 
+myst_enable_extensions = ["colon_fence"]
+
 exclude_patterns = [
     # The following entries are not required when building the documentation via 'bazel
-    # build //docs:docs', as that command runs in a sandboxed environment. However, when
-    # building the documentation via 'bazel run //docs:incremental' or esbonio, these
+    # build //:docs', as that command runs in a sandboxed environment. However, when
+    # building the documentation via 'bazel run //:docs' or esbonio, these
     # entries are required to prevent the build from failing.
     "bazel-*",
-    ".venv_docs",
+    ".venv*",
 ]
 
+# Enable markdown rendering
+source_suffix = {
+    ".rst": "restructuredtext",
+    ".md": "markdown",
+}
+
 templates_path = ["templates"]
 
+
 # Enable numref
 numfig = True
+# needs_builder_filter = ""

src/cpp/BUILD src/cpp/src/BUILDsrc/cpp/BUILD renamed to src/cpp/src/BUILD

@@ -13,16 +13,37 @@
 
 # The filegroup is used to collect all source files for the tests.
 
+cc_library(
+    name = "kvsvalue",
+    srcs = [
+        "kvsvalue.cpp",
+    ],
+    hdrs = ["kvsvalue.hpp"],
+    includes = ["."],
+    visibility = [
+        "//:__pkg__",
+        "//src/cpp/src/internal:__pkg__",
+    ],
+)
+
 cc_library(
     name = "kvs_cpp",
     srcs = [
-        "inc/internal/kvs_helper.hpp",
-        "src/kvs.cpp",
+        "kvs.cpp",
+        "kvsbuilder.cpp",
+    ],
+    hdrs = [
+        "kvs.hpp",
+        "kvsbuilder.hpp",
+    ],
+    implementation_deps = [
+        "//src/cpp/src/internal:kvs_helper",
     ],
-    hdrs = ["inc/kvs.hpp"],
-    includes = ["inc"],
+    includes = ["."],
     visibility = ["//:__pkg__"],
     deps = [
+        ":kvsvalue",
+        "//src/cpp/src/internal:error",
         "@score-baselibs//score/filesystem:filesystem",
         "@score-baselibs//score/json",
         "@score-baselibs//score/result:result",

src/cpp/src/internal/BUILD

@@ -0,0 +1,48 @@
+# *******************************************************************************
+# Copyright (c) 2025 Contributors to the Eclipse Foundation
+#
+# See the NOTICE file(s) distributed with this work for additional
+# information regarding copyright ownership.
+#
+# This program and the accompanying materials are made available under the
+# terms of the Apache License Version 2.0 which is available at
+# https://www.apache.org/licenses/LICENSE-2.0
+#
+# SPDX-License-Identifier: Apache-2.0
+# *******************************************************************************
+
+cc_library(
+    name = "error",
+    srcs = [
+        "error.cpp",
+    ],
+    hdrs = [
+        "error.hpp",
+    ],
+    visibility = [
+        "//src/cpp/src:__pkg__",
+        "//src/cpp/tests:__pkg__",
+    ],
+    deps = [
+        "@score-baselibs//score/result:result",
+    ],
+)
+
+cc_library(
+    name = "kvs_helper",
+    srcs = [
+        "kvs_helper.cpp",
+    ],
+    hdrs = [
+        "kvs_helper.hpp",
+    ],
+    visibility = [
+        "//src/cpp/src:__pkg__",
+        "//src/cpp/tests:__pkg__",
+    ],
+    deps = [
+        ":error",
+        "//src/cpp/src:kvsvalue",
+        "@score-baselibs//score/json",
+    ],
+)