Skip to content

Add startup configuration validation for prebuilt agents#126

Open
CommitHu502Craft wants to merge 1 commit intojentic:mainfrom
CommitHu502Craft:feature/issue-83-config-validation
Open

Add startup configuration validation for prebuilt agents#126
CommitHu502Craft wants to merge 1 commit intojentic:mainfrom
CommitHu502Craft:feature/issue-83-config-validation

Conversation

@CommitHu502Craft
Copy link
Copy Markdown

Summary

  • add fail-fast config validation when creating prebuilt agents
  • require either LLM_MODEL or an explicit model=... value
  • require JENTIC_AGENT_API_KEY for prebuilt agents and AWS_BEARER_TOKEN_BEDROCK for Bedrock variants
  • add focused tests for missing-config and success paths

Closes #83.

Validation

  • uv run pytest tests/agents/test_prebuilt.py -q
  • uv run ruff check agents/prebuilt.py tests/agents/test_prebuilt.py
  • uv run mypy --follow-imports=skip agents/prebuilt.py tests/agents/test_prebuilt.py
  • $env:LLM_MODEL=''test-model''; uv run pytest -q

Notes

  • I also installed the optional observability dependencies and tzdata locally so the full test suite could run cleanly on this Windows environment.
  • No repository files related to dependency locking were included in the patch.

@CommitHu502Craft CommitHu502Craft requested a review from a team as a code owner March 29, 2026 04:12
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.

[Good First Issue] [Enhancement] Add configuration validation on agent startup

1 participant