Skip to content

fix(cli): include type "amend" in post amend envelope#127

Merged
odesenfans merged 1 commit intomainfrom
od/fix-post-amend-type
Mar 24, 2026
Merged

fix(cli): include type "amend" in post amend envelope#127
odesenfans merged 1 commit intomainfrom
od/fix-post-amend-type

Conversation

@odesenfans
Copy link
Copy Markdown
Contributor

The amend envelope was missing the required "type": "amend" field, causing the CCN to reject amend messages with InvalidMessageFormat.

The amend envelope was missing the required "type": "amend" field,
causing the CCN to reject amend messages with InvalidMessageFormat.
Copy link
Copy Markdown

@foxpatch-aleph foxpatch-aleph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a correct and minimal fix for a bug where the amend envelope was missing the required "type": "amend" field, causing CCN rejection. The change adds the field to both the production code (line 77) and the test envelope, and updates the test to verify the field is present rather than absent. The fix is focused, well-tested, and follows the existing code pattern used in handle_post_create which already includes a type field.

@odesenfans odesenfans merged commit d2dc8a2 into main Mar 24, 2026
9 checks passed
@odesenfans odesenfans deleted the od/fix-post-amend-type branch March 24, 2026 15:02
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