Merge branch 'main' into refactor/audio-param-automation-events

Author: mdydek

.github/actions/setup/action.yml

@@ -19,7 +19,7 @@ runs:
 
     - name: Install cpplint
       shell: bash
-      run: pip install 'cpplint==2.0.0'
+      run: pip install 'cpplint==2.0.2'
 
     - name: Install ktlint
       shell: bash
@@ -28,6 +28,17 @@ runs:
         chmod a+x ktlint
         sudo mv ktlint /usr/local/bin/
 
+    # .clang-format uses options added in clang-format 22 (BreakAfterOpenBracket*).
+    # ubuntu-latest still defaults to an older binary; install 22
+    - name: Install clang-format 22
+      shell: bash
+      run: |
+        set -euo pipefail
+        curl -fsSL https://apt.llvm.org/llvm.sh | sudo bash -s -- 22
+        sudo apt-get install -y clang-format-22
+        sudo update-alternatives --install /usr/bin/clang-format clang-format /usr/bin/clang-format-22 200
+        clang-format --version
+
     - name: Cache dependencies
       id: yarn-cache
       uses: actions/cache@v4

.github/workflows/tests.yml

@@ -5,16 +5,28 @@ on:
   workflow_dispatch:
 
 jobs:
-  tests:
+  cpp-tests:
     runs-on: ubuntu-latest
     steps:
-
       - name: Checkout
         uses: actions/checkout@v4
 
       - name: Setup
         uses: ./.github/actions/setup
 
-      - name: Run tests
-        run: yarn test
+      - name: Run C++ tests
+        working-directory: packages/react-native-audio-api
+        run: yarn test:cpp
+
+  js-tests:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup
+        uses: ./.github/actions/setup
 
+      - name: Run JS integration tests
+        working-directory: packages/react-native-audio-api
+        run: yarn test:js

package.json

@@ -16,9 +16,7 @@
     "format": "yarn workspaces foreach -A -p run format",
     "clean": "del-cli packages/**/android/build apps/**/android/build apps/**/android/app/build apps/**/ios/build packages/**/lib node_modules apps/**/node_modules packages/**/node_modules",
     "typecheck": "yarn workspaces foreach -A -p run typecheck",
-    "test": "bash packages/react-native-audio-api/common/cpp/test/RunTests.sh",
-    "test:graph": "bash packages/react-native-audio-api/common/cpp/test/RunTestsGraph.sh",
-    "test:graph:docker": "bash packages/react-native-audio-api/common/cpp/test/RunTestsGraphDocker.sh",
+    "test": "yarn workspace react-native-audio-api run test",
     "check-audio-enum-sync": "bash packages/react-native-audio-api/scripts/check-audio-events-sync.sh"
   },
   "devDependencies": {

packages/react-native-audio-api/.clang-format

@@ -1,7 +1,7 @@
 ---
 ColumnLimit: 100
 AccessModifierOffset: -1
-AlignAfterOpenBracket: AlwaysBreak
+AlignAfterOpenBracket: false
 AlignConsecutiveAssignments: false
 AlignConsecutiveDeclarations: false
 AlignEscapedNewlines: DontAlign
@@ -70,10 +70,14 @@ SpacesInSquareBrackets: false
 UseTab: Never
 ---
 Language: Cpp
+BreakAfterOpenBracketBracedList: true
+BreakAfterOpenBracketFunction: true
 Standard: c++20
 ---
 Language: ObjC
 BreakBeforeBraces: WebKit
 ObjCBlockIndentWidth: 2
 ObjCSpaceAfterProperty: true
 ObjCSpaceBeforeProtocolList: true
+BreakAfterOpenBracketBracedList: true
+BreakAfterOpenBracketFunction: true

packages/react-native-audio-api/.eslintrc.js

@@ -5,6 +5,13 @@ module.exports = {
     {
       files: ['./src/**/*.{ts,tsx}'],
     },
+    {
+      files: ['./tests/**/*.{ts,tsx}'],
+      parserOptions: {
+        project: './tsconfig.test.json',
+        tsconfigRootDir: __dirname,
+      },
+    },
   ],
-  ignorePatterns: ['lib', 'src/web-core/custom/signalsmithStretch' ],
+  ignorePatterns: ['lib', 'src/web-core/custom/signalsmithStretch'],
 };

packages/react-native-audio-api/common/cpp/test/RunTests.sh

@@ -9,7 +9,7 @@ cleanup() {
 
 trap cleanup EXIT
 
-cd packages/react-native-audio-api/common/cpp/test
+cd common/cpp/test
 
 cmake -S . -B build -Wno-dev
 

packages/react-native-audio-api/package.json

@@ -45,7 +45,12 @@
     "app.plugin.js"
   ],
   "scripts": {
-    "test": "jest",
+    "test": "yarn test:js && yarn test:cpp",
+    "test:js": "jest",
+    "test:cpp": "bash common/cpp/test/RunTests.sh",
+    "test:full": "yarn test && yarn test:graph",
+    "test:graph": "bash common/cpp/test/RunTestsGraph.sh",
+    "test:graph:docker": "bash common/cpp/test/RunTestsGraphDocker.sh",
     "typecheck": "tsc --noEmit",
     "lint": "yarn lint:js && yarn lint:cpp && yarn lint:ios && yarn lint:kotlin",
     "lint:js": "eslint src && yarn prettier --check src",

packages/react-native-audio-api/tests/integration.test.ts

@@ -231,8 +231,6 @@ describe('Mock Integration Tests', () => {
         streamPath: 'https://example.com/audio-stream',
       });
 
-      expect(streamer.streamPath).toBe('https://example.com/audio-stream');
-
       // Connect to output
       streamer.connect(context.destination);
 
@@ -242,17 +240,6 @@ describe('Mock Integration Tests', () => {
       streamer.resume();
       streamer.stop();
     });
-
-    it('should handle streamer initialization errors', () => {
-      const context = new MockAPI.AudioContext();
-      const streamer = context.createStreamer();
-
-      // First initialization should succeed
-      expect(streamer.initialize('http://stream1.com')).toBe(true);
-
-      // Second initialization should fail
-      expect(() => streamer.initialize('http://stream2.com')).toThrow();
-    });
   });
 
   describe('System Integration', () => {

packages/react-native-audio-api/tests/mock.test.ts

@@ -204,22 +204,11 @@ describe('React Native Audio API Mocks', () => {
 
     describe('StreamerNode', () => {
       it('should create a StreamerNode', () => {
-        const streamer = context.createStreamer();
+        const streamer = context.createStreamer({
+          streamPath: 'http://example.com/stream',
+        });
         expect(streamer).toBeInstanceOf(MockAPI.StreamerNode);
       });
-
-      it('should support initialization with stream path', () => {
-        const streamer = context.createStreamer();
-        const result = streamer.initialize('http://example.com/stream');
-        expect(result).toBe(true);
-        expect(streamer.streamPath).toBe('http://example.com/stream');
-      });
-
-      it('should throw error on duplicate initialization', () => {
-        const streamer = context.createStreamer();
-        streamer.initialize('http://example.com/stream');
-        expect(() => streamer.initialize('http://example.com/other')).toThrow();
-      });
     });
 
     describe('WorkletNodes', () => {

packages/react-native-audio-api/tsconfig.test.json

@@ -0,0 +1,4 @@
+{
+  "extends": "./tsconfig.json",
+  "include": ["src", "tests"]
+}