Skip to content

fix(runtime): keep operation identifiers redaction-stable - #23

Merged
drewstone merged 1 commit into
mainfrom
fix/redaction-stable-operation-ids
Aug 11, 2026
Merged

fix(runtime): keep operation identifiers redaction-stable#23
drewstone merged 1 commit into
mainfrom
fix/redaction-stable-operation-ids

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Fixes the release-blocking plain-mode crash caused when a timestamp suffix resembled a credential and redaction changed the operation identity.

  • Generate plain-mode operation IDs from UUIDs only.
  • Reject any domain identifier that durable redaction would change.
  • Validate every public run mutation before provider work or journal writes.
  • Add deterministic regression coverage for the original collision and bypass paths.

Verified locally:

  • pnpm run test:unit: 230/230 passed
  • pnpm run test:rpc: 35/35 passed
  • pnpm run typecheck
  • pnpm run boundaries: 466 modules, 0 cycles
  • pnpm run build

No visible UI changed.

@drewstone
drewstone merged commit 053f5b0 into main Aug 11, 2026
4 checks passed
@drewstone
drewstone deleted the fix/redaction-stable-operation-ids branch August 11, 2026 20:36
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