fix some line lengths to satisfy pre-commit

Author: sebastian-goeldi

.gitignore

@@ -132,4 +132,4 @@ dmypy.json
 #css/bootstrap/npm
 package-lock.json
 package.json
-node_modules
+node_modules

.pre-commit-config.yaml

@@ -14,12 +14,8 @@ repos:
         args: ["--pytest-test-first"]
       - id: requirements-txt-fixer
       - id: trailing-whitespace
+        exclude: ^src/kweb/static/bootstrap/
 
-  - repo: https://github.com/hakancelik96/unimport
-    rev: 0.14.1
-    hooks:
-      - id: unimport
-        args: [--remove, --include-star-import]
   - repo: https://github.com/pycqa/isort
     rev: "5.12.0"
     hooks:
@@ -28,7 +24,7 @@ repos:
         args: ["--profile", "black", "--filter-files"]
 
   - repo: https://github.com/psf/black
-    rev: "23.1.0"
+    rev: "23.7.0"
     hooks:
       - id: black
 
@@ -44,7 +40,7 @@ repos:
         files: ".ipynb"
 
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.3.1
+    rev: v3.10.1
     hooks:
       - id: pyupgrade
         args: [--py310-plus, --keep-runtime-typing]
@@ -56,7 +52,7 @@ repos:
   #       args: ["-L TE,TE/TM,te,ba,FPR,fpr_spacing,ro,donot"]
 
   - repo: https://github.com/shellcheck-py/shellcheck-py
-    rev: v0.9.0.2
+    rev: v0.9.0.5
     hooks:
       - id: shellcheck
 
@@ -66,7 +62,7 @@ repos:
   #     - id: python-use-type-annotations
 
   - repo: https://github.com/PyCQA/bandit
-    rev: 1.7.4
+    rev: 1.7.5
     hooks:
       - id: bandit
         args: [--exit-zero]
@@ -137,6 +133,6 @@ repos:
   #     # - id: nbqa-isort
   #     #   args: ["--float-to-top"]
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: "v0.0.253"
+    rev: "v0.0.285"
     hooks:
       - id: ruff

pyproject.toml

@@ -71,13 +71,6 @@ exclude = '''  # Specify the files/dirs that should be ignored by the black form
 )/
 '''
 
-[tool.commitizen]
-name = "cz_conventional_commits"
-version = "0.1.1"
-version_files = [
-    "pyproject.toml:version",
-]
-
 [tool.mypy]
 python_version = "3.10"
 strict = true

src/kweb/main.py

@@ -1,12 +1,10 @@
-import pathlib
-import tempfile
-from glob import glob
+import os
 from pathlib import Path
 from typing import Any
 
 from fastapi import FastAPI, Request
 from fastapi.exceptions import HTTPException
-from fastapi.responses import HTMLResponse, RedirectResponse
+from fastapi.responses import HTMLResponse
 from fastapi.staticfiles import StaticFiles
 from fastapi.templating import Jinja2Templates
 from starlette.routing import WebSocketRoute
@@ -15,8 +13,6 @@
 from kweb import __version__ as version
 from kweb.server import LayoutViewServerEndpoint
 
-import os
-
 # module_path = Path(os.getenv("KWEB_EDAFILES", Path(__file__).parent.resolve()))
 module_path = Path(__file__).parent.absolute()
 home_path = Path.home() / ".gdsfactory" / "extra"
@@ -55,7 +51,8 @@ async def gds_view_static(
     if not exists:
         raise HTTPException(
             status_code=404,
-            detail=f'No gds found with name "{gds_name}". It doesn\'t exist or is not accessible',
+            detail=f'No gds found with name "{gds_name}".'
+            " It doesn't exist or is not accessible",
         )
 
     root_path = request.scope["root_path"]

src/kweb/server.py

@@ -109,9 +109,7 @@ def layer_dump(
                                 ).to_png_data()
                             ).decode("ASCII"),
                             "children": children,
-                            "empty": all(
-                                (c["empty"] for c in children)  # type: ignore[call-overload]
-                            ),
+                            "empty": all(c["empty"] for c in children),
                         }
                     )
                     iter.next_sibling(1)
@@ -171,9 +169,7 @@ def layer_dump(
                                 ).to_png_data()
                             ).decode("ASCII"),
                             "children": children,
-                            "empty": all(
-                                (c["empty"] for c in children)  # type: ignore[call-overload]
-                            ),
+                            "empty": all(c["empty"] for c in children),
                         }
                     )
                     iter.next_sibling(1)
@@ -273,9 +269,10 @@ async def connection(self, websocket: WebSocket, path: str | None = None) -> Non
         asyncio.create_task(self.timer(websocket))
 
     async def timer(self, websocket: WebSocket) -> None:
-        self.layout_view.on_image_updated_event = lambda: self.image_updated(  # type: ignore[attr-defined,assignment]
-            websocket
-        )
+        def update() -> None:
+            self.image_updated(websocket)
+
+        self.layout_view.on_image_updated_event = update  # type: ignore[assignment]
         while True:
             self.layout_view.timer()  # type: ignore[attr-defined]
             await asyncio.sleep(0.01)
@@ -380,11 +377,13 @@ async def reader(self, websocket: WebSocket, data: str) -> None:
                 )
             case "mouse_pressed":
                 self.mouse_event(
-                    self.layout_view.send_mouse_press_event, js  # type: ignore[arg-type]
+                    self.layout_view.send_mouse_press_event,
+                    js,  # type: ignore[arg-type]
                 )
             case "mouse_released":
                 self.mouse_event(
-                    self.layout_view.send_mouse_release_event, js  # type: ignore[arg-type]
+                    self.layout_view.send_mouse_release_event,
+                    js,  # type: ignore[arg-type]
                 )
             case "mouse_enter":
                 self.layout_view.send_enter_event()

src/kweb/static/client.css

@@ -131,4 +131,4 @@ ul, #myUL {
   object-fit: contain;
   width: 80%;
   height: 80%;
-}
+}

src/kweb/static/client.js

@@ -274,7 +274,7 @@ function appendCells(parentelement, cells, current_index, addpadding=false) {
       }
       accordion.id = "cellgroup-" + c.id;
 
-    
+
       cellRow.appendChild(accordion);
 
       accordion_item = document.createElement("div");
@@ -328,7 +328,7 @@ function appendCells(parentelement, cells, current_index, addpadding=false) {
 
       appendCells(accordion_body, c.children, current_index, true);
       lastelement = accordion;
-    
+
     } else {
       let cell_name_button = document.createElement("input");
       cell_name_button.className = "btn-check";
@@ -351,7 +351,7 @@ function appendCells(parentelement, cells, current_index, addpadding=false) {
       accordion_row.className = "row mx-0";
       accordion_row.appendChild(cell_name_button);
       accordion_row.appendChild(cell_name);
-    
+
       let accordion = document.createElement("div");
       if (addpaddings) {
         accordion.className = "accordion accordion-flush ps-2 pe-0";
@@ -422,7 +422,7 @@ function appendLayers(parentelement, layers, addempty=false, addpaddings = false
         }
         accordion.id = "layergroup-" + l.id;
 
-      
+
         layerRow.appendChild(accordion);
 
         accordion_item = document.createElement("div");
@@ -455,7 +455,7 @@ function appendLayers(parentelement, layers, addempty=false, addpaddings = false
         }
 
         layer_image.addEventListener("click", click_layer_img);
-      
+
         img_cont.appendChild(layer_image);
         let layer_name = document.createElement("div");
         layer_name.innerHTML = l.name;
@@ -484,7 +484,7 @@ function appendLayers(parentelement, layers, addempty=false, addpaddings = false
 
         appendLayers(accordion_body, l.children, addempty=addempty);
         lastelement = accordion;
-      
+
       } else {
         let img_cont = document.createElement("div");
         img_cont.className = "col-auto p-0";
@@ -512,7 +512,7 @@ function appendLayers(parentelement, layers, addempty=false, addpaddings = false
         accordion_row.appendChild(img_cont);
         accordion_row.appendChild(layer_name);
         accordion_row.appendChild(layer_source);
-      
+
         let accordion = document.createElement("div");
         if (addpaddings) {
           accordion.className = "accordion accordion-flush px-2";
@@ -605,4 +605,4 @@ window.addEventListener("keydown", function(evt) {
     evt.preventDefault();
     sendKeyEvent(canvas, "keydown", evt);
   }
-});
+});