ci: add integration tests by austinvazquez · Pull Request #74 · containerd/nerdbox

austinvazquez · 2025-12-12T02:37:12Z

No description provided.

Copilot

Pull request overview

This PR adds a new integration test job to the CI workflow to improve test coverage by running integration tests in the build pipeline.

Key changes:

Added a new integration job to the GitHub Actions workflow
Configured the job to build the project and run integration tests using Go

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 8 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/ci.yml

.github/actions/build-kernel/action.yml

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

docker-bake.hcl

.github/workflows/ci.yml

Copilot

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/ci.yml

.github/actions/build-kernel/action.yml

.github/workflows/ci.yml

Copilot

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/ci.yml

Makefile

Copilot

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

austinvazquez · 2025-12-16T22:20:12Z

Downgrade to 6.12.44 (from #74)

Copilot

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-16T22:30:48Z

Makefile

+OS ?= $(shell uname -s | tr '[:upper:]' '[:lower:]')
+ARCH ?= $(shell uname -m)
+export PLATFORM ?= $(OS)/$(ARCH)


The variable name OS may conflict with environment variables or built-in variables in some shells. Consider renaming it to HOST_OS for clarity and consistency with the HOST_OS variable used in the docker-bake.hcl file.

Suggested change

OS ?= $(shell uname -s | tr '[:upper:]' '[:lower:]')

ARCH ?= $(shell uname -m)

export PLATFORM ?= $(OS)/$(ARCH)

HOST_OS ?= $(shell uname -s | tr '[:upper:]' '[:lower:]')

ARCH ?= $(shell uname -m)

export PLATFORM ?= $(HOST_OS)/$(ARCH)

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

Copilot

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copilot · 2026-03-11T21:06:01Z

Makefile

+build:
+	@echo "$(WHALE) $@"
+	HOST_OS=$(OS) ARCH=$(ARCH) $(BUILDX) bake


The build target exports ARCH=$(ARCH) from uname -m (e.g., x86_64, aarch64) into docker-bake.hcl, but the bake file’s ARCH variable is used as a Docker/Go arch (amd64/arm64). This mismatch makes kernelArch() return unknown (and may set GOARCH to an invalid value), breaking builds. Normalize ARCH before passing it (e.g., map x86_64->amd64, aarch64/arm64->arm64), or pass a separate variable for kernel arch vs Go/Docker arch.

Copilot · 2026-03-11T21:06:02Z

docker-bake.hcl

+
 target "_common" {
  args = {
+    TARGETARCH = ARCH


_common.args forcibly sets TARGETARCH = ARCH. Buildx already provides TARGETARCH based on the build platform; overriding it with a custom variable (defaulting to amd64) will produce incorrect artifacts on non-amd64 runners unless every workflow explicitly sets ARCH. If you need a separate knob for kernel naming, consider not overriding TARGETARCH and instead pass only KERNEL_ARCH, or ensure ARCH is always set to amd64/arm64 in all callers (including CI).

Suggested change

TARGETARCH = ARCH

Copilot · 2026-03-11T21:06:02Z

.github/workflows/ci.yml

+
+      - name: Build remaining artifacts (initrd and shim)
+        run: |
+          echo "Building host and guest binaries:"


This step runs docker buildx bake host-binaries guest-binaries without setting ARCH (or HOST_OS). Because docker-bake.hcl defaults ARCH to amd64 and also overrides TARGETARCH, the arm64 matrix job will build amd64 guest artifacts (notably nerdbox-initrd), causing VM startup/tests to fail. Export ARCH/HOST_OS for this step (or stop overriding TARGETARCH in the bake file so Buildx can set it correctly).

Suggested change

echo "Building host and guest binaries:"

echo "Building host and guest binaries:"

if [ "${{ matrix.arch }}" = "x86_64" ]; then

export ARCH=amd64

else

export ARCH=arm64

fi

export HOST_OS=linux

echo "Using ARCH=${ARCH}, HOST_OS=${HOST_OS}"

.github/actions/setup-containerd/action.yml

Copilot · 2026-03-11T21:06:03Z

.github/actions/setup-containerd/action.yml

+    - name: Start containerd
+      shell: bash
+      run: |
+        echo "Starting containerd..."
+
+        # Start containerd in background with logging
+        sudo nohup containerd > /var/log/containerd/containerd.log 2>&1 &
+        CONTAINERD_PID=$!
+
+        echo "Containerd started with PID: $CONTAINERD_PID"
+
+        # Save PID for cleanup
+        echo "$CONTAINERD_PID" | sudo tee /var/run/containerd.pid
+
+        # Wait for containerd to be ready
+        echo "Waiting for containerd to be ready..."
+        for i in {1..30}; do
+          if sudo ctr version >/dev/null 2>&1; then
+            echo "Containerd is ready!"
+            break
+          fi
+          if [ $i -eq 30 ]; then
+            echo "Error: Containerd failed to start within 30 seconds"
+            echo "Containerd logs:"
+            sudo cat /var/log/containerd/containerd.log
+            exit 1
+          fi
+          echo "Waiting... ($i/30)"
+          sleep 1
+        done


Starting containerd with sudo nohup containerd ... & is likely to conflict with any existing containerd already running on the runner (default socket /run/containerd/containerd.sock), and the health check (ctr version) will succeed even if it’s talking to the pre-existing daemon. To make the installed version deterministic, start containerd on an isolated socket/address (and point ctr to it) or stop/disable the preinstalled service before starting the downloaded binary.

Copilot · 2026-03-11T21:06:03Z

.github/actions/setup-containerd/action.yml

+        # Construct download URL
+        TARBALL="containerd-${VERSION}-linux-${ARCH}.tar.gz"
+        URL="https://github.com/containerd/containerd/releases/download/v${VERSION}/${TARBALL}"
+
+        # Download containerd
+        curl -fsSL -o "/tmp/${TARBALL}" "${URL}"
+
+        echo "Downloaded containerd tarball to /tmp/${TARBALL}"
+        ls -lh "/tmp/${TARBALL}"


The downloaded containerd tarball is installed without any integrity verification (checksum/signature). For supply-chain safety, verify the archive against published checksums (or use a provenance-verified source) before extracting as root.

Copilot · 2026-03-11T21:06:03Z

Makefile

+# Detect local platform; 'linux' or 'darwin'
+OS ?= $(shell uname -s | tr '[:upper:]' '[:lower:]')
+ARCH ?= $(shell uname -m)
+export PLATFORM ?= $(OS)/$(ARCH)
+
 LDFLAGS_x86_64_Linux = -lkrun
 LDFLAGS_aarch64_Linux = -lkrun
 LDFLAGS_arm64_Darwin = -L/opt/homebrew/lib -lkrun


OS is now normalized to lowercase ("linux"/"darwin"), but later logic still relies on title-cased OS keys (e.g., LDFLAGS_*_Linux/LDFLAGS_*_Darwin and ifeq ($(OS),Darwin)). With the new OS value, $(LDFLAGS_$(ARCH)_$(OS)) will not resolve and the Darwin codesign blocks will never run. Consider either keeping OS in the original casing, or updating the LDFLAGS variable names and all OS comparisons to match the lowercase convention.

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

Copilot

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copilot · 2026-03-12T00:43:04Z

docker-bake.hcl


+function "kernelArch" {
+  params = []
+  result = ARCH == "amd64" ? "x86_64" : (ARCH == "arm64" ? "arm64" : "unknown")


kernelArch() only handles ARCH values amd64 and arm64. If ARCH is set to common host values like x86_64 or aarch64, it returns "unknown", which will later make the kernel build fail (e.g., missing kernel/config-<version>-unknown). Consider either accepting x86_64/aarch64 here (mapping aarch64 -> arm64) or returning ARCH by default instead of "unknown" so local builds don’t break when ARCH comes from uname -m.

Suggested change

result = ARCH == "amd64" ? "x86_64" : (ARCH == "arm64" ? "arm64" : "unknown")

result = ARCH == "amd64" || ARCH == "x86_64" ? "x86_64" : (ARCH == "arm64" || ARCH == "aarch64" ? "arm64" : ARCH)

Copilot · 2026-03-12T00:43:04Z

.github/actions/setup-containerd/action.yml

+        # Start containerd in background with logging
+        sudo nohup containerd > /var/log/containerd/containerd.log 2>&1 &
+        CONTAINERD_PID=$!
+
+        echo "Containerd started with PID: $CONTAINERD_PID"
+
+        # Save PID for cleanup
+        echo "$CONTAINERD_PID" | sudo tee /var/run/containerd.pid
+


CONTAINERD_PID=$! here captures the PID of the backgrounded sudo process, not the actual containerd daemon PID. This makes /var/run/containerd.pid misleading and can break any cleanup logic that relies on it. Consider starting containerd via sudo sh -c and writing the real child PID, or drop PID-file creation and rely on pgrep/pkill consistently.

Suggested change

# Start containerd in background with logging

sudo nohup containerd > /var/log/containerd/containerd.log 2>&1 &

CONTAINERD_PID=$!

echo "Containerd started with PID: $CONTAINERD_PID"

# Save PID for cleanup

echo "$CONTAINERD_PID" | sudo tee /var/run/containerd.pid

# Start containerd in background with logging and save its PID

sudo sh -c 'nohup containerd > /var/log/containerd/containerd.log 2>&1 & echo $! > /var/run/containerd.pid'

CONTAINERD_PID=$(sudo cat /var/run/containerd.pid)

echo "Containerd started with PID: $CONTAINERD_PID"

# PID is saved for cleanup in /var/run/containerd.pid

Copilot · 2026-03-12T00:43:04Z

.github/actions/build-kernel/action.yml

+    - name: Checkout code
+      uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6.0.2
+


This composite action performs its own actions/checkout, but both callers in this PR (ci.yml and build-kernel.yml) already check out the repo before invoking it. That duplicates work and can also override any checkout options the caller intended (sparse checkout, submodules, custom path, etc.). Prefer removing checkout from the action and requiring callers to checkout, or make it explicitly optional via an input/flag.

Suggested change

- name: Checkout code

uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2

Copilot · 2026-03-12T00:43:04Z

.github/workflows/ci.yml

+        with:
+          containerd_version: ${{ needs.setup.outputs.containerd-version }}
+          architecture: ${{ matrix.arch == 'x86_64' && 'amd64' || 'arm64' }}
+


The integration workflow starts a containerd daemon via ./.github/actions/setup-containerd, but the current go test ./integration/... suite doesn’t appear to talk to containerd (it exercises VM + ttrpc APIs directly). If containerd isn’t required, consider removing this setup to reduce CI time/complexity; if it is required, consider adding/adjusting integration tests so the dependency is validated (otherwise failures here may be confusing and unrelated to the tests).

Suggested change

- name: Validate containerd is running

run: |

echo "Validating containerd is running and responsive..."

# Check containerd version via ctr; this will fail clearly if containerd is not healthy

sudo ctr version

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

Copilot

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copilot · 2026-03-12T03:02:42Z

.github/workflows/ci.yml

+          kernel_version=$(grep -E '^kernel [0-9.]+$' .github/.tool-versions | sed -E 's/^kernel ([0-9.]+)$/\1/')
+          echo "kernel-version=${kernel_version}" >> $GITHUB_OUTPUT
+          containerd_version=$(grep -E '^containerd [0-9.]+$' .github/.tool-versions | sed -E 's/^containerd ([0-9.]+)$/\1/')
+          echo "containerd-version=${containerd_version}" >> $GITHUB_OUTPUT


This step will silently set empty outputs if the grep doesn’t match (because there’s no set -e/validation). That would cause downstream jobs to use empty version strings. Consider enabling set -euo pipefail and explicitly failing when kernel_version or containerd_version is empty.

Suggested change

kernel_version=$(grep -E '^kernel [0-9.]+$' .github/.tool-versions | sed -E 's/^kernel ([0-9.]+)$/\1/')

echo "kernel-version=${kernel_version}" >> $GITHUB_OUTPUT

containerd_version=$(grep -E '^containerd [0-9.]+$' .github/.tool-versions | sed -E 's/^containerd ([0-9.]+)$/\1/')

echo "containerd-version=${containerd_version}" >> $GITHUB_OUTPUT

set -euo pipefail

kernel_version="$(awk '/^kernel [0-9.]+$/ {print $2}' .github/.tool-versions)"

if [ -z "${kernel_version}" ]; then

echo "Failed to determine kernel version from .github/.tool-versions" >&2

exit 1

fi

echo "kernel-version=${kernel_version}" >> "${GITHUB_OUTPUT}"

containerd_version="$(awk '/^containerd [0-9.]+$/ {print $2}' .github/.tool-versions)"

if [ -z "${containerd_version}" ]; then

echo "Failed to determine containerd version from .github/.tool-versions" >&2

exit 1

fi

echo "containerd-version=${containerd_version}" >> "${GITHUB_OUTPUT}"

Copilot · 2026-03-12T03:02:42Z

.github/workflows/ci.yml

+            arch: arm64
+
+    steps:
+      - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8  # v6.0.1


build-kernels checks out the repo, but the referenced ./.github/actions/build-kernel composite action also performs a checkout. This duplicates work and can be confusing if different checkout options are needed. Consider removing one of the checkouts (either rely on the workflow checkout or have the action assume the workspace is already checked out).

Suggested change

- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1

Copilot · 2026-03-12T03:02:42Z

.github/workflows/build-kernel.yml

+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6.0.2
+


This workflow checks out the repo, but the ./.github/actions/build-kernel composite action also checks out the repo, resulting in a redundant checkout. Consider keeping checkout in one place (workflow or action) to reduce runtime and avoid inconsistent checkout behavior.

Suggested change

- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2

Copilot · 2026-03-12T03:02:42Z

.github/.tool-versions

@@ -0,0 +1,3 @@
+golang 1.25.8


The Go version in .github/.tool-versions (used by CI via go-version-file) differs from the Go version pinned in the Dockerfile (GO_VERSION). This can lead to CI/test results not matching Docker-based builds. Consider aligning these versions (or documenting why they intentionally differ).

Copilot AI review requested due to automatic review settings December 12, 2025 02:37

Copilot AI reviewed Dec 12, 2025

View reviewed changes

austinvazquez force-pushed the add-integration-test-to-ci branch 2 times, most recently from 48a516c to 0e0ef9f Compare December 15, 2025 14:32

Copilot AI review requested due to automatic review settings December 15, 2025 14:32

Copilot AI reviewed Dec 15, 2025

View reviewed changes

austinvazquez force-pushed the add-integration-test-to-ci branch from 0e0ef9f to ea4838d Compare December 15, 2025 14:35

Copilot AI review requested due to automatic review settings December 15, 2025 14:37

austinvazquez force-pushed the add-integration-test-to-ci branch from ea4838d to 84890ed Compare December 15, 2025 14:37

Copilot AI reviewed Dec 15, 2025

View reviewed changes

austinvazquez force-pushed the add-integration-test-to-ci branch from 84890ed to a7e64a6 Compare December 15, 2025 17:01

Copilot AI review requested due to automatic review settings December 15, 2025 23:22

austinvazquez force-pushed the add-integration-test-to-ci branch from a7e64a6 to 31f586d Compare December 15, 2025 23:22

austinvazquez mentioned this pull request Dec 15, 2025

Build developers local platform by default #73

Closed

Copilot AI reviewed Dec 15, 2025

View reviewed changes

docker-bake.hcl Outdated Show resolved Hide resolved

.github/workflows/ci.yml Outdated Show resolved Hide resolved

austinvazquez force-pushed the add-integration-test-to-ci branch from 31f586d to 8a9ee83 Compare December 15, 2025 23:24

Copilot AI review requested due to automatic review settings December 16, 2025 14:40

austinvazquez force-pushed the add-integration-test-to-ci branch from 8a9ee83 to 1af3e23 Compare December 16, 2025 14:40

Copilot AI reviewed Dec 16, 2025

View reviewed changes

austinvazquez force-pushed the add-integration-test-to-ci branch from 1af3e23 to f6a5b11 Compare December 16, 2025 14:56

Copilot AI review requested due to automatic review settings December 16, 2025 17:51

austinvazquez force-pushed the add-integration-test-to-ci branch from f6a5b11 to 8a835c3 Compare December 16, 2025 17:51

Copilot AI reviewed Dec 16, 2025

View reviewed changes

.github/workflows/ci.yml Outdated Show resolved Hide resolved

Makefile Show resolved Hide resolved

austinvazquez force-pushed the add-integration-test-to-ci branch from 8a835c3 to 991c714 Compare December 16, 2025 18:16

Copilot AI review requested due to automatic review settings December 16, 2025 18:22

austinvazquez force-pushed the add-integration-test-to-ci branch from 991c714 to a041e1f Compare December 16, 2025 18:22

Copilot AI reviewed Dec 16, 2025

View reviewed changes

austinvazquez force-pushed the add-integration-test-to-ci branch from a041e1f to 9e58257 Compare December 16, 2025 18:31

Copilot AI review requested due to automatic review settings December 16, 2025 22:30

austinvazquez force-pushed the add-integration-test-to-ci branch from 9e58257 to 8a0a1fc Compare December 16, 2025 22:30

Copilot AI reviewed Dec 16, 2025

View reviewed changes

austinvazquez force-pushed the add-integration-test-to-ci branch from 8a0a1fc to 43cb047 Compare March 11, 2026 19:49

austinvazquez added 3 commits March 11, 2026 14:53

ci: add integration tests

79aef6f

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

ci: add build kernel workflow

3b42eac

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

Build developers local platform by default

a8509ab

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

austinvazquez force-pushed the add-integration-test-to-ci branch from 43cb047 to 45a4574 Compare March 11, 2026 21:00

Copilot AI review requested due to automatic review settings March 11, 2026 21:00

Copilot started reviewing on behalf of austinvazquez March 11, 2026 21:00 View session

Copilot AI reviewed Mar 11, 2026

View reviewed changes

austinvazquez added 2 commits March 11, 2026 16:32

ci: setup containerd for integration tests

b2247cc

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

dockerfile: bump containerd v2.2.2

1a0440b

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

austinvazquez force-pushed the add-integration-test-to-ci branch from 45a4574 to 5d56f95 Compare March 12, 2026 00:32

ci: add output to path

9efaabd

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

austinvazquez force-pushed the add-integration-test-to-ci branch from 5d56f95 to 9efaabd Compare March 12, 2026 00:36

Copilot AI review requested due to automatic review settings March 12, 2026 00:36

Copilot started reviewing on behalf of austinvazquez March 12, 2026 00:36 View session

Copilot AI reviewed Mar 12, 2026

View reviewed changes

austinvazquez added 2 commits March 11, 2026 21:39

ci: fix start containerd

4d0efa5

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

ci: fix start containerd containerd#2

a582968

Signed-off-by: Austin Vazquez <austin.vazquez@docker.com>

Copilot AI review requested due to automatic review settings March 12, 2026 02:55

Copilot started reviewing on behalf of austinvazquez March 12, 2026 02:55 View session

Copilot AI reviewed Mar 12, 2026

View reviewed changes

-          echo "Building host and guest binaries:"
+          echo "Building host and guest binaries:"
+          if [ "${{ matrix.arch }}" = "x86_64" ]; then
+            export ARCH=amd64
+          else
+            export ARCH=arm64
+          fi
+          export HOST_OS=linux
+          echo "Using ARCH=${ARCH}, HOST_OS=${HOST_OS}"

	result = ARCH == "amd64" ? "x86_64" : (ARCH == "arm64" ? "arm64" : "unknown")
	result = ARCH == "amd64" \|\| ARCH == "x86_64" ? "x86_64" : (ARCH == "arm64" \|\| ARCH == "aarch64" ? "arm64" : ARCH)

		- name: Checkout code
		uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2

+      - name: Validate containerd is running
+        run: |
+          echo "Validating containerd is running and responsive..."
+          # Check containerd version via ctr; this will fail clearly if containerd is not healthy
+          sudo ctr version

-          kernel_version=$(grep -E '^kernel [0-9.]+$' .github/.tool-versions | sed -E 's/^kernel ([0-9.]+)$/\1/')
-          echo "kernel-version=${kernel_version}" >> $GITHUB_OUTPUT
-          containerd_version=$(grep -E '^containerd [0-9.]+$' .github/.tool-versions | sed -E 's/^containerd ([0-9.]+)$/\1/')
-          echo "containerd-version=${containerd_version}" >> $GITHUB_OUTPUT
+          set -euo pipefail
+          kernel_version="$(awk '/^kernel [0-9.]+$/ {print $2}' .github/.tool-versions)"
+          if [ -z "${kernel_version}" ]; then
+            echo "Failed to determine kernel version from .github/.tool-versions" >&2
+            exit 1
+          fi
+          echo "kernel-version=${kernel_version}" >> "${GITHUB_OUTPUT}"
+          containerd_version="$(awk '/^containerd [0-9.]+$/ {print $2}' .github/.tool-versions)"
+          if [ -z "${containerd_version}" ]; then
+            echo "Failed to determine containerd version from .github/.tool-versions" >&2
+            exit 1
+          fi
+          echo "containerd-version=${containerd_version}" >> "${GITHUB_OUTPUT}"

		- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2

Conversation

austinvazquez commented Dec 12, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

austinvazquez commented Dec 16, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 16, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!