Skip to content

Add Azure Logic Apps Standard CLI skill - #5490

Merged
aaronpowell merged 2 commits into
github:mainfrom
wsilveiranz:add-azure-logic-apps-standard-cli
Oct 25, 2026
Merged

aaronpowell merged 2 commits into
github:mainfrom
wsilveiranz:add-azure-logic-apps-standard-cli

Conversation

@wsilveiranz

@wsilveiranz wsilveiranz commented Oct 25, 2026 •

Copy link
Copy Markdown

Pull Request Checklist

  • I have read and followed the contribution guidelines.
  • I have read the guidance for submissions involving paid services.
  • The new skill is in the correct directory and follows naming conventions.
  • The content is structured with valid frontmatter and bundled references.
  • I have exercised representative installed-skill scenarios with GitHub Copilot (see qualification limits below).
  • I have run npm run build and updated the generated skill catalog.
  • This pull request targets main.

Description

Adds azure-logic-apps-standard-cli, a self-contained skill for Azure Logic Apps
Standard developer workflows through the published Azure CLI logicapp extension.
Consumption-only workflow development is out of scope.

The skill supplies specialized command contracts and operational gates beyond generic
workflow JSON authoring: runtime readiness and process ownership, connector metadata
discovery, separate local/cloud identity authorization, native SDK testing prerequisites,
revision-guarded non-atomic changes, and truthful cancellation/replay outcomes.

The entry point routes agents to five mandatory bundled procedures for runtime,
connections/identity, live operations, testing, and packaging/deployment. All procedures
ship in the skill folder; there are no private wheel or external-skill dependencies.
Existing project layouts and optional workspaces are preserved, and managed identity
is the default for new connections unless explicitly overridden or unsupported.

Type of Contribution

  • New skill.

Validation and qualification limits

  • Repository skill validation and catalog generation completed locally.
  • Bundled relative links, YAML metadata, JSON examples, asset sizes, and source safeguard
    coverage were checked during preparation.
  • Example command paths/options were compared with installed logicapp 1.0.0b1 help.
  • Three fresh Copilot CLI 1.0.103 sessions exercised offline authoring, replay approval,
    and native-test consent using the installed skill. Exact prompts, observed reference
    reads/actions, outcomes, and limitations are recorded in the skill README.
  • The offline session made the requested one-value workflow edit and preserved Stateful
    kind. Replay/testing sessions loaded their mandatory bundled references and did not
    attempt unapproved execution or acceptance flags. Shell execution was denied by the
    evaluation harness; these are limited AI-driven smoke checks, not human acceptance
    testing, unrestricted-tool qualification, or proof of universal adherence.
  • The edited synthetic project passed strict offline validation. A real offline ZIP
    check confirmed host-root-relative exclusions remove custom fixtures, reports, and
    a settings template while retaining required host/workflow content.
  • No Azure resources were mutated, hosts started, workflows invoked, or SDK test packages
    restored as part of preparing this contribution.

Additional Notes

  • The extension baseline is 1.0.0b1 (preview). No production-readiness or universal
    platform/connector qualification is claimed. Existing core Azure CLI commands are
    distinguished from extension-owned preview commands.
  • Local/offline tasks do not authorize cloud mutation. Azure resources may incur charges;
    provisioning, deployment, access changes, and replay require explicit authorization.
  • Native testing requires separately reviewed SDK terms, project trust, and opt-in restore.
    SDK binaries are not bundled and the CLI license does not grant SDK redistribution rights.
  • This is practical community guidance for a Microsoft Azure service, not a product
    announcement, sales material, or a claim of official service endorsement.

By submitting this contribution, I agree to the repository's Code of Conduct and MIT
contribution terms.

Bundle mandatory procedures for runtime, connections, live operations, testing and deployment; add the generated catalog entry.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 96c19dee-940f-4833-abba-f610ef80c697
@wsilveiranz
wsilveiranz requested review from a team as code owners October 25, 2026 02:35
Copilot AI max review requested due to automatic review settings October 25, 2026 02:35
@github-actions github-actions Bot added new-submission PR adds at least one new contribution skills PR touches skills labels Oct 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 7 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

github-actions Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:medium — New or substantially changed resource without privileged execution
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access.

Why this tier
  • Adds 7 file(s), e.g. skills/azure-logic-apps-standard-cli/README.md
  • Changes 1358 lines (low-risk limit is 40)

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ✅ Passed Passed · logs
Skill validation ✅ Passed Passed · logs
Skill lint (vally) ✅ Passed Passed · logs
Risk scan ✅ Passed Passed · logs
Contributor reputation ✅ Passed Passed · logs
Duplicate resource scan ✅ Passed Passed · logs
PR quality signal ⏭️ Skipped Skipped by its workflow · logs

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s)
  • No staffed reviewer pool owns these files yet; any reviewer with write access counts as the domain reviewer.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for 0770499 · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

✅ All checks passed

Scope Checked
Skills 1
Agents 0
Total 1
Severity Count
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ ✅ azure-logic-apps-standard-cli (2/2 checks passed)
ℹ️ ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️ ✓ spec-compliance: All spec checks passed.
ℹ️ ✓ [valid-refs] All file references across 1 skill(s) are valid.
ℹ️ ✓ valid-refs: All file references resolve to existing files within the skill directory.
ℹ️ 1 skill(s) linted, 1 passed
Full linter output
### Linting skills/azure-logic-apps-standard-cli
✅ azure-logic-apps-standard-cli (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

@github-actions github-actions Bot added ready-for-review Submission passed intake validation and is ready for maintainer review and removed awaiting-automation labels Oct 25, 2026
@wsilveiranz wsilveiranz changed the title Add Azure Logic Apps Standard CLI skill 🤖🤖🤖 Add Azure Logic Apps Standard CLI skill Oct 25, 2026
@wsilveiranz

Copy link
Copy Markdown
Author

/request-review

@github-actions

Copy link
Copy Markdown
Contributor

🙋 Added needs-reviewer. Reviewer routing is assigning a reviewer now.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Representative Copilot usage remains unvalidated, and deployment exclusion paths need correction.

3 open issues
What changed in this PR

Adds an Azure Logic Apps Standard CLI skill to the repository’s skill collection, with preview-specific command guidance and explicit operational safeguards.

Changes:

  • Defines skill activation, procedure routing, scaffolding, and validation rules.
  • Bundles five procedures covering runtime, connections, live operations, testing, and deployment.
  • Adds package documentation and a generated skill catalog entry.
File Description
skills/​azure-logic-apps-standard-cli/​SKILL.md Defines metadata, routing, authoring guidance, and safety gates.
skills/​azure-logic-apps-standard-cli/​references/​testing.md Documents native test prerequisites, consent, execution, and results.
skills/​azure-logic-apps-standard-cli/​references/​packaging-deployment.md Covers exclusions, archive inspection, and authorized deployment.
skills/​azure-logic-apps-standard-cli/​references/​local-runtime.md Covers dependencies, readiness, process ownership, and shutdown.
skills/​azure-logic-apps-standard-cli/​references/​live-operations.md Guides guarded workflow changes, execution, diagnosis, and replay.
skills/​azure-logic-apps-standard-cli/​references/​connections.md Covers connector discovery, identity, connections, and access policies.
skills/​azure-logic-apps-standard-cli/​README.md Documents package installation, maintenance, provenance, and qualification limits.
docs/​README.skills.md Registers the skill and its bundled references.

Review effort: Max

Comment thread skills/azure-logic-apps-standard-cli/README.md
Comment thread skills/azure-logic-apps-standard-cli/references/packaging-deployment.md Outdated
Comment thread skills/azure-logic-apps-standard-cli/references/testing.md Outdated
Record installed-skill behavioral checks and correct host-relative packaging and native report paths.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 96c19dee-940f-4833-abba-f610ef80c697
Copilot AI max review requested due to automatic review settings October 25, 2026 20:17
@github-actions github-actions Bot added awaiting-automation and removed ready-for-review Submission passed intake validation and is ready for maintainer review labels Oct 25, 2026
@github-actions github-actions Bot added ready-for-review Submission passed intake validation and is ready for maintainer review and removed awaiting-automation labels Oct 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Version-specific preview contracts for cloud mutations and native testing need maintainer confirmation beyond the documented limited smoke checks.

No open issues 🎉

1 previously missed issue

In code that hasn't changed since last review

Low severity Update README introduction to remain accurate after publication

skills/​azure-logic-apps-standard-cli/​README.md:27

This README ships with the installed skill, but “it is not already published or listed” will be false once this PR adds the catalog entry. Rephrase the introduction as contributor guidance that remains accurate after publication.

3 resolved since last review

Review effort: Max

@aaronpowell aaronpowell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @wsilveiranz. This is specific, useful guidance for the az logicapp extension (target-flag rules, context semantics, separate local and cloud principals, gated replay/cancel), and it complements the existing azure-logic-apps-expert agent. Small cleanup: the README's 'Awesome Copilot publication checklist' is contributor-facing and 'it is not already published or listed' will be wrong after merge; consider a short maintenance note instead. The gate run was cancelled, so please comment /rerun-checks.

@aaronpowell
aaronpowell merged commit 8a8a218 into github:main Oct 25, 2026
35 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-risk:medium needs-reviewer new-submission PR adds at least one new contribution ready-for-review Submission passed intake validation and is ready for maintainer review skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants