Skip to content

Align release infrastructure with mcp-trino's working patterns#12

Merged
cjimti merged 6 commits into
mainfrom
housekeeping/build_updates
Jan 18, 2026
Merged

Align release infrastructure with mcp-trino's working patterns#12
cjimti merged 6 commits into
mainfrom
housekeeping/build_updates

Conversation

@cjimti

@cjimti cjimti commented Jan 18, 2026

Copy link
Copy Markdown
Member

Description

Align release infrastructure with mcp-trino's working patterns.

Updated files:

  • Dockerfile - Simplified for GoReleaser's dockers_v2 (uses linux/${TARGETARCH}/mcp-s3)
  • mcpb/build.sh - Rewritten to match mcp-trino (supports --use-dist flag)
  • mcpb/manifest.json - Updated to manifest_version 0.3 format with proper user_config
  • server.json - Updated to MCP Registry schema with PLACEHOLDER_SHA256 pattern
  • .github/workflows/release.yml - Complete rewrite matching mcp-trino's workflow
  • .goreleaser.yml - Use HOMEBREW_TAP_GITHUB_TOKEN env var, removed skip_upload: auto

Root cause of previous failures:

  1. Dockerfile tried multi-stage build, but GoReleaser's dockers_v2 only provides pre-built binaries
  2. mcpb/build.sh didn't support --use-dist flag needed in CI
  3. release.yml had wrong mcp-publisher install and server.json preparation steps
  4. .goreleaser.yml used wrong env var name for Homebrew token

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • Test improvement (new or updated tests)
  • Documentation update
  • Stability/performance improvement
  • Build/CI improvement

Note: New features are developed by maintainers only. See CONTRIBUTING.md for details.

Related Issues

Fixes release workflow failures:

Testing

  • goreleaser check passes
  • All files match mcp-trino's working patterns

Checklist

  • My code follows the project's style guidelines (make lint)
  • I have read CONTRIBUTING.md
  • This PR is focused and does not include unrelated changes

@cjimti cjimti merged commit 970dff3 into main Jan 18, 2026
5 checks passed
@codecov

codecov Bot commented Jan 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.12%. Comparing base (996d56c) to head (e64bc72).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage   82.12%   82.12%           
=======================================
  Files          33       33           
  Lines        2081     2081           
=======================================
  Hits         1709     1709           
  Misses        285      285           
  Partials       87       87           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cjimti cjimti deleted the housekeeping/build_updates branch February 1, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant