Skip to content

feat(libdatadog): add remote config - #214

Closed
rochdev wants to merge 6 commits into
rochdev/napi-rs-inline-wasmfrom
rochdev/libdatadog-remote-config
Closed

feat(libdatadog): add remote config#214
rochdev wants to merge 6 commits into
rochdev/napi-rs-inline-wasmfrom
rochdev/libdatadog-remote-config

Conversation

@rochdev

@rochdev rochdev commented Aug 27, 2026

Copy link
Copy Markdown
Member

Summary

  • move agentless remote config into the universal libdatadog package
  • expose the same remote config API through native N-API and inline WASM
  • reuse the JavaScript-provided HTTP and timer capabilities on both architectures
  • preserve per-operation async context across native HTTP and timer callbacks with AsyncResource
  • enforce dependency and artifact-size constraints for both backends

Size

  • N-API artifact: 1,356,896 bytes (1325.1 KiB)
  • raw WASM: 1,013,188 bytes (989.4 KiB)
  • final inline WASM: 474,972 bytes (463.8 KiB)

Testing

  • npm --prefix packages/libdatadog test (50 passed, 2 skipped)
  • concurrent AsyncLocalStorage coverage for native and WASM exporters
  • AsyncLocalStorage coverage for native and WASM remote config
  • npm run check:dependencies
  • npm run lint
  • native and wasm32-unknown-unknown Cargo checks and Clippy
  • Cargo formatting

Dependencies

Generated by Codex.

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Overall package size

Self size: 50.65 MB
Deduped: 50.65 MB
No deduping: 50.65 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------|

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@rochdev
rochdev marked this pull request as ready for review August 27, 2026 22:55
@rochdev
rochdev requested review from a team as code owners August 27, 2026 22:55

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0d4abbc0c0

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread crates/libdatadog/src/remote_config.rs
Agentless responses verify against embedded TUF roots, so a local success fixture would require a test-only root override.
Any remote-config dependency previously exempted allowlisted crate names, which hid unrelated duplicate versions from artifact validation.
Host callbacks always use no-op tracing context, so context IDs and retained AsyncResource instances have no consumer.

Agentless sends report completion through the callback and log delivery failures at the wrapper boundary.
@BridgeAR

Copy link
Copy Markdown
Member

Superseded

@BridgeAR BridgeAR closed this Aug 31, 2026
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.

2 participants