Skip to content

SP-316: remove asset-registry methodology command#358

Open
Kastriot Salihu (ksalihu) wants to merge 1 commit into
mainfrom
ksalihu/SP-316
Open

SP-316: remove asset-registry methodology command#358
Kastriot Salihu (ksalihu) wants to merge 1 commit into
mainfrom
ksalihu/SP-316

Conversation

@ksalihu
Copy link
Copy Markdown
Contributor

Summary

Removes the asset-registry methodology subcommand from content-cli. The corresponding methodology endpoint is being phased out from the platform's asset registry surface, so the CLI no longer exposes it.

Linked

https://celonis.atlassian.net/browse/SP-316

What changed

  • src/commands/asset-registry/module.ts — drop the methodology subcommand registration and its private handler.
  • src/commands/asset-registry/asset-registry.service.ts — drop getMethodology() and the endpoints.methodology line in logDescriptorDetail.
  • src/commands/asset-registry/asset-registry-api.ts — drop getMethodology() (GET /pacman/api/core/asset-registry/methodologies/{assetType}).
  • src/commands/asset-registry/asset-registry.interfaces.ts — drop the optional methodology field from AssetEndpoints.
  • Tests: delete asset-registry-methodology.spec.ts; remove methodology fixtures and assertions from the get, list, and module specs.
  • Docs: remove the methodology section from docs/user-guide/asset-registry-commands.md, methodology examples from docs/user-guide/agentic-development-guide.md, and methodology references from .cursor/skills/asset-registry-endpoints/SKILL.md.

Test plan

  • npx jest — full suite (348 tests / 41 suites) passes.
  • npm run build — TypeScript compile succeeds.
  • node dist/content-cli.js asset-registry --helpmethodology no longer listed.
  • node dist/content-cli.js asset-registry methodology --assetType BOARD_V2error: unknown command 'methodology'.
  • Workspace-wide case-insensitive search for methodolog returns zero matches.

Made with Cursor

Removes the `asset-registry methodology` subcommand and its
`/pacman/api/core/asset-registry/methodologies/{assetType}` API call
from content-cli, including the corresponding service method, module
registration, interface field, tests, and user-guide / skill docs.

Includes-AI-Code: true
Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud
Copy link
Copy Markdown

@ksalihu Kastriot Salihu (ksalihu) marked this pull request as ready for review May 22, 2026 15:47
@ksalihu Kastriot Salihu (ksalihu) requested review from a team as code owners May 22, 2026 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants