Skip to content

Preserve recording files and expose typed acquisition metadata - #6

Merged
han-xudong merged 11 commits into
mainfrom
fix/ecosystem-improvements
Oct 8, 2026
Merged

han-xudong merged 11 commits into
mainfrom
fix/ecosystem-improvements

Conversation

@han-xudong

@han-xudong han-xudong commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

Preparing netft-cli 0.3.0 against published SDK v0.3.4 (1ce68a08a9387d9a36d22c3c259e44533199c347). Final required PR checks are running; signed release publication also requires a trusted Git tag signing key. Existing tag signature and publication approval gates remain enforced.

Recording risks overwriting a competing destination and can report success without usable samples. Keep an exclusive handle, promote output without replacement, retain partial files on failure, and return bounded no-data/fault outcomes. Add shared acquisition summaries and a typed command inventory with clean source identity.

Pin the SDK candidate from netft/netft-cpp#22 while preserving the checksum-verified bias/socket adapter. Fix deadline saturation and clang-tidy findings, and replay adaptation bytes independently of Windows autocrlf.

Validation: five native platforms passed run 37609013125; after adding the snapshot guard, four platforms passed run 37610041333. Windows snapshot verification and 30 native suites passed there, with one adaptation-tool failure fixed here. Final focused Windows run 37611614857 passed snapshot verification, all 31 native/tool suites and the PowerShell installer checks. Local affected native/process/tool tests passed; no release is published.

Branch naming update: this draft replaces #5 after the source branch was renamed to fix/ecosystem-improvements. The commit and implementation are unchanged. Original discussion and same-commit acceptance evidence remain on the previous PR; product tests are not repeated solely for the naming change.

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: eaab6246-80ed-4bb7-96db-0e074a357f9c
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@han-xudong
han-xudong marked this pull request as ready for review October 8, 2026 01:31
@han-xudong
han-xudong merged commit 55af6b6 into main Oct 8, 2026
10 checks passed
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.

1 participant