fix: Add task correlation fields and structured log

.gitignore

@@ -150,6 +150,9 @@ webhook-server.private-key.pem
 log-colors.json
 webhook_server/tests/manifests/logs
 .coverage_report.txt
+coverage.json
+webhook-examples
+find_unused_code.py
 
 # AI
 .cursor/
@@ -159,3 +162,4 @@ CLAUDE.md
 .claude/
 .claude-flow/
 .swarm/
+CRUSH.md

.pre-commit-config.yaml

@@ -61,3 +61,13 @@ repos:
       - id: mypy
         exclude: (tests/)
         additional_dependencies: [types-requests, types-PyYAML, types-colorama]
+
+  - repo: https://github.com/pre-commit/mirrors-eslint
+    rev: v9.38.0
+    hooks:
+      - id: eslint
+        files: \.js$
+        exclude: eslint\.config\.js
+        args: [--fix]
+        additional_dependencies:
+          - eslint@9.38.0

Dockerfile

@@ -9,11 +9,12 @@ ENV PATH="$PATH:$BIN_DIR"
 ENV DATA_DIR="$HOME_DIR/data"
 ENV APP_DIR="$HOME_DIR/github-webhook-server"
 
+RUN systemd-machine-id-setup
+
 RUN dnf -y install dnf-plugins-core \
   && dnf -y update \
   && dnf -y install \
   git \
-  hub \
   unzip \
   gcc \
   python3-devel \
@@ -51,6 +52,7 @@ ENV UV_PYTHON=python3.13
 ENV UV_COMPILE_BYTECODE=1
 ENV UV_NO_SYNC=1
 ENV UV_CACHE_DIR=${APP_DIR}/.cache
+ENV PYTHONUNBUFFERED=1
 
 COPY --from=ghcr.io/astral-sh/uv:latest /uv /uvx ${BIN_DIR}/
 RUN uv tool install pre-commit && uv tool install poetry && uv tool install prek
@@ -62,7 +64,13 @@ RUN set -x \
   && chmod +x $BIN_DIR/rosa \
   && rm -rf $BIN_DIR/rosa-linux.tar.gz \
   && curl -L https://github.com/regclient/regclient/releases/latest/download/regctl-linux-amd64 >$BIN_DIR/regctl \
-  && chmod +x $BIN_DIR/regctl
+  && chmod +x $BIN_DIR/regctl \
+  && curl -L https://github.com/mislav/hub/releases/download/v2.14.2/hub-linux-amd64-2.14.2.tgz --output ${BIN_DIR}/hub-linux-amd64.tgz \
+  && tmp_dir="$(mktemp -d)" \
+  && tar xvf ${BIN_DIR}/hub-linux-amd64.tgz -C "${tmp_dir}" \
+  && mv "${tmp_dir}"/hub-linux-amd64-2.14.2/bin/hub ${BIN_DIR}/hub \
+  && chmod +x ${BIN_DIR}/hub \
+  && rm -rf "${tmp_dir}" ${BIN_DIR}/hub-linux-amd64.tgz
 
 WORKDIR $APP_DIR
 

README.md

@@ -57,6 +57,11 @@ GitHub Events → Webhook Server → Repository Management
     └─────────────────────────────────────┘
 ```
 
+**Key Architecture Components:**
+
+- **Performance Optimized**: Repository data fetched efficiently to minimize API calls
+- **Type-Safe**: Full mypy strict mode coverage ensuring code reliability
+
 ## Features
 
 ### 🔧 Repository Management

entrypoint.py

@@ -46,6 +46,13 @@ def run_podman_cleanup() -> None:
     run_podman_cleanup()
 
     result = asyncio.run(repository_and_webhook_settings(webhook_secret=_webhook_secret))
+
+    # Logging Configuration:
+    # - Uvicorn uses default logging which automatically respects FORCE_COLOR environment variable
+    #   for colored terminal output (useful for Docker logs with color support)
+    # - Application logs use simple-logger with console=True for colored output in Docker logs
+    # - Both logging systems work together: uvicorn handles HTTP request logs,
+    #   while simple-logger handles application-level logs with structured formatting
     uvicorn.run(
         "webhook_server.app:FASTAPI_APP",
         host=_ip_bind,

eslint.config.js

@@ -0,0 +1,92 @@
+module.exports = [
+  // Frontend configuration - for browser-based JavaScript
+  {
+    files: ["webhook_server/web/static/**/*.js"],
+    languageOptions: {
+      ecmaVersion: 2022,
+      sourceType: "script",
+      globals: {
+        // Browser environment globals
+        window: "readonly",
+        document: "readonly",
+        console: "readonly",
+        fetch: "readonly",
+        WebSocket: "readonly",
+        localStorage: "readonly",
+        sessionStorage: "readonly",
+        alert: "readonly",
+        confirm: "readonly",
+        prompt: "readonly",
+        setTimeout: "readonly",
+        clearTimeout: "readonly",
+        setInterval: "readonly",
+        clearInterval: "readonly",
+        URLSearchParams: "readonly",
+        AbortController: "readonly",
+      },
+    },
+    rules: {
+      // ESLint recommended rules (manually specified for broader coverage)
+      "constructor-super": "error",
+      "for-direction": "error",
+      "getter-return": "error",
+      "no-async-promise-executor": "error",
+      "no-case-declarations": "error",
+      "no-class-assign": "error",
+      "no-compare-neg-zero": "error",
+      "no-cond-assign": "error",
+      "no-const-assign": "error",
+      "no-constant-condition": "error",
+      "no-control-regex": "error",
+      "no-debugger": "error",
+      "no-delete-var": "error",
+      "no-dupe-args": "error",
+      "no-dupe-class-members": "error",
+      "no-dupe-else-if": "error",
+      "no-dupe-keys": "error",
+      "no-duplicate-case": "error",
+      "no-empty": "error",
+      "no-empty-character-class": "error",
+      "no-empty-pattern": "error",
+      "no-ex-assign": "error",
+      "no-extra-boolean-cast": "error",
+      "no-fallthrough": "error",
+      "no-func-assign": "error",
+      "no-global-assign": "error",
+      "no-import-assign": "error",
+      "no-inner-declarations": "error",
+      "no-invalid-regexp": "error",
+      "no-irregular-whitespace": "error",
+      "no-loss-of-precision": "error",
+      "no-misleading-character-class": "error",
+      "no-new-symbol": "error",
+      "no-obj-calls": "error",
+      "no-octal": "error",
+      "no-prototype-builtins": "error",
+      "no-redeclare": "error",
+      "no-regex-spaces": "error",
+      "no-self-assign": "error",
+      "no-setter-return": "error",
+      "no-shadow-restricted-names": "error",
+      "no-sparse-arrays": "error",
+      "no-this-before-super": "error",
+      "no-unexpected-multiline": "error",
+      "no-unreachable": "error",
+      "no-unsafe-finally": "error",
+      "no-unsafe-negation": "error",
+      "no-unsafe-optional-chaining": "error",
+      "no-unused-labels": "error",
+      "no-useless-backreference": "error",
+      "no-useless-catch": "error",
+      "no-useless-escape": "error",
+      "no-with": "error",
+      "require-yield": "error",
+      "use-isnan": "error",
+      "valid-typeof": "error",
+      // Project-specific overrides
+      "no-unused-vars": "warn",
+      "no-undef": "error",
+      "no-console": "off",
+    },
+  },
+];

pyproject.toml

@@ -14,6 +14,12 @@ line-length = 120
 fix = true
 output-format = "grouped"
 
+[tool.ruff.lint]
+select = ["E", "F", "W", "I", "B", "UP", "PLC0415", "ARG"]
+
+[tool.ruff.lint.per-file-ignores]
+"webhook_server/tests/*" = ["ARG"]
+
 [tool.ruff.format]
 exclude = [".git", ".venv", ".mypy_cache", ".tox", "__pycache__"]
 
@@ -33,20 +39,11 @@ warn_redundant_casts = true
 [tool.hatch.build.targets.wheel]
 packages = ["webhook_server"]
 
-[tool.uv]
-dev-dependencies = [
-  "ipdb>=0.13.13",
-  "ipython>=8.12.3",
-  "types-colorama>=0.4.15.20240311",
-  "types-pyyaml>=6.0.12.20250516",
-  "types-requests>=2.32.4.20250611",
-]
-
 [project]
 name = "github-webhook-server"
 version = "3.0.7"
 requires-python = ">=3.12"
-description = "A webhook server to manage Github reposotories and pull requests."
+description = "A webhook server to manage Github repositories and pull requests."
 readme = "README.md"
 license = "Apache-2.0"
 classifiers = [
@@ -101,6 +98,11 @@ requires = ["hatchling"]
 build-backend = "hatchling.build"
 
 [dependency-groups]
-tests = [
-    "psutil>=7.0.0",
+dev = [
+  "ipdb>=0.13.13",
+  "ipython>=8.12.3",
+  "types-colorama>=0.4.15.20240311",
+  "types-pyyaml>=6.0.12.20250516",
+  "types-requests>=2.32.4.20250611",
 ]
+tests = ["psutil>=7.0.0", "pytest-asyncio>=0.26.0", "pytest-xdist>=3.7.0"]

scripts/generate_changelog.py

@@ -10,17 +10,22 @@ def json_line(line: str) -> dict:
     Format str line to str that can be parsed with json.
 
     In case line is not formatted for json for example:
-    '{"title": "Revert "feat: Use git cliff to generate the change log. (#2322)" (#2324)", "commit": "137331fd", "author": "Meni Yakove", "date": "2025-02-16"}'
-    title have `"` inside the external `"` `"Revert "feat: Use git cliff to generate the change log. (#2322)" (#2324)"`
+    '{"title": "Revert "feat: Use git cliff to generate the change log. (#2322)" '
+    '(#2324)", "commit": "137331fd", "author": "Meni Yakove", "date": "2025-02-16"}'
+    title have `"` inside the external `"` `"Revert "feat: Use git cliff to '
+    'generate the change log. (#2322)" (#2324)"`
     """
     try:
         return json.loads(line)
     except json.JSONDecodeError:
         # split line like by `,`
-        # '{"title": "Revert "feat: Use git cliff to generate the change log. (#2322)" (#2324)", "commit": "137331fd", "author": "Meni Yakove", "date": "2025-02-16"}'
+        # '{"title": "Revert "feat: Use git cliff to generate the change log. '
+        # '(#2322)" (#2324)", "commit": "137331fd", "author": "Meni Yakove", '
+        # '"date": "2025-02-16"}'
         line_split = line.split(",")
 
-        # Pop and save `title key` and `title body` from '{"title": "Revert "feat: Use git cliff to generate the change log. (#2322)" (#2324)"'
+        # Pop and save `title key` and `title body` from '{"title": "Revert '
+        # '"feat: Use git cliff to generate the change log. (#2322)" (#2324)"'
         title_key, title_body = line_split.pop(0).split(":", 1)
 
         if title_body.count('"') > 2: