Skip to content

SP-355: Implement diff with file#362

Merged
Thomas Neumüller (tneum-celonis) merged 6 commits into
mainfrom
SP-355-add-file-diff
May 29, 2026
Merged

SP-355: Implement diff with file#362
Thomas Neumüller (tneum-celonis) merged 6 commits into
mainfrom
SP-355-add-file-diff

Conversation

@tneum-celonis
Copy link
Copy Markdown
Contributor

@tneum-celonis Thomas Neumüller (tneum-celonis) commented May 28, 2026

Description

Extends config nodes diff to diff a local node JSON file against a node version stored in pacman, using the new public endpoint added in SP-144 (POST /pacman/api/core/packages/{packageKey}/nodes/{nodeKey}/diff/configuration/with-file).

  • New -f, --file <file> flag, mutually exclusive with --compareVersion
  • New NodeDiffApi.diffWithFile(...)
  • NodeDiffService reuses the existing rendering (text + --json) for both flows via a shared private helper
  • DOCUMENTATION.md updated

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

@tneum-celonis Thomas Neumüller (tneum-celonis) marked this pull request as ready for review May 28, 2026 09:27
Comment thread tests/commands/configuration-management/config-node-diff.spec.ts
@sonarqubecloud
Copy link
Copy Markdown

Comment thread src/commands/configuration-management/api/node-diff-api.ts
@tneum-celonis Thomas Neumüller (tneum-celonis) merged commit e08c05a into main May 29, 2026
6 checks passed
@tneum-celonis Thomas Neumüller (tneum-celonis) deleted the SP-355-add-file-diff branch May 29, 2026 14:36
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.

4 participants