Merge pull request #24 from eviltester/eviltester/issue23

Eviltester/issue23

Author: eviltester

.babelrc

@@ -0,0 +1,10 @@
+# Changesets
+
+This repository uses [Changesets](https://github.com/changesets/changesets) for versioning and release notes across workspace packages.
+
+Typical flow:
+
+1. `npx changeset` to create a release note.
+2. Merge PR.
+3. `npx changeset version` to update versions.
+4. `npx changeset publish` to publish packages.

.changeset/README.md

@@ -0,0 +1,11 @@
+{
+  "$schema": "https://unpkg.com/@changesets/config@3.0.0/schema.json",
+  "changelog": ["@changesets/cli/changelog", { "repo": "eviltester/grid-table-editor" }],
+  "commit": false,
+  "fixed": [],
+  "linked": [],
+  "access": "public",
+  "baseBranch": "master",
+  "updateInternalDependencies": "patch",
+  "ignore": ["@anywaydata/core-ui"]
+}

.changeset/config.json

@@ -0,0 +1,9 @@
+node_modules
+.git
+.gitignore
+coverage
+build
+test-results
+tests
+docs-src
+*.log

.dockerignore

@@ -4,4 +4,5 @@ docs/
 docs-src/build/
 node_modules/
 js/libs/
+packages/core/js/libs/
 js/gui_components/data-grid-editor/tabulator/customHeader-tabulator.js

.eslintignore

@@ -5,10 +5,44 @@ on:
       tag:
       commit:
         default: 'master'
+permissions:
+  contents: read
 jobs:
   bun-build:
-    name: build-bun-exes
-    runs-on: ubuntu-latest
+    name: build-${{ matrix.name }}
+    runs-on: ${{ matrix.runner }}
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - name: windows-x64
+            runner: windows-latest
+            bun_target: bun-windows-x64
+            bin_relpath: out/windows64/anywaydata.exe
+            zip_name: windows64.zip
+            smoke_command: .\out\windows64\anywaydata.exe --help
+            bun_build_flags: --compile --minify --sourcemap
+          - name: linux-x64
+            runner: ubuntu-latest
+            bun_target: bun-linux-x64
+            bin_relpath: out/linux-x64/anywaydata
+            zip_name: linux-x64.zip
+            smoke_command: ./out/linux-x64/anywaydata --help
+            bun_build_flags: --compile
+          - name: linux-arm64
+            runner: ubuntu-24.04-arm
+            bun_target: bun-linux-arm64
+            bin_relpath: out/linux-arm64/anywaydata
+            zip_name: linux-arm64.zip
+            smoke_command: ./out/linux-arm64/anywaydata --help
+            bun_build_flags: --compile
+          - name: macos-arm64
+            runner: macos-latest
+            bun_target: bun-darwin-arm64
+            bin_relpath: out/macos-arm64/anywaydata
+            zip_name: macos-arm64.zip
+            smoke_command: ./out/macos-arm64/anywaydata --help
+            bun_build_flags: --compile
     defaults:
       run:
         working-directory: ./cli
@@ -19,39 +53,43 @@ jobs:
           bun-version: 1.2.10
       - name: install dependencies
         run: bun install
-      - name: build windows
-        run: |
-          bun build ./index.ts --compile --minify --sourcemap --target=bun-windows-x64 --outfile ${{ github.workspace }}/out/windows64/anywaydata.exe
-          zip -j ${{ github.workspace }}/windows64.zip ${{ github.workspace }}/out/windows64/anywaydata.exe
-      - name: build linux-x64
-        run: |
-          bun build ./index.ts --compile --target=bun-linux-x64 --outfile ${{ github.workspace }}/out/linux-x64/anywaydata
-          zip -j ${{ github.workspace }}/linux-x64.zip ${{ github.workspace }}/out/linux-x64/anywaydata
-      - name: build linux-arm64
-        run: |
-          bun build ./index.ts --compile --outfile ${{ github.workspace }}/out/linux-arm64/anywaydata --target=bun-linux-arm64
-          zip -j ${{ github.workspace }}/linux-arm64.zip ${{ github.workspace }}/out/linux-arm64/anywaydata
-      - name: build mac-x64
-        run: |
-          bun build ./index.ts --compile --outfile ${{ github.workspace }}/out/macos-x64/anywaydata --target=bun-darwin-x64
-          zip -j ${{ github.workspace }}/macos-x64.zip ${{ github.workspace }}/out/macos-x64/anywaydata
-      - name: build mac-arm64
+      - name: build binary
+        shell: bash
         run: |
-          bun build ./index.ts --compile --outfile ${{ github.workspace }}/out/macos-arm64/anywaydata --target=bun-darwin-arm64
-          zip -j ${{ github.workspace }}/macos-arm64.zip ${{ github.workspace }}/out/macos-arm64/anywaydata
-      - name: check dir
-        run: ls -R ${{ github.workspace }}/out
+          bun build ./index.ts ${{ matrix.bun_build_flags }} --target=${{ matrix.bun_target }} --outfile "${{ github.workspace }}/${{ matrix.bin_relpath }}"
+      - name: smoke test binary
+        run: ${{ matrix.smoke_command }}
+      - name: package zip (windows)
+        if: runner.os == 'Windows'
+        shell: powershell
+        run: Compress-Archive -Path "${{ github.workspace }}\${{ matrix.bin_relpath }}" -DestinationPath "${{ github.workspace }}\${{ matrix.zip_name }}" -Force
+      - name: package zip (non-windows)
+        if: runner.os != 'Windows'
+        shell: bash
+        run: zip -j "${{ github.workspace }}/${{ matrix.zip_name }}" "${{ github.workspace }}/${{ matrix.bin_relpath }}"
+      - name: upload zip artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: ${{ matrix.zip_name }}
+          path: ${{ github.workspace }}/${{ matrix.zip_name }}
+          if-no-files-found: error
+  create-draft-release:
+    name: create-draft-release
+    needs: bun-build
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+    steps:
+      - name: download built zips
+        uses: actions/download-artifact@v4
+        with:
+          path: ${{ github.workspace }}/release-assets
       - uses: ncipollo/release-action@v1.16.0
         with:
           tag: ${{ inputs.tag }}
           commit: ${{ inputs.commit }}
           name: Draft Release ${{ github.ref }}
           draft: true
-          artifacts: |
-            ${{ github.workspace }}/windows64.zip
-            ${{ github.workspace }}/linux-x64.zip
-            ${{ github.workspace }}/linux-arm64.zip
-            ${{ github.workspace }}/macos-x64.zip
-            ${{ github.workspace }}/macos-arm64.zip
+          artifacts: ${{ github.workspace }}/release-assets/**/*.zip
           body: 'AnyWayData CLI versions'
           allowUpdates: true

.github/workflows/cli-deploy.yml

@@ -12,7 +12,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x]
+        node-version: [20.x]
 
     steps:
       - name: Check out repository
@@ -30,20 +30,11 @@ jobs:
       - name: Build
         run: npm run build --if-present
 
-      - name: Check formatting
-        run: npm run format:check
-
-      - name: Run linter
-        run: npm run lint
-
-      - name: Run tests with coverage
-        run: npm run testcoverage
-
       - name: Install Playwright browser
         run: npx playwright install --with-deps chromium
 
-      - name: Run browser smoke tests
-        run: npm run test:browser
+      - name: Verify
+        run: npm run verify:ci
 
       - name: Upload coverage artifact
         uses: actions/upload-artifact@v4

.github/workflows/node.js.yml

@@ -0,0 +1,3 @@
+#!/usr/bin/env sh
+
+npm run verify:local

.husky/pre-push

@@ -0,0 +1,23 @@
+# Agent Working Agreement
+
+## Definition of Done for Code Changes
+
+Before reporting a coding task as complete, run:
+
+```bash
+npm run verify:local
+```
+
+and include the result status in the final update.
+
+If verification fails:
+
+- do not mark the task as done
+- report the failing command(s)
+- include the relevant error summary
+- fix issues where possible, then re-run verification
+
+## Notes
+
+- `verify:local` is the canonical local gate for this repo.
+- CI uses `verify:ci` and branch protection should require CI to pass before merge.