Skip to content

chore: gate embed p2p transport on cargo feature#157

Merged
kaichaosun merged 2 commits into
mainfrom
refactor-embed-transport-feature
Jul 2, 2026
Merged

chore: gate embed p2p transport on cargo feature#157
kaichaosun merged 2 commits into
mainfrom
refactor-embed-transport-feature

Conversation

@kaichaosun

Copy link
Copy Markdown
Contributor

Refactor on feature usage in embed p2p transport, use a consistent cargo feature, not env-dependent cfg

@kaichaosun kaichaosun requested review from jazzz and osmaczko July 1, 2026 02:08
@jazzz

jazzz commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

What is the plan for chat-cli? Does this change aim to remove the the transport choice within chat-cli and commit to using embedded λDelivery?

The existing approach, (supporting both cargo feature declaration as wells as envvars) was needed was to support the current setup of chat-cli, which allowed the delivery mechanism to be chosen at compile time via envvars.

@kaichaosun

Copy link
Copy Markdown
Contributor Author

Another PR after this change:
https://github.com/logos-messaging/libchat/compare/refactor-embed-transport-feature...more-transport-features?expand=1

I don't see any hard blocker to keep the env feature.

@jazzz jazzz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't see any hard blocker to keep the env feature.

If you mean that you want to remove the "env feature" from the cli app then I don't see any issue with this change.

I'd further suggest these changes for the future PR.

  • removing the '--transport' flag all together. Embedded delivery needs to be compiled anyway. Remove the added complexity and simplify usage.
  • Logos-delivery should be reserved for logos-delivery via logos-core. Use embedded* to refer to linking directly to the compiled library.

@kaichaosun kaichaosun force-pushed the refactor-embed-transport-feature branch from 41f6577 to a0a2362 Compare July 2, 2026 16:35
@kaichaosun kaichaosun merged commit 943ba29 into main Jul 2, 2026
5 checks passed
@kaichaosun kaichaosun deleted the refactor-embed-transport-feature branch July 2, 2026 17:13
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