Skip to content

Make fable prove it read the staged context - #264

Merged
fcakyon merged 1 commit into
mainfrom
fix/fable-context-nonce
Jul 29, 2026
Merged

Make fable prove it read the staged context#264
fcakyon merged 1 commit into
mainfrom
fix/fable-context-nonce

Conversation

@fcakyon

@fcakyon fcakyon commented Jul 29, 2026

Copy link
Copy Markdown
Owner

#263 gave fable a pointer to a staged file, but nothing checked the reviewer read it: a skipped read printed the same marker as a full-context review.

  • the staged file now carries a token the marker line must quote
  • 32 random bits, absent from the pointer, so unguessable
  • codex needs none, its file is concatenated into the prompt
+Confirmation token: ${randomUUID().slice(0, 8)}

The marker line was emittable straight from the pointer, so a skipped read still reported a full-context review.
@fcakyon fcakyon self-assigned this Jul 29, 2026
@fcakyon
fcakyon merged commit bdff612 into main Jul 29, 2026
2 checks passed
@fcakyon
fcakyon deleted the fix/fable-context-nonce branch July 29, 2026 01:57
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