Skip to content

Add error handling for Q&A trees preload failure#7

Draft
joestein with Copilot wants to merge 2 commits into
graph-qafrom
copilot/sub-pr-5-again
Draft

Add error handling for Q&A trees preload failure#7
joestein with Copilot wants to merge 2 commits into
graph-qafrom
copilot/sub-pr-5-again

Conversation

Copilot AI commented Dec 2, 2025

Copy link
Copy Markdown

Addresses feedback from PR #5: wrap preload_directory() in error handling that logs and exits on failure.

Changes

  • Wrap qna_service.preload_directory() in try-except
  • Log error and call sys.exit(1) on failure to prevent startup in broken state
try:
    await qna_service.preload_directory(config_dir)
except Exception as e:
    logger.error(f"Failed to preload Q&A trees: {e}")
    sys.exit(1)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: joestein <115151+joestein@users.noreply.github.com>
Copilot AI changed the title [WIP] Update to address feedback from review on Q&A graph chat Add error handling for Q&A trees preload failure Dec 2, 2025
Copilot AI requested a review from joestein December 2, 2025 00:35
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.

2 participants