Skip to content

feat: support managed bot autostart#2

Merged
Yanu403 merged 1 commit into
masterfrom
fix/autostart-managed-bot
Jul 11, 2026
Merged

feat: support managed bot autostart#2
Yanu403 merged 1 commit into
masterfrom
fix/autostart-managed-bot

Conversation

@Yanu403

@Yanu403 Yanu403 commented Jul 11, 2026

Copy link
Copy Markdown
Owner

What changed

  • add opt-in DIGITAL_MATE_AUTO_START_BOT dashboard lifecycle setting
  • keep development/default behavior unchanged
  • document the setting and test startup/shutdown lifecycle

Validation

  • python -m pytest tests/test_web.py -q -W error::DeprecationWarning
  • git diff --check

@Yanu403
Yanu403 merged commit 109fc74 into master Jul 11, 2026
2 checks passed
@Yanu403
Yanu403 deleted the fix/autostart-managed-bot branch July 11, 2026 08:25

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

Copy link
Copy Markdown

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: d8a0d2c3d9

ℹ️ 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".

Comment thread digital_mate/web/app.py
Comment on lines +182 to +184
auto_start_bot = os.environ.get(
"DIGITAL_MATE_AUTO_START_BOT", ""
).strip().lower() in {"1", "true", "yes", "on"}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Honor .env for the autostart flag

When DIGITAL_MATE_AUTO_START_BOT=true is added to the project .env (the same place this commit documents the setting), the dashboard still will not autostart the bot because this check reads only os.environ. Other dashboard settings such as DASHBOARD_API_KEY fall back to SettingsService, and the CLI does not load .env into os.environ, so installs that configure the new option in .env and run digital-mate serve will continue to report the bot as stopped unless the variable is separately exported.

Useful? React with 👍 / 👎.

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.

1 participant