Skip to content

Commit 9ba51d9

Browse files
Ajit Pratap Singhclaude
authored andcommitted
fix(ci): add job timeouts and improve binary resolution logging
Add timeout-minutes to build (15m) and publish (10m) jobs to prevent hangs. Log when bundled binary check fails for easier debugging. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 81d1238 commit 9ba51d9

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

.github/workflows/vscode-publish.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ jobs:
1212
build:
1313
name: Build VSIX (${{ matrix.target }})
1414
runs-on: ubuntu-latest
15+
timeout-minutes: 15
1516
strategy:
1617
matrix:
1718
include:
@@ -90,6 +91,7 @@ jobs:
9091
name: Publish to Marketplace
9192
needs: build
9293
runs-on: ubuntu-latest
94+
timeout-minutes: 10
9395
steps:
9496
- name: Setup Node
9597
uses: actions/setup-node@v4

vscode-extension/src/extension.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,8 @@ async function getBinaryPath(): Promise<string> {
5757
await fs.promises.access(bundledPath, process.platform === 'win32' ? fs.constants.F_OK : fs.constants.X_OK);
5858
return bundledPath;
5959
} catch {
60-
// Bundled binary not found or not executable, fall through
60+
// Bundled binary not found or not executable, fall through to PATH lookup
61+
outputChannel?.appendLine(`Bundled binary not found at ${bundledPath}, falling back to PATH`);
6162
}
6263
}
6364

0 commit comments

Comments
 (0)