Skip to content

feat: tighten integration administration APIs#77

Merged
shakkernerd merged 18 commits into
mainfrom
feat/integrations-backend
Jul 14, 2026
Merged

feat: tighten integration administration APIs#77
shakkernerd merged 18 commits into
mainfrom
feat/integrations-backend

Conversation

@shakkernerd

Copy link
Copy Markdown
Member

Summary

  • require workspace managers for integration mutations while preserving connected-account self-revocation
  • add signing-secret rotation for slash commands and event subscriptions
  • add stable cursor pagination for event delivery attempts
  • atomically cascade app uninstall revocations with configurable bot-token handling
  • expose and validate the durable event-type vocabulary
  • update OpenAPI, TypeScript SDK, documentation, and changelog

Verification

  • full pnpm check
  • SQLite and PostgreSQL integration lifecycle tests
  • authorization, secret rotation, HMAC, cascade rollback, and pagination regression coverage
  • sqlc and OpenAPI regeneration checks
  • independent Codex review with all accepted findings fixed

Remote validation: AWS Crabbox cbx_62daafbedec3

@shakkernerd shakkernerd self-assigned this Jul 14, 2026
@shakkernerd shakkernerd merged commit e84d90f into main Jul 14, 2026
9 checks passed
@shakkernerd

Copy link
Copy Markdown
Member Author

Merged using rebase.

What changed after review:

  • Added bot-token coverage for both signing-secret rotation endpoints, restoring the Go coverage gate to 85.1%.
  • Strengthened the tests to assert the handler-specific bot-token rejection instead of accepting any 403.
  • Preserved the reviewed integration contracts, generated artifacts, SDK surface, and documentation.

Verification:

  • Rebased/refreshed onto current origin/main before pushing.
  • AWS Crabbox cbx_5db52fe83def: focused HTTP route tests and the exact Go coverage gate passed.
  • Bot-token rotation requests were verified to fail at the human-session boundary with the expected errors.
  • Auto-review: clean after the accepted test-strength finding was resolved.
  • GitHub checks: all checks green.

Landed on main:

Thanks @shakkernerd!

@shakkernerd shakkernerd deleted the feat/integrations-backend branch July 14, 2026 17:16
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