feat(fiori-mcp-server): Skill productization

[IvoSG]

Title: Productize SAP Fiori ADP Controller Extension Skill

Description

Productizes the SAP Fiori ADP controller extension skill by renaming and enriching the skill metadata, modularizing long guidance into reference documents, and tightening workflow instructions for safer adaptation project changes.

Key updates:

• Renamed the skill to sap-fiori-adp-controller-extension with clearer trigger guidance and metadata.
• Split large sections from SKILL.md into dedicated references:
  • actions-reference.md
  • hitl-gating.md
  • example-session.md
• Updated workflow guidance for navigation, action selection, payload preparation, cleanup, and error handling.
• Standardized MCP tool references to public tool names.
• Improved Chromium fallback and editor-server shutdown instructions across platforms.
• Enhanced ADP controller extension knowledge base rules around:
  • stable IDs, with a Dialog exception
  • controller extension naming and reuse
  • variant-id-based namespaces
  • fragment handler resolution
  • on-demand OData property fetching
  • numeric threshold handling

Type of change

• Bug (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds a new feature)
• Breaking change (Bug or New feature that would cause existing functionality/consumers to not work as expected)
• Non-Breaking chores (Changes to tools, libraries, build process, documentation, etc)
• None of the above (Reviewers might ask for more clarification)

How have you tested?

Documentation and prompt-only changes. Reviewed the updated skill flow, reference links, and knowledge base guidance for consistency and completeness.

Checklist:

• The code conforms to the general development principles

• Supplied as many details as possible on this change

• The code is easy to read and maintainable by others

• Corresponding changes to the documentation has been done

• Already existing and new unit tests pass locally

• I have reviewed and addressed all Hyperspace bot findings (or explicitly explained dismissals)

• I have done an Agentic review

• 🔄 Regenerate and Update Summary

PR Bot Information

Version: 1.31.30

• Correlation ID: 5ef40d40-adbe-11f1-9a1e-0f37fa369d08
• Event Trigger: pull_request.opened
• Output Template: Repository PR Template
• LLM: gpt-5.5
• Summary Prompt: Default Prompt
• File Content Strategy: Full file content

[changeset-bot]

🦋 Changeset detected

Latest commit: 7b84897

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@sap-ux/fiori-mcp-server	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[hyperspace-pr-bot]

I found a couple of issues in the split-out documentation where rules became either over-broad or less deterministic than the original skill text. Overall the refactor is straightforward, but please tighten those references so the productized skill keeps the same runtime behavior.

PR Bot Information

Version: 1.31.30

• Correlation ID: 5ef40d40-adbe-11f1-9a1e-0f37fa369d08
• Event Trigger: pull_request.opened
• Agent Instructions:
  • AGENTS.md
  • CLAUDE.md
• LLM: gpt-5.5
• File Content Strategy: Full file content

[mmilko01]

Just a few small comments

[mmilko01]

Looks good

[nikmace]

Changes look good