Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: add observe api to js sdk #166

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

feat: add observe api to js sdk #166

wants to merge 13 commits into from

Conversation

G4Vi
Copy link
Contributor

@G4Vi G4Vi commented Jul 31, 2024 •
edited
Loading

Resolves #68

  • Add span-enter, span-exit, metric, log, span-tags
  • Add dylibso_observe namespace imports mapping to the new functions
  • Add somewhat easier construction, with instantiateWasm
  • Add new functions to adapters
    • stdout
    • lightstep
    • datadog
    • honeycomb
  • Verify js sdk (not adapters) works on all platforms
    • node
    • web
    • deno
    • workerd?
  • Fix race condition with found in js/packages/observe-sdk-stdout/test/node/index.js with demangling

@G4Vi G4Vi force-pushed the feat-add-observe-api-to-js-sdk branch from 6b924c6 to ce8011d Compare August 8, 2024 20:22
@G4Vi G4Vi force-pushed the feat-add-observe-api-to-js-sdk branch from ce8011d to f3ea78a Compare August 8, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Port Observe API to JS SDK
1 participant
@G4Vi