perf: Speed up fetching services from container

[come-nc]

• Resolves: #

Summary

Speed containers lookup by:

• Storing aliases in a raw array instead of relying on a closure in the container
• Removed useless checks
• Avoid relying on an exception throwing for a common usecase

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)
• Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
• Milestone added for target branch/version (ex: 32.x for stable32)

AI (if applicable)

• The content of this PR was partly or fully generated using AI

[come-nc]

I think the flat array for aliases cannot work, at least not how I implemented it here.
Each container (one for server and one per application) may have its own aliases.
So the alias should be resolved at the last minute, before calling pimple container, never earlier. I’ll have to see if this can be done without too much trouble.
The rest of the changes are not impacted I believe.

[come-nc]

I think the flat array for aliases cannot work, at least not how I implemented it here. Each container (one for server and one per application) may have its own aliases. So the alias should be resolved at the last minute, before calling pimple container, never earlier. I’ll have to see if this can be done without too much trouble. The rest of the changes are not impacted I believe.

I’m actually unsure, I pushed a commit going the other direction for now, I’ll look into this more in-depth later.
To be rock-solid we’d need a test for this, with aliases both on server and app container and see if it works. Or we need to edict some rules about this.