What is the issue and how can we reproduce it?
The skill does not scope the guidance to a specific app type. Nevertheless it seems to be written from a freestyle UI5 perspective. This can lead to recommendations that are wrong in case of e.g. Fiori Elements apps.
A short note could help:
Note: This configuration guide applies to freestyle UI5 OPA5 tests. If you are using SAP Fiori Elements with `sap.fe.test.JourneyRunner`, the setup differs. See the `sap.fe.test` documentation (do we have a good link? Or even better an FE specific skill to reference?) for the correct setup.
This way an AI reading the skill knows upfront whether the guidance applies to the project it's working on, rather than blindly applying freestyle patterns to an FE app.
The skill also currently assumes a classic manual setup and gives no indication that fiori-tools-preview (or similar tooling) can serve the entry point virtually. An AI following the skill literally would either flag the missing opaTests.qunit.js as an error or try to create one unnecessarily.
suggestion:
Note: If you are using `fiori-tools-preview` or similar tooling, the entry point file (`opaTests.qunit.js`) may be served virtually and does not need to exist on disk.
Expected behavior:
Either skip FE opa5 tests or give correct recommendations
Confirmation
What is the issue and how can we reproduce it?
The skill does not scope the guidance to a specific app type. Nevertheless it seems to be written from a freestyle UI5 perspective. This can lead to recommendations that are wrong in case of e.g. Fiori Elements apps.
A short note could help:
This way an AI reading the skill knows upfront whether the guidance applies to the project it's working on, rather than blindly applying freestyle patterns to an FE app.
The skill also currently assumes a classic manual setup and gives no indication that
fiori-tools-preview(or similar tooling) can serve the entry point virtually. An AI following the skill literally would either flag the missingopaTests.qunit.jsas an error or try to create one unnecessarily.suggestion:
Expected behavior:
Either skip FE opa5 tests or give correct recommendations
Confirmation