feat: mongo srv - #28
Merged
Merged
Conversation
Feature/new workflows
The settings registry now serves both components. The agent gains the five variables its documentation lists but the CLI never exposed — DATA_PATH, TMPDIR, RETRY_ATTEMPTS, RETRY_BACKOFF_MS, SSL_CERT_FILE — plus LOG_LEVEL as a choice; agent create derives its flags from the registry like dashboard create does. host_gateway is a compose fact rather than a variable, so the registry allows env-less settings that map to a project attribute. show, set and unset are one implementation shared by both components, parameterised by the project loader and renderer. The agent template renders whichever optional variables are set, so they reach the container without an env_file (which would also expose the managed database credentials). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
… at it The agent uses rustls, which ignores the system CA store, so the only way to trust an internal CA is SSL_CERT_FILE — and the file it names must exist inside the container. Setting the variable alone would have produced an agent that fails at startup. 'agent set NAME ca_bundle ./ca-bundle.crt' now bind-mounts the host file read-only at a fixed path and sets SSL_CERT_FILE to that path. The host path lives in .env as CA_BUNDLE for compose interpolation only; it is not passed to the container. set refuses a path that does not exist, and the help says why the bundle must include the Mozilla roots. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
…es in the mount The host path of the CA bundle is not a container variable, so it no longer goes in .env. It is a compose fact like host_gateway: written as the bind-mount source and read back from the existing compose on load. SSL_CERT_FILE stays in .env with the in-container path, which is what the agent consumes. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
Rename commands/dashboard_auth.py to commands/auth.py. Both arguments of 'dashboard auth add' become optional and are asked interactively when omitted (kind as a choice, OAuth provider from the known list, OIDC id as free text). 'dashboard auth remove' without an id offers the configured providers as a selection list. Non-interactive mode keeps failing with 'Missing --kind' / 'Missing --id'. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
Rename commands/dashboard_auth.py to commands/auth.py. Both arguments of 'dashboard auth add' become optional and are asked interactively when omitted (kind as a choice, OAuth provider from the known list, OIDC id as free text). 'dashboard auth remove' without an id offers the configured providers as a selection list. Non-interactive mode keeps failing with 'Missing --kind' / 'Missing --id'. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbemoDqsmMdRq6PS5Vaup8
update: new CLI
build: replace cryptography with pycryptodome for AES-GCM
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-authored-by: Asuniia <43389096+Asuniia@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.