From bd57155a400e633fc7f014d3f9901adb305bd9a3 Mon Sep 17 00:00:00 2001 From: ivanmilevtues Date: Fri, 25 Sep 2026 03:54:01 +0200 Subject: [PATCH] fix: align PR reviews with the main analysis configuration Replace the temporary Action PR 120 ref with the same released action used by sync, and keep both workflows at depth 2. The old engine and depth 3 created a different base graph for the same merge-base commit. Co-authored-by: Amp Amp-Thread-ID: https://ampcode.com/threads/T-01a0d63c-1b35-732f-bd08-a0ec57f0bff3 --- .github/workflows/codeboarding-review.yml | 7 ++++--- .github/workflows/codeboarding-sync.yml | 2 ++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/codeboarding-review.yml b/.github/workflows/codeboarding-review.yml index 6088188bf..5335d44fb 100644 --- a/.github/workflows/codeboarding-review.yml +++ b/.github/workflows/codeboarding-review.yml @@ -28,9 +28,10 @@ jobs: runs-on: ubuntu-latest timeout-minutes: 60 steps: - # Exercise https://github.com/CodeBoarding/CodeBoarding-action/pull/120. - - uses: CodeBoarding/CodeBoarding-action@refs/pull/120/head + # Keep the action ref, provider, and depth aligned with codeboarding-sync.yml + # so reviews reuse the graph published by sync instead of rebuilding it. + - uses: CodeBoarding/CodeBoarding-action@v1 with: mode: review llm: hosted - depth_cap: 3 + depth_cap: 2 diff --git a/.github/workflows/codeboarding-sync.yml b/.github/workflows/codeboarding-sync.yml index 8dc4b7a33..2e92d7ebc 100644 --- a/.github/workflows/codeboarding-sync.yml +++ b/.github/workflows/codeboarding-sync.yml @@ -38,9 +38,11 @@ jobs: runs-on: ubuntu-latest timeout-minutes: 60 steps: + # Keep the action ref, provider, and depth aligned with codeboarding-review.yml. - uses: CodeBoarding/CodeBoarding-action@v1 with: mode: sync + depth_cap: 2 force_full: ${{ inputs.force_full || false }} target_branch: 'main' # CodeBoarding's free hosted tier. No secret to add: the run authenticates