Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[feature]: Add SSE support #59

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 6, 2025
Merged

[feature]: Add SSE support #59

merged 4 commits into from
May 6, 2025

Conversation

Kylejeong2
Copy link
Contributor

What

Added ability to connect with server side events.

@@ -0,0 +1 @@
import './dist/program.js';
Copy link
Member

Choose a reason for hiding this comment

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

should cli.js this be a .ts file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no

Copy link
Member

Choose a reason for hiding this comment

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

can remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

why remove it? i guess it doesn't really matter whether or not we have but its in the current implementation

{
"mcpServers": {
"playwright": {
"url": "http://localhost:8931/sse",
Copy link
Member

Choose a reason for hiding this comment

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

will there a be a place for flags?

cli commands?

Copy link
Member

Choose a reason for hiding this comment

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

probably can link to config.d.ts

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah ill add more info, flags would be in the args

@@ -0,0 +1,2 @@
import { createServer } from './dist/index.js';
Copy link
Member

Choose a reason for hiding this comment

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

should index.js be .ts

Copy link
Contributor Author

Choose a reason for hiding this comment

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

its js so you can run with node technically i think it can be as long as cli.js is js since thats our entrypoint

@alexdphan alexdphan merged commit 0c38741 into ap/bbplaywright May 6, 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.

2 participants