Skip to content

feat(cli): warn on missing tavily key, add /notifications#2555

Merged
Mason Daugherty (mdrxy) merged 2 commits into
mainfrom
mdrxy/cli/tavily-warn
Apr 8, 2026
Merged

feat(cli): warn on missing tavily key, add /notifications#2555
Mason Daugherty (mdrxy) merged 2 commits into
mainfrom
mdrxy/cli/tavily-warn

Conversation

@mdrxy

@mdrxy Mason Daugherty (mdrxy) commented Apr 8, 2026

Copy link
Copy Markdown
Member

Add a startup warning when TAVILY_API_KEY is not set, alerting users that web search is unavailable. Also introduce a /notifications slash command with a modal settings screen so users can toggle individual startup warnings on or off without hand-editing config.toml.

Changes

  • Add tavily to check_optional_tools — checks settings.has_tavily and the [warnings].suppress config, matching the existing ripgrep pattern
  • Add NotificationSettingsScreen modal with per-warning checkboxes that persist immediately to ~/.deepagents/config.toml via suppress_warning/unsuppress_warning
  • Add unsuppress_warning() in model_config.py as the inverse of suppress_warning, with the same atomic temp-file write strategy
  • Split the old _RIPGREP_SUPPRESS_HINT into _SUPPRESS_HINT_TUI (points to /notifications) and _SUPPRESS_HINT_CLI (shows config.toml snippet with a <key> placeholder), and add Tavily-specific formatting for both output modes
  • Harden suppress_warning and unsuppress_warning against [warnings].suppress not being a list

@github-actions github-actions Bot added cli Related to `deepagents-cli` internal User is a member of the `langchain-ai` GitHub organization size: S 50-199 LOC labels Apr 8, 2026
Comment thread libs/cli/tests/unit_tests/test_main.py Dismissed
@github-actions github-actions Bot added size: M 200-499 LOC and removed size: S 50-199 LOC labels Apr 8, 2026
@mdrxy Mason Daugherty (mdrxy) changed the title (wip) warn for missing tavily api key feat(cli): warn on missing tavily key, add /notifications settings Apr 8, 2026
@mdrxy Mason Daugherty (mdrxy) changed the title feat(cli): warn on missing tavily key, add /notifications settings feat(cli): warn on missing tavily key, add /notifications Apr 8, 2026
@github-actions github-actions Bot added the feature New feature/enhancement or request for one label Apr 8, 2026
@mdrxy
Mason Daugherty (mdrxy) marked this pull request as ready for review April 8, 2026 04:48
@mdrxy
Mason Daugherty (mdrxy) enabled auto-merge (squash) April 8, 2026 04:53
@mdrxy
Mason Daugherty (mdrxy) merged commit 3dff3ed into main Apr 8, 2026
43 of 47 checks passed
@mdrxy
Mason Daugherty (mdrxy) deleted the mdrxy/cli/tavily-warn branch April 8, 2026 04:54
Mason Daugherty (mdrxy) added a commit that referenced this pull request Apr 9, 2026
> [!CAUTION]
> Merging this PR will automatically publish to **PyPI** and create a
**GitHub release**.

For the full release process, see
[`.github/RELEASING.md`](https://github.com/langchain-ai/deepagents/blob/main/.github/RELEASING.md).

---

_Everything below this line will be the GitHub release body._

---


##
[0.0.36](deepagents-cli==0.0.35...deepagents-cli==0.0.36)
(2026-04-09)

### Features

* `deepagents deploy`
([#2491](#2491))
([01dc60e](01dc60e))
* Warn on missing tavily key, add `/notifications`
([#2555](#2555))
([3dff3ed](3dff3ed))

### Bug Fixes

* Fail fast on missing provider credentials
([#2554](#2554))
([50fb8ae](50fb8ae))
* Fix mktemp template in debug script for macOS
([#2603](#2603))
([63fa537](63fa537))

---

_Everything above this line will be the GitHub release body._

---

> [!NOTE]
> A **New Contributors** section is appended to the GitHub release notes
automatically at publish time (see [Release
Pipeline](https://github.com/langchain-ai/deepagents/blob/main/.github/RELEASING.md#release-pipeline),
step 2).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Mason Daugherty <[email protected]>
james8814 pushed a commit to james8814/deepagents that referenced this pull request May 1, 2026
…n-ai#2555)

Add a startup warning when `TAVILY_API_KEY` is not set, alerting users
that web search is unavailable. Also introduce a `/notifications` slash
command with a modal settings screen so users can toggle individual
startup warnings on or off without hand-editing `config.toml`.

- Add `tavily` to `check_optional_tools` — checks `settings.has_tavily`
and the `[warnings].suppress` config, matching the existing `ripgrep`
pattern
- Add `NotificationSettingsScreen` modal with per-warning checkboxes
that persist immediately to `~/.deepagents/config.toml` via
`suppress_warning`/`unsuppress_warning`
- Add `unsuppress_warning()` in `model_config.py` as the inverse of
`suppress_warning`, with the same atomic temp-file write strategy
- Split the old `_RIPGREP_SUPPRESS_HINT` into `_SUPPRESS_HINT_TUI`
(points to `/notifications`) and `_SUPPRESS_HINT_CLI` (shows
`config.toml` snippet with a `<key>` placeholder), and add
Tavily-specific formatting for both output modes
- Harden `suppress_warning` and `unsuppress_warning` against
`[warnings].suppress` not being a list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli Related to `deepagents-cli` feature New feature/enhancement or request for one internal User is a member of the `langchain-ai` GitHub organization size: M 200-499 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants