pook/compliancebot
1
0
Fork 0
Code Issues 1.1k Pull requests 82 Projects Releases Packages Wiki Activity Actions

feat: centralized error handler with request ID tracking #349

Closed
pook wants to merge 45 commits from feat/centralized-error-handler into main
pull from: feat/centralized-error-handler
merge into: pook:main
Conversation 1 Commits 45 Files changed 59 +348 -189
pook commented 2026-04-10 11:49:35 -04:00
Owner
Copy link

Closes #299

Closes #299
feat: add centralized error handler with request ID and structured logging
Some checks are pending
CI Quality Gate / Lint / Typecheck / Test / Build (pull_request) Waiting to run
Details
fd6d848c6f 
Unhandled errors now return consistent JSON ({ error, requestId }) without
leaking stack traces. HTTPException status codes are forwarded; plain errors
with a .status property also propagate their status. All errors are logged
as structured JSON via a new logger utility. A request-id middleware assigns
a UUID to every request for correlation.

Closes #299

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
feat: issue #346 issue-299-stalled-specific-fix-add-a-4-p (agent task agent-ta)
Some checks failed
CI Quality Gate / Lint / Typecheck / Test / Build (pull_request) Has been cancelled
Details
ac5b458991
pook commented 2026-04-10 15:04:13 -04:00
Author
Owner
Copy link

Closed 2026-04-10 during pipeline triage.

Why: This PR has merge conflicts with current main and is blocking the CEO agent's view of the backlog. The compliancebot repo had 50 open PRs (most conflicting) and 141 open agent-task issues — CEO couldn't see progress and kept duplicating work.

What was fixed upstream:

  • Git-push race in agent-worker executor (dispatch jobs collided on agent/dispatch/* branch name). RunId now threaded through shim → dispatch → executor for unique branches.
  • Shim /shim/ceo route now injects current open issues/PRs into the CEO prompt + refuses dispatch if backlog exceeds 20.

Reopen / resubmit this PR against current main if the work is still relevant — CEO will now see it and avoid duplicating.

Closed 2026-04-10 during pipeline triage. **Why:** This PR has merge conflicts with current `main` and is blocking the CEO agent's view of the backlog. The compliancebot repo had 50 open PRs (most conflicting) and 141 open agent-task issues — CEO couldn't see progress and kept duplicating work. **What was fixed upstream:** - Git-push race in agent-worker executor (dispatch jobs collided on `agent/dispatch/*` branch name). RunId now threaded through shim → dispatch → executor for unique branches. - Shim `/shim/ceo` route now injects current open issues/PRs into the CEO prompt + refuses dispatch if backlog exceeds 20. **Reopen / resubmit** this PR against current main if the work is still relevant — CEO will now see it and avoid duplicating.
pook closed this pull request 2026-04-10 15:04:15 -04:00
Some checks failed
CI Quality Gate / Lint / Typecheck / Test / Build (pull_request) Has been cancelled
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
agent-task

Task for autonomous agent execution

  
agent-task

Paperclip autonomous agent task

No labels
agent-task
agent-task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pook/compliancebot!349
No description provided.