docs: clarify pt_security_logger is standalone, does not extend pt_logging_system

[Copilot]

Review comment r3295308736 flagged that the PR description called pt_security_logger an extension of pt_logging_system, contradicting the module docstring which correctly states no such dependency exists. Updated the docstring to make the ownership model unambiguous at both the summary line and the explanatory note.

Description

Tightened app/pt_security_logger.py module docstring to eliminate the contradiction between the PR description ("extends pt_logging_system") and the module's actual design (fully standalone):

• Opening line: appended (does NOT extend pt_logging_system) so the standalone nature is visible at a glance
• Note block: strengthened from "does not depend on" to "does NOT extend, import, or depend on ... in any way. It is fully self-contained"

"""
PowerTraderAI+ Security & Audit Logging
Standalone security event logging module (does NOT extend pt_logging_system)
providing:
...

Note: This module does NOT extend, import, or depend on pt_logging_system in
any way. It is fully self-contained and manages its own logging handler
independently, so security events are always written to a dedicated audit
file regardless of the application's root logger configuration.
"""

Related Issues

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
• 📚 Documentation update
• 🔒 Security fix
• 🔧 Refactoring (no functional changes)
• ⚡ Performance improvement
• 🧪 Test coverage improvement

Testing

Testing Performed

• Unit tests added/updated
• Integration tests added/updated
• Manual testing completed
• Performance testing completed
• Security testing completed

Test Results

Docstring-only change; no logic altered. No test output applicable.

Security Checklist

• No hardcoded credentials or API keys
• Input validation implemented where needed
• Error handling doesn't expose sensitive information
• Authentication/authorization considered
• Security review completed (if security-related changes)

Code Quality Checklist

• Code follows project style guidelines
• Code is well-commented and documented
• No debugging code or console.log statements left
• Error handling is appropriate
• Performance impact considered
• Backward compatibility maintained (or breaking change documented)

Deployment Considerations

• Database migrations included (if needed)
• Environment variables documented (if new ones added)
• Configuration changes documented
• Rollback plan documented (for significant changes)

Documentation

• README updated (if needed)
• API documentation updated (if applicable)
• Inline code comments added/updated
• Changelog updated

Screenshots (if applicable)

Additional Notes

No behavioural changes. Pure docstring clarification to resolve the reviewer-flagged ambiguity about whether pt_security_logger extends pt_logging_system (it does not).

---

Review Checklist for Reviewers

• Code logic is sound and efficient
• Security implications reviewed
• Test coverage is adequate
• Documentation is clear and complete
• No performance regressions introduced
• Breaking changes are properly documented

[Copilot]

Pull request overview

Updates/introduces the standalone pt_security_logger module and its tests, with the stated goal of clarifying that it does not extend or depend on pt_logging_system.

Changes:

• Adds a standalone security/audit logging implementation with correlation ID support, JSONL audit output, and rotating secure file handling.
• Adds a comprehensive unittest suite covering correlation context behavior, event serialization, and logger I/O helpers.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
app/pt_security_logger.py	Implements standalone security audit logger + correlation context; docstring emphasizes no dependency on pt_logging_system.
app/test_security_logger.py	Adds unittests for correlation IDs, event objects, audit file behavior, and recent-event reading.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.