Skip to content
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

Error executing browser tool: page.evaluate: ReferenceError: __name is not defined #276

Open
rajeshgautamphd opened this issue Jan 17, 2025 · 2 comments

Comments

@rajeshgautamphd
Copy link

What

Getting this error:

Error executing browser tool: page.evaluate: ReferenceError: __name is not defined
at eval (eval at evaluate (:234:30), :1:781)
at UtilityScript.evaluate (:236:17)
at UtilityScript. (:1:44)
at BrowserTool.getNormalizedComponentStringByCoords (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/browser/core/browser-tool.ts:801:33)
at (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/ai/client.ts:168:43)
at AIClient.makeRequest (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/ai/client.ts:144:31)
at AIClient.processAction (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/ai/client.ts:49:16)
at TestRunner.executeTest (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/core/runner/index.ts:279:20)
at TestRunner.executeTestFile (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/core/runner/index.ts:364:26)
at TestRunner.runTests (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/core/runner/index.ts:410:7)
at main (/Users/admin/Downloads/shortest-file-upload/shortest/packages/shortest/src/cli/bin.ts:152:5)

Why

Why are we shipping it? (Intention behind behavior change)

The less words the better, but "save others time" – provide all of the necessary context in order to grok the issue and contribute.

How

What's the right approach, atomically, to ship this?

@rmarescu
Copy link
Member

Hey @rajeshgautamphd, thanks for the report. I believe is the same issue discussed on #256 (comment).

I'll remove the script that uses the source code (pkg:test:src) until it gets fixed.

Meanwhile, could you try building the package first?

pnpm pkg:build
pnpm shortest

rmarescu added a commit that referenced this issue Jan 17, 2025
@rajeshgautamphd
Copy link
Author

thanks @rmarescu, that works.

@rmarescu rmarescu added bug and removed bug labels Jan 18, 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

No branches or pull requests

2 participants