Decide provider and daemon execution model#111
Conversation
Hermes-omt
left a comment
There was a problem hiding this comment.
Blocked pending branch refresh: head 0fca069 is behind current main d7f88dc and GitHub reports mergeable=false, mergeable_state=dirty. Please rebase/update the PR branch, resolve conflicts, and wait for fresh CI before re-review. Exact-head local inspection and swift test passed (110 tests); public reviews and inline review comments are empty.
Expose a deterministic metadata-only provider registry through JSON and text output, covering every current Apple-data command family with maturity, source, permission, sensitivity, capability, and polling declarations. Document the v1 compatibility policy.\n\nCloses #89.
Expose a versioned external manifest derived from the provider registry, document bounded local wrapper policy, and cover it with synthetic contract tests. Co-authored-by: Hermes <Hermes-omt@protonmail.com>
Build universal macOS DMGs with stable code identity, notarization, stapling, checksums, release evidence, and governed secrets. Reconcile v0.2 versioning, bootstrap release policy, onboarding, installation, and security decisions.
Compare CLI, daemon, helper, and delegated models across lifecycle, IPC, TCC, isolation, archives, signing, and recovery. Classify every provider and keep daemon mode and mutations outside the default boundary pending human approval.
0fca069 to
8dc58ce
Compare
Hermes-omt
left a comment
There was a problem hiding this comment.
Exact-head macOS review completed for 8dc58ce7e7cef18a39f626009102c5f1f13ec44a. The branch is current with main, mergeable/clean, auto-merge is enabled (squash), and no unresolved review comments were found. I inspected the full diff, execution-model call paths, release scripts, and adjacent parser tests; git diff --check and local fast checks passed (110 tests). GitHub Actions is successful, but the claude and cloudflare-workers-and-pages check suites remain queued after a bounded wait. Please let those checks settle, then rerun exact-head review before approval or merge.
Summary
Governing Issue
Refs #83
This issue is explicitly human-required. The PR proposes a decision but does not mark the ADR accepted or close the issue before non-author product, architecture, and security approval.
Validation
Relevant local checks passed
Required PR checks are expected to satisfy
CI GateSkipped checks are explained below
git diff --check— passedbash scripts/ci/run-fast-checks.sh— passed; 110 tests, 88.58% source coverage, mutation checks, debug and release buildsNo checks skipped.
Bootstrap Governance
CONTRIBUTING.md,.github/PULL_REQUEST_TEMPLATE.md, anddocs/bootstrap/onboarding.mdwhen applicableNo contributor guidance changed.
Flow Contract
Flow Merge Readiness
Next actors: product, architecture, and security reviewers must accept or amend the proposed decision.
Merge Automation
gh pr merge --auto --squash, or the reason it is unavailable/unsafe is noted belowAuto-merge is intentionally disabled for the human-required architecture decision.
Notes