Live Share SDK 2.0.0

.github/CODEOWNERS

@@ -1 +1 @@
-@arun2k17 @corinagum @halbondmsft @pradeepananth @ryanbliss @siduppal @Stevenic
+@pradeepananth @ryanbliss @siduppal @huntj88

.github/PULL_REQUEST_TEMPLATE/pull_request_template.md

@@ -1,9 +1,11 @@
 ## Linked issues
+
 > What issue(s) does this Pull Request address?
 
 Issues: # (issue number)
 
 ## Details
+
 #### PR type:
 
 - [ ] Bug fix
@@ -12,25 +14,26 @@ Issues: # (issue number)
 - [ ] Samples or Test coverage
 
 #### Change details
+
 > Describe your changes, with screenshots and code snippets as appropriate
 
 **code snippets**:
 
 **screenshots**:
 
+## Checklist
 
-## Checklist 
 <!-- - [ ] My code follows the style guidelines of this project -->
+
 - [ ] I have checked for/fixed spelling, linting, and other errors
 - [ ] I have commented my code for clarity
 - [ ] I have made corresponding changes to the documentation (we use [TypeDoc](https://typedoc.org/) to document our code)
 - [ ] My changes generate no new warnings
 - [ ] I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:
-   - [ ] Local testing
-   - [ ] E2E testing in Teams
+    - [ ] Local testing
+    - [ ] E2E testing in Teams
 - [ ] New and existing unit tests pass locally with my changes
 
 ### Additional information
 
 > Feel free to add other relevant information below
-

.github/dependabot.yml

@@ -1,21 +1,21 @@
 version: 2
 updates:
-  - package-ecosystem: "npm"
-  # Only specify the root so dependabot will update all manifests once per package
-  # See https://github.com/dependabot/dependabot-core/issues/5226#issuecomment-1179434437
-    directory: "/"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    # Ignore the following packages because
-    # 1. Semver for these packages is not always followed
-    # 1. These packages MUST be updated all at the same time (same PR)
-    ignore:
-      - "fluid-framework",
-      - "@fluidframework/test-client-utils"
-      - "@fluidframework/test-utils"
-      - "@fluidframework/test-runtime-utils"
-      - "@fluidframework/azure-client",
-      - "@microsoft/teams-js"
-    # Updates both the package-lock and package.json, not just package-lock
-    versioning-strategy: increase
+    - package-ecosystem: "npm"
+      # Only specify the root so dependabot will update all manifests once per package
+      # See https://github.com/dependabot/dependabot-core/issues/5226#issuecomment-1179434437
+      directory: "/"
+      schedule:
+          interval: "weekly"
+          day: "monday"
+      # Ignore the following packages because
+      # 1. Semver for these packages is not always followed
+      # 1. These packages MUST be updated all at the same time (same PR)
+      ignore:
+          - dependency-name: "fluid-framework"
+          - dependency-name: "@fluidframework/test-client-utils"
+          - dependency-name: "@fluidframework/test-utils"
+          - dependency-name: "@fluidframework/test-runtime-utils"
+          - dependency-name: "@fluidframework/azure-client"
+          - dependency-name: "@microsoft/teams-js"
+      # Updates both the package-lock and package.json, not just package-lock
+      versioning-strategy: increase

.github/workflows/checkFormatting.sh

@@ -0,0 +1,15 @@
+#!/bin/bash
+cd ../..
+npm run doctor
+diff=`git diff`
+if [ -z "$diff"  ]
+then
+    echo "everything formatted"
+    exit 0
+else
+    echo "" # empty line
+    echo "" # empty line
+    echo "FORMATTING NEEDED"
+    echo "Please run 'npm run doctor'"
+    exit 1
+fi

.github/workflows/codeql.yaml

@@ -9,67 +9,68 @@
 # the `language` matrix defined below to confirm you have the correct set of
 # supported CodeQL languages.
 #
-name: 'CodeQL'
+name: "CodeQL"
 
 on:
-  push:
-    branches: ['main', main*]
-  pull_request:
-    # The branches below must be a subset of the branches above
-    branches: ['main']
-  schedule:
-    - cron: '0 2 * * 6'
+    push:
+        branches: [main, mainv2]
+    pull_request:
+        # The branches below must be a subset of the branches above
+        branches: [main, mainv2]
+    schedule:
+        - cron: "0 2 * * 6"
+    workflow_dispatch:
 
 jobs:
-  analyze:
-    name: Analyze
-    runs-on: ubuntu-latest
-    permissions:
-      actions: read
-      contents: read
-      security-events: write
+    analyze:
+        name: Analyze
+        runs-on: ubuntu-latest
+        permissions:
+            actions: read
+            contents: read
+            security-events: write
 
-    strategy:
-      fail-fast: false
-      matrix:
-        language: ['javascript']
-        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby' ]
-        # Use only 'java' to analyze code written in Java, Kotlin or both
-        # Use only 'javascript' to analyze code written in JavaScript, TypeScript or both
-        # Learn more about CodeQL language support at https://aka.ms/codeql-docs/language-support
+        strategy:
+            fail-fast: false
+            matrix:
+                language: ["javascript"]
+                # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby' ]
+                # Use only 'java' to analyze code written in Java, Kotlin or both
+                # Use only 'javascript' to analyze code written in JavaScript, TypeScript or both
+                # Learn more about CodeQL language support at https://aka.ms/codeql-docs/language-support
 
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v3
+        steps:
+            - name: Checkout repository
+              uses: actions/checkout@v4
 
-      # Initializes the CodeQL tools for scanning.
-      - name: Initialize CodeQL
-        uses: github/codeql-action/init@v2
-        with:
-          languages: ${{ matrix.language }}
-          # If you wish to specify custom queries, you can do so here or in a config file.
-          # By default, queries listed here will override any specified in a config file.
-          # Prefix the list here with "+" to use these queries and those in the config file.
+            # Initializes the CodeQL tools for scanning.
+            - name: Initialize CodeQL
+              uses: github/codeql-action/init@v2
+              with:
+                  languages: ${{ matrix.language }}
+                  # If you wish to specify custom queries, you can do so here or in a config file.
+                  # By default, queries listed here will override any specified in a config file.
+                  # Prefix the list here with "+" to use these queries and those in the config file.
 
-          # Details on CodeQL's query packs refer to : https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
-          # queries: security-extended,security-and-quality
+                  # Details on CodeQL's query packs refer to : https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
+                  # queries: security-extended,security-and-quality
 
-      # Autobuild attempts to build any compiled languages  (C/C++, C#, Go, or Java).
-      # If this step fails, then you should remove it and run the build manually (see below)
-      - name: Autobuild
-        uses: github/codeql-action/autobuild@v2
+            # Autobuild attempts to build any compiled languages  (C/C++, C#, Go, or Java).
+            # If this step fails, then you should remove it and run the build manually (see below)
+            - name: Autobuild
+              uses: github/codeql-action/autobuild@v2
 
-      # ℹ️ Command-line programs to run using the OS shell.
-      # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
+            # ℹ️ Command-line programs to run using the OS shell.
+            # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
 
-      #   If the Autobuild fails above, remove it and uncomment the following three lines.
-      #   modify them (or add more) to build your code if your project, please refer to the EXAMPLE below for guidance.
+            #   If the Autobuild fails above, remove it and uncomment the following three lines.
+            #   modify them (or add more) to build your code if your project, please refer to the EXAMPLE below for guidance.
 
-      # - run: |
-      #     echo "Run, Build Application using script"
-      #     ./location_of_script_within_repo/buildscript.sh
+            # - run: |
+            #     echo "Run, Build Application using script"
+            #     ./location_of_script_within_repo/buildscript.sh
 
-      - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@v2
-        with:
-          category: '/language:${{matrix.language}}'
+            - name: Perform CodeQL Analysis
+              uses: github/codeql-action/analyze@v2
+              with:
+                  category: "/language:${{matrix.language}}"

.github/workflows/live-share-build-samples.yaml

@@ -1,36 +1,39 @@
 name: Build Live Share SDK samples
 
 on:
-  push:
-    branches: [ main ]
-  pull_request:
-    branches: [ main ]
+    push:
+        branches: [main, mainv2]
+    pull_request:
+        branches: [main, mainv2]
+    workflow_dispatch:
 
 jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [18.x]
-
-    steps:
-      - uses: actions/checkout@v3
-      - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v3
-        with:
-          node-version: ${{ matrix.node-version }}
-      - run: npm install
-      - run: npm install jest
-        working-directory: samples/javascript/02.react-video
-
-      - name: "build packages and samples"
-        run: npm run build
-
-      # TODO: get scenario_test.sh working
-
-      # - name: "test 02.react-video sample"
-      #   shell: "bash"
-      #   run: sh ../../../.github/workflows/scenario_test.sh
-      #   working-directory: samples/javascript/02.react-video
+    build:
+        runs-on: ubuntu-latest
+
+        strategy:
+            matrix:
+                node-version: [18.x, 22.x]
+
+        steps:
+            - uses: actions/checkout@v4
+            - name: Use Node.js ${{ matrix.node-version }}
+              uses: actions/setup-node@v4
+              with:
+                  node-version: ${{ matrix.node-version }}
+                  cache: "npm"
+            - run: npm --loglevel verbose ci --ignore-scripts
+              timeout-minutes: 20
+
+            - run: npm install jest
+              working-directory: samples/javascript/02.react-video
+
+            - name: "build packages and samples"
+              run: npm run build
+
+            # TODO: get scenario_test.sh working
+
+            # - name: "test 02.react-video sample"
+            #   shell: "bash"
+            #   run: sh ../../../.github/workflows/scenario_test.sh
+            #   working-directory: samples/javascript/02.react-video

.github/workflows/live-share-formatting.yaml

@@ -0,0 +1,31 @@
+name: Check formatting
+
+on:
+    push:
+        branches: [main, mainv2]
+    pull_request:
+        branches: [main, mainv2]
+    workflow_dispatch:
+
+jobs:
+    build:
+        runs-on: ubuntu-latest
+
+        strategy:
+            matrix:
+                node-version: [18.x, 22.x]
+
+        steps:
+            - uses: actions/checkout@v4
+            - name: Use Node.js ${{ matrix.node-version }}
+              uses: actions/setup-node@v4
+              with:
+                  node-version: ${{ matrix.node-version }}
+                  cache: "npm"
+
+            - run: npm --loglevel verbose ci --ignore-scripts
+              timeout-minutes: 20
+
+            - name: "check formatting"
+              run: "bash checkFormatting.sh"
+              working-directory: .github/workflows