Skip to content

Conversation

@maheshmurag
Copy link
Contributor

No description provided.

Copy link
Member

@jspahrsummers jspahrsummers left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! Most of the changes look good, but I'm not sure about moving to a class. What are your thoughts?

},
];

class PuppeteerServer {
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this OO style is very conventional for TypeScript servers (if we look at Node.js' built-in HTTP, Express, the Slack SDK, etc).

Can we leave it as free functions?

@maheshmurag maheshmurag merged commit 018c7d8 into main Nov 19, 2024
5 checks passed
@maheshmurag maheshmurag deleted the mahesh/update-puppeteer branch November 19, 2024 23:00
HarleyCoops pushed a commit to HarleyCoops/ClaudeServers that referenced this pull request Nov 28, 2024
…ahesh/update-puppeteer

Update Puppeteer (Code & README)
jspahrsummers pushed a commit that referenced this pull request Dec 12, 2024
Add mcp-server-rememberizer to community servers in README.md
laith-abood pushed a commit to laith-abood/mcp-claude that referenced this pull request Jan 13, 2025
…mberizer-server

Add mcp-server-rememberizer to community servers in README.md
CDRPico pushed a commit to CDRPico/mcp-servers that referenced this pull request Mar 10, 2025
PazerOP referenced this pull request in PazerOP/mcp-template Jul 15, 2025
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