Add size reports

Author: Marcos Boyington

.github/actions/setup-size-reports/action.yaml

@@ -0,0 +1,15 @@
+name: Setup size reports
+description: Setup size reports
+inputs:
+  gh-context:
+    description: "GH Context"
+    required: true
+
+runs:
+  using: "composite"
+  steps:
+  - name: Set up environment for size reports
+    shell: bash
+    env:
+      GH_CONTEXT: ${{ inputs.gh-context }}
+    run: matter_cpp/repo/scripts/tools/memory/gh_sizes_environment.py "${GH_CONTEXT}"

.github/actions/upload-size-reports/action.yaml

@@ -0,0 +1,17 @@
+name: upload-size-reports
+description: upload-size-reports
+inputs:
+  platform-name:
+    description: "Platform name Name"
+    required: true
+
+runs:
+  using: "composite"
+  steps:
+  - name: Uploading Size Reports
+    uses: actions/upload-artifact@v4
+    if: ${{ !env.ACT }}
+    with:
+      name: Size,${{ inputs.platform-name }}-Examples,${{ env.GH_EVENT_PR }},${{ env.GH_EVENT_HASH }},${{ env.GH_EVENT_PARENT }},${{ github.event_name }}
+      path: |
+        /tmp/bloat_reports/

.github/workflows/ci.yml

@@ -62,13 +62,34 @@ jobs:
       - name: Benchmark
         run: cd rs-matter; cargo bench --no-default-features --features ${{matrix.crypto-backend}},${{matrix.features}},log
 
+      - name: Set up environment for size reports
+        uses: ./.github/actions/setup-size-reports
+        if: ${{ !env.ACT }}
+        with:
+          gh-context: ${{ toJson(github) }}
+
       - name: Test
         if: matrix.features == 'os'
         run: cd rs-matter; cargo test --no-default-features --features ${{matrix.crypto-backend}},${{matrix.features}},log
 
       - name: Examples
         if: matrix.features == 'os' && matrix.crypto-backend == 'rustcrypto'
-        run: cargo build --examples --no-default-features --features ${{matrix.crypto-backend}},${{matrix.features}},log
+        run: cargo build --examples --release --no-default-features --features ${{matrix.crypto-backend}},${{matrix.features}},log
+
+      - name: Prepare bloat report from the previous builds
+        run: |
+            .environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
+              linux rs-matter onoff-light-bt \
+              target/release/onoff_light_bt \
+              /tmp/bloat_reports/
+            .environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
+              linux rs-matter dimmable_light \
+              target/release/dimmable_light \
+              /tmp/bloat_reports/
+            .environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
+              linux rs-matter speaker \
+              target/release/speaker \
+              /tmp/bloat_reports/
 
       - name: Xtask Fmt
         if: matrix.features == 'os' && matrix.crypto-backend == 'rustcrypto'
@@ -77,3 +98,9 @@ jobs:
       - name: Xtask Clippy
         if: matrix.features == 'os' && matrix.crypto-backend == 'rustcrypto'
         run: cd xtask; cargo clippy --no-deps -- -Dwarnings
+
+      - name: Uploading Size Reports
+          uses: ./.github/actions/upload-size-reports
+          if: ${{ !env.ACT }}
+          with:
+            platform-name: linux

.gitmodules

@@ -0,0 +1,4 @@
+[submodule "matter_cpp/repo"]
+	path = matter_cpp/repo
+	url = https://github.com/project-chip/connectedhomeip.git
+	branch = v1.3-branch

matter_cpp/repo

@@ -9,3 +9,8 @@ From the rs-matter root directory:
 ```bash
 cargo xtask --help
 ```
+
+To get necessary environment configs (such as `gn`), you may also need to run:
+```base
+source matter_cpp/repo/scripts/bootstrap.sh
+```

xtask/README.md

@@ -43,10 +43,8 @@ const DEFAULT_TESTS: &[&str] = &[
     "TestCommissioningWindow",
 ];
 
-/// The default Git reference to use for the Chip repository
-pub const CHIP_DEFAULT_GITREF: &str = "v1.3.0.0"; //"master";
 /// The directory where the Chip repository will be cloned
-const CHIP_DIR: &str = ".build/itest/connectedhomeip";
+const CHIP_DIR: &str = "matter_cpp/repo";
 
 /// The tooling that is checked for presence in the command line
 const REQUIRED_TOOLING: &[&str] = &[
@@ -138,8 +136,8 @@ impl ITests {
     /// - Clone the Chip repo if it doesn't exist, or updates it if it does
     /// - Activate the Chip environment
     /// - Build `chip-tool`
-    pub fn setup(&self, chip_gitref: Option<&str>, force_rebuild: bool) -> anyhow::Result<()> {
-        self.setup_chip_tool(chip_gitref, force_rebuild)
+    pub fn setup(&self, force_rebuild: bool) -> anyhow::Result<()> {
+        self.setup_chip_tool(force_rebuild)
     }
 
     /// Build the executable (`chip-tool-tests`) that is to be tested with the Chip integration tests.
@@ -166,64 +164,25 @@ impl ITests {
 
     fn setup_chip_tool(
         &self,
-        chip_gitref: Option<&str>,
         force_rebuild: bool,
     ) -> anyhow::Result<()> {
         warn!("Setting up Chip environment...");
 
         let chip_dir = self.workspace_dir.join(CHIP_DIR);
-        let chip_gitref = chip_gitref.unwrap_or(CHIP_DEFAULT_GITREF);
 
         // Check system dependencies
         self.check_tooling()?;
 
-        // Clone or update Chip repository
-        if !chip_dir.exists() {
-            info!("Cloning Chip repository...");
-
-            // Ensure parent directories exist
-            if let Some(parent) = chip_dir.parent() {
-                fs::create_dir_all(parent)
-                    .context("Failed to create parent directories for Chip")?;
-            }
-
-            let mut cmd = Command::new("git");
-
-            cmd.arg("clone")
-                .arg("https://github.com/project-chip/connectedhomeip.git")
-                .arg(&chip_dir);
-
-            if !self.print_cmd_output {
-                cmd.arg("--quiet");
-            }
-
-            self.run_command(&mut cmd)?;
-        } else {
-            info!("Chip repository already exists");
-
-            if force_rebuild {
-                info!("Force rebuild requested, cleaning build artifacts...");
+        if force_rebuild {
+            info!("Force rebuild requested, cleaning build artifacts...");
 
-                let out_dir = chip_dir.join("out");
-                if out_dir.exists() {
-                    fs::remove_dir_all(&out_dir)
-                        .context("Failed to remove existing out directory")?;
-                }
+            let out_dir = chip_dir.join("out");
+            if out_dir.exists() {
+                fs::remove_dir_all(&out_dir)
+                    .context("Failed to remove existing out directory")?;
             }
         }
 
-        // Checkout the specified reference
-        info!("Checking out Chip GIT reference: {chip_gitref}...");
-
-        let mut cmd = Command::new("git");
-
-        cmd.current_dir(&chip_dir).arg("checkout").arg(chip_gitref);
-
-        if !self.print_cmd_output {
-            cmd.arg("--quiet");
-        }
-
-        self.run_command(&mut cmd)?;
 
         // Detect host platform for selective submodule initialization
         let platform = self.host_platform()?;

xtask/src/itest.rs

@@ -95,7 +95,7 @@ impl Command {
                 ITests::new(workspace_dir(), print_cmd_output).print_packages()
             }
             Command::ItestSetup(args) => ITests::new(workspace_dir(), print_cmd_output)
-                .setup(Some(&args.gitref), args.force_setup),
+                .setup(args.force_setup),
             Command::ItestExe(args) => ITests::new(workspace_dir(), print_cmd_output).build(
                 &args.profile,
                 &args.target,
@@ -171,9 +171,6 @@ impl Verbosity {
 /// Arguments for the `itest-setup` command
 #[derive(Parser, Debug, Clone)]
 struct ItestSetupArgs {
-    /// Chip repository reference (branch/tag/commit)
-    #[arg(long, default_value = itest::CHIP_DEFAULT_GITREF)]
-    gitref: String,
     /// Force setup even if cached
     #[arg(long)]
     force_setup: bool,