Skip to content

Allow clearing agent profile roles - #48

Merged
sametbasbug merged 1 commit into
mainfrom
equinox/v0.5-role-clear
Aug 8, 2026
Merged

sametbasbug merged 1 commit into
mainfrom
equinox/v0.5-role-clear

Conversation

@sametbasbug

Copy link
Copy Markdown
Owner

Summary

  • allow the optional agent role field to be cleared back to an empty string
  • keep role trim-normalized and bounded to 80 code points
  • align ProfilePatch with the persisted/read profile model, where an empty role is already valid
  • apply the same behavior to credential-based and MCP-native profile updates
  • preserve ETag/version conflict handling

Regression coverage

  • normal agent profile can set a role, reject a stale write, then clear the role with the current ETag
  • MCP-native profile can set a role, reject a stale ETag, then restore the role to empty without changing bio/accent/pin

Validation

  • npm run check: passed (0 errors, 0 warnings; 2 existing hints)
  • API contract tests: passed
  • production-live Worker dry-run: passed
  • focused Slice 1: the two modified profile flows passed; the suite still hit the two known unrelated local Wrangler ECONNRESET flakes in sign-in-trace tests

Treat the optional role field as a trim-normalized 0-80 character string across normal agent and MCP-native profile updates. Align the public ProfilePatch schema with persisted/read profile state and add regression coverage for clearing role while preserving ETag concurrency.
@sametbasbug
sametbasbug marked this pull request as ready for review August 8, 2026 18:30
@sametbasbug
sametbasbug merged commit 578dd9c into main Aug 8, 2026
4 checks passed
@sametbasbug
sametbasbug deleted the equinox/v0.5-role-clear branch August 8, 2026 18:31
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