Skip to content

Fix cloud package artifact root layout - #189

Draft
valerymoskalenko wants to merge 1 commit into
fscpscollaborative:developfrom
ciellosinc:fix/cloud-package-pac-layout
Draft

valerymoskalenko wants to merge 1 commit into
fscpscollaborative:developfrom
ciellosinc:fix/cloud-package-pac-layout

Conversation

@valerymoskalenko

@valerymoskalenko valerymoskalenko commented Sep 3, 2026 •

Copy link
Copy Markdown

Preview: validation required before merge

This is a preview implementation. Validate the generated package in CI and with PAC CLI before merging into the develop branch.

References: https://github.com/fscpscollaborative/fscps.ado/issues/1

Change

  • Normalize the cloud-package source root when the package creator returns a CloudDeployablePackage wrapper directory.
  • Copy the package contents into the artifact directory so TemplatePackage.dll, PackageAssets, and the package metadata are at the expected root.
  • Add focused regression tests for direct-root, nested-root, and missing-path behavior.

Validation completed

  • Focused Pester tests: 3 passed, 0 failed.
  • PowerShell parsing passed for all touched files.
  • git diff --check passed.

Validation still required

The full solution compile could not run on the development machine because the .NET Framework 4.5.2 reference assemblies are not installed. CI validation must compile the solution and exercise the cloud-package path before merge.

The hard-coded solution filenames reported in issue #1 are generated inside the bundled Microsoft.PowerPlatform.VSShared.Util.dll; this preview addresses the extra archive-root nesting. Filename generation requires a follow-up change in the package-generation component if the binary cannot be replaced or configured.

This implementation was generated with assistance from the GPT-5.6 Luna model.

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