Skip to content

feat(dictyon): trace wire noise transport phases - #38

Merged
forkwright merged 1 commit into
mainfrom
codex/issue-20-wire-noise-tracing
May 25, 2026
Merged

feat(dictyon): trace wire noise transport phases#38
forkwright merged 1 commit into
mainfrom
codex/issue-20-wire-noise-tracing

Conversation

@forkwright

@forkwrightforkwright commented May 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • add structured tracing::debug! events in dictyon wire, transport, and noise paths
  • cover connection phase transitions, Noise handshake frame lengths, and encrypted frame sizes
  • avoid logging key material, auth material, payload bytes, ciphertext bytes, or handshake header values

Scope

Progresses #20 by covering the remaining transport/wire/noise tracing instrumentation slice. This does not close#20; transport/wire API cleanup and the NodeKeyExpiry/OldNodeKey lifecycle remain open.

Gates

  • cargo fmt --all --check
  • cargo check --workspace --all-targets
  • cargo test --workspace --all-targets
  • cargo clippy --workspace --all-targets
  • git diff --check

Note: clippy still prints the existing PeerRemovalWire enum-prefix warning from current main, but exits successfully.

Refs #20

Add structured debug events for the remaining dictyon wire, transport, and noise paths under #20. Events cover connection phase transitions, handshake frame lengths, and encrypted frame sizes without logging key material, auth material, or payload bytes.
Refs #20
Gate-Passed: cargo fmt --all --check
Gate-Passed: cargo check --workspace --all-targets
Gate-Passed: cargo test --workspace --all-targets
Gate-Passed: cargo clippy --workspace --all-targets
Gate-Passed: git diff --check
@forkwright
forkwright merged commit 28fffa9 into mainMay 25, 2026
6 checks passed
@forkwright
forkwright deleted the codex/issue-20-wire-noise-tracing branch May 25, 2026 19:46
forkwright added a commit that referenced this pull request May 29, 2026
Add structured debug events for the remaining dictyon wire, transport, and noise paths under #20. Events cover connection phase transitions, handshake frame lengths, and encrypted frame sizes without logging key material, auth material, or payload bytes.
Refs #20
Gate-Passed: cargo fmt --all --check
Gate-Passed: cargo check --workspace --all-targets
Gate-Passed: cargo test --workspace --all-targets
Gate-Passed: cargo clippy --workspace --all-targets
Gate-Passed: git diff --check
@github-actionsgithub-actionsBot mentioned this pull request Jul 16, 2026
forkwright pushed a commit that referenced this pull request Jul 28, 2026
🤖 I have created a release *beep* *boop*
---
<details><summary>0.2.0</summary>
## [0.2.0](v0.1.0...v0.2.0)
(2026-07-28)
### Features
* **_llm:** add T0 corpus per
[#667](https://github.com/forkwright/hamma/issues/667) /
[#673](https://github.com/forkwright/hamma/issues/673) fleet rollout
([#10](#10))
([0568f51](0568f51))
* **control:** instrument async control client entry points
([#37](#37))
([e80622f](e80622f)),
closes [#20](#20)
* **control:** support zstd map responses
([dd5ab90](dd5ab90))
* **dictyon:** add TCP/TLS connection, registration, and map streaming
([a311d8a](a311d8a))
* **dictyon:** control protocol types and map response parser
([dfc25c7](dfc25c7))
* **dictyon:** migrate tracing init to koinon
([e5a4260](e5a4260))
* **dictyon:** Noise IK handshake, key types, HTTP transport skeleton
([aafea4f](aafea4f))
* **dictyon:** trace wire noise transport phases
([#38](#38))
([2d80231](2d80231)),
closes [#20](#20)
### Bug Fixes
* **cargo:** track lockfile for pinned rust toolchain
([#30](#30))
([e6a2f01](e6a2f01)),
closes [#29](#29)
* **ci:** resolve cargo-deny + MSRV + binary smoke failures
([#13](#13))
([9bb4533](9bb4533))
* **ci:** waive gate attestation by PR author, not by github.actor
([#69](#69))
([764422e](764422e)),
closes [#68](#68)
* **control:** accept node id peer removals
([7debc6b](7debc6b))
* **control:** apply peer patch map deltas
([#33](#33))
([dfc3731](dfc3731))
* **core:** parse peer patch map fields
([0686a66](0686a66))
* **deps:** clear RUSTSEC-2026-0190 via anyhow lockfile bump
([#56](#56))
([2423485](2423485))
* **lint:** add non_exhaustive to public error enums, mark public-key
fields
([0355297](0355297))
* **lint:** mechanical wins — allow→expect, indexing/slicing, casts,
http→https
([9e339db](9e339db))
* **lint:** resolve clippy warnings in hamma-core and wire integration
test ([#42](#42))
([079ad5e](079ad5e))
* **lint:** suppress pub-visibility for library API surface
([6d54674](6d54674))
* **lint:** unblock kanon gate
([f93ff63](f93ff63))
* **release:** bump the internal hamma-core pin and Cargo.lock with the
release ([#71](#71))
([dfdfc35](dfdfc35)),
closes [#70](#70)
* resolve 1 lint violations via local
([#8](#8))
([0d4aa84](0d4aa84))
### Refactoring
* **dictyon:** replace expect with ? and rename test helper
([#9](#9))
([655b783](655b783))
* **lint:** split oversized modules; add hamma-core integration tests
([fa8f54f](fa8f54f))
* rename plegma→hamma, plegma-core→hamma-core
([a67f792](a67f792))
### Documentation
* add CLAUDE.md precedence preamble
(forge[#153](https://github.com/forkwright/hamma/issues/153))
([e91ebea](e91ebea))
* add CONTRIBUTING.md for 05e cutover
([#1](#1))
([9dd5f87](9dd5f87))
* add llms.txt per kanon doc standards (refs
[#10](#10))
([#11](#11))
([e869fe1](e869fe1))
* **agents:** add AGENTS.md per fleet repo-structure standard
([#40](#40))
([0382da1](0382da1))
* **hamma:** align pre-alpha status
([#6](#6))
([873a6c3](873a6c3))
* **hamma:** replace standards copy with kanon pointer
([#8](#8))
([324ff18](324ff18))
* sanitize local bootstrap docs
([4c57d40](4c57d40))
* **standards:** add canonical standards from kanon
([#1](#1))
([81e5007](81e5007))
</details>
---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for freeto 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.

Phase 3 audit: missing PeersChangedPatch + Node fields, no zstd, transport duplication, no tracing, no map-stream integration test

1 participant

@forkwright