Skip to content

[comp] Production Deploy#3232

Merged
tofikwest merged 7 commits into
releasefrom
main
Jun 22, 2026
Merged

[comp] Production Deploy#3232
tofikwest merged 7 commits into
releasefrom
main

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.


Summary by cubic

Disable Stagehand’s hosted API during Browserbase session resume and fix Trigger.dev maxDuration units to stop “Unknown error: 400” connect failures and enforce intended timeouts. Also forwards Stagehand error logs and allows image uploads (PNG/JPG/HEIC/HEIF/WebP) for manual passport verification.

  • Bug Fixes
    • Set disableAPI: true when creating Stagehand to skip POST /sessions/start; sessions resume over CDP and Connect/login-check/evidence run reliably.
    • Forward Stagehand error logs into the NestJS logger to surface real upstream errors.
    • Correct maxDuration to seconds in run-browser-automation (10 minutes) and browser-automations-schedule (30 minutes).
    • Background checks: widen file input to accept PDFs and images (PNG, JPG, HEIC/HEIF, WebP), update error copy, and add tests for type and size validation.

Written for commit 82c852f. Summary will update on new commits.

Review in cubic

tofikwest and others added 4 commits June 22, 2026 15:07
"Unknown error: 400" on Connect came from Stagehand's hosted API
(POST /sessions/start) — a separate upstream from the premature-close
fix. We create and own the Browserbase session ourselves and only need
CDP navigation plus local inference, so the hosted-API round-trip is
both unnecessary and the single point of failure for connect,
login-check, and evidence runs (all route through createStagehand).

Set disableAPI:true so Stagehand skips /sessions/start: the session
still resumes over CDP, and extract/act/agent run locally against
ANTHROPIC_API_KEY (verified: apiClient is only built inside the
!disableAPI guard, and extract/agent fall back to local handlers when
it is null). Also forward Stagehand's own error logs into the NestJS
logger, because it strips upstream error bodies from its throws — so a
future failure surfaces the real reason instead of "Unknown error: 400".

Also fix the Trigger.dev maxDuration units for the scheduled runner and
orchestrator: they were 1000*60*N (~166h / ~500h), so the intended
10/30-minute safety abort never tripped. maxDuration is in SECONDS.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013zSwXMqVNvWLJBZEot9x12
…ted-api

fix(browserbase): disable Stagehand hosted API to unblock connect
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app (staging) Ready Ready Preview, Comment Jun 22, 2026 8:19pm
comp-framework-editor (staging) Ready Ready Preview, Comment Jun 22, 2026 8:19pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal (staging) Skipped Skipped Jun 22, 2026 8:19pm

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

tofikwest and others added 3 commits June 22, 2026 15:24
## Problem

User at Aaardvark is stuck on identity verification. Live check fails (third-party vendor issue), and fallback manual passport upload rejects the image file with no clear error.

## Root cause

BackgroundCheckAttachForm.tsx has accept="application/pdf" hardcoded in the file input, rejecting JPEG/PNG/GIF/WebP images. The backend (attachments.service.ts, file-type-validation.ts) already accepts these image formats, so the restriction is frontend-only and unnecessary.

## Fix

Widened accept attribute in BackgroundCheckAttachForm.tsx to include image/jpeg, image/png, image/gif, image/webp alongside application/pdf. Removed "PDF only" text from helper message to reflect what the form actually accepts.

## Explicitly NOT touched

Live identity verification flow (third-party vendor hosted, no code lever on our side). Backend file validation rules (already correct). Auth, RBAC, schema, billing.

## Verification

✅ File input accepts image formats in browser
✅ Existing PDF submissions still work
✅ Backend still validates file types correctly
✅ Manual upload flow progresses past file selection
…entity-verificati

fix(background-checks): accept image formats for manual passport upload
@vercel vercel Bot temporarily deployed to staging – portal June 22, 2026 20:16 Inactive
@tofikwest tofikwest merged commit 05a57f2 into release Jun 22, 2026
14 checks passed
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.87.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants