merge

Author: paulius-valiunas

.github/CODEOWNERS

@@ -9,4 +9,4 @@
 # @global-owner1 and @global-owner2 will be requested for
 # review when someone opens a pull request.
 
-*       @calebmshafer @evelynpreslar-bentley @aruniverse @ben-polinsky @anmolshres98 @MichaelSwigerAtBentley
+*       @aruniverse @ben-polinsky @anmolshres98 @MichaelSwigerAtBentley

.github/workflows/ci.yaml

@@ -28,12 +28,12 @@ jobs:
           git config --local user.name imodeljs-admin
 
       - name: Setup node
-        uses: actions/setup-node@v2
+        uses: actions/setup-node@v4
         with:
-          node-version: "18.12.0"
+          node-version: 20.x
 
       - name: Setup pnpm
-        run: corepack enable
+        uses: pnpm/action-setup@v4
 
       - name: Install packages
         run: pnpm install

.github/workflows/codeql-analysis.yaml

@@ -20,7 +20,7 @@ jobs:
 
     steps:
       - name: Checkout branch
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
 

.github/workflows/create-github-release.yaml

@@ -0,0 +1,41 @@
+name: Create GitHub Release
+
+on:
+  push:
+    tags:
+      - '@itwin/*_v[0-9]+.[0-9]+.[0-9]+'
+  workflow_dispatch:
+
+jobs:
+  create-release:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout tag
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.ref }} # checkouts the branch that triggered the workflow
+          fetch-depth: 0
+
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20.x
+
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
+
+      - name: Install dependencies
+        run: pnpm install
+
+      - name: Configure git
+        run: |
+          git config --local user.email imodeljs-admin@users.noreply.github.com
+          git config --local user.name imodeljs-admin
+
+      - name: Create GitHub release
+        run: |
+          echo "Creating GitHub release for tag ${{ github.ref }}"
+          bash .github/workflows/scripts/create-release.sh ${{ github.ref }}
+        env:
+          GITHUB_TOKEN: ${{ secrets.IMJS_ADMIN_GH_TOKEN }}

.github/workflows/create-release.yaml .github/workflows/publish-packages.yaml.github/workflows/create-release.yaml renamed to .github/workflows/publish-packages.yaml

@@ -1,6 +1,4 @@
-# Workflow that will create a GitHub release a when a new release tag is pushed
-
-name: Create Release
+name: Publish Packages to npm
 
 on: workflow_dispatch
 
@@ -11,26 +9,26 @@ jobs:
     name: Release packages
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           token: ${{ secrets.IMJS_ADMIN_GH_TOKEN }}
 
-      - name: Use Node.js 18
-        uses: actions/setup-node@v3
+      - name: Setup Node
+        uses: actions/setup-node@v4
         with:
-          node-version: 18.16.x
+          node-version: 20.x
           registry-url: https://registry.npmjs.org/
 
-      - name: Setup pnpm
-        run: corepack enable
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
 
       - name: Install dependencies
         run: pnpm install
 
       - name: Build, Lint, Test
         run: pnpm lage build lint cover
 
-      - name: Publish packages
+      - name: Publish packages and create git tags
         run: |
           git config --local user.email imodeljs-admin@users.noreply.github.com
           git config --local user.name imodeljs-admin

.github/workflows/scripts/create-release.sh

@@ -0,0 +1,66 @@
+#!/bin/bash
+
+if [ -z "$1" ]; then
+  echo "No refName was supplied"
+  exit 1
+fi
+
+refName=$1
+echo "Ref name passed in: $refName"
+
+tagName=$(echo $refName | sed 's/refs\/tags\///')
+echo "Tag name was parsed as: $tagName"
+packageName=$(echo $tagName | sed 's/_v.*//')
+echo "Package name was parsed as: $packageName"
+packageVersion=$(echo $tagName | sed 's/.*_v//')
+echo "Package version was parsed as: $packageVersion"
+
+# The packageDirectory variable is determined by searching the ./packages directory for a subdirectory
+# that contains a package.json file with a name matching the parsed packageName.
+# determine package directory so that we can zip it up and also find the changelog
+packageDirectory=$(find ./packages -maxdepth 1 -type d | while read dir; do
+  if [ -f "$dir/package.json" ]; then
+    packageNameInJson=$(jq -r '.name' "$dir/package.json")
+    if [ "$packageNameInJson" == "$packageName" ]; then
+      echo "$dir"
+      break
+    fi
+  fi
+done)
+
+if [ -z "$packageDirectory" ]; then
+  echo "No package directory found for package name: $packageName"
+  exit 1
+else
+  echo "Package directory was determined as: $packageDirectory"
+fi
+
+cd $packageDirectory
+
+changelogMd="CHANGELOG.md"
+if [ ! -f "$changelogMd" ]; then
+  echo "Changelog file not found: $changelogMd"
+  exit 1
+fi
+
+# build the package and create consumable to release
+pnpm build
+
+# Extract the changelog text
+releaseNoteText=$(awk -v version="$packageVersion" '$0 ~ version {flag=1; print; next} /^## /{flag=0} flag' "$changelogMd")
+echo "Release note text was extracted as: $releaseNoteText"
+
+# remove the @itwin/ or @bentley/ scope from the tag name to create the zip file name since they have special characters
+zipFileName=$(echo "$tagName" | sed 's/@itwin\///; s/@bentley\///')
+echo "Zip file name was parsed as: $zipFileName"
+
+# Zip the package deliverables - lib and dist directories
+zip -r "$zipFileName.zip" lib dist
+tar -czvf "$zipFileName.tar.gz" lib dist
+
+# Create a release and upload assets
+gh release create "$tagName" \
+  "$zipFileName.zip" \
+  "$zipFileName.tar.gz" \
+  --notes "$releaseNoteText" \
+  --verify-tag

.gitignore

@@ -9,7 +9,6 @@ dist
 
 # testing
 .nyc_output
-.parcel-cache
 junit_results.xml
 /**/test-results/**/*
 generated-docs

.pipelines/build-docs.yaml

@@ -33,12 +33,12 @@ stages:
             clean: true
 
           - task: NodeTool@0
-            displayName: Use Node 18
+            displayName: Use Node 20
             inputs:
-              versionSpec: 18
+              versionSpec: 20
               checkLatest: true
 
-          - script: corepack enable
+          - script: npm install -g corepack && corepack enable
             displayName: Install pnpm
 
           - script: |

.pipelines/integration-test.yaml

@@ -12,9 +12,11 @@ variables:
   - group: iTwin.js non-secret config variables
   - group: iTwin.js Integration Test Users
   - group: iTwin.js Auth Client Integration tests
+  - name: packages
+    value: browser-authorization, electron-authorization, oidc-signin-tool
 
 jobs:
-  - job: BuildPackages
+  - job: IntegrationTests
     strategy:
       matrix:
         linux:
@@ -32,9 +34,9 @@ jobs:
       - task: NodeTool@0
         displayName: Use Node 18
         inputs:
-          versionSpec: 18.x
+          versionSpec: 18.16.x
 
-      - script: corepack enable
+      - script: npm install -g corepack && corepack enable
         displayName: Install pnpm
 
       - bash: |
@@ -53,7 +55,7 @@ jobs:
           test_azuread_user_password: $(IMJS_TEST_AZUREAD_USER_PASSWORD)
         condition: and(succeeded(), ne(variables['Agent.OS'], 'Windows_NT'))
 
-      # For debugging
+        # For debugging
       - bash: |
           export
 
@@ -106,13 +108,14 @@ jobs:
           IMJS_TEST_REGULAR_USER_PASSWORD: $(IMJS_TEST_REGULAR_USER_PASSWORD)
         condition: and(succeeded(), eq(variables['Agent.OS'], 'Linux'))
 
-      - script: pnpm test:integration
-        displayName: pnpm test:integration
-        env:
-          # Couldn't get this variable to be ingested in the steps above, but this works
-          IMJS_TEST_AZUREAD_USER_PASSWORD: $(IMJS_TEST_AZUREAD_USER_PASSWORD)
-          IMJS_TEST_REGULAR_USER_PASSWORD: $(IMJS_TEST_REGULAR_USER_PASSWORD)
-        condition: and(succeeded(), ne(variables['Agent.OS'], 'Linux'))
+      - ${{ each package in split(variables.packages, ', ')}}:
+          - script: pnpm --filter ${{package}} test:integration
+            displayName: run ${{package}} tests
+            env:
+              # Couldn't get this variable to be ingested in the steps above, but this works
+              IMJS_TEST_AZUREAD_USER_PASSWORD: $(IMJS_TEST_AZUREAD_USER_PASSWORD)
+              IMJS_TEST_REGULAR_USER_PASSWORD: $(IMJS_TEST_REGULAR_USER_PASSWORD)
+            condition: and(succeeded(), ne(variables['Agent.OS'], 'Linux'))
 
       - task: PublishTestResults@2
         displayName: "Publish Integration Test Results"