Skip to content

build(packaging): align cargo binary identities - #334

Draft
sandersonstabo wants to merge 4 commits into
agent/backend-conversation-commit-notifier-port-55fdb03from
agent/portable-cargo-binary-identity-port-17d9694
Draft

build(packaging): align cargo binary identities#334
sandersonstabo wants to merge 4 commits into
agent/backend-conversation-commit-notifier-port-55fdb03from
agent/portable-cargo-binary-identity-port-17d9694

Conversation

@sandersonstabo

@sandersonstabo sandersonstabo commented Aug 29, 2026

Copy link
Copy Markdown
Collaborator

Dependency

Scope

  • Freeze the four approved Cargo binary output identities with a source-only acceptance test.
  • Keep the stable launcher as ae.
  • Rename the installer output from ae-installer to installer.
  • Make the desktop and backend Cargo outputs explicit as editor and forge.
  • Register the focused packaging test in package and root lint/format/test aggregates.

Exclusions

  • Does not add the four Bazel binary targets; that is the next packaging packet.
  • Does not assemble/install archives, alter versioned layout, sign artifacts, or change runtime behavior.
  • Cargo package names remain internal crate identities.

Verification

Exact-head jobs=1 gate on 9c32c38bf0a4c88cedd6654b730a89bcbe844447:

  • //tests/packaging:cargo_binary_identity_test build
  • focused uncached test
  • fresh direct Rust Clippy aspect
  • //:format_test
  • cargo metadata --locked --offline --no-deps
  • exact seven-path hash/status guards
  • Bazel shutdown exit 0

Receipt: D:/artisan-editor-gate-logs/portable-cargo-binary-identity-20260829-gate2/receipt.json.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Aligns the four Cargo binary output identities so packaging no longer relies on Cargo defaults. The installer binary is renamed from ae-installer to installer, and the frontend and backend binaries are now explicit editor and forge; the launcher remains ae. Anything referencing ae-installer must update to installer.

Test coverage

  • Adds a source-only test that parses each manifest's [[bin]] block and rejects ambiguous or invalid identities.
  • Registers the test in the root lint, format, and test aggregates.

Written for commit 9c32c38. Summary will update on new commits.

Review in cubic

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