-
Notifications
You must be signed in to change notification settings - Fork 1
stabilize: decision-proxy-evidence schema + template (Amara action #3) #222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
| @@ -0,0 +1,73 @@ | ||||
| # Worked example — backfilled retroactively to seed the format. | ||||
| # Documents the Otto-66 decision to apply minimum-viable branch | ||||
| # protection on AceHack/Zeta (force-push + deletion blocks only; | ||||
| # no richer gates). See README.md for schema semantics. | ||||
|
|
||||
| decision_id: DP-001 | ||||
| timestamp_utc: 2026-04-23T23:45:00Z | ||||
|
|
||||
| requested_by: Aaron | ||||
| proxied_by: Otto | ||||
| task_class: settings-change | ||||
| authority_level: retroactive | ||||
| escalation_required: false | ||||
|
|
||||
| repo_canonical: AceHack/Zeta | ||||
| head_commit: "5b2f1ac" | ||||
|
|
||||
| model: | ||||
| vendor: anthropic | ||||
| snapshot: claude-opus-4-7 | ||||
| prompt_bundle_hash: null | ||||
| loaded_memory_files: | ||||
| - "./CLAUDE.md" | ||||
| - "~/.claude/CLAUDE.md" | ||||
|
|
||||
| consulted_views: | ||||
| - memory/CURRENT-aaron.md | ||||
| - memory/CURRENT-amara.md | ||||
|
|
||||
| consulted_memory_ids: | ||||
| - feedback_agent_owns_all_github_settings_and_config_all_projects_zeta_frontier_poor_mans_mode_default_budget_asks_require_scheduled_backlog_and_cost_estimate_2026_04_23 | ||||
| - feedback_lfg_free_actions_credits_limited_acehack_is_poor_man_host_big_batches_to_lfg_not_one_for_one_2026_04_23 | ||||
| - feedback_honor_those_that_came_before | ||||
|
|
||||
| live_state_checks: | ||||
| - "gh api repos/Lucent-Financial-Group/Zeta/branches/main/protection" | ||||
| - "gh api repos/AceHack/Zeta --jq '.fork // .parent.full_name'" | ||||
| - "gh api users/AceHack/events (repo-level create/delete scan)" | ||||
|
|
||||
| decision_summary: > | ||||
| Applied minimum-viable branch protection on AceHack/Zeta: | ||||
| allow_force_pushes=false + allow_deletions=false. Left richer | ||||
| LFG-style gates OFF (required_status_checks, review_required, | ||||
| required_linear_history, required_conversation_resolution) | ||||
| because Amara's authority-axis split names AceHack as | ||||
| experimentation-frontier where heavier gates slow iteration; | ||||
| canonical-decision substrate lives on LFG. The protection | ||||
| asymmetry is load-bearing, not a consistency defect. | ||||
|
|
||||
| disagreements: | ||||
| present: false | ||||
| conflict_row: null | ||||
|
|
||||
| outputs_touched: | ||||
| - memory/project_acehack_branch_protection_minimal_applied_prior_zeta_archaeology_inconclusive_2026_04_23.md | ||||
|
||||
| - memory/project_acehack_branch_protection_minimal_applied_prior_zeta_archaeology_inconclusive_2026_04_23.md |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,267 @@ | ||||||||||||||||||||||
| # Decision-proxy evidence records | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| **Stage:** Stabilize (Otto-67 Amara 4th ferry absorb, PR #221) | ||||||||||||||||||||||
| **Companion template:** [`_template.yaml`](_template.yaml) | ||||||||||||||||||||||
| **External-maintainer-decision-proxy ADR:** `docs/DECISIONS/2026-04-22-external-maintainer-decision-proxy-adr.md` | ||||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
This reference points to Useful? React with 👍 / 👎.
|
||||||||||||||||||||||
| **External-maintainer-decision-proxy ADR:** `docs/DECISIONS/2026-04-22-external-maintainer-decision-proxy-adr.md` | |
| **External-maintainer-decision-proxy ADR:** `docs/DECISIONS/2026-04-23-external-maintainer-decision-proxy-pattern.md` |
Copilot
AI
Apr 24, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The filename example uses DP-001 on 2026-04-24, but this PR already introduces 2026-04-23-DP-001-.... Keeping the example in sync with the actual first record (or using a different DP number) will prevent confusion about whether IDs are global-monotonic vs per-date.
| Example: `docs/decision-proxy-evidence/2026-04-24-DP-001-acehack-branch-protection-minimal.yaml` | |
| Example: `docs/decision-proxy-evidence/2026-04-24-DP-002-acehack-branch-protection-minimal.yaml` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Define status field for escalation lifecycle
The schema text says escalation_required controls when status can move to landed, but no status field is defined anywhere in required/optional fields or in the template. For any escalated decision, this makes the lifecycle state unrepresentable and will force authors to invent inconsistent out-of-band tracking.
Useful? React with 👍 / 👎.
Copilot
AI
Apr 24, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P1: This section asserts there are "4 courier ferries" and points to docs/aurora/2026-04-23-amara-*.md, but currently only two matching Amara absorb docs exist under docs/aurora/. Either add the missing absorb docs referenced in the PR description or adjust the wording to match what is actually present in the tree so the documentation doesn’t drift.
| - **Amara's 4 courier ferries** are not themselves | |
| decision-proxy records — they are proposals. Decisions | |
| made BASED ON those ferries should cite them via | |
| `consulted_memory_ids` pointing to the absorb docs | |
| (`docs/aurora/2026-04-23-amara-*.md`). | |
| - **Amara's courier ferries** are not themselves | |
| decision-proxy records — they are proposals. Decisions | |
| made BASED ON those ferries should cite them via | |
| `consulted_memory_ids` pointing to the relevant absorb | |
| docs under `docs/aurora/`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P0:
consulted_memory_idsincludes memory slugs that don’t appear to exist undermemory/(e.g.,feedback_lfg_free_actions_credits_limited_acehack_is_poor_man_host_big_batches_to_lfg_not_one_for_one_2026_04_23andfeedback_honor_those_that_came_before). Since this file is meant to be an auditable evidence record, these should either be corrected to existing memory IDs (matching actual filenames/slugs) or the missing memory files should be added in the same PR.