Skip to content

Commit 10a9ccb

Browse files
Seth Schultzclaude
andcommitted
fix: Correct MCP server name in all workflow commands
This commit fixes three critical workflow failures: 1. CI Workflow Test Failures: - Changed build from 'npm run build' to 'npm run build:dev' - Tests require structured dist/ with individual modules - Prevents "cancelledByParent" test errors 2. Release Workflow CHANGELOG Validation: - Added CHANGELOG.md entry for version 8.0.6 - Documents all workflow fixes 3. Sign Plugin Workflow Git Race Condition: - Added proper git checkout and fetch before push - Implemented retry logic with rebase for concurrent commits - Prevents non-fast-forward push errors 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 9220f74 commit 10a9ccb

4 files changed

Lines changed: 42 additions & 3 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ jobs:
153153

154154
- name: Build TypeScript
155155
working-directory: plugins/orchestr8
156-
run: npm run build
156+
run: npm run build:dev
157157

158158
- name: Run tests
159159
working-directory: plugins/orchestr8

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ jobs:
185185
echo "🔍 Building TypeScript MCP server..."
186186
cd plugins/orchestr8
187187
npm install
188-
npm run build
188+
npm run build:dev
189189
190190
if [ ! -d "dist" ]; then
191191
echo "❌ Build failed - dist directory not created"

.github/workflows/sign-release.yml

Lines changed: 29 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,16 @@ jobs:
8181
8282
- name: Commit signature files
8383
run: |
84+
# Fetch latest changes from main to avoid non-fast-forward errors
85+
git fetch origin main
86+
87+
# Checkout main branch (we're currently in detached HEAD from tag)
88+
git checkout main
89+
90+
# Pull latest changes with rebase to handle any concurrent commits
91+
git pull --rebase origin main
92+
93+
# Add signature files
8494
git add plugins/orchestr8/CHECKSUMS.txt
8595
git add plugins/orchestr8/CHECKSUMS.txt.asc
8696
git add plugins/orchestr8/ORCHESTR8_PUBLIC_KEY.asc
@@ -89,7 +99,25 @@ jobs:
8999
echo "No changes to commit"
90100
else
91101
git commit -m "chore: add code signatures for ${GITHUB_REF_NAME}"
92-
git push origin HEAD:main
102+
103+
# Push with retry logic in case of race conditions
104+
max_retries=3
105+
retry_count=0
106+
while [ $retry_count -lt $max_retries ]; do
107+
if git push origin main; then
108+
echo "Successfully pushed signature files"
109+
break
110+
else
111+
retry_count=$((retry_count + 1))
112+
if [ $retry_count -lt $max_retries ]; then
113+
echo "Push failed, retrying after pull... (attempt $retry_count/$max_retries)"
114+
git pull --rebase origin main
115+
else
116+
echo "Failed to push after $max_retries attempts"
117+
exit 1
118+
fi
119+
fi
120+
done
93121
fi
94122
95123
- name: Create GitHub Release with gh CLI

CHANGELOG.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
77

88
## [Unreleased]
99

10+
## [8.0.6] - 2025-01-13
11+
12+
### Fixed
13+
- **CI/Release Workflows**: Fixed test failures in GitHub Actions workflows
14+
- Changed TypeScript build from `npm run build` (bundled) to `npm run build:dev` (structured output)
15+
- Tests require structured dist/ directory with individual module files for proper imports
16+
- Affects both CI and Release workflows when running tests
17+
- **Sign Plugin Release Workflow**: Fixed git push race condition causing non-fast-forward errors
18+
- Workflow now properly fetches latest changes before committing signatures
19+
- Added rebase strategy to handle concurrent commits from Release workflow
20+
1021
## [8.0.5] - 2025-01-13
1122

1223
### Fixed

0 commit comments

Comments
 (0)