Skip to content

Conversation

@l5yth
Copy link
Owner

@l5yth l5yth commented Jan 10, 2026

Config loading now merges optional TOML with CLI/env/secret inputs, adds container-aware defaults, and ships a new clap-based CLI so the bridge can be configured dynamically without hard‑coding Config.toml. I also updated container entrypoint defaults, docs, and the Rust toolchain to 1.92.

  • Added a clap CLI with config/state/override flags and wired it into bootstrap (matrix/src/cli.rs, matrix/src/main.rs)
  • Reworked configuration loading to merge TOML + CLI/env + secret files, added container detection/defaults, and expanded unit tests for precedence and container behavior (matrix/src/config.rs)
  • Updated container entrypoint to set container awareness + secrets dir defaults without forcing a config path (matrix/docker-entrypoint.sh)
  • Documented precedence, env vars, secret files, and container defaults in matrix/README.md
  • Bumped the builder image to Rust 1.92 and regenerated the lockfile (matrix/Dockerfile, matrix/Cargo.lock)
  • Replaces matrix: add docker env boilerplate #605

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d1182bbe47

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@codecov
Copy link

codecov bot commented Jan 10, 2026

Codecov Report

❌ Patch coverage is 95.82505% with 21 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
matrix/src/config.rs 95.82% 21 Missing ⚠️

📢 Thoughts on this report? Let us know!

@l5yth l5yth merged commit fed8b9e into main Jan 10, 2026
8 checks passed
@l5yth l5yth deleted the l5y-matrix-docker-enw branch January 10, 2026 22:39
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