From 5a305f4853e107eeb0222b67fd2ad8a7621f175c Mon Sep 17 00:00:00 2001 From: Cristian Dominguez Date: Thu, 21 Aug 2025 17:11:17 -0300 Subject: [PATCH 1/7] chore: workflows cleanup --- .github/workflows/test.yml | 6 ++---- .husky/commit-msg | 4 ---- .husky/pre-commit | 4 ---- .husky/pre-push | 4 ---- 4 files changed, 2 insertions(+), 16 deletions(-) delete mode 100755 .husky/commit-msg delete mode 100755 .husky/pre-commit delete mode 100755 .husky/pre-push diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 792f445e..b59dd95a 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -11,12 +11,10 @@ jobs: # Since the Windows unit tests take much longer, we run the linux unit tests first and then run the windows unit tests in parallel with NUTs linux-unit-tests: needs: yarn-lockfile-check -# uses: salesforcecli/github-workflows/.github/workflows/unitTestsLinux.yml@main - uses: ./.github/workflows/tempUnitTestsLinux.yml # temporary replacement + uses: salesforcecli/github-workflows/.github/workflows/unitTestsLinux.yml@cd/remove-wireit-workaround windows-unit-tests: needs: linux-unit-tests -# uses: salesforcecli/github-workflows/.github/workflows/unitTestsWindows.yml@main - uses: ./.github/workflows/tempUnitTestsWindows.yml # temporary replacement + uses: salesforcecli/github-workflows/.github/workflows/unitTestsWindows.yml@cd/remove-wireit-workaround # Uncomment to enable NUT testing in Github Actions # nuts: diff --git a/.husky/commit-msg b/.husky/commit-msg deleted file mode 100755 index c8e9f1d8..00000000 --- a/.husky/commit-msg +++ /dev/null @@ -1,4 +0,0 @@ -#!/bin/sh -. "$(dirname "$0")/_/husky.sh" - -yarn commitlint --edit diff --git a/.husky/pre-commit b/.husky/pre-commit deleted file mode 100755 index 4fbfe02f..00000000 --- a/.husky/pre-commit +++ /dev/null @@ -1,4 +0,0 @@ -#!/bin/sh -. "$(dirname "$0")/_/husky.sh" - -yarn lint && yarn pretty-quick --staged diff --git a/.husky/pre-push b/.husky/pre-push deleted file mode 100755 index 56b63c31..00000000 --- a/.husky/pre-push +++ /dev/null @@ -1,4 +0,0 @@ -#!/bin/sh -. "$(dirname "$0")/_/husky.sh" - -yarn build && yarn test From 055f69b31a176689960c3ba06f75edcf203de402 Mon Sep 17 00:00:00 2001 From: Cristian Dominguez Date: Thu, 21 Aug 2025 17:14:19 -0300 Subject: [PATCH 2/7] chore: remove temp workflows --- .github/workflows/tempUnitTestsLinux.yml | 72 ---------------------- .github/workflows/tempUnitTestsWindows.yml | 64 ------------------- 2 files changed, 136 deletions(-) delete mode 100644 .github/workflows/tempUnitTestsLinux.yml delete mode 100644 .github/workflows/tempUnitTestsWindows.yml diff --git a/.github/workflows/tempUnitTestsLinux.yml b/.github/workflows/tempUnitTestsLinux.yml deleted file mode 100644 index 1c9fd5e4..00000000 --- a/.github/workflows/tempUnitTestsLinux.yml +++ /dev/null @@ -1,72 +0,0 @@ -# THIS FILE IS A MODIFIED VERSION OF https://github.com/salesforcecli/github-workflows/blob/main/.github/workflows/unitTestsLinux.yml -# THIS SHOULD BE TEMPORARILY USED JUST UNTIL WE CAN GET THINGS WORKING AGAIN - -on: - workflow_call: - inputs: - skipTsDepCheck: - type: boolean - required: false - default: false - description: skip `prevent-typescript-dependency`. Use it for devDeps that ship TS - -jobs: - determine-node-versions: - outputs: - nodeVersions: ${{ steps.determine-node-versions.outputs.nodeVersions }} - runs-on: ubuntu-latest - steps: - - uses: salesforcecli/github-workflows/.github/actions/determineNodeVersions@main - id: determine-node-versions - with: - nodeVersionOverride: ${{ vars.NODE_VERSION_OVERRIDE }} # default is 'lts/*' - nodeDisableVersions: ${{ vars.NODE_DISABLE_VERSIONS }} - - prevent-typescript-dependency: - if: ${{ inputs.skipTsDepCheck == false }} - uses: salesforcecli/github-workflows/.github/workflows/preventTypescriptDep.yml@main - - linux-unit-tests: - needs: determine-node-versions - strategy: - matrix: - node_version: ${{ fromJSON(needs.determine-node-versions.outputs.nodeVersions) }} - fail-fast: false - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v4 - - - uses: actions/setup-node@v4 - with: - node-version: ${{ matrix.node_version }} - cache: yarn - -# - uses: google/wireit@setup-github-actions-caching/v2 -# continue-on-error: true - - - name: Cache node modules - id: cache-nodemodules - uses: actions/cache@v4 - env: - cache-name: cache-node-modules - with: - path: "**/node_modules" - key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/yarn.lock') }} - - - uses: salesforcecli/github-workflows/.github/actions/yarnInstallWithRetries@main - if: ${{ steps.cache-nodemodules.outputs.cache-hit != 'true' }} - -# # This is a temporary workaround to ensure wireit is >= 0.14.12 -# # Once all plugins/libraries that use this workflow are updated, this can be removed -# # See: https://github.com/google/wireit/issues/1297#issuecomment-2794737569 -# - name: Install wireit -# run: yarn add wireit@^0.14.12 - - - run: yarn build - - - name: yarn test - uses: salesforcecli/github-workflows/.github/actions/retry@main - with: - command: yarn test - env: - SF_DISABLE_TELEMETRY: true \ No newline at end of file diff --git a/.github/workflows/tempUnitTestsWindows.yml b/.github/workflows/tempUnitTestsWindows.yml deleted file mode 100644 index 3be9f661..00000000 --- a/.github/workflows/tempUnitTestsWindows.yml +++ /dev/null @@ -1,64 +0,0 @@ -# THIS FILE IS A MODIFIED VERSION OF https://github.com/salesforcecli/github-workflows/blob/main/.github/workflows/unitTestsLinux.yml -# THIS SHOULD BE TEMPORARILY USED JUST UNTIL WE CAN GET THINGS WORKING AGAIN - -on: - workflow_call: - -jobs: - determine-node-versions: - outputs: - nodeVersions: ${{ steps.determine-node-versions.outputs.nodeVersions }} - runs-on: ubuntu-latest - steps: - - uses: salesforcecli/github-workflows/.github/actions/determineNodeVersions@main - id: determine-node-versions - with: - nodeVersionOverride: ${{ vars.NODE_VERSION_OVERRIDE }} # default is 'lts/*' - nodeDisableVersions: ${{ vars.NODE_DISABLE_VERSIONS }} - - windows-unit-tests: - needs: determine-node-versions - strategy: - matrix: - node_version: ${{ fromJSON(needs.determine-node-versions.outputs.nodeVersions) }} - fail-fast: false - runs-on: windows-latest - steps: - - run: git config --system core.longpaths true - - - uses: actions/checkout@v4 - -# - uses: google/wireit@setup-github-actions-caching/v2 -# continue-on-error: true - - - uses: actions/setup-node@v4 - with: - node-version: ${{ matrix.node_version }} - cache: yarn - - - name: Cache node modules - id: cache-nodemodules - uses: actions/cache@v4 - env: - cache-name: cache-node-modules - with: - path: "**/node_modules" - key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/yarn.lock') }} - - - uses: salesforcecli/github-workflows/.github/actions/yarnInstallWithRetries@main - if: ${{ steps.cache-nodemodules.outputs.cache-hit != 'true' }} - -# # This is a temporary workaround to ensure wireit is >= 0.14.12 -# # Once all plugins/libraries that use this workflow are updated, this can be removed -# # See: https://github.com/google/wireit/issues/1297#issuecomment-2794737569 -# - name: Install wireit -# run: yarn add wireit@^0.14.12 - - - run: yarn build - - - name: yarn test - uses: salesforcecli/github-workflows/.github/actions/retry@main - with: - command: yarn test - env: - SF_DISABLE_TELEMETRY: true \ No newline at end of file From a93db42218516267eb63a91ab575562079937f29 Mon Sep 17 00:00:00 2001 From: Cristian Dominguez Date: Thu, 21 Aug 2025 17:58:56 -0300 Subject: [PATCH 3/7] chore: dependabot only bump our pkgs --- .github/dependabot.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/dependabot.yml b/.github/dependabot.yml index 0ef035df..e125230f 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -1,7 +1,9 @@ version: 2 updates: - package-ecosystem: 'npm' - directory: '/' + directories: + - '/packages/mcp' + - '/packages/mcp-provider-api' schedule: interval: 'weekly' day: 'saturday' From d87fd2e0069c23501aeb08a9f9b52853c9e5bf81 Mon Sep 17 00:00:00 2001 From: Cristian Dominguez Date: Fri, 22 Aug 2025 15:50:51 -0300 Subject: [PATCH 4/7] chore: restore git hooks --- .husky/commit-msg | 4 ++++ .husky/pre-commit | 4 ++++ .husky/pre-push | 4 ++++ 3 files changed, 12 insertions(+) create mode 100644 .husky/commit-msg create mode 100644 .husky/pre-commit create mode 100644 .husky/pre-push diff --git a/.husky/commit-msg b/.husky/commit-msg new file mode 100644 index 00000000..c8e9f1d8 --- /dev/null +++ b/.husky/commit-msg @@ -0,0 +1,4 @@ +#!/bin/sh +. "$(dirname "$0")/_/husky.sh" + +yarn commitlint --edit diff --git a/.husky/pre-commit b/.husky/pre-commit new file mode 100644 index 00000000..4fbfe02f --- /dev/null +++ b/.husky/pre-commit @@ -0,0 +1,4 @@ +#!/bin/sh +. "$(dirname "$0")/_/husky.sh" + +yarn lint && yarn pretty-quick --staged diff --git a/.husky/pre-push b/.husky/pre-push new file mode 100644 index 00000000..56b63c31 --- /dev/null +++ b/.husky/pre-push @@ -0,0 +1,4 @@ +#!/bin/sh +. "$(dirname "$0")/_/husky.sh" + +yarn build && yarn test From 4fc2b5d44fec15000a31c8010adac84e3abcc0ad Mon Sep 17 00:00:00 2001 From: Cristian Dominguez Date: Mon, 25 Aug 2025 12:33:02 -0300 Subject: [PATCH 5/7] chore: add example pkg to dependabot config [skip ci] --- .github/dependabot.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/dependabot.yml b/.github/dependabot.yml index e125230f..4b831fd8 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -4,6 +4,7 @@ updates: directories: - '/packages/mcp' - '/packages/mcp-provider-api' + - '/packages/EXAMPLE-MCP-PROVIDER' schedule: interval: 'weekly' day: 'saturday' From d71743ebc0a6b21a1c48c9410bac78c5ec0c367a Mon Sep 17 00:00:00 2001 From: Cristian Dominguez <6853656+cristiand391@users.noreply.github.com> Date: Thu, 28 Aug 2025 16:38:22 -0300 Subject: [PATCH 6/7] Apply suggestion from @iowillhoit [skip ci] Co-authored-by: Willhoit --- .github/dependabot.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/dependabot.yml b/.github/dependabot.yml index 4b831fd8..1eb6f764 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -4,6 +4,7 @@ updates: directories: - '/packages/mcp' - '/packages/mcp-provider-api' + - '/packages/mcp-provider-dx-core' - '/packages/EXAMPLE-MCP-PROVIDER' schedule: interval: 'weekly' From d39e037e42dd79f02e6c9aeffa135bbd4559e548 Mon Sep 17 00:00:00 2001 From: Cristian Dominguez Date: Fri, 29 Aug 2025 10:19:36 -0300 Subject: [PATCH 7/7] chore: add dependabot automation note [skip ci] --- DEVELOPING.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/DEVELOPING.md b/DEVELOPING.md index 758d938c..7b257622 100644 --- a/DEVELOPING.md +++ b/DEVELOPING.md @@ -137,6 +137,10 @@ The server release workflow will automatically update to the latest provider ver - For server wiring details and how providers are loaded, see `packages/mcp/DEVELOPING.md`. +## Dependabot automation +We have Dependabot enabled in this repo for the the packages listed in `.github/dependabot.yml`, if a PR passes all tests it will be automatically merged (cron job checks PRs opens every day). +This automation is opt-in by default (just add your package dir to the dependabot config), just note that you can't enable dependabot PRs and disable automerges yet (will be addressed in the future) + ## Running the MCP server locally Quick options; see `packages/mcp/DEVELOPING.md` for full details and troubleshooting.