computer-use-ootb-internal 0.0.115__py3-none-any.whl → 0.0.117__py3-none-any.whl

computer_use_ootb_internal/app_teachmode.py

@@ -11,11 +11,48 @@ import os # Import os for path joining
 import logging # Import logging
 import importlib # For dynamic imports
 import pkgutil # To find modules
+import sys # For logging setup
+import traceback # For logging setup
+from logging.handlers import RotatingFileHandler # For logging setup
 from fastapi import FastAPI, Request
 from fastapi.responses import JSONResponse
 from fastapi.middleware.cors import CORSMiddleware
 from computer_use_ootb_internal.computer_use_demo.tools.computer import get_screen_details
 from computer_use_ootb_internal.run_teachmode_ootb_args import simple_teachmode_sampling_loop
+import uvicorn # Assuming uvicorn is used to run FastAPI
+
+# --- App Logging Setup ---
+try:
+    # Log to user's AppData directory for better accessibility
+    log_dir_base = os.environ.get('APPDATA', os.path.expanduser('~'))
+    log_dir = os.path.join(log_dir_base, 'OOTBAppLogs')
+    os.makedirs(log_dir, exist_ok=True)
+    log_file = os.path.join(log_dir, 'ootb_app.log')
+
+    log_format = '%(asctime)s - %(levelname)s - %(process)d - %(threadName)s - %(message)s'
+    log_level = logging.INFO # Or logging.DEBUG for more detail
+
+    # Use rotating file handler
+    handler = RotatingFileHandler(log_file, maxBytes=5*1024*1024, backupCount=2, encoding='utf-8')
+    handler.setFormatter(logging.Formatter(log_format))
+
+    # Configure root logger
+    logging.basicConfig(level=log_level, handlers=[handler])
+
+    # Add stream handler to see logs if running interactively (optional)
+    # logging.getLogger().addHandler(logging.StreamHandler(sys.stdout))
+
+    logging.info("="*20 + " OOTB App Starting " + "="*20)
+    logging.info(f"Running with args: {sys.argv}")
+    logging.info(f"Python Executable: {sys.executable}")
+    logging.info(f"Working Directory: {os.getcwd()}")
+    logging.info(f"User: {os.getenv('USERNAME')}")
+
+except Exception as log_setup_e:
+    print(f"FATAL: Failed to set up logging: {log_setup_e}")
+    # Fallback logging might be needed here if file logging fails
+
+# --- End App Logging Setup ---
 
 app = FastAPI()
 
@@ -56,6 +93,8 @@ class RateLimiter:
 
 
 def log_ootb_request(server_url, ootb_request_type, data):
+    logging.info(f"OOTB Request: Type={ootb_request_type}, Data={data}")
+    # Keep the requests post for now if it serves a specific purpose
     logging_data = {
         "type": ootb_request_type,
         "data": data,
@@ -65,7 +104,10 @@ def log_ootb_request(server_url, ootb_request_type, data):
         server_logging_url = server_url + "/update_ootb_logging"
     else:
         server_logging_url = server_url
-    requests.post(server_logging_url, json=logging_data)
+    try:
+        requests.post(server_logging_url, json=logging_data, timeout=5)
+    except Exception as req_log_e:
+        logging.warning(f"Could not log ootb request to server {server_logging_url}: {req_log_e}")
 
 
 class SharedState:
@@ -138,35 +180,41 @@ def prepare_environment(state):
 
 @app.post("/update_params")
 async def update_parameters(request: Request):
-    data = await request.json()
-    
-    if 'task' not in data:
-        return JSONResponse(
-            content={"status": "error", "message": "Missing required field: task"},
-            status_code=400
-        )
-    
-    shared_state.args = argparse.Namespace(**data)
-    shared_state.task_updated = True
-    
-    # Update shared state when parameters change
-    shared_state.model = getattr(shared_state.args, 'model', "teach-mode-gpt-4o")
-    shared_state.task = getattr(shared_state.args, 'task', "Following the instructions to complete the task.")
-    shared_state.selected_screen = getattr(shared_state.args, 'selected_screen', 0)
-    shared_state.user_id = getattr(shared_state.args, 'user_id', "hero_cases")
-    shared_state.trace_id = getattr(shared_state.args, 'trace_id', "build_scroll_combat")
-    shared_state.api_keys = getattr(shared_state.args, 'api_keys', "sk-proj-1234567890")
-    shared_state.server_url = getattr(shared_state.args, 'server_url', "http://ec2-44-234-43-86.us-west-2.compute.amazonaws.com")
-    
-    log_ootb_request(shared_state.server_url, "update_params", data)
+    logging.info("Received request to /update_params")
+    try:
+        data = await request.json()
+        
+        if 'task' not in data:
+            return JSONResponse(
+                content={"status": "error", "message": "Missing required field: task"},
+                status_code=400
+            )
+        
+        shared_state.args = argparse.Namespace(**data)
+        shared_state.task_updated = True
+        
+        # Update shared state when parameters change
+        shared_state.model = getattr(shared_state.args, 'model', "teach-mode-gpt-4o")
+        shared_state.task = getattr(shared_state.args, 'task', "Following the instructions to complete the task.")
+        shared_state.selected_screen = getattr(shared_state.args, 'selected_screen', 0)
+        shared_state.user_id = getattr(shared_state.args, 'user_id', "hero_cases")
+        shared_state.trace_id = getattr(shared_state.args, 'trace_id', "build_scroll_combat")
+        shared_state.api_keys = getattr(shared_state.args, 'api_keys', "sk-proj-1234567890")
+        shared_state.server_url = getattr(shared_state.args, 'server_url', "http://ec2-44-234-43-86.us-west-2.compute.amazonaws.com")
+        
+        log_ootb_request(shared_state.server_url, "update_params", data)
 
-    # Call the (now dynamic) preparation function here, after parameters are updated
-    prepare_environment(shared_state)
+        # Call the (now dynamic) preparation function here, after parameters are updated
+        prepare_environment(shared_state)
 
-    return JSONResponse(
-        content={"status": "success", "message": "Parameters updated", "new_args": vars(shared_state.args)},
-        status_code=200
-    )
+        logging.info("Parameters updated successfully.")
+        return JSONResponse(
+            content={"status": "success", "message": "Parameters updated", "new_args": vars(shared_state.args)},
+            status_code=200
+        )
+    except Exception as e:
+        logging.error("Error processing /update_params:", exc_info=True)
+        return JSONResponse(content={"status": "error", "message": "Internal server error"}, status_code=500)
 
 @app.post("/update_message")
 async def update_message(request: Request):
@@ -332,6 +380,8 @@ async def get_status(request: Request):
     )
 
 def process_input():
+    global shared_state
+    logging.info("process_input thread started.")
     shared_state.is_processing = True
     shared_state.should_stop = False
     shared_state.is_paused = False
@@ -412,8 +462,11 @@ def process_input():
         shared_state.is_paused = False
         shared_state.stop_event.clear()
         print("Processing completed, ready for new tasks")
+    logging.info("process_input thread finished.")
 
 def main():
+    # Logging is set up at the top level now
+    logging.info("App main() function starting setup.")
     global app, shared_state, rate_limiter
     
     parser = argparse.ArgumentParser(
@@ -431,36 +484,35 @@ def main():
         default="http://ec2-44-234-43-86.us-west-2.compute.amazonaws.com",
         help="Server URL for the session"
     )
+    parser.add_argument("--host", type=str, default="127.0.0.1", help="Host to bind the server to")
+    parser.add_argument("--port", type=int, default=8000, help="Port to bind the server to") # Default port
 
     args = parser.parse_args()
     shared_state = SharedState(args)
     rate_limiter = RateLimiter(interval_seconds=2)
 
-    import uvicorn
-    import platform
-    import os
-    
-    # Default port
-    port = 7888
-    
-    # Determine port based on Windows username
-    if platform.system() == "Windows":
-        username = os.environ["USERNAME"].lower()
-        if username == "altair":
-            port = 14000
-        elif username.startswith("guest") and username[5:].isdigit():
-            num = int(username[5:])
-            if 1 <= num <= 10:
-                port = 14000 + num
-            else:
-                port = 7888
-        else:
-            port = 7888
-    
-    uvicorn.run(app, host="0.0.0.0", port=port)
+    # Validate args or set defaults if needed
+    if not hasattr(args, 'model'): args.model = "default_model"
+    if not hasattr(args, 'task'): args.task = "default_task"
+    if not hasattr(args, 'selected_screen'): args.selected_screen = 0
+    if not hasattr(args, 'user_id'): args.user_id = "unknown_user"
+    if not hasattr(args, 'trace_id'): args.trace_id = "unknown_trace"
+    if not hasattr(args, 'api_keys'): args.api_keys = "none"
+    if not hasattr(args, 'server_url'): args.server_url = "none"
+
+    logging.info(f"Shared state initialized. Host={args.host}, Port={args.port}")
+
+    try:
+        logging.info(f"Starting Uvicorn server on {args.host}:{args.port}")
+        uvicorn.run(app, host=args.host, port=args.port)
+        logging.info("Uvicorn server stopped.")
+    except Exception as main_e:
+        logging.error("Error in main execution:", exc_info=True)
+    finally:
+        logging.info("App main() function finished.")
 
 if __name__ == "__main__":
-    # main()
+    main()
 
     # Test log_ootb_request
     log_ootb_request("http://ec2-44-234-43-86.us-west-2.compute.amazonaws.com", "test_request", {"message": "Test message"})

computer_use_ootb_internal/guard_service.py

@@ -34,7 +34,7 @@ _LISTEN_HOST = "0.0.0.0" # Listen on all interfaces
 _LISTEN_PORT = 14000 # Port for server to POST commands TO
 # _SHARED_SECRET = "YOUR_SECRET_HERE" # !! REMOVED !! - No secret check implemented now
 # --- End Server POST Configuration ---
-_SERVER_STATUS_REPORT_URL = "http://52.160.105.102:7000/api/guard/status" # URL to POST status back TO
+_SERVER_STATUS_REPORT_URL = "http://52.160.105.102:7000/guard/status" # URL to POST status back TO (Path changed)
 _LOG_FILE = pathlib.Path(os.environ['PROGRAMDATA']) / "OOTBGuardService" / "guard_post_mode.log" # Different log file
 # --- End Configuration ---
 
@@ -427,18 +427,34 @@ class GuardService(win32serviceutil.ServiceFramework):
                      token = win32ts.WTSQueryUserToken(session_id)
                      env = win32profile.CreateEnvironmentBlock(token, False)
                      startup = win32process.STARTUPINFO()
+                     # Simplify startup flags: Run hidden, don't explicitly set desktop
                      startup.dwFlags = win32process.STARTF_USESHOWWINDOW
-                     startup.wShowWindow = win32con.SW_SHOW # Try showing the window
-                     startup.lpDesktop = 'winsta0\\default'
+                     startup.wShowWindow = win32con.SW_HIDE
+                     # startup.lpDesktop = 'winsta0\\default' # Removed
+
                      creation_flags = win32process.CREATE_NEW_CONSOLE | win32process.CREATE_UNICODE_ENVIRONMENT
+                     # Define cwd as user's profile directory if possible
+                     user_profile_dir = win32profile.GetUserProfileDirectory(token)
 
                      hProcess, hThread, dwPid, dwTid = win32process.CreateProcessAsUser(
                          token, self.python_exe, self.ootb_command,
-                         None, None, False, creation_flags, env, None, startup
+                         None, None, False, creation_flags, env, 
+                         user_profile_dir, # Set current directory
+                         startup
                      )
-                     log_info(f"Successfully started OOTB for user '{user}' (PID: {dwPid}).")
-                     started_count += 1
-                     target_users_started.add(user)
+                     log_info(f"CreateProcessAsUser call succeeded for user '{user}' (PID: {dwPid}).")
+
+                     # Add post-start check
+                     time.sleep(1) # Small delay
+                     if psutil.pid_exists(dwPid):
+                         log_info(f"Process PID {dwPid} confirmed to exist shortly after creation.")
+                         started_count += 1
+                         target_users_started.add(user)
+                     else:
+                         log_warning(f"Process PID {dwPid} reported by CreateProcessAsUser does NOT exist shortly after creation. It likely exited immediately.")
+                         users_failed_to_start.add(user)
+                         # Attempt to get exit code? Difficult without waiting.
+
                      win32api.CloseHandle(hProcess)
                      win32api.CloseHandle(hThread)
 

computer_use_ootb_internal/service_manager.py

@@ -107,19 +107,29 @@ def run_service_command(command_args, check_errors=True):
 
 def install_and_start():
     """Installs and starts the Guard Service."""
-    print(f"Attempting to install/update service: '{_SERVICE_NAME}' ('{_SERVICE_DISPLAY_NAME}')")
-    # Use 'update' which handles install or updates display name/path if needed
-    if run_service_command(['--startup', 'auto', 'update']): # Set startup to auto, then update
-        print(f"\nAttempting to start service: '{_SERVICE_NAME}' (waiting a few seconds first)")
-        time.sleep(3) # Give SCM time to register the update
-        if run_service_command(['start']):
-            print(f"\nService '{_SERVICE_NAME}' installed/updated and started successfully.")
+    print(f"Attempting to install service: '{_SERVICE_NAME}' ('{_SERVICE_DISPLAY_NAME}')")
+    # Call 'install' command first.
+    # We pass check_errors=True to stop if installation fails fundamentally.
+    install_success = run_service_command(['--startup', 'auto', 'install'], check_errors=True)
+
+    if install_success:
+        # Note: Even if install_success is True, pywin32 might have printed internal errors
+        # like 'service already installed'. We proceed to start anyway in that case.
+        print(f"\nInstallation command finished. Attempting to start service: '{_SERVICE_NAME}' (waiting a few seconds first)")
+        time.sleep(3) # Give SCM time to register the install/update
+        start_success = run_service_command(['start'], check_errors=True)
+
+        if start_success:
+            # Similar caveat: start might succeed according to subprocess, but pywin32 could print internal errors.
+            print(f"\nService '{_SERVICE_NAME}' install command executed and start command executed.")
+            print(f"Please verify service status in 'services.msc' and check logs.")
         else:
-             print(f"\nWarning: Service '{_SERVICE_NAME}' installed/updated but failed to start.", file=sys.stderr)
-             print(f" Check service logs ('C:\ProgramData\OOTBGuardService\guard.log') or Windows Event Viewer for details.", file=sys.stderr)
-             print(f" You may need to start it manually via 'services.msc' or `net start {_SERVICE_NAME}`.")
+             # This path is taken if run_service_command returned False (subprocess error occurred)
+             print(f"\nService '{_SERVICE_NAME}' installed/updated but the 'start' command failed with an error.", file=sys.stderr)
+             print(f" Check output above, service logs ('C:\ProgramData\OOTBGuardService\guard_post_mode.log'), or Windows Event Viewer.", file=sys.stderr)
     else:
-        print(f"\nService '{_SERVICE_NAME}' installation/update failed.", file=sys.stderr)
+        # This path is taken if the initial 'install' command failed critically (subprocess error)
+        print(f"\nService '{_SERVICE_NAME}' installation failed critically. See errors above.", file=sys.stderr)
 
 
 def stop_and_remove():

