Greptile Summary

This PR separates the monolithic CI workflow into distinct pipelines for the core Go binary and the browser extension, adds a new reusable extension-checks.yml called by extension.yml, introduces a dedicated integration.yml, renames build.yml/lint.yml to core-* variants with paths-ignore: extension/**, removes extension zip artifacts from GoReleaser (handled by the new extension release job), and removes several unrelated workflows and tooling scripts.

• core-build.yml's push.branches trigger is missing paths-ignore: extension/** — extension-only merges to main will needlessly spin up the full 3-OS Go test matrix, inconsistent with core-lint.yml which correctly excludes extension changes on both triggers.
• close-stale-prs.yml, flaky-tests.yml, analyze.py, and benchmark.py are silently deleted without explanation, bundling unrelated removals into a workflow-separation PR.

Confidence Score: 5/5

Safe to merge — all remaining findings are P2 style/efficiency suggestions with no blocking correctness issues

The structural workflow changes are sound: reusable extension-checks, release job properly guarded with if: startsWith(ext-v), build correctly gated on checks, and GoReleaser decoupled from extension artifacts. The two flagged items are a minor paths-ignore inconsistency on push-to-main and bundled unrelated deletions — neither blocks correct CI behaviour.

.github/workflows/core-build.yml — missing paths-ignore on the push.branches trigger

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Push / PR] -->|ext-v* tag| B[extension.yml]
    A -->|main branch push or PR\nextension/** changes| B
    A -->|workflow_dispatch| B
    A -->|main branch push or PR\nnon-extension changes| C[core-build.yml]
    A -->|v* tag| C

    B --> D[checks job\ncalls extension-checks.yml]
    D --> E[lint]
    D --> F[type-check]
    D --> G[test]
    E & F & G --> H[build matrix\nchrome / firefox / edge]
    H --> I{startsWith\next-v?}
    I -->|yes| J[release job\nsoftprops/action-gh-release]
    I -->|no| K[artifacts uploaded only]

    C --> L[test matrix\nubuntu / macos / windows]
    L --> M{startsWith\nrefs/tags/v?}
    M -->|yes| N[GoReleaser release]
    M -->|no| O[build check only]

    P[extension/** or .go changes] --> Q[integration.yml\nExtension ↔ Backend tests]

This is a comment left during a code review.
Path: .github/workflows/core-build.yml
Line: 5-11

Comment:
**Missing `paths-ignore` on `push.branches` trigger**

`paths-ignore: extension/**` was added for `pull_request` but not for `push.branches: [main]`. When an extension-only PR is merged to `main`, this workflow will still spin up the full 3-OS Go test matrix unnecessarily. `core-lint.yml` was updated consistently on both triggers — the same fix should apply here.

```suggestion
  push:
    branches:
      - main
    paths-ignore:
      - "extension/**"
    tags:
      - "v*"
      - "!ext-v*"
```

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: .github/workflows/close-stale-prs.yml
Line: 1

Comment:
**Unrelated deletions bundled with workflow-separation changes**

`close-stale-prs.yml`, `flaky-tests.yml`, `analyze.py`, and `benchmark.py` (along with `benchmark.yml`) are all removed in this PR, but none of these are related to separating extension and surge CI pipelines. Mixing functional removals (stale-PR automation, flaky-test detection, diagnostic tooling) with a structural refactor makes the diff harder to review and harder to revert cleanly if any part needs to be rolled back.

**Rule Used:** What: Flag commits that bundle unnecessary changes... ([source](https://app.greptile.com/review/custom-context?memory=74a28159-60fc-4201-a661-331594ebaf90))

How can I resolve this? If you propose a fix, please make it concise.

_{Reviews (6): Last reviewed commit: "chore: update workflow concurrency setti..." | Re-trigger Greptile}

Context used:

• Rule used - What: Flag commits that bundle unnecessary changes... (source)