Skip to content

SP-40: remove redundant base path from asset reg list summary#345

Open
Patrick Ruhkopf (restfulhead) wants to merge 1 commit intomainfrom
SP-40
Open

SP-40: remove redundant base path from asset reg list summary#345
Patrick Ruhkopf (restfulhead) wants to merge 1 commit intomainfrom
SP-40

Conversation

@restfulhead
Copy link
Copy Markdown
Collaborator

@restfulhead Patrick Ruhkopf (restfulhead) commented Apr 22, 2026

Description

With the introduction of the public proxy API endpoints, the basePath of the public API is always the same and therefore redundant.

Output of asset-registry list before this change:

info:    ai-annotations-agent-widget - Annotation Builder LLM [AI/ML Tools] (basePath: /pacman/api)
info:    BOARD_V2 - View [DASHBOARDS] (basePath: /pacman/api)
...

Output after this change:

info:    ai-annotations-agent-widget - Annotation Builder LLM [AI/ML Tools]
info:    BOARD_V2 - View [DASHBOARDS]
...

Relevant links

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

Note

Low Risk
Low risk: this only changes CLI log formatting for asset-registry list by removing the displayed basePath, with no API calls or data handling changes.

Overview
Simplifies the asset-registry list console output by removing the redundant (basePath: ...) suffix from each asset type summary line.

Detailed output (getType/logDescriptorDetail) is unchanged and still includes Base Path when requested.

Reviewed by Cursor Bugbot for commit 30b37e9. Bugbot is set up for automated code reviews on this repo. Configure here.

@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant