From a9eac4b3de65348e8425d8d456febaa1a6d01779 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 3 Apr 2026 14:36:56 +0000 Subject: [PATCH 1/2] Add PR CI workflow and bump changeset to minor - Add ci.yml: runs build+test on PRs targeting main/dev (previously tests only ran after merge, allowing broken code through) - Bump changeset from patch to minor: result shape changes are breaking for existing consumers https://claude.ai/code/session_017mqanCkMvA1VU8MVD7hkA1 --- .changeset/fix-maxcount-result-mapping.md | 2 +- .github/workflows/ci.yml | 29 +++++++++++++++++++++++ 2 files changed, 30 insertions(+), 1 deletion(-) create mode 100644 .github/workflows/ci.yml diff --git a/.changeset/fix-maxcount-result-mapping.md b/.changeset/fix-maxcount-result-mapping.md index 6c1bf64..6712426 100644 --- a/.changeset/fix-maxcount-result-mapping.md +++ b/.changeset/fix-maxcount-result-mapping.md @@ -1,5 +1,5 @@ --- -"@_linked/core": patch +"@_linked/core": minor --- Fix maxCount-aware result mapping for single-value and multi-value properties diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml new file mode 100644 index 0000000..28dd24b --- /dev/null +++ b/.github/workflows/ci.yml @@ -0,0 +1,29 @@ +name: CI + +on: + pull_request: + branches: + - main + - dev + +jobs: + build-and-test: + name: Build & Test + runs-on: ubuntu-latest + steps: + - name: Checkout + uses: actions/checkout@v4 + + - name: Setup Node.js + uses: actions/setup-node@v4 + with: + node-version: 22 + + - name: Install dependencies + run: npm ci --legacy-peer-deps + + - name: Build + run: npm run build + + - name: Test + run: npm test From 8368c86fa109e4b7d5d25a4ad83b481b96727891 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 3 Apr 2026 14:38:48 +0000 Subject: [PATCH 2/2] Remove redundant test step from publish workflow MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Tests now run on PRs via ci.yml. The post-merge publish workflow only needs to build before publishing — testing is redundant since the code was already tested before merge. https://claude.ai/code/session_017mqanCkMvA1VU8MVD7hkA1 --- .github/workflows/publish.yml | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml index 8b2897e..a0d6acd 100644 --- a/.github/workflows/publish.yml +++ b/.github/workflows/publish.yml @@ -9,8 +9,8 @@ on: concurrency: ${{ github.workflow }}-${{ github.ref }} jobs: - build-and-test: - name: Build & Test + build: + name: Build runs-on: ubuntu-latest steps: - name: Checkout @@ -28,12 +28,9 @@ jobs: - name: Build run: npm run build - - name: Test - run: npm test - release: name: Publish Stable Release - needs: build-and-test + needs: build if: github.ref == 'refs/heads/main' runs-on: ubuntu-latest permissions: @@ -98,7 +95,7 @@ jobs: dev-release: name: Publish Dev Release - needs: build-and-test + needs: build if: github.ref == 'refs/heads/dev' runs-on: ubuntu-latest permissions: