lint

packages/trace-viewer/src/sw/main.ts

@@ -80,7 +80,7 @@ async function doFetch(event: FetchEvent): Promise<Response> {
 
   if (event.request.headers.get('x-pw-serviceworker') === 'forward') {
     const request = new Request(event.request);
-    request.headers.delete('x-pw-serviceworker')
+    request.headers.delete('x-pw-serviceworker');
     return fetch(request);
   }
 

packages/trace-viewer/src/ui/aiConversation.tsx

@@ -1,5 +1,20 @@
+/**
+ * Copyright (c) Microsoft Corporation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { useCallback, useState } from 'react';
-import Markdown from 'react-markdown'
+import Markdown from 'react-markdown';
 import './aiConversation.css';
 import { clsx } from '@web/uiUtils';
 import type { Conversation, LLMMessage } from './llm';
@@ -15,50 +30,50 @@ export function AIConversation({ history, conversation }: { history: LLMMessage[
   }, [conversation]);
 
   return (
-    <div className="chat-container">
-      <div className="messages-container">
+    <div className='chat-container'>
+      <div className='messages-container'>
         {history.filter(({ role }) => role !== 'developer').map((message, index) => (
           <div
             key={'' + index}
             className={clsx('message', message.role === 'user' && 'user-message')}
           >
             {message.role === 'assistant' && (
-              <div className="message-icon">
-                <img src="playwright-logo.svg" />
+              <div className='message-icon'>
+                <img src='playwright-logo.svg' />
               </div>
             )}
-            <div className="message-content">
+            <div className='message-content'>
               <Markdown>{message.displayContent ?? message.content}</Markdown>
             </div>
           </div>
         ))}
       </div>
 
-      <div className="input-form">
+      <div className='input-form'>
         <textarea
           name='content'
           value={input}
-          onChange={(e) => setInput(e.target.value)}
+          onChange={e => setInput(e.target.value)}
           onKeyDown={e => {
             if (e.key === 'Enter' && !e.shiftKey) {
               e.preventDefault();
               onSubmit();
             }
           }}
-          placeholder="Ask a question..."
-          className="message-input"
+          placeholder='Ask a question...'
+          className='message-input'
         />
         {conversation.isSending() ? (
-          <button type="button" className="send-button" onClick={(evt) => {
-            evt.preventDefault()
+          <button type='button' className='send-button' onClick={evt => {
+            evt.preventDefault();
             conversation.abortSending();
           }}>
             Cancel
           </button>
         ) : (
-          <button className="send-button" disabled={!input.trim()} onClick={onSubmit}>
+          <button className='send-button' disabled={!input.trim()} onClick={onSubmit}>
             Send
-          </button>  
+          </button>
         )}
       </div>
     </div>

packages/trace-viewer/src/ui/errorsTab.tsx

@@ -127,11 +127,11 @@ function Error({ message, error, errorId, sdkLanguage, pageSnapshot, revealInSou
       </div>}
       <span style={{ position: 'absolute', right: '5px' }}>
         {llmAvailable
-          ? <ToolbarButton onClick={() => setShowLLM(v => !v)} style={{ width: "96px", justifyContent: 'center' }} title="Fix with AI" className='copy-to-clipboard-text-button'>{showLLM ? 'Hide AI' : 'Fix with AI'}</ToolbarButton>
+          ? <ToolbarButton onClick={() => setShowLLM(v => !v)} style={{ width: '96px', justifyContent: 'center' }} title='Fix with AI' className='copy-to-clipboard-text-button'>{showLLM ? 'Hide AI' : 'Fix with AI'}</ToolbarButton>
           : <CopyPromptButton error={message} pageSnapshot={pageSnapshot} diff={diff} />}
       </span>
     </div>
-    
+
     <ErrorMessage error={message} />
 
     {showLLM && <AIErrorConversation error={message} pageSnapshot={pageSnapshot} conversationId={errorId} diff={diff} />}
@@ -146,7 +146,7 @@ export const ErrorsTab: React.FunctionComponent<{
   revealInSource: (error: ErrorDescription) => void,
 }> = ({ errorsModel, sdkLanguage, revealInSource, actions, wallTime }) => {
   const pageSnapshot = usePageSnapshot(actions);
-  
+
   if (!errorsModel.errors.size)
     return <PlaceholderPanel text='No errors' />;
 
@@ -160,12 +160,12 @@ export const ErrorsTab: React.FunctionComponent<{
 
 export function AIErrorConversation({ conversationId, error, pageSnapshot, diff }: { conversationId: string, error: string, pageSnapshot?: string, diff?: string }) {
   const [history, conversation] = useLLMConversation(
-    conversationId,
-    [
-      `My Playwright test failed. What's going wrong?`,
-      `Please give me a suggestion how to fix it, and then explain what went wrong. Be very concise and apply Playwright best practices.`,
-      `Don't include many headings in your output. Make sure what you're saying is correct, and take into account whether there might be a bug in the app.`
-    ].join('\n')
+      conversationId,
+      [
+        `My Playwright test failed. What's going wrong?`,
+        `Please give me a suggestion how to fix it, and then explain what went wrong. Be very concise and apply Playwright best practices.`,
+        `Don't include many headings in your output. Make sure what you're saying is correct, and take into account whether there might be a bug in the app.`
+      ].join('\n')
   );
 
   React.useEffect(() => {

packages/trace-viewer/src/ui/llm.tsx

@@ -224,33 +224,32 @@ export function LLMProvider({ children }: React.PropsWithChildren<{}>) {
   const cookiePairs = useCookies();
   const chat = React.useMemo(() => {
     const cookies = Object.fromEntries(cookiePairs);
-    console.log({ cookies })
     if (cookies.openai_api_key)
       return new LLMChat(new OpenAI(cookies.openai_api_key, cookies.openai_base_url));
     if (cookies.anthropic_api_key)
       return new LLMChat(new Anthropic(cookies.anthropic_api_key, cookies.anthropic_base_url));
   }, [cookiePairs]);
   return <llmContext.Provider value={chat}>{children}</llmContext.Provider>;
-};
+}
 
 export function useLLMChat() {
-    return React.useContext(llmContext);
-};
+  return React.useContext(llmContext);
+}
 
 export function useLLMConversation(id: string, systemPrompt: string) {
-    const chat = useLLMChat();
-    if (!chat)
-        throw new Error('No LLM chat available, make sure theres a LLMProvider above');
-    const conversation = React.useMemo(() => chat.getConversation(id, systemPrompt), [chat, id]);
-    const [history, setHistory] = React.useState(conversation.history);
-    React.useEffect(() => {
-      function update() {
-        setHistory([...conversation.history]);
-      }
-      update();
-      const subscription = conversation.onChange.event(update);
-      return subscription.dispose;
-    }, [conversation]);
+  const chat = useLLMChat();
+  if (!chat)
+    throw new Error('No LLM chat available, make sure theres a LLMProvider above');
+  const conversation = React.useMemo(() => chat.getConversation(id, systemPrompt), [chat, id]);
+  const [history, setHistory] = React.useState(conversation.history);
+  React.useEffect(() => {
+    function update() {
+      setHistory([...conversation.history]);
+    }
+    update();
+    const subscription = conversation.onChange.event(update);
+    return subscription.dispose;
+  }, [conversation]);
 
-    return [history, conversation] as const;
-};
+  return [history, conversation] as const;
+}

packages/web/src/uiUtils.ts

@@ -250,8 +250,8 @@ export function useFlash(): [boolean, EffectCallback] {
 }
 
 export function useCookies() {
-  return document.cookie.split("; ").filter(v => v.includes("=")).map(kv => {
-    const separator = kv.indexOf("=");
+  return document.cookie.split('; ').filter(v => v.includes('=')).map(kv => {
+    const separator = kv.indexOf('=');
     return [kv.substring(0, separator), kv.substring(separator + 1)];
-  }) 
+  });
 }