Skip to content

Conversation

@github-actions
Copy link
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.

github-actions bot and others added 2 commits January 19, 2026 21:23
…2019)

* feat(automation): add error sanitization for user-friendly messages

* fix(automation): enhance error sanitization with fallback for AI failures

* refactor(automation): simplify error extraction and enhance fallback handling

---------

Co-authored-by: Tofik Hasanov <annexcies@gmail.com>
@vercel
Copy link

vercel bot commented Jan 20, 2026

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

2 Skipped Deployments
Project Deployment Review Updated (UTC)
app (staging) Skipped Skipped Jan 20, 2026 10:02pm
portal (staging) Skipped Skipped Jan 20, 2026 10:02pm

Request Review

@cursor
Copy link

cursor bot commented Jan 20, 2026

PR Summary

Improves reliability of vector syncs and user-facing automation errors.

  • Vector sync: sync-policies, sync-context, sync-knowledge-base, and manual answers now return/aggregate { status, lastEmbeddingId }, store lastUpsertedEmbeddingId in SyncStats, and after syncing run verifyEmbeddingIsReady() (queries Upstash with retries/backoff) to ensure embeddings are indexed/queryable
  • Manual answers: returns full SyncStats, filters empty texts before upsert, and tracks the last actually upserted ID
  • Sync utils: SyncStats extended with lastUpsertedEmbeddingId; initSyncStats updated accordingly
  • App (automation): adds server action sanitize-error using AI; use-task-automation-execution sanitizes errors (including nested), suppresses internal logs, and provides safe messages; removes noisy console log in use-task-automation

Written by Cursor Bugbot for commit 1f6e635. This will update automatically on new commits. Configure here.

cursor[bot]

This comment was marked as outdated.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

cursor[bot]

This comment was marked as outdated.

* refactor(automation): update error handling and remove internal logs

* refactor(automation): enhance error message for invalid regex input

---------

Co-authored-by: Tofik Hasanov <annexcies@gmail.com>
* refactor(sync): update sync functions to return detailed results with last embedding ID

* refactor(sync): filter valid items before upserting embeddings

---------

Co-authored-by: Tofik Hasanov <annexcies@gmail.com>
@vercel vercel bot temporarily deployed to staging – portal January 20, 2026 22:02 Inactive
@vercel vercel bot temporarily deployed to staging – app January 20, 2026 22:02 Inactive
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

});

const result = text.trim() || 'The automation encountered an error. Please check your script and try again.';
console.log('[sanitizeErrorMessage] SYSTEM AI response:', result);
Copy link

Choose a reason for hiding this comment

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

Debug console.log statement in production code

Low Severity

A console.log statement logging [sanitizeErrorMessage] SYSTEM AI response: is present in the new sanitize-error.ts file. This logs every sanitized error message from the AI to the console in production, which adds noise to logs and could expose error details (even if sanitized) that aren't meant for end-user browser consoles. This appears to be debugging code that was inadvertently left in.

Fix in Cursor Fix in Web

@tofikwest tofikwest merged commit a28a900 into release Jan 20, 2026
13 of 14 checks passed
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.

3 participants