Skip to content

fix: recover repair and exact-review workers#596

Merged
hxy91819 merged 1 commit into
mainfrom
agent/fix-online-automation-failures
Jul 15, 2026
Merged

fix: recover repair and exact-review workers#596
hxy91819 merged 1 commit into
mainfrom
agent/fix-online-automation-failures

Conversation

@hxy91819

Copy link
Copy Markdown
Member

Root causes

  • Blacksmith exposes mount_setattr(2) as ENOSYS; validation treated the missing optional syscall as containment failure even though per-mount read-only remounting is available.
  • Exact-review lease claims treated expected conflict outcomes as fatal, then downstream reviewer/publisher steps consumed absent outputs.

Changes

  • Fall back only on ENOSYS to /proc/self/mountinfo plus bind-remount-readonly; all other containment errors remain fail-closed.
  • Classify only known lease-conflict reasons as clean skips, initialize safe outputs, guard every downstream claim-dependent step, and keep unknown 409/4xx/malformed responses fatal.
  • Make identical claim retries replay the successful decision.

Validation

  • corepack enable
  • pnpm install
  • pnpm run check
  • focused repair/exact-review tests: 150 passed
  • autoreview: 0 findings (confidence 0.86)

No live apply/close command was run and no workflow was disabled.

@hxy91819
hxy91819 requested a review from a team as a code owner July 15, 2026 12:37
@hxy91819
hxy91819 merged commit 4eff9b5 into main Jul 15, 2026
9 checks passed
@hxy91819
hxy91819 deleted the agent/fix-online-automation-failures branch July 15, 2026 13:17
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.

1 participant