Skip to content

Conversation

@HollywoodTonight
Copy link
Contributor

@HollywoodTonight HollywoodTonight commented Nov 20, 2025

This PR triggers the semantic-release for the trace ID propagation feature that was merged in #1097 but didn't get released due to incorrect commit message format.

Changes

  • Empty commit with proper feat: prefix to trigger release

Related

Once merged, this will release:

This commit triggers the release of PR #1097 which adds end-to-end trace ID propagation:
- Enhanced logWrapper to include traceId in logs
- SQS message propagation of trace IDs
- HTTP header support for x-trace-id
- New addTraceIdHeader utility function

Ref: #1097
@HollywoodTonight HollywoodTonight changed the title feat(http-utils): add trace ID propagation for HTTP headers chore: trigger release for trace ID propagation (PR #1097) Nov 20, 2025
@iuliag iuliag changed the title chore: trigger release for trace ID propagation (PR #1097) feat: release for trace ID propagation (PR #1097) Nov 20, 2025
@iuliag iuliag self-requested a review November 20, 2025 10:14
@HollywoodTonight HollywoodTonight merged commit c1d878e into main Nov 20, 2025
5 checks passed
@HollywoodTonight HollywoodTonight deleted the release-trigger-trace-id branch November 20, 2025 10:50
HollywoodTonight added a commit to adobe/spacecat-audit-worker that referenced this pull request Nov 20, 2025
Replace gitpkg temporary dependency with official release version.

Update to @adobe/spacecat-shared-utils@^1.76.0 which includes:
- Enhanced logWrapper with automatic trace ID inclusion
- SQS message trace ID propagation
- HTTP header trace ID support

Note: package-lock.json will be updated once 1.76.0 is published.

Depends on: adobe/spacecat-shared#1152
HollywoodTonight added a commit that referenced this pull request Nov 20, 2025
This commit triggers the release for trace ID propagation feature from PR #1097.

Changes include:
- Enhanced logWrapper to automatically include traceId in log messages
- SQS message trace ID propagation via sendMessage/sqsEventAdapter
- HTTP header trace ID extraction via x-trace-id header
- New addTraceIdHeader utility function

This enables end-to-end distributed tracing across SpaceCat services.

Related: #1097, #1152
HollywoodTonight added a commit that referenced this pull request Nov 20, 2025
## Problem

PR #1152 was merged with a proper `feat:` commit message, but
semantic-release did not trigger a release. This happened because:

- **Empty commits are ignored by `semantic-release-monorepo`**
- The monorepo plugin filters commits to only include those that modify
files within each package directory
- Since #1152 was an empty commit with no file changes, it wasn't
associated with any package
- Result: Both `spacecat-shared-utils` and `spacecat-shared-http-utils`
were not released despite having new features from #1097

## Solution

This PR adds documentation updates to both affected packages:
- **`spacecat-shared-utils`**: Added release note for trace ID
propagation
- **`spacecat-shared-http-utils`**: Added release note for trace ID
extraction

The commit includes:
- Proper `feat(utils,http-utils):` prefix following conventional commits
- Actual file modifications in both package directories
- Will be correctly recognized by `semantic-release-monorepo`

## Changes from PR #1097

The original trace ID propagation feature includes:
- Enhanced `logWrapper` to automatically include `traceId` in log
messages
- SQS message trace ID propagation via
`sendMessage()`/`sqsEventAdapter()`
- HTTP header trace ID extraction via `x-trace-id` header
- New `addTraceIdHeader()` utility function

This enables end-to-end distributed tracing across SpaceCat services.

## Expected Releases

Once merged, semantic-release will publish:
- `@adobe/[email protected]` (currently 1.75.0)
- `@adobe/[email protected]` (currently 1.18.2)

## Related

- Implements: #1097
- Supersedes: #1152
- Needed for: adobe/spacecat-audit-worker#1520
solaris007 pushed a commit that referenced this pull request Nov 20, 2025
solaris007 pushed a commit that referenced this pull request Nov 20, 2025
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.

3 participants