Skip to content

feat(ui5-modernization): Add UI5 modernization plugin#77

Open
devtomtom wants to merge 16 commits into
mainfrom
add-ui5-modernization-plugin
Open

feat(ui5-modernization): Add UI5 modernization plugin#77
devtomtom wants to merge 16 commits into
mainfrom
add-ui5-modernization-plugin

Conversation

@devtomtom

@devtomtom devtomtom commented Jun 22, 2026

Copy link
Copy Markdown
Member

Adds the new ui5-modernization plugin with 20 skills for automated UI5 app modernization.

The plugin provides agent skills that fix UI5 linter findings and modernize legacy patterns:

Skills fixing UI5 linter findings (on top of the fixing capabilities of UI5 linter):

  • fix-bootstrap-params,
  • fix-component-async,
  • fix-control-renderer,
  • fix-csp-compliance,
  • fix-cyclic-deps,
  • fix-deprecated-controls,
  • fix-fiori-elements-extensions,
  • fix-js-globals,
  • fix-library-init,
  • fix-linter-blind-spots,
  • fix-manifest-json,
  • fix-partially-deprecated-apis,
  • fix-pseudo-modules,
  • fix-table-row-mode,
  • fix-xml-globals,
  • fix-xml-native-html

Skills applying modernization best practices:

  • modernize-ui5-app,
  • modernize-test-starter,
  • modernize-flp-sandbox

JIRA: CPOUI5FRAMEWORK-895

@devtomtom devtomtom requested review from a team and flovogt June 22, 2026 08:13
@cla-assistant

cla-assistant Bot commented Jun 22, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@devtomtom devtomtom force-pushed the add-ui5-modernization-plugin branch from fed22a0 to b16d8b2 Compare June 22, 2026 09:17
Comment thread plugins/ui5-modernization/README.md
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
stopcoder
stopcoder previously approved these changes Jun 22, 2026
Comment thread plugins/ui5-modernization/.github/plugin/plugin.json Outdated
Comment thread plugins/ui5-modernization/plugin.json Outdated
Comment thread knip.json Outdated
Comment thread plugins/ui5-modernization/.mcp.json
Comment thread plugins/ui5-modernization/README.md
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Co-authored-by: Florian Vogt <florian.vogt@sap.com>

@stopcoder stopcoder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good to me.

@KlattG KlattG 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.

A few minor changes, otherwise LGTM.
NB: Only the README.md was checked.

Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
Co-authored-by: Günter Klatt <guenter.klatt@sap.com>
@devtomtom devtomtom requested a review from KlattG June 22, 2026 15:02
@flovogt flovogt mentioned this pull request Jun 22, 2026
KlattG
KlattG previously approved these changes Jun 23, 2026

@KlattG KlattG 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.

README.md -- LGTM

Comment thread plugins/ui5-modernization/README.md Outdated
Comment thread plugins/ui5-modernization/README.md Outdated
@devtomtom devtomtom requested a review from flovogt June 23, 2026 08:22

@flovogt flovogt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you please add the new plugin to the plugin-overview?

@devtomtom

Copy link
Copy Markdown
Member Author

Could you please add the new plugin to the plugin-overview?

Done.

Comment thread knip.config.js Outdated
@devtomtom devtomtom requested a review from margopolo June 23, 2026 10:33
@devtomtom devtomtom requested a review from flovogt June 23, 2026 11:48
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.

6 participants