Skip to content

Conversation

@Snyssfx
Copy link
Contributor

@Snyssfx Snyssfx commented Jul 11, 2025

No description provided.

@gemini-code-assist
Copy link
Contributor

Important

Installation incomplete: to start using Gemini Code Assist, please ask the organization owner(s) to visit the Gemini Code Assist Admin Console and sign the Terms of Services.

@notion-workspace
Copy link

@changeset-bot
Copy link

changeset-bot bot commented Jul 11, 2025

🦋 Changeset detected

Latest commit: c792fcd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@milaboratories/pl-deployments Patch
@milaboratories/pl-middle-layer Patch
@platforma-sdk/bootstrap Patch
@platforma-sdk/test Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Snyssfx Snyssfx force-pushed the MILAB-3485_builtin_better_errors branch from db16770 to c792fcd Compare July 11, 2025 14:01
@Snyssfx Snyssfx changed the title Milab 3485 builtin better errors MILAB-3485: pl-deployments: local: better errors Jul 11, 2025
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.

Bug: Error Handling Breaks Debugging and Upstream Processing

The catch block now throws a new Error instead of rethrowing the original error. This discards the original error's stack trace, type, and custom properties, making debugging significantly harder as the stack trace points to the catch block. It also breaks upstream error handling expecting specific error types.

lib/node/pl-deployments/src/local/trace.ts#L26-L29

} catch (e: any) {
const msg = `${e} while doing traced operation, state: ${JSON.stringify(t)}`;
logger.error(msg);
throw new Error(msg);

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant