fix(pipeline): remove temp branch pin left over from PR #9453 testing

[Elaina-Lee]

Commit Type

• feature - New functionality
• fix - Bug fix
• refactor - Code restructuring without behavior change
• perf - Performance improvement
• docs - Documentation update
• test - Test-related changes
• chore - Maintenance/tooling

Risk Level

• Low - Minor changes, limited scope
• Medium - Moderate changes, some user impact
• High - Major changes, significant user/system impact

What & Why

PR #9453 temporarily pinned the LogicAppsUX template repository resource in .azure-pipelines/1esmain.yml to refs/heads/elaina/update-pipeline so test runs would exercise the CFS registry changes in setup.yml. The pin was marked TEMP (testing only, remove before merge) but was accidentally left in when the PR merged. This PR removes the comment block and the ref: line, restoring the pre-#9453 behavior where template references like setup.yml@LogicAppsUX resolve to the default branch (main).

Impact of Change

• Users: None.
• Developers: None.
• System: The 1ES release pipeline resolves its @LogicAppsUX templates from main again instead of the now-merged feature branch. Since elaina/update-pipeline was merged into main, both refs currently have identical template content, so there is no behavior change today — this removes the risk of the pipeline silently drifting or breaking if the feature branch is deleted or diverges.

Test Plan

• Unit tests added/updated
• E2E tests added/updated
• Manual testing completed
• Tested in: Verified the diff exactly reverts the block to its pre-fix(pipeline): route npm/pnpm package restores through CFS for SFI Network Isolation (SR21) #9453 state (git show 6fcbd144^:.azure-pipelines/1esmain.yml); YAML-only change to the ADO pipeline resource definition, not exercised by GitHub CI.

Contributors

Screenshots/Videos

[github-actions]

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

✅ PR Title

• Current: fix(pipeline): remove temp branch pin left over from PR #9453 testing
• Issue: None — uses a valid fix(scope): prefix and is clear and descriptive.
• Recommendation: No change needed.

---

✅ Commit Type

• Exactly one type selected (fix), which matches the change (removing an accidentally-merged temporary pin).
• Note: Good — single type, no overloading.

---

✅ Risk Level

• Low is selected in the body and the risk:low label is present, and this matches the advised estimate. The change only removes a temporary ref: pin from an ADO pipeline resource definition (.azure-pipelines/1esmain.yml). It is CI/pipeline automation that does not ship to users, does not touch security/auth, and does not alter repo governance/permissions — correctly low.

---

✅ What & Why

• Current: Clear explanation that PR fix(pipeline): route npm/pnpm package restores through CFS for SFI Network Isolation (SR21) #9453 left a TEMP (testing only) branch pin in place, and this PR removes it to restore default-branch template resolution.
• Issue: None.
• Recommendation: No change needed.

---

✅ Impact of Change

• All three audiences addressed (Users: none, Developers: none, System: 1ES pipeline resolves @LogicAppsUX templates from main again).
• Recommendation:
  • Users: As stated — none.
  • Developers: As stated — none.
  • System: As stated — clear and accurate.

---

✅ Test Plan

• Manual testing is checked with an adequate explanation: this is a YAML-only ADO pipeline resource change not exercised by GitHub CI, verified by comparing against the pre-fix(pipeline): route npm/pnpm package restores through CFS for SFI Network Isolation (SR21) #9453 content. Automated tests correctly do not apply.

---

⚠️ Contributors

• Section is blank. Not required, but consider crediting any reviewers/PMs who helped identify the leftover pin.

---

✅ Screenshots/Videos

• Not required — no UI files (libs/designer-ui/src/, libs/designer/src/, apps/vs-code-react/src/) are touched. This is a pipeline config change only.

---

Summary Table

Section	Status	Recommendation
Title	✅	No change needed
Commit Type	✅	No change needed
Risk Level	✅	low is correct
What & Why	✅	No change needed
Impact of Change	✅	No change needed
Test Plan	✅	Manual testing justified
Contributors	⚠️	Optionally credit collaborators
Screenshots/Videos	✅	N/A — no visual changes

---

All required checks pass. This PR is compliant and ready to merge. Consider removing the needs-pr-update label.

---

Powered by: Copilot CLI (claude-opus-4.8) | Last updated: Wed, 29 Jul 2026 20:43:29 GMT

[github-actions]

📊 Coverage Check

No source files changed in this PR.

[Copilot]

Pull request overview

This PR restores the intended Azure DevOps pipeline behavior by removing a temporary branch pin on the LogicAppsUX GitHub repository resource, so setup.yml@LogicAppsUX templates resolve from the default branch (main) again.

Changes:

• Remove the temporary comment block and ref: refs/heads/elaina/update-pipeline from the LogicAppsUX repository resource in the 1ES pipeline YAML.