fix(mcpb): resolve binary naming conflict and update macOS runner

Critical fixes for MCPB bundle creation:

1. Binary Naming Conflict Resolution:
   - Binaries: mcpb-{platform} (no .mcpb extension)
   - Bundles: mcpb-{platform}.mcpb (ZIP with .mcpb extension)
   - Prevents bundle from overwriting binary during creation
   - Fixes 0-byte binary issue inside .mcpb bundles

2. macOS Runner Update:
   - Updated darwin-x64 from deprecated macos-13 to macos-15-intel
   - Addresses GitHub Actions deprecation warning

3. Workflow Updates:
   - Updated verify step to use new binary naming
   - Updated create-bundle step to use new binary naming
   - Bundle artifact paths remain unchanged (.mcpb extension)

This ensures MCPB bundles contain valid binaries and can be installed
in Claude Desktop.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Test

.github/workflows/release-mcpb.yml

@@ -17,7 +17,7 @@ jobs:
         include:
           - os: ubuntu-latest
             platform: linux-x64
-          - os: macos-13  # Intel Mac
+          - os: macos-15-intel  # Intel Mac (was macos-13, now deprecated)
             platform: darwin-x64
           - os: macos-latest  # Apple Silicon Mac (M1/M2)
             platform: darwin-arm64
@@ -52,11 +52,11 @@ jobs:
 
       - name: Verify binary
         run: |
-          python scripts/build_binary.py verify --binary dist/mcpb-${{ matrix.platform }}${{ matrix.platform == 'windows-x64' && '.exe' || '.mcpb' }} --version ${{ steps.get_version.outputs.version }}
+          python scripts/build_binary.py verify --binary dist/mcpb-${{ matrix.platform }}${{ matrix.platform == 'windows-x64' && '.exe' || '' }} --version ${{ steps.get_version.outputs.version }}
 
       - name: Create bundle
         run: |
-          python scripts/build_binary.py create-bundle --platform ${{ matrix.platform }} --binary dist/mcpb-${{ matrix.platform }}${{ matrix.platform == 'windows-x64' && '.exe' || '.mcpb' }} --version ${{ steps.get_version.outputs.version }} --output-dir dist
+          python scripts/build_binary.py create-bundle --platform ${{ matrix.platform }} --binary dist/mcpb-${{ matrix.platform }}${{ matrix.platform == 'windows-x64' && '.exe' || '' }} --version ${{ steps.get_version.outputs.version }} --output-dir dist
 
       - name: Upload artifact
         uses: actions/upload-artifact@v4

scripts/build_binary.py

@@ -217,7 +217,9 @@ def create_bundle(
         )
         command = f"server/mcpb-{platform_id}.exe"
     else:
-        server_binary_name = f"server/mcpb-{platform_id}.mcpb"
+        server_binary_name = (
+            f"server/mcpb-{platform_id}"  # No .mcpb extension inside bundle
+        )
         entry_point = server_binary_name
         command = entry_point
 
@@ -288,8 +290,7 @@ def build_binary(output_dir: Path, platform_id: Optional[str] = None) -> Path:
     binary_name = f"mcpb-{platform_id}"
     if platform_id.startswith("windows"):
         binary_name += ".exe"
-    else:
-        binary_name += ".mcpb"
+    # Don't add .mcpb here - only the bundle gets .mcpb extension
 
     # Find spec file
     spec_file = Path(__file__).parent.parent / "pyinstaller.spec"

src/code_indexer/__init__.py

@@ -6,5 +6,5 @@
 HNSW graph indexing (O(log N) complexity).
 """
 
-__version__ = "8.2.2"
+__version__ = "8.2.3"
 __author__ = "Seba Battig"

src/code_indexer/mcpb/bridge.py

@@ -217,9 +217,7 @@ def main():  # pragma: no cover
     # Handle --diagnose flag
     if args.diagnose:
         try:
-            result = diagnose_configuration(
-                config_path=args.config, use_env=True
-            )
+            result = diagnose_configuration(config_path=args.config, use_env=True)
             print(result.format_output())
             sys.exit(0)
         except Exception as e:

src/code_indexer/mcpb/config.py

@@ -6,7 +6,6 @@
 
 import logging
 import os
-import stat
 from dataclasses import dataclass
 from pathlib import Path
 from typing import Optional
@@ -111,7 +110,6 @@ def load_config(
 
         # Check file permissions (Story #517)
         file_stat = os.stat(path)
-        file_mode = stat.filemode(file_stat.st_mode)
         # Get octal permissions (last 3 digits)
         file_perms = file_stat.st_mode & 0o777
 

src/code_indexer/mcpb/manifest.py

@@ -60,9 +60,7 @@ class PlatformManifest(BaseModel):
     name: str = Field(..., description="Package name")
     version: str = Field(..., description="Version string")
     description: str = Field(..., description="Package description")
-    mcp_version: str = Field(
-        default="2024-11-05", description="MCP protocol version"
-    )
+    mcp_version: str = Field(default="2024-11-05", description="MCP protocol version")
     platforms: dict[Platform, BinaryMetadata] = Field(
         default_factory=dict, description="Platform-specific binaries"
     )

tests/mcpb/test_config.py

@@ -219,7 +219,9 @@ class TestHTTPSValidation:
     def test_config_requires_https_url(self):
         """Test that non-HTTPS URL raises validation error."""
         with pytest.raises(ValueError, match="server_url must use HTTPS"):
-            BridgeConfig(server_url="http://cidx.example.com", bearer_token="test-token")
+            BridgeConfig(
+                server_url="http://cidx.example.com", bearer_token="test-token"
+            )
 
     def test_config_allows_https_url(self):
         """Test that HTTPS URL is accepted."""

tests/mcpb/test_diagnostics.py

@@ -7,11 +7,9 @@
 import json
 import os
 import tempfile
-from unittest.mock import AsyncMock, MagicMock, patch
+from unittest.mock import patch
 
-import pytest
 
-from code_indexer.mcpb.config import BridgeConfig
 from code_indexer.mcpb.diagnostics import (
     DiagnosticsResult,
     diagnose_configuration,

tests/unit/configuration/test_legacy_config_detection.py

@@ -141,7 +141,6 @@ def test_reject_project_ports_config_in_json(self, tmp_path: Path):
         )
         assert "v8.0" in error_message or "8.0" in error_message
 
-
     def test_reject_invalid_embedding_provider(self, tmp_path: Path):
         """Test that non-voyageai embedding providers are rejected."""
         config_path = tmp_path / ".code-indexer" / "config.json"