Skip to content

simplify(release): retire unpublished-crate distribution - #4266

Open
ScriptedAlchemy wants to merge 11 commits into
masterfrom
simplify/retire-crate-distribution
Open

ScriptedAlchemy wants to merge 11 commits into
masterfrom
simplify/retire-crate-distribution

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

The workspace maintained a second package/extract/recompile path for private Rust crates, although releases ship Bazel-built CLI archives, MCP bundles, and the npm SDK.

Remove the dispatch-only crate distribution workflow, its 998-line battery, package feature/profile/layout helpers, private Cargo package whitelists, and the extracted-crate dashboard embedding path. Remove the distribution-only binary override from the Rust test launcher. Keep production archive/MCPB packaging, checksums/signatures, immutable release recovery, installer coverage, and npm tarball conformance.

Move the existing MCP conformance journey and LSP initialize handshake to shipped-cli, using its production binary in temporary isolated profiles. Declare the packaged LSP tool's Python import root and add its existing framing/platform helper checks as a Bazel test so missing helper imports fail before the production smoke step.

Verification:

  • bazel test //scripts:packaged_lsp_bridge_test --test_output=errors passed.
  • Both bazel run //scripts:packaged_lsp_bridge and bazel run //scripts:mcp-conformance-smoke.sh start and reach their argument validation; the latter exported executable source target is runnable.
  • Reproduced ModuleNotFoundError: lib.process_control before the import-root fix; the corrected executable reaches its usage error.
  • git diff --check passed. Full production transport journeys run in hosted shipped-cli CI.

Closes #4197.
Closes #4218.

@changeset-bot

changeset-bot Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 353231f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy
ScriptedAlchemy marked this pull request as ready for review October 18, 2026 07:44

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Devin Review: 2 flags

Not posted on this PR by your GitHub settings — view them in Devin Review. (Configure)

Devin Review

…/retire-crate-distribution

# Conflicts:
#	.github/workflows/ci.yml
#	scripts/BUILD.bazel

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant