Skip to content

feat: add Slack draft approvals - #861

Closed
e1ektr0 wants to merge 2 commits into
yc-software:mainfrom
e1ektr0:feature/message-approval-surfaces
Closed

feat: add Slack draft approvals#861
e1ektr0 wants to merge 2 commits into
yc-software:mainfrom
e1ektr0:feature/message-approval-surfaces

Conversation

@e1ektr0

@e1ektr0 e1ektr0 commented Aug 30, 2026

Copy link
Copy Markdown

Summary

  • add durable Slack draft approval cards with approve, edit, and reject actions
  • resume approved drafts in the original session under normal tool authorization
  • enforce exact approved message content across MCP preview, edit, and completion workflows
  • add crash recovery, lease fencing, private DM routing, and bounded redaction

Validation

  • focused approval, Slack, harness, run-store, and privacy suites passed
  • TypeScript typecheck passed
  • ESLint and oxlint passed
  • changed-file Prettier and git diff checks passed
  • Codex/OpenCode Linux adapter checks passed under WSL where available

Notes

  • repository-wide formatting and knip retain unrelated baseline failures
  • no schema migration is required

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Maxim Feofilov added 2 commits August 31, 2026 00:04
…roval-surfaces

# Conflicts:
#	src/harness/claude-harness.ts
#	src/harness/codex-harness.ts
#	src/harness/pi-harness.ts
#	test/codex-harness.test.ts
@e1ektr0

e1ektr0 commented Aug 30, 2026

Copy link
Copy Markdown
Author

Closing this organization-specific delivery branch; no upstream contribution is requested at this time.

@e1ektr0 e1ektr0 closed this Aug 30, 2026
@e1ektr0
e1ektr0 deleted the feature/message-approval-surfaces branch August 30, 2026 21:32
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