browser://console should be single resource (#34)
Returning one resource per log line is flooding the Claude UI: <img width="1061" alt="Screenshot 2025-03-19 at 16 01 45" src="https://github.com/user-attachments/assets/1779374e-6b9d-44d7-b916-c521933e1085" /> Returning one big resource with all lines feels better. original PR: https://github.com/microsoft/playwright/pull/35276
This commit is contained in:
@@ -14,7 +14,7 @@
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
import type { Resource, ResourceResult } from './resource';
|
||||
import type { Resource } from './resource';
|
||||
|
||||
export const console: Resource = {
|
||||
schema: {
|
||||
@@ -24,14 +24,12 @@ export const console: Resource = {
|
||||
},
|
||||
|
||||
read: async (context, uri) => {
|
||||
const result: ResourceResult[] = [];
|
||||
for (const message of await context.ensureConsole()) {
|
||||
result.push({
|
||||
uri,
|
||||
mimeType: 'text/plain',
|
||||
text: `[${message.type().toUpperCase()}] ${message.text()}`,
|
||||
});
|
||||
}
|
||||
return result;
|
||||
const messages = await context.ensureConsole();
|
||||
const log = messages.map(message => `[${message.type().toUpperCase()}] ${message.text()}`).join('\n');
|
||||
return [{
|
||||
uri,
|
||||
mimeType: 'text/plain',
|
||||
text: log
|
||||
}];
|
||||
},
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user