Skip to content

Gracefully handle missing config file in watcher - #14

Open
karlTGA wants to merge 1 commit into
mainfrom
codex/modify-watch_config-to-handle-missing-files-3uexwk
Open

Gracefully handle missing config file in watcher#14
karlTGA wants to merge 1 commit into
mainfrom
codex/modify-watch_config-to-handle-missing-files-3uexwk

Conversation

@karlTGA

@karlTGA karlTGA commented Aug 31, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Continue watching when the config file is removed, logging a warning and skipping cron generation until it returns
  • Add regression test simulating config file removal and recreation without Docker or network dependencies
  • Start watcher even if the config file is missing at startup and skip initial cron generation with warning
  • Simplify test runner to execute all test scenarios
  • Merge latest upstream changes with strict bash options

Testing

  • bash tests/config_parser_test.sh
  • bash tests/watch_config_test.sh
  • bash test.sh

https://chatgpt.com/codex/tasks/task_e_68b42c2b8158832f812b93748b77f615

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant