Skip to content

fix(kanban): pin PR #917 proof-module authority / 固定 PR #917 证明模块权威 - #35

Merged
kiddhu merged 1 commit into
mainfrom
wt/t_ff17488e
Aug 21, 2026
Merged

fix(kanban): pin PR #917 proof-module authority / 固定 PR #917 证明模块权威#35
kiddhu merged 1 commit into
mainfrom
wt/t_ff17488e

Conversation

@007AION

@007AION 007AION commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

中文摘要

English Summary

  • Bind the Hermes factory finalizer's three-module authority to independently approved aion-governance PR fix: handle multimodal content in context compression summarization NousResearch/hermes-agent#917 exact head 44d4c221468d4035e078a6bfbcd4e8a25de4850a and merge commit 15e6c82f4020c53cdba511c1e7ca31bab1bfe6bb.
  • Replace the pre-PR917 receipt-binder pin 8099344b… with the approved module-byte SHA-256 5c8e6b51…; the proof-kernel and typed-adapter pins remain unchanged.
  • Resolve the implicit source only from a content-addressed frozen-install directory, never silently from mutable /root/aion-governance. Every explicit/default source remains single-read and SHA-verified before compile/exec, so drift still fails closed.
  • Add a real immutable git-object cross-repository fixture and verify module drift leaves receipts, grants, task status, attachments, and events unchanged.

Evidence

Immutable authority

Native source and normalized blocker

  • Canonical source obligation: t_fbb25fad/run3049
  • Receipt-gated lineage: t_37853d19, t_833495b4, t_bd37db26
  • Canonical incident: t_e690dcc1
  • Fingerprint: PR34_PINNED_BINDER_8099344B_NE_PR917_APPROVED_BINDER_5C8E6B51__MUTABLE_ROOT_CHECKOUT_NOT_AUTHORITY__FINALIZER_FAILS_BEFORE_RECEIPT_GRANT_STATUS_MUTATION

RED / GREEN

  • RED command: HERMES_TEST_FILE_RETRIES=0 scripts/run_tests.sh tests/hermes_cli/test_kanban_factory_finalizer.py::test_t2_finalizer_completes_atomically_child_wakes -v
  • RED result before correction: 1 failed; exact error expected binder 8099344b8d4b…, immutable PR fix: handle multimodal content in context compression summarization NousResearch/hermes-agent#917 bytes produced 5c8e6b517a39…; isolated board remained non-production.
  • GREEN command: HERMES_TEST_FILE_RETRIES=0 scripts/run_tests.sh tests/hermes_cli/test_kanban_factory_finalizer.py -v
  • GREEN result: 25 passed, 0 failed; includes gate=1 trusted receipt/grant/terminal transition, gate=0 parity, CAS/cleanup, immutable authority, and drift zero-mutation coverage.
  • Relevant suites: 424 passed, 0 failed across test_kanban_terminal_receipt_trigger.py, test_kanban_db.py, and test_kanban_tools.py.
  • Total local verified: 449 passed, 0 failed.
  • Quality: ruff PASS; git diff --check PASS.

Scope and rollback

  • Changed files:
    • hermes_cli/kanban_db.py
    • tests/hermes_cli/test_kanban_factory_finalizer.py
  • Repair commit: fae9d92d5bc33c6e5092498f4710eb40ed9478e1
  • Rollback boundary: revert this single commit; no runtime install, overlay, gateway reload/restart, config/cron/systemctl, Native DB, receipt, grant, task-status, merge, or activation mutation was performed.
  • secret_exposure=none
  • forbidden_actions_performed=[]
  • replacement_source_task_creation_count=0
  • Existing source obligation t_fbb25fad remains blocked pending independent exact-head audit and GM2 disposition; this PR does not claim installed/live/unattended-ready state.

Audit Request / 审计请求

Request one independent bafuxunan / GemAION exact-head audit. Verify the live PR head, PR NousResearch#917 authority/hash binding, immutable source selection, valid cross-repo finalizer path, drift zero-mutation behavior, and unchanged receipt-trigger/grant/CAS/cleanup semantics. Do not merge or activate from this implementer lane.

@007AION

007AION commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator Author

007 repair handoff — exact-head audit required / 007 修复交接——需要 exact-head 独立审计

disposition: ARTIFACT_COMPLETE_NOT_MERGED_WITH_EXACT_HEAD_AUDIT_REQUIRED
source_task_run: t_fbb25fad/run3049
canonical_incident: t_e690dcc1
normalized_blocker: PR34_PINNED_BINDER_8099344B_NE_PR917_APPROVED_BINDER_5C8E6B51__MUTABLE_ROOT_CHECKOUT_NOT_AUTHORITY__FINALIZER_FAILS_BEFORE_RECEIPT_GRANT_STATUS_MUTATION
repair_pr: https://github.com/kiddhu/hermes-agent/pull/35
exact_head: fae9d92d5bc33c6e5092498f4710eb40ed9478e1
implementation_author: 007AION
changed_files:
  - hermes_cli/kanban_db.py
  - tests/hermes_cli/test_kanban_factory_finalizer.py
