chore: add Danger support

.github/workflows/android-build-and-distribute.yml

@@ -1,13 +1,15 @@
 name: Android
 
-on: pull_request
+on:
+  workflow_run:
+    workflows: ["Danger (JS)", "Danger (Swift)", "Danger (Kotlin)"]
+    types: [completed]
 
 jobs:
   android-distribute-to-appetize:
     runs-on: ubuntu-latest
     name: "Build and upload app to Appetize 🚀"
     steps:
-
       - name: Cancel previous jobs
         uses: styfle/cancel-workflow-action@85880fa0301c86cca9da44039ee3bb12d3bedbfa # v0.12.1
         with:

.github/workflows/danger-js.yml

@@ -0,0 +1,32 @@
+name: Danger (JS)
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - synchronize
+
+concurrency: 
+  group: ${{ github.ref }}-danger-js
+  cancel-in-progress: true
+
+jobs:
+  danger-js:
+    runs-on: ubuntu-latest
+    name: "Run Danger JS"
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+
+      - name: Set up Node.js
+        uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2
+        with:
+          node-version: 18
+
+      - name: Install Dependencies
+        run: yarn install
+
+      - name: Run Danger JS
+        run: npx danger ci --id js --dangerfile dangerfile.ts --failOnErrors --removePreviousComments
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/danger-kotlint.yml

@@ -0,0 +1,52 @@
+name: Danger (Kotlin)
+
+on:
+  workflow_run:
+    workflows: ["Danger (JS)"]
+    types: [completed]
+  pull_request:
+    types:
+      - opened
+      - edited
+      - synchronize
+
+concurrency: 
+  group: ${{ github.ref }}-danger-kotlin
+  cancel-in-progress: true
+
+jobs:
+  danger-kotlin:
+    runs-on: ubuntu-latest
+    name: "Run Danger Kotlin"
+    container:
+      image: docker://ghcr.io/danger/danger-kotlin:1.3.3
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+
+      - name: Install Yarn
+        run: npm install -g yarn
+
+      - name: Install dependencies with Yarn
+        run: yarn install
+        working-directory: packages/example
+
+      # Override container JDK
+      - name: Set up JDK 17
+        run: |
+          apt-get update
+          apt-get install -y openjdk-17-jdk
+
+      - name: Set up Android SDK
+        uses: android-actions/setup-android@v3
+
+      - name: Set sdk.dir in local.properties
+        run: echo "sdk.dir=/usr/lib/android-sdk" > packages/example/android/local.properties
+
+      - name: Run detekt
+        run: ./gradlew -Dorg.gradle.java.home=/usr/lib/jvm/java-17-openjdk-amd64 detekt || echo "Detekt completed with errors, running Danger..."
+        working-directory: packages/example/android
+
+      - name: Run Danger Kotlin
+        run: danger-kotlin ci --id=kt --failOnErrors --no-publish-check --remove-previous-comments
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/danger-swift.yml

@@ -0,0 +1,29 @@
+name: Danger (Swift)
+
+on:
+  workflow_run:
+    workflows: ["Danger (JS)", "Danger (Kotlin)"]
+    types: [completed]
+  pull_request:
+    types:
+      - opened
+      - edited
+      - synchronize
+
+concurrency: 
+  group: ${{ github.ref }}-danger-swift
+  cancel-in-progress: true
+
+jobs:
+  danger-swift:
+    runs-on: ubuntu-latest
+    name: "Run Danger"
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+
+      - name: Danger
+        uses: docker://ghcr.io/danger/danger-swift-with-swiftlint:3.15.0
+        with:
+            args: --id=swift --failOnErrors --no-publish-check --remove-previous-comments
+        env:
+          GITHUB_TOKEN: ${{ github.token }}

.github/workflows/ios-build-and-distribute.yml

@@ -1,6 +1,10 @@
 ---
 name: iOS
-on: pull_request
+on:
+  workflow_run:
+    workflows: ["Danger (JS)", "Danger (Swift)", "Danger (Kotlin)"]
+    types: [completed]
+
 jobs:
 
   build-and-upload-to-appetize:

