feat: add Backstage Software Catalog entities for platform, chart, an…#7
Conversation
![amazon-q-developer[bot]](https://avatars.githubusercontent.com/in/1220912?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review Summary
This PR adds Backstage Software Catalog entities for the platform infrastructure, but contains critical logic errors that block merge:
Critical Issues Requiring Fix:
- Circular Dependency: The
simple-app-infra-codeandapp-chartcomponents have a circulardependsOnrelationship that will break Backstage catalog resolution - Incorrect API Relationships: The application components (
app-bandsimple-app) incorrectly claim to provide thetenant-contractAPI when they actually consume it
These issues must be resolved before merge as they will cause Backstage to fail loading and processing the catalog entities.
Required Changes:
- Remove circular dependency declarations between infrastructure repository and chart
- Fix API provider/consumer relationships for application components
You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.
| dependsOn: | ||
| - component:simple-app-infra-code |
There was a problem hiding this comment.
🛑 Logic Error: Circular dependency with simple-app-infra-code. This component declares dependsOn: component:simple-app-infra-code, while the infra code (catalog-info.yaml:17) declares dependsOn: component:app-chart. This circular relationship will cause Backstage catalog resolution to fail. The chart should not depend on the infra repository - remove this dependency.
| dependsOn: | |
| - component:simple-app-infra-code | |
| system: platform | |
| providesApis: |
…scaffolder template
1 participant
No description provided.