{computer_use_ootb_internal-0.0.115.dist-info → computer_use_ootb_internal-0.0.117.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: computer-use-ootb-internal
-Version: 0.0.115
+Version: 0.0.117
 Summary: Computer Use OOTB
 Author-email: Siyuan Hu <siyuan.hu.sg@gmail.com>
 Requires-Python: >=3.11

{computer_use_ootb_internal-0.0.115.dist-info → computer_use_ootb_internal-0.0.117.dist-info}/RECORD

@@ -1,12 +1,12 @@
 computer_use_ootb_internal/README.md,sha256=FxpW95lyub2iX73ZDfK6ML7SdEKg060H5I6Grub7li4,31
 computer_use_ootb_internal/__init__.py,sha256=Nqnn8clbgv-5l0PgxcTOldg8mkMKrFn4TvPL-rYUUGg,1
-computer_use_ootb_internal/app_teachmode.py,sha256=U0vHAEHeDMDioYatLQkzesT1Hy0aI6kKjhAlV86zxfc,17902
+computer_use_ootb_internal/app_teachmode.py,sha256=GkgvBLRwp_kWM-wkTXPoSnxagfol_51Hqsga21LGFk4,20971
 computer_use_ootb_internal/app_teachmode_gradio.py,sha256=cmFpBrkdlZxOQADWveVdIaaNqaBD8IVs-xNLJogU7F8,7909
 computer_use_ootb_internal/dependency_check.py,sha256=y8RMEP6RXQzTgU1MS_1piBLtz4J-Hfn9RjUZg59dyvo,1333
-computer_use_ootb_internal/guard_service.py,sha256=_tPhlciqXmtWViOeZSsiNhYdBsvqrN9yoFhI9Zojc64,22575
+computer_use_ootb_internal/guard_service.py,sha256=z9GQdPJCd_Ds1xwCUn7hJBqPuto_FLhE1F5xqrvgh98,23496
 computer_use_ootb_internal/requirements-lite.txt,sha256=5DAHomz4A_P2BmTIXNkNqkHbnIF0AyZ4_1XAlb1LaYs,290
 computer_use_ootb_internal/run_teachmode_ootb_args.py,sha256=7Dj0iY4GG7P03tRKYJ2x9Yvt-PE-b7uyjCAed3SaF3Y,7086
-computer_use_ootb_internal/service_manager.py,sha256=sesbSUBBqm6W77Dykaw-HGARU-yNdK9DqOHPiR2TbkE,6920
+computer_use_ootb_internal/service_manager.py,sha256=xbLIxQ4jzMr8AdZdxKw-QcAGm2Y86vudDzSf3hGjOM4,7595
 computer_use_ootb_internal/computer_use_demo/animation/click_animation.py,sha256=tP1gsayFy-CKk10UlrE9RlexwlHWiHQUe5Ogg4vQvSg,3234
 computer_use_ootb_internal/computer_use_demo/animation/icons8-select-cursor-transparent-96.gif,sha256=4LfwsfFQnREXrNRs32aJU2jO65JXianJoL_8q7-8elg,30966
 computer_use_ootb_internal/computer_use_demo/animation/test_animation.py,sha256=2R1u98OLKYalSZ5nt5vvyZ71FL5R5vLv-n8zM8jVdV8,1183
@@ -34,7 +34,7 @@ computer_use_ootb_internal/computer_use_demo/tools/run.py,sha256=xhXdnBK1di9muaO
 computer_use_ootb_internal/computer_use_demo/tools/screen_capture.py,sha256=L8qfvtUkPPQGt92N-2Zfw5ZTDBzLsDps39uMnX3_uSA,6857
 computer_use_ootb_internal/preparation/__init__.py,sha256=AgtGHcBpiTkxJjF0xwcs3yyQ6SyUvhL3G0vD2XO-zJw,63
 computer_use_ootb_internal/preparation/star_rail_prepare.py,sha256=s1VWszcTnJAKxqCHFlaOEwPkqVSrkiFx_yKpWSnSbHs,2649
-computer_use_ootb_internal-0.0.115.dist-info/METADATA,sha256=FRm9Sd0aHBSdRsRGeeYfoIBtMSbcMjeaSkx4uGHoNU0,1048
-computer_use_ootb_internal-0.0.115.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-computer_use_ootb_internal-0.0.115.dist-info/entry_points.txt,sha256=bXfyAU_qq-G1EiEgAQEioXvgEdRCFxaTooqdDD9Y4OA,258
-computer_use_ootb_internal-0.0.115.dist-info/RECORD,,
+computer_use_ootb_internal-0.0.117.dist-info/METADATA,sha256=mUDjRU51iIamJDr2479tUSAiXKrCTTzpGQ-g4ZYfcWU,1048
+computer_use_ootb_internal-0.0.117.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+computer_use_ootb_internal-0.0.117.dist-info/entry_points.txt,sha256=bXfyAU_qq-G1EiEgAQEioXvgEdRCFxaTooqdDD9Y4OA,258
+computer_use_ootb_internal-0.0.117.dist-info/RECORD,,