Conversation
Direnv uses .envrc for environment variables which may contain secrets. This file should not be committed to the repository. 👾 Generated with [Letta Code](https://letta.com) Co-Authored-By: Letta <noreply@letta.com>
Implements Tchap/Matrix integration using matrix-bot-sdk. - Adds src/channels/tchap.ts with E2EE, DM pairing, and editing support - Updates main.ts to load Tchap adapter - Adds configuration to .env.example - Fixes WhatsApp adapter build error (legacy export)
…roduce linting and formatting scripts.
cpfiffer
left a comment
cpfiffer
left a comment
There was a problem hiding this comment.
Thanks for this contribution! The adapter is well-implemented with good E2EE support, French localization, and clean integration with the existing patterns.
Suggestion: Rename from "Tchap" to "Matrix"
Since the adapter works with any Matrix homeserver (matrix.org, self-hosted Synapse, Tchap, etc.), I'd suggest marketing it as a Matrix adapter rather than Tchap-specific:
Renames:
TchapAdapter→MatrixAdaptersrc/channels/tchap.ts→src/channels/matrix.tsTCHAP_*env vars →MATRIX_*(e.g.,MATRIX_HOMESERVER_URL,MATRIX_ACCESS_TOKEN)TchapConfig→MatrixConfig- Channel id:
"tchap"→"matrix"
Docs:
- Rename
docs/tchap-setup.md→docs/matrix-setup.md - Title: "Matrix Setup Guide (including Tchap)"
- Mention Tchap as a specific use case for French government users
This makes the adapter more discoverable and useful to the broader Matrix community while still supporting Tchap users.
The French localization for pairing messages is a nice touch - we could keep that as a config option or detect based on homeserver URL.
Let me know what you think!
…gration and updating related scripts and documentation.
…s for generic Matrix support
…and remove explicit Tchap configuration.
|
How is this PR going? I'm happy to take it over. |
|
Closing this draft for stale triage cleanup. It has been inactive/conflicting for over a month and is largely superseded by current Matrix-focused work in #430. If you want to continue this line of work, please reopen with a refreshed branch (or open a new PR from latest main). |
2 participants
Adds support for Tchap (French government Matrix instance) and general Matrix servers.
Includes:
Tested on matrix.i.tchap.gouv.fr and matrix.dev01.tchap.incubateur.net.