Skip to content

feat: make context injection into functions optional when we trace - #71

Merged
tabkram merged 2 commits into
mainfrom
feature/trace-context
Aug 8, 2025
Merged

feat: make context injection into functions optional when we trace#71
tabkram merged 2 commits into
mainfrom
feature/trace-context

Conversation

@tabkram

@tabkramtabkram commented Aug 8, 2025•
edited
Loading

Copy link
Copy Markdown
Owner

feat: Make context injection optional in trace decorator.

This change introduces an injectContextInArgs option to the trace decorator, defaulting to false, allowing users to control whether the trace context is automatically injected into the decorated function's arguments.
The change aims to provide greater flexibility when tracing code execution.

@tabkramtabkram changed the title feat: make context injection into functions optional by default when we tracefeat: make context injection into functions optional when we traceAug 8, 2025
@github-actions

github-actionsBot commented Aug 8, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Coverage report

St.❔
CategoryPercentageCovered / Total
🟢Statements
96.13% (-0.05% 🔻)
397/413
🟢Branches
87.78% (+0.32% 🔼)
237/270
🟢Functions94.74%108/114
🟢Lines
96.68% (-0.04% 🔻)
379/392

Test suite run success

85 tests passing in 12 suites.

Report generated by 🧪jest coverage report action from 1aa31a0

@tabkramtabkram added the deploy deploy a canary version to NPM label Aug 8, 2025
@tabkram
tabkramforce-pushed the feature/trace-context branch from 0408d68 to af2f788CompareAugust 8, 2025 15:34
@github-actions

github-actionsBot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

🚫 Unpublished versions: ∅
✅ Created Version:3.5.0-canary.71.ba5348f
📦 Published on NPM:View on NPM

⚠️Note: This version is temporary and will be unpublished on the next PR update or merge.

@tabkram
tabkramforce-pushed the feature/trace-context branch from af2f788 to b34a8d6CompareAugust 8, 2025 15:40
@github-actions

github-actionsBot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

🚫 Unpublished versions: 3.5.0-canary.71.ba5348f
✅ Created Version:3.5.0-canary.71.8f4337b
📦 Published on NPM:View on NPM

⚠️Note: This version is temporary and will be unpublished on the next PR update or merge.

@tabkram
feat: make context injection into functions optional by default when …
1e9cf2a
…we `trace`
feat: make context injection into functions optional by default when we `trace`
@tabkram
tabkramforce-pushed the feature/trace-context branch from b34a8d6 to 1e9cf2aCompareAugust 8, 2025 15:47
@tabkram
fix: unnecessary npm script
1aa31a0
Removes the `prepare` script which only ran `npm audit`.
This script is unnecessary as `npm audit` is not needed in the publish lifecycle.
@github-actions

github-actionsBot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

🚫 Unpublished versions: 3.5.0-canary.71.8f4337b
✅ Created Version:3.5.0-canary.71.586ccae
📦 Published on NPM:View on NPM

⚠️Note: This version is temporary and will be unpublished on the next PR update or merge.

@tabkram
tabkram merged commit f764fe5 into mainAug 8, 2025
8 checks passed
@github-actions

github-actionsBot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

🚫 Unpublished versions: 3.5.0-canary.71.586ccae

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

deploydeploy a canary version to NPM

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tabkram