Skip to content

Conversation

@alexweininger
Copy link
Member

Needed for signing the helper extension for /azure which lives in a subdirectory of the root extension.

AI summary:

This pull request updates the Azure Pipelines signing workflow to support specifying a custom working directory for extension signing operations. This makes the pipeline more flexible and robust when handling extensions located in subdirectories or non-standard locations.

Pipeline configuration improvements:

  • Added a new workingDirectory parameter to the sign.yml template, allowing customization of the directory where signing operations and file lookups occur.
  • Updated all relevant script and task paths in sign.yml to use the new workingDirectory parameter, ensuring that file operations such as manifest generation, signing, and verification are correctly scoped to the specified directory. [1] [2] [3]

Integration with main pipeline:

  • Passed the workingDirectory parameter from the main pipeline stage configuration to the signing template, enabling end-to-end support for directory customization.

@alexweininger alexweininger requested a review from a team as a code owner December 9, 2025 19:47
condition: and(succeeded(), eq(variables['signprojExists'], True))
name: package
displayName: "\U0001F449 Get extension info from package.json"
workingDirectory: $(Build.SourcesDirectory)
Copy link
Contributor

Choose a reason for hiding this comment

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

It's a little confusing why this step happens at the root, but subsequent ones at $(Build.SourcesDirectory)/${{ parameters.workingDirectory }}

Copy link
Member Author

Choose a reason for hiding this comment

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

Weird because it seems like it worked fine but I would've assumed this would break it https://dev.azure.com/devdiv/DevDiv/_build/results?buildId=12924123&view=artifacts&pathAsName=false&type=publishedArtifacts

@alexweininger
Copy link
Member Author

alexweininger commented Dec 10, 2025

Got this working now and tested for RGs, and both extensions in /azure. Doesn't require changes to the signproj file

@alexweininger alexweininger merged commit ef4ed15 into main Dec 11, 2025
4 checks passed
@alexweininger alexweininger deleted the alex/signing-dir branch December 11, 2025 16:00
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.

3 participants