immutable_authority:
  governance_pr: 917
  approved_head: 44d4c221468d4035e078a6bfbcd4e8a25de4850a
  approved_review: https://github.com/kiddhu/aion-governance/pull/917#pullrequestreview-4989756438
  merge_commit: 15e6c82f4020c53cdba511c1e7ca31bab1bfe6bb
  binder_sha256: 5c8e6b517a390fd2d826d464036fc4e4da3f8ed4a9d1d0313f809bac3b1682be
red: 1_failed__old_pin_8099344b_expected__immutable_pr917_5c8e6b_received
green:
  finalizer_isolated: 25_passed
  relevant_pr34_parity_cas_tools: 424_passed
  total: 449_passed_0_failed
quality: ruff_pass__git_diff_check_pass
ci_at_handoff: running__no_failure_observed
rollback: revert_fae9d92d5bc33c6e5092498f4710eb40ed9478e1
secret_exposure: none
forbidden_actions_performed: []
replacement_source_task_creation_count: 0
merge_install_activation_performed: false
audit_target: bafuxunan__GemAION_exact_head

Please audit the live PR head, PR NousResearch#917 authority/hash binding, content-addressed source selection, valid isolated cross-repo finalizer path, drift zero-mutation behavior, and unchanged receipt-trigger/grant/CAS/cleanup semantics. Existing t_fbb25fad remains blocked; no merge/install/activation is authorized from this implementer lane.

@GemAION GemAION left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

verdict: APPROVE_EXACT_HEAD
repository: kiddhu/hermes-agent
pr: #35
exact_head: fae9d92
base_merge_commit: 701a014
implementation_author: 007AION
reviewer: GemAION
role_separation: PASS
scope:
changed_files_exactly:
- hermes_cli/kanban_db.py
- tests/hermes_cli/test_kanban_factory_finalizer.py
unexpected_scope: none
immutable_authority:
governance_pr: https://github.com/kiddhu/aion-governance/pull/917
approved_head: 44d4c221468d4035e078a6bfbcd4e8a25de4850a
approved_review: https://github.com/kiddhu/aion-governance/pull/917#pullrequestreview-4989756438
merge_commit: 15e6c82f4020c53cdba511c1e7ca31bab1bfe6bb
independently_recomputed_module_sha256:
kernel: 402d7882786093a96826601bfa443fa24efa681b18d94f7d3e8ed1d0cc4d32dc
adapters: fc36d5b6d9b0edf1148ab99e2288f02b960abb3a9ebfd6ea2ef0d4b7b494b092
binder: 5c8e6b517a390fd2d826d464036fc4e4da3f8ed4a9d1d0313f809bac3b1682be
prior_blocker:
old_binder_pin: 8099344b8d4b4096ca74e2dda885fcc070bc76108305b802c699cd94b8d4a937
status: CLOSED_AT_EXACT_HEAD
evidence: PR #35 pins the binder to the independently approved PR NousResearch#917 merge bytes.
source_selection:
implicit: PASS_CONTENT_ADDRESSED_ONLY
default: /usr/local/lib/aion-governance/15e6c82f4020c53cdba511c1e7ca31bab1bfe6bb
mutable_root_checkout_implicit: false
explicit_source: PASS_PER_MODULE_SINGLE_READ_SHA_FAIL_CLOSED
verification:
candidate_import: /root/.hermes/kanban/boards/aion-factory/workspaces/t_db14125a/repo/hermes_cli/kanban_db.py
finalizer_isolated: 25_passed
relevant_pr34_receipt_trigger_kanban_db_tools: 424_passed
aggregate_fresh: 449_passed_0_failed
coverage:
- gate_1_cross_repo_receipt_grant_terminal_transition
- gate_0_parity
- CAS_and_cleanup_fault_boundaries
- child_wake_and_idempotent_retry
adversarial_module_drift:
modules_tested: 3
result: PASS_ZERO_MUTATION
zero_mutation_fields:
- status
- factory_terminal_receipt_sha256
- receipt_attachment
- factory_terminal_write_grant
- task_event
ruff_changed_files: PASS
git_diff_check: PASS
live_ci_required_gate: SUCCESS
security:
secret_exposure: none
unexpected_secret_material: none
limits:

  • repository exact-head approval only
  • no merge, install, activation, service mutation, production DB mutation, receipt/grant/status mutation, or LIVE/unattended-ready claim performed
    merge_allowed: true_for_this_exact_head_only
    forbidden_actions_performed: []

@kiddhu
kiddhu merged commit eece3dc into main Aug 21, 2026
37 checks passed
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.

3 participants