.prettierignore

@@ -0,0 +1,11 @@
+# Ignore everything recursively
+*
+
+# But not the following
+!**/packages/sdk/src/**/*.ts
+!**/packages/sdk/src/**/*.tsx
+!**/packages/sdk/src/**/*.js
+!**/packages/sdk/src/**/*.jsx
+
+# Check subdirectories too
+!*/

packages/example/.prettierrc.js → .prettierrc.js

@@ -1,7 +1,11 @@
 module.exports = {
   arrowParens: 'avoid',
   bracketSameLine: true,
-  bracketSpacing: false,
   singleQuote: true,
   trailingComma: 'all',
+  quoteProps: 'consistent',
+  tabWidth: 2,
+  trailingComma: 'es5',
+  useTabs: false,
+  printWidth: 120,
 };

Dangerfile.df.kts

@@ -0,0 +1,49 @@
+@file:DependsOn("xyz.pavelkorolev.danger.detekt:plugin:1.2.0")
+
+import java.io.File
+import systems.danger.kotlin.*
+import systems.danger.kotlin.models.github.*
+import xyz.pavelkorolev.danger.detekt.DetektPlugin
+
+register plugin DetektPlugin
+
+danger(args) {
+    val allSourceFiles = git.modifiedFiles + git.createdFiles
+
+    onGitHub {
+        // region PR Contains Tests
+        val kotlinFilesContainingChanges =
+                allSourceFiles.filter { path: String ->
+                    path.endsWith(".kt") &&
+                            File(path).readText().contains("import org.junit.jupiter.api.Test")
+                }
+        if (kotlinFilesContainingChanges.isEmpty()) {
+            warn(
+                "This PR doesn't seem to contain any updated Unit Test for Kotlin 🤔. Please consider double checking it 🙏"
+            )
+        }
+        // endregion
+
+        // region Detekt
+        parseDetektReport()
+        // endregion
+    }
+}
+
+fun parseDetektReport() {
+    val file = File("packages/sdk/android/build/reports/detekt/detekt.xml")
+    if (!file.exists()) {
+        warn("🙈 No detekt report found at ${file.path}")
+        return
+    }
+    with(DetektPlugin) {
+        val report = parse(file)
+        val count = report.count
+        report(report)
+        if (count == 0) {
+            message("✅ No detekt violations found.")
+        } else {
+            fail("🙁 Found **${report.count}** detekt violations.")
+        }
+    }
+}

Dangerfile.swift

@@ -0,0 +1,61 @@
+import Danger
+import Foundation
+
+let danger = Danger()
+let pr = danger.github.pullRequest
+let isReleasePr = pr.head.ref.hasPrefix("release")
+let allCreatedAndModifiedFiles = danger.git.modifiedFiles + danger.git.createdFiles
+let sdkEditedFiles = allCreatedAndModifiedFiles
+    .filter { $0.name != "Dangerfile.swift" }
+    .filter { !$0.hasPrefix("packages/example/ios/") }
+
+// You can use these functions to send feedback:
+// message("Highlight something in the table")
+// warn("Something pretty bad, but not important enough to fail the build")
+// fail("Something that must be changed")
+// markdown("Free-form markdown that goes under the table, so you can do whatever.")
+
+// MARK: - Copyright
+
+// Checks whether new files have "Copyright / Created by" mentions
+
+let swiftFilesWithCopyright = sdkEditedFiles.filter {
+    $0.fileType == .swift &&
+        danger.utils.readFile($0).contains("//  Created by")
+}
+
+// if swiftFilesWithCopyright.count > 0 {
+//    let files = swiftFilesWithCopyright.joined(separator: ", ")
+//    warn("In Danger we don't include copyright headers, found them in: \(files)")
+// }
+
+// MARK: - PR Contains Tests
+
+// Raw check based on created / updated files containing `import XCTest`
+
+let swiftTestFilesContainChanges = sdkEditedFiles.filter {
+    $0.fileType == .swift &&
+        danger.utils.readFile($0).contains("import XCTest")
+}
+
+if swiftTestFilesContainChanges.isEmpty {
+    warn("This PR doesn't seem to contain any updated Unit Test for Swift 🤔. Please consider double checking it 🙏")
+}
+
+// MARK: - SwiftLint
+
+// Use a different path for SwiftLint
+
+let filesToLint = sdkEditedFiles.filter { $0.fileType == .swift }
+let violations = SwiftLint.lint(.files(filesToLint), inline: true, configFile: "packages/example/ios/.swiftlint.yml")
+
+if violations.isEmpty {
+    message("✅ No SwiftLint violations found.")
+} else {
+    fail("🙁 Found **\(violations.count)** SwiftLint violations.")
+}
+
+// MARK: - Check Coverage
+
+// Coverage.xcodeBuildCoverage(.derivedDataFolder("Build"),
+//                            minimumCoverage: 30)

dangerfile.ts

@@ -0,0 +1,87 @@
+import { danger, warn, fail, message } from 'danger';
+import { ESLint } from 'eslint';
+
+const pr = danger.github.pr;
+const directory = process.cwd();
+
+// #region PR Length
+const bigPrThreshold = 600;
+const additions = pr.additions || 0;
+const deletions = pr.deletions || 0;
+
+if (additions + deletions > bigPrThreshold) {
+  warn(
+    'Pull Request size seems relatively large. If this Pull Request contains multiple changes, please split each into separate PRs for a faster, easier review.'
+  );
+}
+// #endregion
+
+// #region WIP PR
+if (pr.draft || pr.title.toLowerCase().includes('wip')) {
+  warn('PR is classed as Work in Progress');
+}
+// #endregion
+
+// #region PR Assignee
+if (pr.assignees.length === 0) {
+  warn('Please assign someone aside from CODEOWNERS (@checkout-pci-reviewers) to review this PR.');
+}
+// #endregion
+
+// #region Conventional Commit Title
+const validPrefixes = [
+  'fix',
+  'feat',
+  'chore',
+  'ci',
+  'refactor',
+  'docs',
+  'perf',
+  'test',
+  'build',
+  'revert',
+  'style',
+  'BREAKING CHANGE',
+];
+const isConventionalCommitTitle = validPrefixes.some(prefix => pr.title.startsWith(prefix));
+
+if (!isConventionalCommitTitle) {
+  fail(
+    'Please use a conventional commit title for this PR. See [Conventional Commits and SemVer](https://www.notion.so/primerio/Automating-Version-Bumping-and-Changelog-Creation-c13e32fea11447069dea76f966f4b0fb?pvs=4#c55764aa2f2748eb988d581a456e61e7)'
+  );
+}
+// #endregion
+
+// #region ESLint
+async function eslint() {
+  const eslint = new ESLint({
+    overrideConfigFile: 'eslint.config.mjs',
+  });
+  const results = await eslint.lintFiles('.');
+
+  let errorCount = 0;
+  results.forEach(result => {
+    result.messages.forEach(eslintMessage => {
+      let actualFilePath = removePrefix(result.filePath, directory + '/');
+      let errorMessage = `** ESLint **: ${eslintMessage.message} - (${eslintMessage.ruleId})`;
+      if (eslintMessage.severity === 1) {
+        errorCount++;
+        warn(errorMessage, actualFilePath, eslintMessage.line);
+      } else if (eslintMessage.severity === 2) {
+        errorCount++;
+        fail(errorMessage, actualFilePath, eslintMessage.line);
+      }
+    });
+  });
+
+  if (errorCount == 0) {
+    message('✅ No ESLint violations found.');
+  } else {
+    fail(`🙁 Found **${errorCount}** ESLint violations.`);
+  }
+}
+
+eslint().catch(error => fail(`❌ ESLint execution failed: ${error.message}`));
+// #endregion
+
+const removePrefix = (str: string, prefix: string) => (str.startsWith(prefix) ? str.slice(prefix.length) : str);