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

[Agent] Issue #683: same critical reliability gap as contrac #686

Open
pook wants to merge 63 commits from agent-task/683 into main
pull from: agent-task/683
merge into: pook:main
Conversation 0 Commits 63 Files changed 66 +6534 -69
pook commented 2026-04-11 12:21:04 -04:00
Owner
Copy link

Closes #683

Changes

feat: issue #683 same-critical-reliability-gap-as-contrac (agent task liancebot683)

Acceptance Criteria

Same critical reliability gap as contractpilot — without these handlers, the server crashes silently on unhandled promise rejections, losing in-flight document generation requests.

Implementation:

  1. In the Express app entry point, add:
    • process.on('unhandledRejection', (reason, promise) => { logger.error('Unhandled Rejection', { reason }); })
    • process.on('uncaughtException', (error) => { logger.error('Uncaught Exception', { error }); process.exit(1); })
  2. Use the existing logger module.
  3. uncaughtException handler must call process.exit(1) for clean container restart.
  4. Add unit test verifying handlers are registered and log correctly.

Acceptance: Both handlers registered, test passing, no new dependencies.

Generated by CEO Planner (priority: 2)

Tokens: 0 in / 0 out

Closes #683 ## Changes feat: issue #683 same-critical-reliability-gap-as-contrac (agent task liancebot683) ## Acceptance Criteria Same critical reliability gap as contractpilot — without these handlers, the server crashes silently on unhandled promise rejections, losing in-flight document generation requests. Implementation: 1. In the Express app entry point, add: - `process.on('unhandledRejection', (reason, promise) => { logger.error('Unhandled Rejection', { reason }); })` - `process.on('uncaughtException', (error) => { logger.error('Uncaught Exception', { error }); process.exit(1); })` 2. Use the existing logger module. 3. uncaughtException handler must call `process.exit(1)` for clean container restart. 4. Add unit test verifying handlers are registered and log correctly. Acceptance: Both handlers registered, test passing, no new dependencies. --- *Generated by CEO Planner (priority: 2)* Tokens: 0 in / 0 out
feat: issue #683 same-critical-reliability-gap-as-contrac (agent task liancebot683)
Some checks failed
CI Quality Gate / Lint / Typecheck / Test / Build (pull_request) Has been cancelled
Details
920a981ef0
Some checks failed
CI Quality Gate / Lint / Typecheck / Test / Build (pull_request) Has been cancelled
Details
This pull request has changes conflicting with the target branch.
  • .forgejo/workflows/ci.yml
  • bun.lock
  • package.json
  • packages/api/src/db/schema.ts
  • packages/api/src/index.ts
  • packages/api/src/middleware/csrf.ts
  • packages/api/src/middleware/rate-limit.ts
  • packages/api/src/middleware/security-headers.ts
  • packages/api/src/routes/admin.ts
  • packages/api/src/routes/billing.ts
  • packages/api/src/routes/generate-tos.ts
  • packages/api/src/routes/generate.ts
  • packages/api/src/routes/health.ts
  • packages/api/src/routes/questionnaire.ts
  • packages/api/src/services/document-generator.ts
  • packages/api/src/services/llm.ts
  • packages/api/src/templates/index.ts
  • packages/api/tsconfig.json
  • packages/shared/src/types.ts
  • packages/web/src/app/questionnaire/page.tsx
  • packages/web/src/components/documents/DocumentList.tsx
  • packages/web/src/components/questionnaire/ReviewStep.tsx
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin agent-task/683:agent-task/683
git switch agent-task/683
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
1 participant
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!686
No description provided.