WIP: Track dwell time counter for time in area project

.coderabbit.yaml

@@ -71,10 +71,26 @@ reviews:
         AXIS OS 9.80), it should be avoided unless the AXIS OS limitations are explicitly
         mentioned. If not using `local`, it can be good to use a function specific prefix
         for the variable names to avoid conflicts with other variables.
+
         Clearly document in the head of the file which environment variables they expect
         and if they are optional or required.
+
         It's normally good to use `set -eu` for stricter error handling, but `-o pipefail`
         is not supported in the Axis devices' shell.
+
+        When an error occurs in a helper script that is used with the exec or execd plugin of
+        Telegraf, then the script should at minimum log an descriptive error message to stderr
+        and exit with a non-zero error code. The exit code should be unique enough to identify
+        the problem type and the error codes should be documented in the top of the script.
+        When errors are logged to stderr, Telegraf will only show the first line that was logged
+        followed by three dots. Therefore, a longer line might make more sense than multiple
+        shorter lines. The error logs should not end with a newline.
+
+        It is also good if the script respects the `TELEGRAF_DEBUG` variable, and if this is set
+        to "true", then the script should log debug messages to a file in the `HELPER_FILES_DIR`
+        with the .debug suffix. This log can be more verbose and make use of multiple lines.
+        When TELEGRAF_DEBUG is not set, logging to file should not be enabled since it can fill
+        the flash storage or wear out the storage chip.
     - path: "**/*.md"
       instructions: >-
         Documentation files should clearly communicate the dual audience: (1) server-side

.github/actions/python-quality-check/action.yml

@@ -0,0 +1,79 @@
+name: "Python Quality Check"
+description: "Run Python linting and type checking tools"
+inputs:
+  working-directory:
+    description: "Working directory for the Python project"
+    required: true
+  additional-types:
+    description: "Additional type packages to install (space-separated)"
+    required: false
+    default: ""
+  pylint-options:
+    description: "Additional options for pylint"
+    required: false
+    default: ""
+
+runs:
+  using: "composite"
+  steps:
+    - name: Setup Python
+      uses: actions/setup-python@v5
+      with:
+        python-version: "3.13"
+
+    - name: Install test dependencies
+      shell: bash
+      run: |
+        python -m pip install --upgrade pip
+        pip install \
+          black==25.1.0 \
+          flake8==7.3.0 \
+          isort==6.0.1 \
+          mypy==1.17.1 \
+          pylint==3.3.8 \
+          types-click==7.1.8 \
+          ${{ inputs.additional-types }}
+
+    - name: Install project requirements
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        pip install -r requirements.txt
+
+    - name: Run flake8
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        # E203: whitespace before ':' (conflicts with black)
+        # W503: line break before binary operator (conflicts with black)
+        flake8 . --max-line-length=100 --extend-ignore=E203,W503
+
+    - name: Run mypy
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        mypy .
+
+    - name: Run pylint
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        pylint *.py ${{ inputs.pylint-options }}
+
+    - name: Run isort
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        isort --check-only --diff --profile black .
+
+    - name: Run black
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        black --check --diff .
+
+    - name: Run doctests
+      shell: bash
+      working-directory: ${{ inputs.working-directory }}
+      run: |
+        python -m doctest *.py -v

.github/workflows/project-time-in-area-analytics-python-quality.yml

@@ -0,0 +1,30 @@
+name: Python Code Quality Check for the time-in-area-analytics project
+
+on:
+  push:
+    paths:
+      - "project-time-in-area-analytics/test_scripts/**"
+  pull_request:
+    paths:
+      - "project-time-in-area-analytics/test_scripts/**"
+
+permissions:
+  contents: read
+
+jobs:
+  python-quality:
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: project-time-in-area-analytics/test_scripts
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Run Python Quality Check
+        uses: ./.github/actions/python-quality-check
+        with:
+          working-directory: project-time-in-area-analytics/test_scripts
+          # Disable E1101 for OpenCV false positives (no-member errors for cv2)
+          pylint-options: --disable=E1101