Code Standards

• ask me to review before going to each next step (mention n step out of x) (AI only)
• before starting, prepare implementation plan (AI only)
• ask me to review it and ask any clarifying questions first
• add test creation as last step - follow repo architecture patterns - do not place in the root of /test
• code has to be maintainable, no duplicate code
• follow DRY principle - maintainability of code is more important than speed of implementation
• code files should be less than 500 LOC for better maintainability
• DB columns must not contain underscores, use camelCase instead (e.g., deviceInstanceId, not deviceinstanceid)
• treat DB as temporary storage for stats, long-term configuration should be stored in the /config folder, the /config folder should allow you to restore most of your functionality (excluding historical data)
• never access DB directly from application layers, always use helper functions in server/db/dbhelper.py and implement new functionality in handlers (e.g., DeviceInstance in server/models/deviceinstance.py)
• always validate and normalize MAC addresses before writing to DB (use normalizemac from pluginhelper.py)
• all subprocess calls must set explicit timeouts
• use timeNowUTC from utils.datetime_utils for all time-related operations and DB timestamps (store all timestamps in UTC)
• use sanitizers from server/helper.py for user input before storing in DB
• reuse shared mocks and factories from test/dbtesthelpers.py for tests, never redefine them locally
• use environment variables for runtime paths, never hardcode paths or use relative paths
• follow existing code style and structure, and ensure backward compatibility with existing installations when submitting PRs
• all code needs to be scalable to handle large networks with thousands of devices (10k+) without performance degradation
• no inline imports, all imports must be at the top of the file
• when using server/logger.py mylog(), only use valid levels: none, minimal, verbose, debug, trace; invalid levels silently degrade to none

File Length

Keep code files under 500 lines. Split larger files into modules.

DRY Principle

Do not re-implement functionality. Reuse existing methods or refactor to create shared methods.

Database Access

• Never access DB directly from application layers
• Use server/db/dbhelper.py functions (e.g., gettable_json)
• Implement new functionality in handlers (e.g., DeviceInstance in server/models/device_instance.py)

MAC Address Handling

Always validate and normalize MACs before DB writes:

from plugin_helper import normalize_mac

mac = normalize_mac(raw_mac)

Subprocess Safety

MANDATORY: All subprocess calls must set explicit timeouts.

result = subprocess.run(cmd, timeout=60)  # Minimum 60s

Nested subprocess calls need their own timeout—outer timeout won't save you.

Time Utilities

from utils.datetime_utils import timeNowUTC

timestamp = timeNowUTC()

This is the ONLY function that calls datetime.datetime.now() in the entire codebase.

⚠️ CRITICAL: ALL database timestamps MUST be stored in UTC This is the SINGLE SOURCE OF TRUTH for current time in NetAlertX Use timeNowUTC() for DB writes (returns UTC string by default) Use timeNowUTC(as_string=False) for datetime operations (scheduling, comparisons, logging)

String Sanitization

Use sanitizers from server/helper.py before storing user input. MAC addresses are always lowercased and normalized. IP addresses should be validated.

Devcontainer Constraints

• Never chmod or chown during operations
• Everything is already writable
• If permissions needed, fix .devcontainer/scripts/setup.sh

Test Helpers — No Duplicate Mocks

Reuse shared mocks and factories from test/dbtesthelpers.py. Never redefine DummyDB, make_db, or inline DDL in individual test files.

import sys, os
sys.path.insert(0, os.path.join(os.path.dirname(__file__), ".."))
from db_test_helpers import make_db, DummyDB, insert_device, minutes_ago

If a helper you need doesn't exist yet, add it to dbtesthelpers.py — not locally in the test file.

Stubbing Modules in Standalone-Capable Tests

If a test stubs NetAlertX modules into sys.modules so a script can be imported outside the container (see test/plugins/testntfycustom_headers.py), pop each stubbed name back out of sys.modules right after the one-time import that needed it. Otherwise the fake module leaks into every other test file collected in the same pytest session and shadows the real module (see testing-workflow skill for the full pattern and reproduction steps).

MAC Literals in Tests — ALWAYS Lowercase

MANDATORY: Every MAC address literal used in test fixtures, parametrize decorators, assertions, or comments must be lowercase hex:

# Correct
make_device_dict("aa:bb:cc:dd:ee:01", ...)

# Wrong — will be rejected in review
make_device_dict("AA:BB:CC:DD:EE:01", ...)
make_device_dict("Aa:Bb:Cc:Dd:Ee:01", ...)

This applies to hardcoded strings in assert, pytest.mark.parametrize, docstrings, and comments too. There are no exceptions.

Path Hygiene

• Use environment variables for runtime paths
• /data for persistent config/db
• /tmp for runtime logs/api/nginx state
• Never hardcode /data/db or use relative paths