Remove MTX Sidecar Requirement

[eric-pSAP]

Remove requirement for mtx to be in sidecar. Addresses issue #218

[hyperspace-insights]

Summary

The following content is AI-generated and provides a summary of the pull request:

---

Remove MTX Sidecar Profile Requirement

Changes

♻️ Refactor: Simplified multitenancy detection by removing the dependency on the mtx-sidecar profile check.

Changes

• lib/mtx/server.js: Replaced profile-based check (profiles.includes('mtx-sidecar')) with a direct multitenancy requirement check (cds.requires.multitenancy). This simplifies the conditional logic for object store handling and removes the need for explicit sidecar profile configuration.

---

• 🔄 Regenerate and Update Summary
• ✏️ Insert as PR Description (deletes this comment)
• 🗑️ Delete comment

PR Bot Information

Version: 1.17.50 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

• Summary Prompt: Default Prompt
• Output Template: Default Template
• Correlation ID: cde6d5a0-01bd-11f1-9bdb-ab328ef08a35
• Event Trigger: pull_request.opened
• LLM: anthropic--claude-4.5-sonnet

---

💌 Have ideas or want to contribute? Create an issue and share your thoughts with us!
📑 Check out the documentation for more information.
📬 Subscribe to our mailing list to get the latest announcements and pilot features!

Made with ❤️ by Hyperspace.

[hyperspace-insights]

The pull request replaces profile-based detection with a multitenancy check, but contains a logic error: it accesses cds.requires.multitenancy instead of the correct cds.env.requires.multitenancy configuration path.

PR Bot Information

Version: 1.17.50 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

• LLM: anthropic--claude-4.5-sonnet
• Event Trigger: pull_request.opened
• Correlation ID: cde6d5a0-01bd-11f1-9bdb-ab328ef08a35