Skip to content

Fix broken CI, scan workflow, and add local bundle target#2

Merged
mgoldsborough merged 4 commits intomainfrom
fix-broken-ci-and-bundle
Mar 5, 2026
Merged

Fix broken CI, scan workflow, and add local bundle target#2
mgoldsborough merged 4 commits intomainfrom
fix-broken-ci-and-bundle

Conversation

@JoeCardoso13
Copy link
Contributor

Summary

  • Fix ci.yml bundle job: pip install mcpbnpx @anthropic-ai/mcpb pack (mcpb is npm, not pip)
  • Fix scan.yml: graceful skip when mpak-scanner isn't available yet
  • Fix pyproject.toml: ty>=0.1.0 doesn't exist → ty>=0.0.17
  • Add make bundle target for local bundle validation
  • Add deps/ to .gitignore and .mcpbignore

Closes #1

Test plan

  • CI passes (especially the bundle job)
  • make bundle produces a .mcpb file locally
  • make check passes (lint, typecheck, test)

@JoeCardoso13
Copy link
Contributor Author

PR covers template issues 1-4 at the bottom of NimbleBrainInc/contributor-toolkit#21 (comment)

@mgoldsborough
Copy link
Contributor

CI is broken.

mgoldsborough
mgoldsborough previously approved these changes Mar 5, 2026
JoeCardoso13 and others added 3 commits March 4, 2026 15:38
- ci.yml: replace `pip install mcpb && mcpb pack` with `npx @anthropic-ai/mcpb pack`
  (mcpb is an npm package, not pip)
- scan.yml: fix mcpb install, make mpak-scanner step skip gracefully since
  mpak-scanner is not yet an installable package (scanning happens server-side)
- pyproject.toml: fix ty version constraint (0.1.0 doesn't exist, use 0.0.17)
- Makefile: add `bundle` target for local bundle creation (vendors deps + mcpb pack)
- .gitignore/.mcpbignore: add deps/ directory used by bundle target

Closes #1
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mgoldsborough mgoldsborough force-pushed the fix-broken-ci-and-bundle branch from 64d3241 to cb507b0 Compare March 5, 2026 01:38
@mgoldsborough mgoldsborough merged commit d5eb59e into main Mar 5, 2026
4 checks passed
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.

Add build bundle command

2 participants