From fa41d90c2d14fdb69dea2329752ef75e087e51f2 Mon Sep 17 00:00:00 2001 From: Steven Enamakel Date: Tue, 10 Feb 2026 16:41:29 +0530 Subject: [PATCH] Update subproject commit reference and enhance GitHub workflows - Updated the subproject commit reference in the skills directory. - Modified the build workflow to enable submodule fetching. - Standardized quotation styles in the package-and-publish workflow for consistency. - Refactored the `setSkillOAuthCredential` function for improved readability. - Added an attribute to the `Error` variant in the `SkillMessage` enum to suppress warnings for unused code. --- .github/workflows/build.yml | 1 + .github/workflows/package-and-publish.yml | 24 +++++++++++------------ skills | 2 +- src-tauri/src/runtime/types.rs | 1 + src/store/skillsSlice.ts | 5 ++++- 5 files changed, 19 insertions(+), 14 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 20b74bb96..e782a7e3f 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -21,6 +21,7 @@ jobs: uses: actions/checkout@v4 with: fetch-depth: 1 + submodules: true - name: Setup Node.js 24.x uses: actions/setup-node@v4 diff --git a/.github/workflows/package-and-publish.yml b/.github/workflows/package-and-publish.yml index fa2b0a5c0..c10f56289 100644 --- a/.github/workflows/package-and-publish.yml +++ b/.github/workflows/package-and-publish.yml @@ -14,7 +14,7 @@ on: branches: - develop workflow_run: - workflows: ["Version Bump"] + workflows: ['Version Bump'] types: - completed branches: @@ -169,15 +169,15 @@ jobs: fail-fast: false matrix: settings: - - platform: "macos-latest" - args: "--target aarch64-apple-darwin" - target: "aarch64-apple-darwin" - - platform: "macos-latest" - args: "--target x86_64-apple-darwin" - target: "x86_64-apple-darwin" - - platform: "ubuntu-22.04" - args: "" - target: "" + - platform: 'macos-latest' + args: '--target aarch64-apple-darwin' + target: 'aarch64-apple-darwin' + - platform: 'macos-latest' + args: '--target x86_64-apple-darwin' + target: 'x86_64-apple-darwin' + - platform: 'ubuntu-22.04' + args: '' + target: '' runs-on: ${{ matrix.settings.platform }} steps: - name: Checkout @@ -197,7 +197,7 @@ jobs: uses: actions/setup-node@v4 with: node-version: 24.x - cache: "yarn" + cache: 'yarn' - name: Install Rust stable uses: dtolnay/rust-toolchain@stable @@ -320,7 +320,7 @@ jobs: # macOS 10.15+ required for std::filesystem used by llama.cpp MACOSX_DEPLOYMENT_TARGET: ${{ matrix.settings.platform == 'macos-latest' && '10.15' || '' }} with: - args: "-c ${{ steps.config-overrides.outputs.json }} ${{ matrix.settings.args }}" + args: '-c ${{ steps.config-overrides.outputs.json }} ${{ matrix.settings.args }}' includeDebug: ${{ needs.get-version.outputs.should-publish == '' && inputs.forceRelease != 'true' }} includeRelease: ${{ needs.get-version.outputs.should-publish != '' || inputs.forceRelease == 'true' }} # TDLib dylibs are now bundled natively via build.rs + tauri.conf.json macOS.frameworks diff --git a/skills b/skills index 8030aef02..2dbb87011 160000 --- a/skills +++ b/skills @@ -1 +1 @@ -Subproject commit 8030aef027b7227b2fd63bf1e31bdf9d8e806383 +Subproject commit 2dbb87011a097086fa82f79d6d559a315b0b3c19 diff --git a/src-tauri/src/runtime/types.rs b/src-tauri/src/runtime/types.rs index 7616e6065..3beb3d646 100644 --- a/src-tauri/src/runtime/types.rs +++ b/src-tauri/src/runtime/types.rs @@ -87,6 +87,7 @@ pub enum SkillMessage { reply: tokio::sync::oneshot::Sender>, }, /// Notify the skill of an error from an async operation. + #[allow(dead_code)] Error { error_type: String, message: String, diff --git a/src/store/skillsSlice.ts b/src/store/skillsSlice.ts index b1d7f1f8a..fcf3b2c7a 100644 --- a/src/store/skillsSlice.ts +++ b/src/store/skillsSlice.ts @@ -58,7 +58,10 @@ const skillsSlice = createSlice({ } }, - setSkillOAuthCredential(state, action: PayloadAction<{ skillId: string; credential: OAuthCredential | undefined }>) { + setSkillOAuthCredential( + state, + action: PayloadAction<{ skillId: string; credential: OAuthCredential | undefined }> + ) { const { skillId, credential } = action.payload; if (state.skills[skillId]) { state.skills[skillId].oauthCredential = credential;