Skip to content

chore(sap-ux-fiori-tools): bump to 1.16.13 - #6324

Closed
IainSAP wants to merge 2 commits into
github:mainfrom
IainSAP:bump/sap-ux-fiori-tools-1.16.13
Closed

IainSAP wants to merge 2 commits into
github:mainfrom
IainSAP:bump/sap-ux-fiori-tools-1.16.13

Conversation

@IainSAP

@IainSAP IainSAP commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Automated version bump for sap-ux-fiori-tools from 1.16.7 to 1.16.13.

Source: https://github.com/SAP/open-ux-tools/tree/main/plugins-coding-agents/fiori

@IainSAP
IainSAP requested a review from a team as a code owner November 9, 2026 10:59
Copilot AI auto review requested due to automatic review settings November 9, 2026 10:59
@github-actions github-actions Bot added external-plugin Public external plugin submission plugin PR touches plugins labels Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 2 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 Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:high — Privileged execution, automation, or review-policy change
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access, including a maintainer with admin or maintain permission.

Why this tier
  • plugins/external.json is a high-risk path (automation, scripts, MCP config, hooks, or review policy)

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ✅ Passed Passed · logs
Plugin and extension validation ✅ Passed Passed · logs
Plugin structure ✅ 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
Canvas/plugin smoke test ✅ Passed Passed · 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); an approval from a maintainer with admin or maintain permission
  • The core-maintainers pool is not staffed yet; an approver with admin or maintain permission is required instead.

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 7b9de4b · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions github-actions Bot added the ready-for-review Submission passed intake validation and is ready for maintainer review label Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

✅ External plugin PR checks passed

  • Changed entries detected: 1
  • Workflow state label: ready-for-review
  • Full logs: Download quality gate artifact
  • Status legend: ✅ pass · ⚠️ warning · 🛑 fail

Per-plugin quality summary

Plugin spec compliance vally lint install smoke test version match ref/sha consistency canvas structure overall source tree
sap-ux-fiori-tools ✅ pass ✅ pass ✅ pass ✅ pass ⚪ not_run ⚪ not_run ✅ pass cc8d106cf79f77c5f105750ee1c79bcf35bdcbc9

Gate output details

sap-ux-fiori-tools - spec compliance (✅ pass)

Agent Plugins v1.0.0 manifest checks passed for plugin.json.
sap-ux-fiori-tools - vally lint (✅ pass)

✅ sap-fiori-add-visual-filter (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.
✅ sap-fiori-analytical-chart (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.
✅ sap-fiori-app-development (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.
✅ sap-fiori-create-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.
✅ sap-fiori-eslint-plugin (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.
✅ sap-fiori-opa5-test-development (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.
✅ sap-fiori-tree-table (2/2 checks passed)

...output truncated...
sap-ux-fiori-tools - install smoke test (✅ pass)

Install smoke test succeeded. Verified /tmp/external-plugin-quality-2mYsH7/copilot-home/.copilot/installed-plugins/external-plugin-intake/sap-ux-fiori-tools/plugin.json.
sap-ux-fiori-tools - version match (pass)

- cc8d106cf79f77c5f105750ee1c79bcf35bdcbc9: matched version "1.16.13" at "plugins-coding-agents/fiori/plugin.json".
sap-ux-fiori-tools - ref/sha consistency (not_run)

Ref/SHA consistency gate skipped because one of source.ref or source.sha was not provided.
sap-ux-fiori-tools - canvas structure (not_run)

Canvas structure gate skipped because plugin is not tagged with "canvas".
- sap-ux-fiori-tools: spec=pass, vally-lint=pass, install-smoke=pass, version-match=pass, ref-sha-consistency=not_run, canvas-structure=not_run, overall=pass

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

The canonical plugin entry was updated without regenerating and committing the marketplace copy, so repository users would still install the older version.

1 open issue
What changed in this PR

Updates the sap-ux-fiori-tools external plugin from version 1.16.7 to 1.16.13, pinned to upstream commit cc8d106cf79f77c5f105750ee1c79bcf35bdcbc9.

Changes:

  • Bumps the plugin version to 1.16.13.
  • Updates the immutable upstream commit SHA.
  • The generated marketplace manifest remains stale and should be regenerated.

This pull request requires changes.

File Summary
plugins/​external.json Updates the SAP Fiori Tools plugin version and source commit. The generated .github/plugin/marketplace.json still references version 1.16.7 and the previous SHA, preventing users from receiving the updated revision.

Review effort: Lite (auto)

Comment thread plugins/external.json
"name": "sap-ux-fiori-tools",
"description": "MCP server and skills for SAP Fiori development. Helps AI coding assistants build, modify, and test SAP Fiori Elements applications connected to CAP or ABAP OData services. Provides 7 skills covering app generation, analytical charts, visual filters, OPA5 testing, ESLint, tree tables, and the Fiori tools CLI.",
"version": "1.16.7",
"version": "1.16.13",
Copilot AI auto review requested due to automatic review settings November 9, 2026 11:07

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.

🟢 Approval recommended

The plugin version and pinned upstream commit are updated consistently in both the external registry and marketplace metadata, with no active review issues identified.

1 open issue

Review effort: Lite (auto)

@IainSAP

IainSAP commented Nov 9, 2026

Copy link
Copy Markdown
Contributor Author

testing bot

@IainSAP IainSAP closed this Nov 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external-plugin Public external plugin submission merge-risk:high plugin PR touches plugins ready-for-review Submission passed intake validation and is ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants