Skip to content

fix(simply-package): sort package aliases in dependencies manage - #139

Merged
ClayChipps merged 1 commit into
mainfrom
fix/dependencies-manage-sort-package-aliases
Aug 28, 2026
Merged

fix(simply-package): sort package aliases in dependencies manage#139
ClayChipps merged 1 commit into
mainfrom
fix/dependencies-manage-sort-package-aliases

Conversation

@ClayChipps

Copy link
Copy Markdown
Contributor

Summary

  • `sf simply package dependencies manage` now writes `packageAliases` in `sfdx-project.json` sorted alphabetically by key, instead of leaving newly added/updated aliases in insertion order.
  • Sorting happens right before the write in `applyChanges` (`sfdxProjectService.ts`).

Test plan

  • Added a unit test in `sfdxProjectService.test.ts` asserting written `packageAliases` keys come out sorted.
  • `npx vitest run` for `simply-package`'s `sfdxProjectService` and `dependencies/manage` test files — all passing.
  • `npx tsc --noEmit` and `eslint` on changed files — clean.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

https://claude.ai/code/session_014oTGUGohBiqrzETHCs25AB

Writes sfdx-project.json's packageAliases alphabetically by key after
sf simply package dependencies manage applies changes, instead of
leaving them in insertion order.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014oTGUGohBiqrzETHCs25AB
@ClayChipps
ClayChipps enabled auto-merge August 28, 2026 21:15
@ClayChipps
ClayChipps merged commit 7f80d29 into main Aug 28, 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.

1 participant