Skip to content

Conversation

@github-actions
Copy link
Contributor

This is an automated pull request to merge tofik/logger-credential-filtering into dev.
It was created by the [Auto Pull Request] action.

@vercel
Copy link

vercel bot commented Jan 19, 2026

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

Project Deployment Review Updated (UTC)
app Ready Ready Preview, Comment Jan 19, 2026 4:27pm
1 Skipped Deployment
Project Deployment Review Updated (UTC)
portal Skipped Skipped Jan 19, 2026 4:27pm

Request Review

@cursor
Copy link

cursor bot commented Jan 19, 2026

PR Summary

Introduces a filtered logging layer and updates action logging to rely on it.

  • Adds utils/logger.ts with LoggerValidatorLayer, default skip rules (e.g., skip logs when cloudProvider: 'gcp'), and safe param formatting to avoid leaking credentials and format-string issues
  • Replaces explicit JSON.stringify in safe-action.ts with raw object logging (logger.info/warn) for Input, Result, and validation errors, leveraging the new filtering

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

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 2 potential issues.

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

@vercel vercel bot temporarily deployed to Preview – portal January 19, 2026 16:25 Inactive
@tofikwest tofikwest merged commit 147ec73 into main Jan 19, 2026
10 checks passed
@tofikwest tofikwest deleted the tofik/logger-credential-filtering branch January 19, 2026 16:33
@claudfuen
Copy link
Contributor

🎉 This PR is included in version 1.78.0 🎉

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.

3 participants