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.

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

Support wires in NockApps #124

Merged
merged 5 commits into from
Jan 27, 2025
Merged

Support wires in NockApps #124

merged 5 commits into from
Jan 27, 2025

Conversation

ryjm
Copy link
Contributor

@ryjm ryjm commented Jan 23, 2025

Adds Wire trait to standardize (driver/module)-kernel communication format. Implements Wire for all drivers (npc, http, file, timer, one_punch) with versioned wire formats.

Updates kernel poke interface to accept wire parameter. Note that there is no enforcement on the format of this noun, though the kernel will expect a path noun.

Adds NpcWire enum with variants for different NPC message types (Poke, Pack, Nack, Bind) that implements the Wire trait. Each variant includes a process ID (pid) and follows a standard wire format: [source=npc version=1 tag pid].

Adds tests for the npc driver.

@ryjm ryjm marked this pull request as ready for review January 23, 2025 22:40
@ryjm ryjm changed the title wip: Support wires in NockApps Support wires in NockApps Jan 23, 2025
ryjm added 5 commits January 24, 2025 15:38
Adds Wire trait to standardize (driver/module)-kernel communication
format. Implements Wire for all drivers (npc, http, file, timer,
one_punch) with versioned wire formats.

Updates kernel poke interface to accept wire parameter. Note that there
is no enforcement on the format of this noun, though the kernel will
expect a path noun.

Adds NpcWire enum with variants for different NPC message types (Poke,
Pack, Nack, Bind) that implements the Wire trait. Each variant includes a
process ID (pid) and follows a standard wire format: [source=npc
version=1 tag pid].

Adds tests for the npc driver.
@eamsden eamsden merged commit 37084bd into master Jan 27, 2025
1 check 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.

2 participants