Skip to content

Conversation

@aholstrup1
Copy link
Collaborator

@aholstrup1 aholstrup1 commented Feb 6, 2026

❔What, Why & How

Problem
Right now, the "Publish to Environment" and "Create Online Dev. Environment" workflows will fail when no Authcontext is provided. AL-Go is supposed to use the DeviceCode flow when we're lacking an Authcontext but in order to run DownloadAndImportBcContainerHelper we need to import the DebugHelper.psm1. Right now we don't, so we see failures around "OutputDebug" not being recognized.

Proposed Solution
Both workflow use an inline powershell script. We could move this to its own AL-Go action where the script is invoked with Invoke-AlGoAction.ps1. That way we ensure that DebugLogHelper is imported.

Related to issue: #2113

✅ Checklist

  • Add tests (E2E, unit tests)
  • Update RELEASENOTES.md
  • Update documentation (e.g. for new settings or scenarios)
  • Add telemetry

@aholstrup1 aholstrup1 requested a review from a team as a code owner February 6, 2026 08:45
Copilot AI review requested due to automatic review settings February 6, 2026 08:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses issue #2113 where the "Create Online Dev. Environment" and "Publish to Environment" workflows fail when no AuthContext is provided due to missing DebugLogHelper imports. The solution extracts inline PowerShell authentication logic into a new reusable AL-Go action called CheckAuthContext, which is invoked using Invoke-AlGoAction.ps1 to ensure proper module imports.

Changes:

  • Created new CheckAuthContext action to handle authentication context verification and device code flow initialization
  • Updated CreateOnlineDevelopmentEnvironment and PublishToEnvironment workflows to use the new action
  • Added comprehensive unit tests for the new action

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Actions/CheckAuthContext/CheckAuthContext.ps1 Core script that checks for auth secrets and initiates device code flow if needed
Actions/CheckAuthContext/action.yaml Action definition with inputs and outputs, invokes script via Invoke-AlGoAction.ps1
Actions/CheckAuthContext/README.md Documentation for the new action including inputs, outputs, and usage
Tests/CheckAuthContext.Action.Test.ps1 Comprehensive unit tests covering secret discovery, fallback logic, and device login scenarios
RELEASENOTES.md Added issue #2113 to release notes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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