Skip to content

fix(config): add YAML config support for daemon auto-sync settings#1294

Merged
steveyegge merged 1 commit intosteveyegge:mainfrom
irbull:config-auto-sync
Jan 27, 2026
Merged

fix(config): add YAML config support for daemon auto-sync settings#1294
steveyegge merged 1 commit intosteveyegge:mainfrom
irbull:config-auto-sync

Conversation

@irbull
Copy link
Contributor

@irbull irbull commented Jan 24, 2026

Add ability to configure daemon.auto-sync, daemon.auto-commit, daemon.auto-push, and daemon.auto-pull via config.yaml.

Priority order: CLI flag > env var > YAML config > database config

Individual YAML settings allow fine-grained control without triggering the legacy "enable everything" behavior. Tests cover unified and individual settings, env var overrides, and CLI flag overrides.

Add ability to configure daemon.auto-sync, daemon.auto-commit,
daemon.auto-push, and daemon.auto-pull via config.yaml.

Priority order: CLI flag > env var > YAML config > database config

Individual YAML settings allow fine-grained control without triggering
the legacy "enable everything" behavior. Tests cover unified and
individual settings, env var overrides, and CLI flag overrides.
@irbull
Copy link
Contributor Author

irbull commented Jan 25, 2026

Any thoughts on this? Essentially the problem is that the daemon needs to be started with the --auto-commit because it doesn't read it from the config.yaml. However, the daemon won't start with this setting if you have auto-start on. Took me a while to figure out how to even get this to work, because each time I killed the daemon (so I could start it with auto-commit) it automatically started without this setting.

There is bd config set daemon.auto-commit true but this just sets the flag in the config.yaml, which isn't read (until this is merged ;).

Copy link
Owner

@steveyegge steveyegge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean implementation of YAML config support for daemon auto-sync settings. The priority order (CLI > env > YAML > database) is consistent with existing patterns. Good test coverage for unified and individual settings, plus env var overrides. LGTM.

🤖 Reviewed by PR Sheriff (beads/crew/emma)

@steveyegge steveyegge merged commit 7448754 into steveyegge:main Jan 27, 2026
groblegark pushed a commit to groblegark/beads that referenced this pull request Jan 31, 2026
…teveyegge#1294)

Add ability to configure daemon.auto-sync, daemon.auto-commit,
daemon.auto-push, and daemon.auto-pull via config.yaml.

Priority order: CLI flag > env var > YAML config > database config

Individual YAML settings allow fine-grained control without triggering
the legacy "enable everything" behavior. Tests cover unified and
individual settings, env var overrides, and CLI flag overrides.
groblegark pushed a commit to groblegark/beads that referenced this pull request Jan 31, 2026
…teveyegge#1294)

Add ability to configure daemon.auto-sync, daemon.auto-commit,
daemon.auto-push, and daemon.auto-pull via config.yaml.

Priority order: CLI flag > env var > YAML config > database config

Individual YAML settings allow fine-grained control without triggering
the legacy "enable everything" behavior. Tests cover unified and
individual settings, env var overrides, and CLI flag overrides.
groblegark pushed a commit to groblegark/beads that referenced this pull request Jan 31, 2026
…teveyegge#1294)

Add ability to configure daemon.auto-sync, daemon.auto-commit,
daemon.auto-push, and daemon.auto-pull via config.yaml.

Priority order: CLI flag > env var > YAML config > database config

Individual YAML settings allow fine-grained control without triggering
the legacy "enable everything" behavior. Tests cover unified and
individual settings, env var overrides, and CLI flag overrides.
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

Comments