Adds tail attributes from caller to tail workers

[Caio-Nogueira]

Adds tail tags to the Trace object and mirror these tags with attributes on the OnSet event

[ask-bonk]

ResolveMessage: Cannot find module '@opencode-ai/plugin' from '/home/runner/work/workerd/workerd/.opencode/tools/bazel-deps.ts'

github run

[ask-bonk]

@Caio-Nogueira Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

[github-actions]

The generated output of @cloudflare/workers-types matches the snapshot in types/generated-snapshot 🎉

[codecov-commenter]

Codecov Report

❌ Patch coverage is 9.57447% with 85 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.75%. Comparing base (bc0bc2a) to head (2425b02).
⚠️ Report is 12 commits behind head on main.

Files with missing lines	Patch %	Lines
src/workerd/api/trace.c++	9.25%	49 Missing ⚠️
src/workerd/io/tracer.c++	10.00%	26 Missing and 1 partial ⚠️
src/workerd/io/trace.c++	0.00%	7 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6254      +/-   ##
==========================================
- Coverage   70.80%   70.75%   -0.06%     
==========================================
  Files         420      420              
  Lines      112299   112391      +92     
  Branches    18417    18443      +26     
==========================================
+ Hits        79513    79520       +7     
- Misses      21789    21871      +82     
- Partials    10997    11000       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[petebacondarwin]

The PR calls these "tail tags" but the code calls them "tail attributes". Is that just the PR being outdated or am I missing the connection?

[Caio-Nogueira]

The PR calls these "tail tags" but the code calls them "tail attributes". Is that just the PR being outdated or am I missing the connection?

Yeah, the PR description was outdated. We are using the term "tail attributes" for these tags now

[fhanau]

This should not be needed – we don't do memory allocation for the given bool/double/integer types, and the type we have here (e.g. bool vs. double) doesn't matter since the size of tracing::Attribute is fixed.