Repository navigation
feat: forward job and question events to a webhook - #27
Merged
Merged
Conversation
Store repo_id on api_tokens and quota_ledger so usage can be attributed per repository. This is internal plumbing and does not change user-visible behavior. Docs-not-needed: internal attribution columns; budgets are documented separately.
Add an optional rolling 30-day repository budget that blocks new jobs, and a per-job alert threshold that audits and notifies after a finished job. Cost is attributed through quota_ledger.repo_id.
Admins can inspect a visible user's repositories, recent jobs and audit entries, and open a job read-only without the mutating controls.
Admins can filter jobs and audit entries within their visibility scope and export the current result as CSV; exports are audited.
Add an optional Slack-compatible outbound webhook with a bearer secret. Job success/failure and team questions are posted; failures are logged without blocking the job.
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
HugoXK
commented
Nov 4, 2026
HugoXK
left a comment
HugoXK
left a comment
Owner
Author
There was a problem hiding this comment.
Verdict: PASS at d6926ec
Written-by: Reviewer
Posted-by: HugoXK
Ran:
npm run check— pass (109 tests, build)node scripts/changelog.mjs check— pass
Not run: production deploy
Feature-only PR for the outbound notification webhook.
# Conflicts: # src/jobs/service.ts # test/jobs.test.ts
HugoXK
commented
Nov 4, 2026
HugoXK
left a comment
HugoXK
left a comment
Owner
Author
There was a problem hiding this comment.
Verdict: PASS at 10941ca79ae29fe5ea289e5f176d281a4d469afa
Written-by: Reviewer
Posted-by: HugoXK
Ran:
npm run check— pass (109 tests, build)node scripts/changelog.mjs check— pass
Not run: production deploy
Feature-only PR for the outbound notification webhook.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Written-by: HugoXK
Adds an optional outbound notification webhook.
NOTIFY_WEBHOOK_URL/NOTIFY_WEBHOOK_SECRETpost a Slack-compatible payload..env.example, deploy docs, tests and a changelog fragment.Ran:
npm run check— pass (109 tests, build)node scripts/changelog.mjs check— passNot run: production deploy.