mirror of
https://github.com/openclaw/openclaw.git
synced 2026-06-06 14:01:24 +08:00
8b29ff5f16
Summary: - This PR adds opt-in first-parent merge-head diff-base handling for CI changed-scope, changed-lanes, and OpenGrep PR scans, plus synthetic merge coverage and small lint/type cleanups. - PR surface: Source +6, Tests +204, Config +1, Other +179. Total +390 across 15 files. - Reproducibility: yes. The synthetic merge tests and PR body live-ref proof show the stale payload-base path can include main-only files, and first-parent mode narrows it to PR-owned paths. Automerge notes: - PR branch already contained follow-up commit before automerge: fix(ci): update workflow guard expectations - PR branch already contained follow-up commit before automerge: fix(ci): resolve plugin guardrail lint failures - PR branch already contained follow-up commit before automerge: fix(ci): preserve plugin run context typing - PR branch already contained follow-up commit before automerge: fix(ci): scope PR merge diff checks to first parent Validation: - ClawSweeper review passed for head40235e8c3d. - Required merge gates passed before the squash merge. Prepared head SHA:40235e8c3dReview: https://github.com/openclaw/openclaw/pull/90287#issuecomment-4621155576 Co-authored-by: Mason Huang <masonxhuang@tencent.com> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: hxy91819 Co-authored-by: hxy91819 <8814856+hxy91819@users.noreply.github.com>
102 lines
3.5 KiB
YAML
102 lines
3.5 KiB
YAML
name: OpenGrep — PR Diff
|
|
|
|
# Runs the high-precision OpenGrep rule super-config against only first-party
|
|
# source paths changed by a pull request. Keeping PR scans diff-scoped makes
|
|
# findings attributable to the proposed change instead of surfacing unrelated
|
|
# repository-wide backlog.
|
|
#
|
|
# For a repository-wide scan, use the manual OpenGrep — Full workflow.
|
|
|
|
on:
|
|
pull_request:
|
|
types: [opened, synchronize, reopened, ready_for_review]
|
|
paths:
|
|
- ".github/actions/ensure-base-commit/**"
|
|
- ".github/workflows/opengrep-precise.yml"
|
|
- ".github/workflows/opengrep-precise-full.yml"
|
|
- ".semgrepignore"
|
|
- "apps/**"
|
|
- "extensions/**"
|
|
- "packages/**"
|
|
- "scripts/**"
|
|
- "security/opengrep/**"
|
|
- "src/**"
|
|
|
|
concurrency:
|
|
group: opengrep-pr-diff-${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}
|
|
cancel-in-progress: true
|
|
|
|
env:
|
|
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: "true"
|
|
|
|
permissions:
|
|
contents: read
|
|
security-events: write
|
|
|
|
jobs:
|
|
scan:
|
|
name: Scan changed paths (precise)
|
|
if: ${{ !github.event.pull_request.draft }}
|
|
runs-on: blacksmith-4vcpu-ubuntu-2404
|
|
timeout-minutes: 30
|
|
steps:
|
|
- name: Checkout
|
|
uses: actions/checkout@v6
|
|
with:
|
|
ref: ${{ github.sha }}
|
|
fetch-depth: 2
|
|
fetch-tags: false
|
|
persist-credentials: false
|
|
submodules: false
|
|
|
|
- name: Ensure PR base commit
|
|
uses: ./.github/actions/ensure-base-commit
|
|
with:
|
|
base-sha: ${{ github.event.pull_request.base.sha }}
|
|
fetch-ref: ${{ github.event.pull_request.base.ref }}
|
|
|
|
- name: Install opengrep
|
|
env:
|
|
# Pin both the install script (by commit SHA) and the binary version.
|
|
# The script SHA must match the v1.22.0 release tag in opengrep/opengrep
|
|
# so a compromised or force-pushed `main` cannot RCE in our CI runner.
|
|
# Bump both together when upgrading.
|
|
OPENGREP_VERSION: v1.22.0
|
|
OPENGREP_INSTALL_SHA: f458d7f0d52cc58eae1ca3cf3d5caf101e637519
|
|
run: |
|
|
curl -fsSL "https://raw.githubusercontent.com/opengrep/opengrep/${OPENGREP_INSTALL_SHA}/install.sh" \
|
|
| bash -s -- -v "$OPENGREP_VERSION"
|
|
echo "$HOME/.opengrep/cli/latest" >> "$GITHUB_PATH"
|
|
|
|
- name: Verify opengrep
|
|
run: opengrep --version
|
|
|
|
- name: Run opengrep on PR diff
|
|
env:
|
|
OPENCLAW_OPENGREP_BASE_REF: ${{ github.event.pull_request.base.sha }}...HEAD
|
|
OPENCLAW_OPENGREP_MERGE_HEAD_FIRST_PARENT: "1"
|
|
# Findings from precise rules block this workflow. Pull requests scan
|
|
# changed first-party source paths only so findings stay attributable to
|
|
# the PR diff. Test/fixture/QA path exclusions live in `.semgrepignore`
|
|
# at the repo root and are picked up automatically.
|
|
run: |
|
|
mkdir -p .opengrep-out
|
|
scripts/run-opengrep.sh --changed --sarif --error
|
|
|
|
- name: Upload SARIF to GitHub Code Scanning
|
|
uses: github/codeql-action/upload-sarif@v4
|
|
# Only upload if the scan actually produced a SARIF file.
|
|
if: always() && hashFiles('.opengrep-out/precise.sarif') != ''
|
|
with:
|
|
sarif_file: .opengrep-out/precise.sarif
|
|
category: opengrep-pr-diff
|
|
|
|
- name: Upload SARIF as workflow artifact
|
|
if: always()
|
|
uses: actions/upload-artifact@v7
|
|
with:
|
|
name: opengrep-pr-diff-sarif
|
|
path: .opengrep-out/precise.sarif
|
|
if-no-files-found: warn
|
|
retention-days: 30
|