Skip to content

[gardener] Daemon sync sweep should honor --open-issues via sync_mode config knob #319

@serenakeyitan

Description

@serenakeyitan

Follow-up narrowed from #184 after @bingran-you's review.

What's already shipped

  • Pull-mode daemon loop: `gardener start/stop/status/run-once` (launchd-backed) — feat(gardener): pull-mode daemon (start/stop/status/run-once) #243, see `src/products/gardener/engine/daemon/loop.ts:168`.
  • `lastReconciledSourceCommit` per-binding cursor in `.first-tree/source.json` drives the since-window scan (`sync.ts` L1933/1972/1997/2306).

What's missing

`defaultRunSweep` currently runs sync in its default mode only. It doesn't know how to run `--open-issues`, so the proactive owner-assignment path that #184 wanted only fires when a human invokes the CLI with that flag.

Proposal

Add a `sync_mode` knob in `~/.gardener/config.json` (values: `propose` | `apply` | `open-issues`; default `propose` to preserve today's behavior). Wire it into `defaultRunSweep` at `src/products/gardener/engine/daemon/loop.ts` so the daemon honors the configured mode on each sweep.

Per-binding override can come later if needed — global knob is enough for the first cut.

@bingran-you offered to own this in the #184 thread.

Metadata

Metadata

Assignees

Labels

breeze:wipbreeze is actively working on it

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions