arrow_backRetour aux issues
LearningCircuit/local-deep-research
#5676
Débutant
Ouvrirarrow_forward
Débutant
Ouvrirarrow_forward
Débutant
Ouvrirarrow_forward
tests: import-validation docstring cites brave default_params.time_period, which never gained an options list
ecoDébutant
documentation
good first issue
priority: low
tests
descriptionDescription
Follow-up to #5597 (fixes #5589).
The module docstring of `tests/settings/test_import_settings_validation.py:6` cites the motivating example as:
> a free-text `search.engine.web.brave.default_params.time_period` value from before the field gained an options list
But that key has not gained an options list. In the current defaults:
| key | `ui_element` | `options` |
|---|---|---|
| `search.engine.web.brave.default_params.time_period` | `text` | `null` |
| `search.engine.web.serpapi.default_params.time_period` | `text` | `null` |
| `search.engine.web.serper.default_params.time_period` | `select` | day / week / month / year / null |
| `search.engine.web.tavily.default_params.time_period` | `select` | d / w / m / y |
**Impact:** a maintainer tracing the regression to the cited key finds brave's `time_period` is still free-text, which undercuts the documented motivation and sends reproduction effort to the wrong setting.
**Scope:** docstring only. The tests themselves correctly exercise `search.time_period` and the validation logic is unaffected.
**Suggested fix:** point the example at a key that actually carries an options list — `search.engine.web.serper.default_params.time_period` or the tavily equivalent — or drop the engine qualifier so the example matches the changelog's generic `default_params.time_period` phrasing.
Issues similaires
calkit/calkit
star53
Poids du dépôt moyen
VS Code extension should be robust to YAML parser errors
Seeing this error: ``` Failed to read calkit.yaml: YAMLParseError: A block sequence may not be used as an implicit map…
Python
bug
good first issue
fu351/Doberman-Core
star211
Poids du dépôt léger
dash: a manual Refresh control
The dashboard polls: `refreshStats()` (`src/doberman/dash/app.py:408`) every 5 s and `refreshPending()` (`:546`) every …
Python
enhancement
good first issue
fu351/Doberman-Core
star211
Poids du dépôt léger
dash: "Copy details" button on each pending-approval card
Each pending-approval card in the dashboard (`renderPending`, `src/doberman/dash/app.py:448-544`) shows the risk badge,…
Python
enhancement
good first issue