Skip to content

fix(sdk): remove console.log from listener#1787

Merged
neelneelneel merged 1 commit intodevfrom
remove-console-log
Aug 28, 2025
Merged

fix(sdk): remove console.log from listener#1787
neelneelneel merged 1 commit intodevfrom
remove-console-log

Conversation

@tevanburen
Copy link
Copy Markdown
Contributor

Description

Removes the console.log from listener.ts

@tevanburen tevanburen self-assigned this Aug 28, 2025
@tevanburen tevanburen requested a review from a team as a code owner August 28, 2025 19:24
@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /describe

@QodoAI-Agent
Copy link
Copy Markdown

Title

fix(sdk): remove console.log from listener


User description

Description

Removes the console.log from listener.ts


PR Type

Bug fix


Description

  • Remove debug console.log from listener

  • Preserve message handling logic unchanged


Diagram Walkthrough

flowchart LR
  A["Window message listener"] -- "before" --> B["Log to console + handle message"]
  A -- "after" --> C["Handle message without logging"]
Loading

File Walkthrough

Relevant files
Bug fix
listener.ts
Strip debug logging from window message listener                 

libs/sdk/src/listener.ts

  • Remove console.log of incoming messages
  • Keep SMSS_INIT_TOOL handling intact
+0/-2     

@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /review

@QodoAI-Agent
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /improve

@QodoAI-Agent
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@neelneelneel neelneelneel merged commit 59c800b into dev Aug 28, 2025
4 checks passed
@neelneelneel neelneelneel deleted the remove-console-log branch August 28, 2025 19:49
@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /update_changelog

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.

3 participants