Skip to content

fix(desktop): make persona deployment idempotent - #7719

Open
bmurphy201 wants to merge 1 commit into
block:mainfrom
bmurphy201:marshall/agent-create-idempotency-20260917
Open

bmurphy201 wants to merge 1 commit into
block:mainfrom
bmurphy201:marshall/agent-create-idempotency-20260917

Conversation

@bmurphy201

Copy link
Copy Markdown

Summary

  • reuse an active managed identity only when the team, persona, runtime, provider, model, and configuration binding match exactly
  • fail closed on ambiguous or mismatched candidates while preserving explicit force-new behavior
  • add a locked backend persistence guard for stale or concurrent retries, including rollback coverage

Verification

CI-equivalent full package gates passed:

  • frontend: 6,518/6,518
  • Rust desktop: 3,197 passed, 19 ignored, 0 failed; auxiliary workspace suites passed
  • mobile: 2,121/2,121
  • typecheck, lint, format, Clippy, cargo check, production builds, and git diff --check passed

The initial managed-runtime run inherited BUZZ_ACP_SESSION_POLICY=thread, and an untouched emoji prewarm test produced one transient file-level failure. Rerunning with the inherited override removed and then rerunning the complete desktop suite passed.

No live duplicate-cleanup or identity-archive manifest is included.

Signed-off-by: Marshall <command@colonyspark.com>
@bmurphy201
bmurphy201 requested a review from a team as a code owner September 18, 2026 04:15
@github-actions

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is 779af8886caae1317b4de962082429867ab61503...ce3112de9a2f3013ddba0abb6b7f75207d8f01de.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review ce3112de9a2f3013ddba0abb6b7f75207d8f01de to authorize a new review.
Any previous review applies only to its recorded range.

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