Skip to content

Add npm, NuGet, and PyPI distribution for the xcrs binary - #47

Closed
setoelkahfi with Copilot wants to merge 16 commits into
add-xcrs-npm-nuget-pypi-distributionfrom
copilot/sync-with-latest-development
Closed

Add npm, NuGet, and PyPI distribution for the xcrs binary#47
setoelkahfi with Copilot wants to merge 16 commits into
add-xcrs-npm-nuget-pypi-distributionfrom
copilot/sync-with-latest-development

Conversation

Copilot AI commented Aug 9, 2026

Copy link
Copy Markdown

Related Issue

Description

This branch was behind development and had drifted on release metadata. The update brings the PR current with the latest release/tooling changes and fixes the version-sync path so the xcrs package manifests stay aligned with the workspace version.

  • Development sync

    • Merged the latest development into the PR branch
    • Pulled in current release/versioning updates already landed on the base branch
  • Release metadata alignment

    • Updated xcrs npm and NuGet release metadata from 0.5.0 to 0.5.1
    • Kept the standalone xcrs distribution artifacts consistent with the workspace release version
  • Version sync hardening

    • Replaced the workspace dependency version regex with section-aware entry parsing
    • Handles reordered version / path fields
    • Handles multiline inline-table formatting
    • Fails explicitly on malformed dependency entries instead of silently skipping them
if (entry.includes('path = "crates/') && entry.includes("version = ")) {
  return entry.replace(/version\s*=\s*"[^"]+"/g, `version = "${version}"`);
}

Type of Change

  • Bug fix
  • New feature
  • Refactor (no functional change)
  • Documentation update
  • CI / tooling change

How Has This Been Tested?

Validated the merged branch against the relevant release/versioning checks and CI-equivalent workspace/package builds.

  • cargo test --all-features
  • Manual testing (smb <command>)

Checklist

  • cargo check --workspace passes
  • cargo clippy --workspace --tests -- -D warnings passes
  • cargo fmt --all -- --check passes
  • No new unwrap() or expect() calls in production code
  • No new mod.rs files introduced
  • New dependencies added to root Cargo.toml and inherited with { workspace = true }
  • Error messages are user-friendly and surfaced to the terminal

Release Notes

  • Improved xcrs release metadata synchronization so npm and NuGet artifacts stay aligned with the current workspace version after syncing with development.

setoelkahfi and others added 16 commits August 3, 2026 11:32
xcrs previously had no --version flag; `xcrs --version` errored. Add
`#[command(version)]` so it reports its crate version, matching `smb`
and the version recorded in the MCP registry.

Also document that `xcrs` is installed via `cargo install xcrs` from
crates.io, not Homebrew, since `brew install smbcloud-cli` ships only the
`smb` binary.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…tall-docs

Add --version flag to xcrs and document cargo install
…ith-latest-development

Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Co-authored-by: setoelkahfi <1797197+setoelkahfi@users.noreply.github.com>
Copilot AI requested a review from setoelkahfi August 9, 2026 14:24
@setoelkahfi setoelkahfi closed this Aug 9, 2026
An error occurred while trying to automatically change base from add-xcrs-npm-nuget-pypi-distribution to development August 9, 2026 15:10
@setoelkahfi
setoelkahfi deleted the copilot/sync-with-latest-development branch August 10, 2026 07:20
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.

2 participants