Skip to content

refactor(lading): drop expect_used quarantines and annotate bins#1901

Draft
goxberry wants to merge 1 commit into
goxberry/expect-cleanup-lading-observer-targetfrom
goxberry/expect-cleanup-lading-bins-drop-quarantines
Draft

refactor(lading): drop expect_used quarantines and annotate bins#1901
goxberry wants to merge 1 commit into
goxberry/expect-cleanup-lading-observer-targetfrom
goxberry/expect-cleanup-lading-bins-drop-quarantines

Conversation

@goxberry
Copy link
Copy Markdown
Contributor

@goxberry goxberry commented May 23, 2026

Summary

Final sub-PR of the lading-crate cleanup. Annotates the three bin entry points (lading, captool, payloadtool), drops the four #![allow(clippy::expect_used)] quarantines, and annotates the previously-masked production sites in generator/{process_tree,tcp_rr,udp} that the quarantine drop surfaces.

After this PR, the lading crate has no remaining crate-level expect_used quarantines — every .expect() is either covered by a fn-level #[expect] with a documented reason or has been mechanically converted to .unwrap_or_else(|| unreachable!(...)) in an earlier sub-PR of this stack.

Quarantines dropped:

  • `lading/src/lib.rs`
  • `lading/src/bin/lading.rs`
  • `lading/src/bin/captool/main.rs`
  • `lading/src/bin/payloadtool.rs`

New fn-level annotations:

  • `bin/lading.rs::{CliKeyValues::from_str, get_config, inner_main}`
  • `bin/captool/main.rs::{main, validate_capture, report_validation_result}`
  • `bin/payloadtool.rs::{generate_and_check, check_generator}`
  • `generator/process_tree.rs::spawn_tree`
  • `generator/tcp_rr.rs::{spin, client_thread_main}` (FIXME on spin — user-supplied addr)
  • `generator/udp.rs::{Udp::new, UdpWorker::spin}` (FIXME on Udp::new — user-supplied addr)

Stacked on #1900. With this stack landed, the lading crate is ready for clippy::expect_used = "deny" enforcement to be flipped on workspace-wide.

Test plan

  • `./ci/validate` passes
  • CI green on this PR

Annotate the lading bin entry points (lading, captool, payloadtool),
drop the `#![allow(clippy::expect_used)]` quarantine from lib.rs and
the three bin entrypoints, and annotate previously-masked production
sites in process_tree, tcp_rr, and udp generators that the quarantine
drop surfaces.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor Author

goxberry commented May 23, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

Pipelines

Fix all issues with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

Changelog Check | changelog-check   View in Datadog   GitHub Actions

🛟 This job is unlikely to succeed on retry. Please review your pipeline configuration. No changes to CHANGELOG.md detected. Add 'no-changelog' label if this is intentional.

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ac59396 | Docs | Datadog PR Page | Give us feedback!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant