chore(ci): add a separate PR workflow for forks (#915)

Author: nirinchev

.github/workflows/test-and-build-from-fork.yaml

@@ -0,0 +1,52 @@
+name: Test and Build (from fork)
+on:
+  pull_request_target:
+    branches:
+      - main
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  test-and-build:
+    name: Test and Build
+    if: github.event.pull_request.user.login == 'dependabot[bot]' || github.event.pull_request.head.repo.full_name != github.repository
+
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+      fail-fast: false
+
+    runs-on: ${{ matrix.os }}
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node.js Environment
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20.16.0
+
+      - name: Install Dependencies
+        run: npm ci --omit=optional
+
+      - name: Run Checks
+        run: npm run check
+        # the glob here just fails
+        if: ${{ runner.os != 'Windows' }}
+
+      - name: Run Tests
+        env:
+          NODE_OPTIONS: "--max_old_space_size=4096"
+          SEGMENT_KEY: ${{ secrets.SEGMENT_KEY_DEV }}
+        run: npm run test
+
+      - name: Enable auto-merge for Dependabot PRs
+        if: github.event.pull_request.user.login == 'dependabot[bot]'
+        run: gh pr merge --auto --merge "$PR_URL"
+        env:
+          PR_URL: ${{github.event.pull_request.html_url}}
+          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

.github/workflows/test-and-build.yaml

@@ -16,6 +16,7 @@ permissions:
 jobs:
   test-and-build:
     name: Test and Build
+    if: github.event.pull_request.user.login != 'dependabot[bot]' && github.event.pull_request.head.repo.full_name == github.repository
 
     strategy:
       matrix:

src/commands/launchMongoShell.ts

@@ -12,7 +12,7 @@ const launchMongoDBShellWithEnv = ({
   mdbConnectionString: string;
   envVariableString: string;
   parentHandle?: string;
-}) => {
+}): void => {
   const mongoDBShell = vscode.window.createTerminal({
     name: 'MongoDB Shell',
     env: {
@@ -29,19 +29,19 @@ const launchMongoDBShellWithEnv = ({
   mongoDBShell.show();
 };
 
-const getPowershellEnvString = () => {
+const getPowershellEnvString = (): string => {
   return '$Env:MDB_CONNECTION_STRING';
 };
 
-const getCmdEnvString = () => {
+const getCmdEnvString = (): string => {
   return '%MDB_CONNECTION_STRING%';
 };
 
-const getGitBashEnvString = () => {
+const getGitBashEnvString = (): string => {
   return '$MDB_CONNECTION_STRING';
 };
 
-const getBashEnvString = () => {
+const getBashEnvString = (): string => {
   return '$MDB_CONNECTION_STRING';
 };
 

src/editors/documentIdStore.ts

@@ -32,7 +32,7 @@ export default class DocumentIdStore {
     return newDocument.documentIdReference;
   }
 
-  get(documentIdReference: string) {
+  get(documentIdReference: string): any {
     const existingDocument = this._documents.find(
       (item) => item.documentIdReference === documentIdReference
     );

src/explorer/databaseTreeItem.ts

@@ -129,7 +129,8 @@ export default class DatabaseTreeItem
       const sortFunction = (
         collectionA: CollectionDetailsType,
         collectionB: CollectionDetailsType
-      ) => (collectionA.name || '').localeCompare(collectionB.name || '');
+      ): number =>
+        (collectionA.name || '').localeCompare(collectionB.name || '');
 
       const collectionTreeEntries = [
         ...otherCollections.sort(sortFunction),

src/featureFlags.ts

@@ -4,14 +4,14 @@ const FEATURE_FLAGS = {
 
 export type FeatureFlag = keyof typeof FEATURE_FLAGS;
 
-export const getFeatureFlag = (flag: FeatureFlag) => {
+export const getFeatureFlag = (flag: FeatureFlag): boolean => {
   if (typeof window === 'object') {
     return (window as any).MDB_FEATURE_FLAGS?.[flag];
   }
   return FEATURE_FLAGS[flag];
 };
 
-export const getFeatureFlagsScript = (nonce: string) => {
+export const getFeatureFlagsScript = (nonce: string): string => {
   return `
     <script nonce="${nonce}">window['MDB_FEATURE_FLAGS']=${JSON.stringify(
     FEATURE_FLAGS

src/logging.ts

@@ -57,7 +57,7 @@ class Logger implements ILogger {
     );
   }
 
-  private append(type: string, message: string) {
+  private append(type: string, message: string): void {
     // https://code.visualstudio.com/api/references/vscode-api#window.createOutputChannel
 
     Logger.channel.appendLine(

src/storage/storageController.ts

@@ -91,7 +91,7 @@ export default class StorageController {
     return Promise.resolve();
   }
 
-  getUserIdentity() {
+  getUserIdentity(): { anonymousId: string } {
     let anonymousId = this.get(StorageVariables.GLOBAL_ANONYMOUS_ID);
 
     // The anonymousId becomes required with analytics-node v6.

src/test/runTest.ts

@@ -8,7 +8,7 @@ import { TEST_DATABASE_PORT } from './suite/dbTestHelper';
 
 // More information on vscode specific tests: https://github.com/microsoft/vscode-test
 
-async function startTestMongoDBServer() {
+async function startTestMongoDBServer(): Promise<MongoCluster> {
   console.log('Starting MongoDB server on port', TEST_DATABASE_PORT);
   return await MongoCluster.start({
     topology: 'standalone',
@@ -19,7 +19,7 @@ async function startTestMongoDBServer() {
 
 let testMongoDBServer: MongoCluster;
 
-function cleanup() {
+function cleanup(): void {
   console.log('Stopping MongoDB server on port', TEST_DATABASE_PORT);
   void testMongoDBServer?.close();
 }

src/test/setup-webview.ts

@@ -64,9 +64,9 @@ Object.assign(global, { TextDecoder, TextEncoder });
 
 (global as any).vscodeFake = {
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  postMessage: (message: unknown) => {},
+  postMessage: (message: unknown): void => {},
 };
 
-(global as any).acquireVsCodeApi = () => {
+(global as any).acquireVsCodeApi = (): any => {
   return (global as any).vscodeFake;
 };