Skip to content

feat(improvement): expose exact author shot evidence#552

Merged
drewstone merged 1 commit into
mainfrom
feat/improvement-shot-evidence
Jul 15, 2026
Merged

feat(improvement): expose exact author shot evidence#552
drewstone merged 1 commit into
mainfrom
feat/improvement-shot-evidence

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Problem

Author-shot receipts retained output hashes but discarded the exact stdout, stderr, and post-run worktree decision. A clean or rejected attempt could therefore become unauditable after its temporary worktree was removed.

Solution

  • pass a frozen exact execution snapshot as the second argument to the awaited onShotCompleted callback
  • add an awaited onShotDisposition callback while the worktree is still available
  • classify every completed attempt as clean, rejected, accepted, or setup-error, including exact rejection feedback and the live worktree path
  • abort the candidate when either persistence callback fails
  • keep existing one-argument onShotCompleted callbacks source-compatible
  • bump the package and generated API reference to 0.94.11

Verification

  • pnpm test: 1,574 passed, 2 skipped across 152 files
  • focused agentic-generator tests: 25 passed
  • pnpm typecheck: passed, including examples
  • pnpm lint: 441 files passed
  • pnpm build: JavaScript and declaration builds passed
  • pnpm verify:package: passed
  • pnpm docs:check: passed
  • git merge-tree --write-tree origin/main HEAD: clean

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Auto-approved drewstone PR — 335d4114

This PR was opened by the trusted drewstone account.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: drewstone_author · 2026-07-15T02:01:45Z

@drewstone
drewstone merged commit 61309d8 into main Jul 15, 2026
2 checks passed
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.

2 participants