Skip to content

Conversation

@HollywoodTonight
Copy link
Contributor

Problem

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

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/spacecat-shared-utils@1.76.0 (currently 1.75.0)
  • @adobe/spacecat-shared-http-utils@1.19.0 (currently 1.18.2)

Related

HollywoodTonight and others added 3 commits November 20, 2025 11:09
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
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
@github-actions
Copy link

This PR will trigger a minor release when merged.

@HollywoodTonight HollywoodTonight merged commit fb48149 into main Nov 20, 2025
7 checks passed
@HollywoodTonight HollywoodTonight deleted the feat-trigger-trace-id-release branch November 20, 2025 11:39
solaris007 pushed a commit that referenced this pull request Nov 20, 2025
@solaris007
Copy link
Member

🎉 This PR is included in version @adobe/spacecat-shared-http-utils-v1.19.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

solaris007 pushed a commit that referenced this pull request Nov 20, 2025
@solaris007
Copy link
Member

🎉 This PR is included in version @adobe/spacecat-shared-utils-v1.76.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

HollywoodTonight added a commit to adobe/spacecat-audit-worker that referenced this pull request Nov 20, 2025
Updated package-lock.json after @adobe/spacecat-shared-utils@1.76.0 was officially released.

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to HollywoodTonight/spacecat-coralogix-feeder 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

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to HollywoodTonight/spacecat-api-service that referenced this pull request Nov 20, 2025
- Add logWrapper to imports from spacecat-shared-utils
- Add logWrapper to wrapper chain after enrichPathInfo
- Bump @adobe/spacecat-shared-utils to ^1.76.0
- Logs will now automatically include AWS X-Ray trace IDs and HTTP x-trace-id headers

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit that referenced this pull request Nov 20, 2025
The logWrapper was prepending trace/job IDs as a separate argument instead
of concatenating them with the format string. This caused format placeholders
(%s, %d, etc.) to not be interpolated, appearing literally in logs.

Changes:
- Merge marker string with format string (first arg) instead of prepending as separate arg
- Add tests to verify format string placeholder interpolation works correctly

Before: log.info('[traceId=123]', 'Found %d items', 42)
After:  log.info('[traceId=123] Found %d items', 42)

Fixes format string interpolation in all log levels (info, error, debug, warn, etc.)

Related: #1154
HollywoodTonight added a commit that referenced this pull request Nov 20, 2025
The logWrapper was prepending trace/job IDs as a separate argument instead
of concatenating them with the format string. This caused format placeholders
(%s, %d, etc.) to not be interpolated, appearing literally in logs.

Changes:
- Merge marker string with format string (first arg) instead of prepending as separate arg
- Add tests to verify format string placeholder interpolation works correctly

Before: log.info('[traceId=123]', 'Found %d items', 42)
After:  log.info('[traceId=123] Found %d items', 42)

Fixes format string interpolation in all log levels (info, error, debug, warn, etc.)

Related: #1154
HollywoodTonight added a commit that referenced this pull request Nov 20, 2025
The logWrapper was prepending trace/job IDs as a separate argument instead
of concatenating them with the format string. This caused format placeholders
(%s, %d, etc.) to not be interpolated, appearing literally in logs.

Changes:
- Merge marker string with format string (first arg) instead of prepending as separate arg
- Add tests to verify format string placeholder interpolation works correctly

Before: log.info('[traceId=123]', 'Found %d items', 42)
After:  log.info('[traceId=123] Found %d items', 42)

Fixes format string interpolation in all log levels (info, error, debug, warn, etc.)

Related: #1154
HollywoodTonight added a commit to adobe/spacecat-audit-worker that referenced this pull request Nov 20, 2025
Updated package-lock.json after @adobe/spacecat-shared-utils@1.76.0 was officially released.

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to HollywoodTonight/spacecat-coralogix-feeder 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

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to adobe/spacecat-audit-worker that referenced this pull request Nov 20, 2025
Updated package-lock.json after @adobe/spacecat-shared-utils@1.76.0 was officially released.

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to adobe/spacecat-coralogix-feeder 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

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to adobe/spacecat-coralogix-feeder 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

Related: adobe/spacecat-shared#1154
HollywoodTonight added a commit to adobe/spacecat-audit-worker that referenced this pull request Nov 20, 2025
Updated package-lock.json after @adobe/spacecat-shared-utils@1.76.0 was officially released.

Related: adobe/spacecat-shared#1154
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants